On 2025-07-27 12:52, David Yat Sin wrote:
> NV10 mask used for gfx12. Fix it.
> Put back DCC flag and default mtype to MTYPE_NC.
>
> Fixes: b8c76c59987a ("drm/amdgpu: rework how PTE flags are generated")
>
> Suggested-by: Felix Kuehling <[email protected]>
> Co-authored-by: Harish Kasiviswanathan <[email protected]>
> Signed-off-by: David Yat Sin <[email protected]>
> Signed-off-by: Harish Kasiviswanathan <[email protected]>
> Tested-by: Mark Broadworth <[email protected]>Reviewed-by: Felix Kuehling <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c > b/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c > index fb3fb31724a8..feb92e107af8 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c > @@ -514,10 +514,11 @@ static void gmc_v12_0_get_vm_pte(struct amdgpu_device > *adev, > *flags &= ~AMDGPU_PTE_VALID; > } > > - if (bo && bo->flags & (AMDGPU_GEM_CREATE_COHERENT | > - AMDGPU_GEM_CREATE_EXT_COHERENT | > - AMDGPU_GEM_CREATE_UNCACHED)) > - *flags = AMDGPU_PTE_MTYPE_NV10(*flags, MTYPE_UC); > + if (bo && bo->flags & AMDGPU_GEM_CREATE_GFX12_DCC) > + *flags |= AMDGPU_PTE_DCC; > + > + if (bo && bo->flags & AMDGPU_GEM_CREATE_UNCACHED) > + *flags = AMDGPU_PTE_MTYPE_GFX12(*flags, MTYPE_UC); > } > > static unsigned gmc_v12_0_get_vbios_fb_size(struct amdgpu_device *adev)
