Thanks Focus, so the lockup issue should be gone.
> Hi Sean and all, > QA side had tried the H264 playback(Three resolutions: 1080P, 480P, > CIF, each resolution was playbacked with 10 times ) by using the > yamidecoder as the test tool on the latest libva driver (master > branch). The kernel version which I used is 4.8 RC3 mainline kernel. > Both of BDW and HSW platforms were covered. > It worked well, and without any issue found. > > Thanks > Focus > > -----Original Message----- > From: Sean V Kelley [mailto:[email protected]] > Sent: Friday, November 18, 2016 12:45 AM > To: Xiang, Haihao <[email protected]>; [email protected] > Cc: [email protected]; Luo, Focus <[email protected]> > Subject: Re: [Libva] [PATCH] va/x11: Require synchronisation to > vblank with DRI2SwapBuffers > > On Thu, 2016-11-17 at 08:15 +0000, Xiang, Haihao wrote: > > But in Lukas's comment: > > > > "Had to revert it because of number of hard lockups." > > > > Did you have other patch to fix the lockup issue in other component > > or > > does the issue has gone away? If not, We have to revert the patch > > otherwise user will suffer lockup issue with libva. > > > +Focus > > You are making too many assumptions here. I don't have a problem > with the patch per se and if it will alleviate your concern we can > run some smoke tests on it. Ambiguous and unreproduced error reports > from a patch does not a revert make...So ask Focus to run some tests. > > > Thanks, > > Sean > > > > Thanks > > Haihao > > > > > > > On Thu, Nov 17, 2016 at 01:22:28AM +0000, Xiang, Haihao wrote: > > > > > > > > Hi, > > > > > > > > Someone got lockup with this patch and I don't see the issue is > > > > resolved, please check the thread below > > > > > > > > https://lists.freedesktop.org/archives/intel-gfx/2015-December/ > > > > 08 > > > > 35 > > > > 72.h > > > > tml ([Intel-gfx] vsync + vaapi question). > > > > > > > > and this is the last response: > > > > > > > > https://lists.freedesktop.org/archives/intel-gfx/2015-December/ > > > > 08 > > > > 38 > > > > 62.h > > > > tml > > > > > > > > I will revert this patch until we root the cause. > > > > > > This is not the cause of the lockup. > > > -Chris > > > > > > > _______________________________________________ > > Libva mailing list > > [email protected] > > https://lists.freedesktop.org/mailman/listinfo/libva _______________________________________________ Libva mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libva
