[AMD Official Use Only - Internal Distribution Only] Please split the patch into two, one to update the emit IB sequence, and one to bump the minor version. That way we can make sure older kernels get the new sequence. Also do we need an sdma fw version check for the new packet in the emi IB function?
Alex ________________________________ From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> on behalf of Christian König <christian.koe...@amd.com> Sent: Sunday, April 26, 2020 4:52 AM To: Marek Olšák <mar...@gmail.com> Cc: amd-gfx mailing list <amd-gfx@lists.freedesktop.org> Subject: Re: drm/amdgpu: invalidate L2 before SDMA IBs (on gfx10) Thought so, we should try to get this get it committed ASAP. And maybe add a CC: stable tag as well. Patch is Reviewed-by: Christian König <christian.koe...@amd.com><mailto:christian.koe...@amd.com>. Thanks, Christian. Am 26.04.20 um 02:28 schrieb Marek Olšák: Not without a mandatory firmware update. The gcr packet support for IBs was added into the sdma firmware just two weeks ago. Marek On Sat., Apr. 25, 2020, 11:04 Christian König, <ckoenig.leichtzumer...@gmail.com<mailto:ckoenig.leichtzumer...@gmail.com>> wrote: Could we do this in userspace as well? Am 25.04.20 um 10:52 schrieb Marek Olšák: This should fix SDMA hangs on gfx10. Marek _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> https://lists.freedesktop.org/mailman/listinfo/amd-gfx<https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Calexander.deucher%40amd.com%7Cc88f73e068bb483bb21f08d7e9bf2cd4%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637234879980330283&sdata=LZ2GMq0a0n6cm74ulhIkMq%2Fyj6XlG1lj9AH0InU1%2BdQ%3D&reserved=0>
_______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx