<div dir="ltr"><div><div>Hi Christian,<br><br>I have 2 remarks:<br><br></div><div>1: I am sure you have the clear answer to the following questions, so maybe add a comment in the commit message:<br></div><div>Isn't the thread safety delegated to the application that use vaapi ? like it is designed for many libraries. I tried to find answer in vaapi-intel-driver and libva but it is not obvious. I'll try the mailing list. <br></div>For vdpau you are right: "All VDPAU functionality is fully thread-safe; any number of threads may call into any VDPAU functions at any time. VDPAU may not be called from signal-handlers."<br></div>I could not find something equivalent for vaapi but I guess it would make sense if it was the same.<br><div><div><div><div><br></div><div>2: Sometimes lock/unlock is just surrounding the "get". Shouldn't the mutex be kept locked until the function is done accessing the resources returned by this "get". (if it was refcounted you would not unref before you continue to access it)<br></div><div>(Also I cannot see any lock/unlock in vlVaQueryVideoProcPipelineCaps)<br><br></div><div>Otherwise this patch is:<br>Reviewed-by: Julien Isorce <<a href="mailto:j.isorce@samsung.com">j.isorce@samsung.com</a>><br>Tested-by: Julien Isorce <<a href="mailto:j.isorce@samsung.com">j.isorce@samsung.com</a>><br><br></div><div>Cheers<br></div><div>Julien<br></div><div><br></div></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 18 December 2015 at 15:08, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Christian,<br>
<br>
I've just sent a few comments/suggestions. With or without the<br>
nitpicks (but the remaining addressed) the series is<br>
Reviewed-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Emil<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div>