[pulseaudio-discuss] [PATCH] tunnel-{sink, source}-new: Fix assertion when used with module-loopback

[Georg Chini]

On 17.07.2017 19:32, Tanu Kaskinen wrote:
> On Sun, 2017-07-16 at 11:42 +0200, Georg Chini wrote:
>> Currently pulseaudio crashes with an assertion in pa_rtpoll_item_new_asyncmsgq_read()
>> if a loopback is applied to a tunnel-new sink/source because tunnel-{sink,source}-new
>> do not set thread_info.rtpoll. Bug was reported on IRC.
>>
>> This patch fixes the problem by initializing thread_info.rtpoll properly.
> Did you test this patch? The tunnel devices don't run the rtpoll that
> you create, so I would expect that the loopback won't work.
>
> This is a known bug that I started working on in the past:
> https://bugs.freedesktop.org/show_bug.cgi?id=73429
>
> I made some patches, issues got pointed out in review, and then I never
> finished v2 of the patches. I haven't given up on that, but it's been a
> year since I last worked on it... In case you're interested in the
> current state of the v2 patches, I pushed the code to
> git://people.freedesktop.org/~tanuk/pulseaudio branch "rtpoll-mainloop-
> v2".
>
I haven't tested it myself, but the bug reporter on IRC (tar-dingens)
tested them and said it worked. The patch looked like the easiest way to
make module-loopback happy.