[PATCH v0] loopback: Fix cork state not updated after move

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2013-07-01 at 10:27 +0200, Mikel Astiz wrote:
> From: Mikel Astiz <mikel.astiz at bmw-carit.de>
> 
> The source output and sink inputs should be corked if the corresponding
> sink/source is suspended, as handled during module initialization. This
> also needs to be handled during stream move, because the suspend state
> of the destination sink/source might be different to the previous one.
> 
> This fixes the issue with an infinite number of "Requesting rewind due
> to end of underrun" traces after a stream move.

module-loopback should certainly update the stream cork states when the
streams are moved, but this patch looks like it will cause crashing. Did
you test the patch? The moving() callback is called when i->sink is
NULL, and pa_sink_input_cork() looks like it will crash in that
situation.

As an aside, it's not clear to me why there would be endless rewind
requests if the cork state is wrong. I would expect there to be just
loads of "Could not peek into queue" messages.

-- 
Tanu



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux