<div dir="ltr">Boyuan, gentle ping ?<div class="gmail_extra"><br><div class="gmail_quote">On 26 July 2017 at 19:36, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So that means Reviewed-and-tested-by: Zhang, Boyuan <<a href="mailto:Boyuan.Zhang@amd.com" target="_blank">Boyuan.Zhang@amd.com</a>>?<br>
<br>
Christian.<div><div class="h5"><br>
<br>
Am 26.07.2017 um 20:29 schrieb Zhang, Boyuan:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
The code looks good to me. I also did a few local vaapi-enc tests and confirmed that patch works fine.<br>
<br>
Regards,<br>
Boyuan<br>
<br>
-----Original Message-----<br>
From: Liu, Leo<br>
Sent: July-26-17 12:01 PM<br>
To: Andy Furniss; Koenig, Christian; Julien Isorce; <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><wbr>; Zhang, Boyuan<br>
Subject: Re: [Mesa-dev] [PATCH] st/va: change frame_idx from array to hash table<br>
<br>
<br>
<br>
On 07/25/2017 06:04 PM, Andy Furniss wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Christian König wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Leo and Boyuan can you take a quick look as well? On first glance<br>
looks totally sane to me.<br>
</blockquote>
This reminds me .....<br>
<br>
I don't know what's special about my setup, but I haven't been able to<br>
use gst + vce properly since March.<br>
<br>
As I said at the time -<br>
<br>
<a href="https://lists.freedesktop.org/archives/mesa-dev/2017-March/148216.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>archives/mesa-dev/2017-March/1<wbr>48216.html</a><br>
<br>
I did ping as well, but never quite got round to doing a bug.<br>
<br>
I guess devs who tests vce with gstreamer don't see this?<br>
</blockquote>
Personally, haven't been working on encode too much recently. esp for VA-API encode.<br>
<br>
Cheers,<br>
Leo<br>
<br>
<br></div></div>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote>
<br>
<br>
</blockquote></div><br></div></div>