Please answer my second question about why we aren't parsing audio metadata in a separate process.
On Sun, 2014-10-19 at 00:35 +0000, re-cy...@hushmail.com wrote: > "I think crashing on totally crap data is not completely > unreasonable as long as we can figure out some way of making it > debuggable." > > *raises hand* > > As someone who has used Mixxx for live gigs, where it crashed > enough times during various sets to sabotage my reputation as a > promoter and DJ, I want to remind people that software used in live > performances has to be held to the highest possible non-crash > standard. > > I don't care if the thing looks like it was made by kindergarten > students as a class project as long as it queues music properly and > NEVER CRASHES LIVE. > > ~RAWRR > > > On Sat, 18 Oct 2014 23:12:36 +0000 "Owen Williams" > <owilli...@mixxx.org> wrote: > >Is taglib really that crash-prone? Is this really a common issue? > > I > >think crashing on totally crap data is not completely unreasonable > >as > >long as we can figure out some way of making it debuggable. > > > >I don't see how processing files for cover art is any different > >than > >processing files for audio data, and we don't process audio data > >in a > >separate kernel process. > > > >On Sat, 2014-10-18 at 15:09 -0400, RJ Ryan wrote: > >> The main limitation with QtConcurrent is that it uses a single > >global > >> thread pool -- and currently we use 4 worker threads in our > >global > >> thread pool (for parsing 3rd-party libraries, etc). There is no > >notion > >> of auto-scaling or the ability for sensing whether we are > >overloading > >> the system inherent in QtConcurrent -- it's just a wrapper > >around a > >> thread pool. For something as common and compute heavy as > >processing > >> cover art we need more deliberate control. > >> > >> TagLib crashes will happen given the extreme diversity of > >> library-from-hell files out there. We really do need a > >> separate-process isolation feature for processing untrusted data > >but > >> I'm pretty sure we don't have time for that for 1.12 if we want > >to > >> release in 2014 :). > >> > >> > >> > >> > >> On Fri, Oct 17, 2014 at 7:33 PM, Daniel Schürmann > ><dasch...@mixxx.org> wrote: > >> > Am 18.10.2014 um 00:31 schrieb Max Linke: > >> >> I haven't noticed any taglib crashes with mixxx. Plus the > >covers are all > >> >> extracted in seperate threads. > >> > I have not experienced any crashes during the tests as well, > >so the > >> > issue is hopefully none. > >> > I was just trying to collect the point that might be risky. > >> > By the way: Segfaults in a extra thread will have the same > >effect than a > >> > segfault in the main thread, Mixxx will crash. > >> > So we can either rely on the quality of the new taglib > >version, or setup > >> > a quarantine process, that read the tags. > >> > > >> > > >> > --------------------------------------------------------------- > >--------------- > >> > Comprehensive Server Monitoring with Site24x7. > >> > Monitor 10 servers for $9/Month. > >> > Get alerted through email, SMS, voice calls or mobile push > >notifications. > >> > Take corrective actions from your mobile device. > >> > http://p.sf.net/sfu/Zoho > >> > _______________________________________________ > >> > Get Mixxx, the #1 Free MP3 DJ Mixing software Today > >> > http://mixxx.org > >> > > >> > > >> > Mixxx-devel mailing list > >> > Mixxx-devel@lists.sourceforge.net > >> > https://lists.sourceforge.net/lists/listinfo/mixxx-devel > >> > >> ----------------------------------------------------------------- > >------------- > >> Comprehensive Server Monitoring with Site24x7. > >> Monitor 10 servers for $9/Month. > >> Get alerted through email, SMS, voice calls or mobile push > >notifications. > >> Take corrective actions from your mobile device. > >> http://p.sf.net/sfu/Zoho > >> _______________________________________________ > >> Get Mixxx, the #1 Free MP3 DJ Mixing software Today > >> http://mixxx.org > >> > >> > >> Mixxx-devel mailing list > >> Mixxx-devel@lists.sourceforge.net > >> https://lists.sourceforge.net/lists/listinfo/mixxx-devel > > > > > > > >------------------------------------------------------------------- > >----------- > >Comprehensive Server Monitoring with Site24x7. > >Monitor 10 servers for $9/Month. > >Get alerted through email, SMS, voice calls or mobile push > >notifications. > >Take corrective actions from your mobile device. > >http://p.sf.net/sfu/Zoho > >_______________________________________________ > >Get Mixxx, the #1 Free MP3 DJ Mixing software Today > >http://mixxx.org > > > > > >Mixxx-devel mailing list > >Mixxx-devel@lists.sourceforge.net > >https://lists.sourceforge.net/lists/listinfo/mixxx-devel ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://p.sf.net/sfu/Zoho _______________________________________________ Get Mixxx, the #1 Free MP3 DJ Mixing software Today http://mixxx.org Mixxx-devel mailing list Mixxx-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mixxx-devel