[AMD Official Use Only] > -----Original Message----- > From: Borislav Petkov <bp@xxxxxxxxx> > Sent: Thursday, October 14, 2021 5:01 PM > To: Quan, Evan <Evan.Quan@xxxxxxx> > Cc: Alex Deucher <alexdeucher@xxxxxxxxx>; amd-gfx list <amd- > gfx@xxxxxxxxxxxxxxxxxxxxx>; LKML <linux-kernel@xxxxxxxxxxxxxxx>; Deucher, > Alexander <Alexander.Deucher@xxxxxxx>; Pan, Xinhui > <Xinhui.Pan@xxxxxxx>; Chen, Guchun <Guchun.Chen@xxxxxxx> > Subject: Re: bf756fb833cb ("drm/amdgpu: add missing cleanups for Polaris12 > UVD/VCE on suspend") > > On Thu, Oct 14, 2021 at 02:02:48AM +0000, Quan, Evan wrote: > > [Quan, Evan] Yes, but not(apply them) at the same time. One by one as > you did before. > > - try the patch1 first > > Ok, first patch worked fine. > > > - undo the changes of patch1 and try patch2 > > Did that, worked fine too except after the first resume cycle, the video didn't > continue playing. > > Then I restarted the video and did a couple more suspend cycles to see if it > would not continue again. In the subsequent tries it would resume fine and > the video would continue playing too. > > So I'm going to chalk that single case of halted video with the second patch to > a resume glitch or so. > > Btw, I don't have pm-suspend on that box but I did suspend to RAM > assuming this is what you wanted, which is done as root with two > commands: > > # echo "suspend" > /sys/power/disk > # echo "mem" > /sys/power/state [Quan, Evan] Yes, that also works. > > If you want me to do more extensive testing, just shoot. [Quan, Evan] Thanks! That's enough for now. I will prepare an official solution for the issue. BR Evan > > HTH. > > -- > Regards/Gruss, > Boris. > > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpeo > ple.kernel.org%2Ftglx%2Fnotes-about- > netiquette&data=04%7C01%7CEvan.Quan%40amd.com%7C08df3d5453 > d64ad40dfa08d98ef119ec%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C > 0%7C637697988457790715%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLj > AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000& > sdata=zmFVlmUBv6byoDYyUhSgk9J9Zmvexz5IqG7xBxwiR3M%3D&rese > rved=0