Bug 326871 - zynaddsubfx improper blocking call in Jack process function
Summary: zynaddsubfx improper blocking call in Jack process function
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: zynaddsubfx
Version: rawhide
Hardware: All
OS: Linux
medium
high
Target Milestone: ---
Assignee: Anthony Green
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-10-10 19:41 UTC by Fernando Lopez-Lezcano
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-10-10 21:36:50 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
unzombify patch (1001 bytes, patch)
2007-10-10 19:41 UTC, Fernando Lopez-Lezcano
no flags Details | Diff
spec file changes (1.29 KB, patch)
2007-10-10 19:42 UTC, Fernando Lopez-Lezcano
no flags Details | Diff

Description Fernando Lopez-Lezcano 2007-10-10 19:41:19 UTC
Description of problem:

ZynAddSubFX was not very well designed to work with the Jack audio server, the
jack process call can block on a mutex (it should not) and then the jack client
delays processing the whole jack graph. This patch by Lars Luthman avoids the
problem and zynaddsubfx becomes a lot more reliable, specially with the current
version of svn jack which is much more sensitive to hiccups in the clients. 

Version-Release number of selected component (if applicable):
Current version is 2.2.1-14.fcx

How reproducible:
Always

Steps to Reproduce:
1. For example, change the current patch, you should observe a glitch in the
Jack output. If the glitch is long enough ZynAddSubFX gets kicked out of the
Jack processing graph. 
  
Actual results:
Xruns in the Jack audio server

Expected results:
No xruns in the Jack audio server

Additional info:
Patch originally by Lars Luthman, tweaked to patch on zyn with no preexisting
Jack midi patches.

Attached files: zynaddsubfx-unzombify.patch

Comment 1 Fernando Lopez-Lezcano 2007-10-10 19:41:19 UTC
Created attachment 223291 [details]
unzombify patch

Comment 2 Fernando Lopez-Lezcano 2007-10-10 19:42:08 UTC
Created attachment 223301 [details]
spec file changes

Comment 3 Anthony Green 2007-10-10 20:25:42 UTC
Thanks.  I agree that ZynAddSubFX has problems in this area.    I looked into it
about a year ago with an eye to have it use lockless buffers.   I'll push out a
new zyn for devel, F7 and FC-6.


Comment 4 Fernando Lopez-Lezcano 2007-10-10 20:30:35 UTC
(In reply to comment #3)
> Thanks.  I agree that ZynAddSubFX has problems in this area.    I looked into it
> about a year ago with an eye to have it use lockless buffers.   I'll push out a
> new zyn for devel, F7 and FC-6.

Thanks! Obviously it'd be good to have a proper fix but this patch improves the
situation a lot. I'm testing a new svn jack and zyn without this is not really
usable. 


Comment 5 Anthony Green 2007-10-10 21:36:50 UTC
I've kicked off builds for FE-6, F-7 and devel.  The next updates should include
this patch.  I'm closing this bug.  Thanks nando!



Note You need to log in before you can comment on or make changes to this bug.