The code looks good to me. I also did a few local vaapi-enc tests and confirmed that patch works fine.
Regards, Boyuan -----Original Message----- From: Liu, Leo Sent: July-26-17 12:01 PM To: Andy Furniss; Koenig, Christian; Julien Isorce; [email protected]; Zhang, Boyuan Subject: Re: [Mesa-dev] [PATCH] st/va: change frame_idx from array to hash table On 07/25/2017 06:04 PM, Andy Furniss wrote: > Christian König wrote: > >> Leo and Boyuan can you take a quick look as well? On first glance >> looks totally sane to me. > > This reminds me ..... > > I don't know what's special about my setup, but I haven't been able to > use gst + vce properly since March. > > As I said at the time - > > https://lists.freedesktop.org/archives/mesa-dev/2017-March/148216.html > > I did ping as well, but never quite got round to doing a bug. > > I guess devs who tests vce with gstreamer don't see this? Personally, haven't been working on encode too much recently. esp for VA-API encode. Cheers, Leo _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
