Hi Col,<div><br></div><div>Pls find attached updated patch, it works fine on my Meego platform.</div><div>Feel free to let me know if there&#39;s any risk.</div><div><br></div><div>--xingchao</div><div><br>2011/8/11 Colin Guthrie &lt;<a href="mailto:gmane@colin.guthr.ie">gmane@colin.guthr.ie</a>&gt;:<br>
&gt; &#39;Twas brillig, and Colin Guthrie at 05/08/11 12:41 did gyre and gimble:<br>&gt;&gt; &#39;Twas brillig, and xing wang at 05/08/11 05:46 did gyre and gimble:<br>&gt;&gt;&gt; Hi,<br>&gt;&gt;&gt;<br>&gt;&gt;&gt; Does anyone has comment on this patch?<br>
&gt;&gt;<br>&gt;&gt; Sorry, I forgot about this one. I had it marked as unread, but it fell<br>&gt;&gt; of the bottom of my unread list. Thanks very much for pinging it up again.<br>&gt;&gt;<br>&gt;&gt; I think it looks good, but I&#39;ll go over it with a few people in the next<br>
&gt;&gt; few days! It&#39;s certainly a problem, so it needs fixed and this patch<br>&gt;&gt; looks like the right fix on first glance.<br>&gt;<br>&gt; OK, so this causes the following assert:<br>&gt;<br>&gt; E: [alsa-sink] sink-input.c: Assertion<br>
&gt; &#39;PA_SINK_INPUT_IS_LINKED(i-&gt;thread_info.state)&#39; failed at<br>&gt; ../../src/pulsecore/sink-input.c:1049, function<br>&gt; pa_sink_input_update_max_request(). Aborting.<br>&gt;<br>&gt;<br>&gt; This is done when a stream is in a START_CORKED state I believe (paplay<br>
&gt; triggered it).<br>&gt;<br>&gt; Either way this cannot be done yet.... can you come up with a better<br>&gt; patch that doesn&#39;t break paplay and friends?<br>&gt;<br>&gt; Col<br>&gt;<br>&gt;<br>&gt;<br>&gt;<br>&gt;<br>
&gt; --<br>&gt;<br>&gt; Colin Guthrie<br>&gt; gmane(at)<a href="http://colin.guthr.ie">colin.guthr.ie</a><br>&gt; <a href="http://colin.guthr.ie/">http://colin.guthr.ie/</a><br>&gt;<br>&gt; Day Job:<br>&gt;  Tribalogic Limited [<a href="http://www.tribalogic.net/">http://www.tribalogic.net/</a>]<br>
&gt; Open Source:<br>&gt;  Mageia Contributor [<a href="http://www.mageia.org/">http://www.mageia.org/</a>]<br>&gt;  PulseAudio Hacker [<a href="http://www.pulseaudio.org/">http://www.pulseaudio.org/</a>]<br>&gt;  Trac Hacker [<a href="http://trac.edgewall.org/">http://trac.edgewall.org/</a>]<br>
&gt;<br>&gt; _______________________________________________<br>&gt; pulseaudio-discuss mailing list<br>&gt; <a href="mailto:pulseaudio-discuss@lists.freedesktop.org">pulseaudio-discuss@lists.freedesktop.org</a><br>&gt; <a href="http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss">http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss</a><br>
&gt;<br><br></div>