[PATCH] drm/amdkfd: Insert missing TLB flush on GFX10 and later
Felix Kuehling
felix.kuehling at amd.com
Tue Sep 12 00:55:36 UTC 2023
On 2023-09-11 15:00, Harish Kasiviswanathan wrote:
> Heavy-weight TLB flush is required after unmap on all GPUs for
> correctness and security.
>
> Signed-off-by: Harish Kasiviswanathan <Harish.Kasiviswanathan at amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> index b315311dfe2a..b9950074aee0 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> @@ -1466,8 +1466,7 @@ void kfd_flush_tlb(struct kfd_process_device *pdd, enum TLB_FLUSH_TYPE type);
>
> static inline bool kfd_flush_tlb_after_unmap(struct kfd_dev *dev)
> {
> - return KFD_GC_VERSION(dev) == IP_VERSION(9, 4, 3) ||
> - KFD_GC_VERSION(dev) == IP_VERSION(9, 4, 2) ||
> + return KFD_GC_VERSION(dev) > IP_VERSION(9, 4, 2) ||
> (KFD_GC_VERSION(dev) == IP_VERSION(9, 4, 1) && dev->sdma_fw_version >= 18) ||
> KFD_GC_VERSION(dev) == IP_VERSION(9, 4, 0);
> }
More information about the amd-gfx
mailing list