Wednesday, 2016-12-07

*** trebmuh has quit IRC00:39
*** Yruama_Lairba has quit IRC02:30
*** ColaEuphoria_ has quit IRC05:16
*** ColaEuphoria_ has joined #lv205:31
*** grejppi has quit IRC05:55
*** grejppi has joined #lv206:31
*** trebmuh has joined #lv209:09
*** sigma6 has joined #lv209:55
*** Yruama_Lairba has joined #lv210:44
*** edogawa has joined #lv211:50
*** SianaGearz has quit IRC11:56
*** SianaGearz has joined #lv211:56
*** SianaGearz has joined #lv211:56
*** ricardocrudo has joined #lv212:28
ventosusrgareus: does ardour still run 'state_restore' and 'run' concurrently?12:48
ventosusI'm investigating a race condition in my plug in 'state_restore'12:49
*** son0p has joined #lv212:50
falktxsomething was added to lv2 because of this12:57
drobillaYeah, state:threadSafeRestore is the proper way13:22
drobillaI don't remember if the normal case was fixed in ardour13:22
ventosusseems not, and it does not explicitely check for the feature either13:27
rgareusdrobilla: I thought you fixed it when adding  state:threadSafeRestore13:27
rgareusventosus: loading presets pushes a different worker-thread (state_worker)13:29
drobillargareus: I don't think I figured out the "stop processing while this happens" bit13:32
rgareusventosus: which plugin(s) are affected by that, anyway?13:33
drobillaAlso github search apparently doesn't search commit messages, which is impressively useless13:35
drobillaA smart handle would be nice, RAII "if you have this, the plugin isn't being run"13:37
drobillaWaiting for the first/next cycle to arrive, but also not blocking forever if it never comes, is the annoying part13:37
ventosusrgareus: moony. I can easily switch to state:threadSafeRestore, though, no big deal13:40
rgareusdrobilla: smells like TRY_LOCK to me.13:41
rgareusdrobilla: is that actually in the spec, that  run() and  state save/restore must not be called concurrently?13:43
rgareusit's indirectly mentioned in http://lv2plug.in/ns/ext/state/state.html#threadSafeRestore13:44
rgareushttp://lv2plug.in/doc/html/group__state.html#ac6d423f49c55a6df24ef687450d3cf1a  has it.  good13:46
rgareusdrobilla: RAII will be more work.  at the end of   LV2Plugin::set_state()  there's  a  latency_compute_run() which needs to run.13:48
drobillargareus: I guess that will do14:00
drobillargareus: Not really more difficult in that respect, if (locked) vs if (should_run_flag).  Acquiring the mutex is much simpler though14:02
*** ricardocrudo has quit IRC14:22
ssj71reMID.lv2 does a bunch of allocation in the state restore funciton fwiw15:46
rgareusthat should be fine.  state load/save is not meant to be rt-safe, is it?15:47
rgareusit's instantiation threading class.15:48
ssj71oh duh, thread safe and rt safe are independent15:49
ssj71my bad15:50
rgareuswell, a lot of plugins don't care if   state save/restore  is called concurrently with run().     but for those that do care it's usually fatal.15:50
drobillaFor some value of "don't care" anyway.  Almost all can cause an audible disaster15:53
drobillaIf not a crashy one15:53
rgareusif it's only GUI state.. no.15:57
*** ssj71 has quit IRC16:35
drobillaWell, sure16:40
*** ssj71 has joined #lv216:40
*** son0p has quit IRC16:52
*** sigma6 has quit IRC17:18
*** deva has joined #lv218:51
*** ricardocrudo has joined #lv219:26
*** oofus has quit IRC20:03
*** oofus has joined #lv220:08
*** HarryHaaren has joined #lv220:30
HarryHaarendrobilla: ping, you have an interesting nanogui repo fork - I'm trying to compile but its failing to link - any advice?20:31
*** ricardocrudo has quit IRC20:36
drobillaHarryHaaren: link harder? :P20:38
drobilla(gonna need more information than that)20:38
HarryHaarendrobilla: heh good point. http://pastebin.com/vMnq1zVQ  that's already build the library, just linking the examples20:39
HarryHaarenVERBOSE=1 output http://pastebin.com/wLKvqED920:39
HarryHaarenand cmake .. output http://pastebin.com/QAWaJpzv20:41
drobillaHarryHaaren: You appear to have no pugl lib, or an out of date one, or some such20:42
HarryHaarenyep, but its the latest git cloned, and installed in /usr20:42
drobillaHarryHaaren: Old one in /usr/local?20:43
drobilla... hm, though I get a compile error with most recent20:46
HarryHaarendrobilla: yeah I "hacked" around that puglGetModifier one20:46
HarryHaarenbut then hit the linking error which I haven't been able to solve yet..20:47
HarryHaarenno old versions as far as I can see here - nuked all of them and reinstalled but no joy20:47
* HarryHaaren tries clean cmake .. && recompile20:47
HarryHaarenit is *finding* libpugl-0, because if i manually change the command it fails differently20:49
drobillaI guess your pugl build could be botched somehow and not exporting symbols..20:49
drobillaI'd exhaustively look around for libpugls and objdump them for said symbols etc20:50
HarryHaarengood plan, i'll get to it20:50
drobillaFixed the compile issue20:51
drobillaFor some value of "fixed" that means it compiles at least20:51
HarryHaareni'll do a pull20:54
HarryHaarendrobilla: did you push?20:54
drobillammhmm20:54
HarryHaarendo you use Github or your own Git? I don't see the commit on either..20:55
*** deva has quit IRC20:57
* HarryHaaren pulls commit - took a while to show up on Github. I'm a bit impatient that way :)21:02
HarryHaarenstrange - only the examples fail to link21:04
HarryHaarenthe python + nanogui.so link fine, and ldd shows them to be fine too21:04
JaVelDahttp://www.premierguitar.com/articles/24993 Nice article, but links to non informative subscribe-page about MOD21:31
ssj71interesting. IMO it would have been a good idea to launch the new website before a press release21:33
ssj71but what do I know21:33
JaVelDafalktx ; there must be more than this by now.21:41
ssj71JaVelDa: forums.moddevices.com21:42
*** oofus has quit IRC21:42
ssj71oops, forum.moddevices.com21:42
*** HarryHaaren has quit IRC21:42
*** oofus has joined #lv221:45
JaVelDassj71 : That more like it, but http://moddevices.com/ should also link to this.21:46
ssj71well, were it my company, I wouldn't link to it, but I sure wouldn't have just a subscribe box either21:48
ssj71they had a decent front-page before21:48
ssj71JaVelDa: there's #moddevices too, if you want to talk to more of the mod team21:49
JaVelDa ssj71 I think falktx is on the mod team so all I wanted to do was to warn him of the "dead link" in the fresh new article I found outside Linux/Foss usual places.21:57
JaVelDaIn best wishes for them21:58
rgareusyeah it's an odd choice to remove all links from the mod front-page22:03
*** edogawa has quit IRC23:00
*** JaVelDa has left #lv223:30
*** JaVelDa has joined #lv223:31
*** grejppi has quit IRC23:31
*** grejppi has joined #lv223:46

Generated by irclog2html.py 2.13.0 by Marius Gedminas - find it at mg.pov.lt!