<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 21 December 2015 at 15:10, Christian König <span dir="ltr"><<a href="mailto:deathsimple@vodafone.de" target="_blank">deathsimple@vodafone.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On 21.12.2015 11:05, Andy Furniss wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Julien Isorce wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Hi Christian,<br>
<br>
I have 2 remarks:<br>
<br>
1: I am sure you have the clear answer to the following questions, so<br>
maybe add a comment in the commit message: Isn't the thread safety<br>
delegated to the application that use vaapi ? like it is designed for<br>
many libraries. I tried to find answer in vaapi-intel-driver and<br>
libva but it is not obvious. I'll try the mailing list. For vdpau you<br>
are right: "All VDPAU functionality is fully thread-safe; any number<br>
of threads may call into any VDPAU functions at any time. VDPAU may<br>
not be called from signal-handlers." I could not find something<br>
equivalent for vaapi but I guess it would make sense if it was the<br>
same.<br>
</blockquote>
<br>
Oh, interesting as ffmpeg recently decided hwdec should be forced to<br>
single thread for some reason. It hurts perf for using ffmpeg cli.<br>
</blockquote>
<br></span>
Well I had to add this because of the simple reason MPV failed miserable in some circumstances because it assumed that VA-API is thread safe.<br>
<br>
And yes Julien your suspicion is right that VA-API simply doesn't define this properly like VDPAU does.<span><br></span></blockquote><div><br></div><div>I guess you saw the conversation on libva mailing list, what you did matches what it is done in intel-vaapi-driver so that's great.<br></div><div><br>I had time to go further, the series is:<br>Reviewed-by: Julien Isorce <<a href="mailto:j.isorce@samsung.com" target="_blank">j.isorce@samsung.com</a>><br>Tested-by: Julien Isorce <<a href="mailto:j.isorce@samsung.com" target="_blank">j.isorce@samsung.com</a>><br><br><br></div></div></div></div>