[PATCH] drm/amdkfd: enable heavy-weight TLB flush on Arcturus

Christian König christian.koenig at amd.com
Wed Jan 19 07:31:13 UTC 2022


Am 18.01.22 um 21:13 schrieb Eric Huang:
> I understand Alex's concern. I think usually we only check the version 
> when a feature is only available in a specific version, and other 
> version or newer version doesn't have.
>
> In case of FW fix, we assume the driver and FWs have to be 
> synchronous. If we have driver backward compatibility for FWs, there 
> must be a lot of redundant codes for FW version check. So this patch 
> and SDMA fix will be pushed into ROCm 5.1 release branch at the same 
> time.

Others replied as well, but just to make it clear this approach is an 
absolutely clear NAK.

Driver backward compatibility is a documented mandatory feature.

So if you already pushed this to ROCm 5.1 then please revert that ASAP.

Regards,
Christian.

>
> Regards,
> Eric



More information about the amd-gfx mailing list