Re: [PATCH] drm/amdgpu: don't read DF register for SRIOV
Hi Yintian, sorry I have not the slightest idea how this part of the hw works. Maybe try to explain more what the DF register is actually doing in the commit message. I suspect that it is only about detecting which VRAM connection is used, but I'm not 100% sure. Regards, Christian. Am 17.05.19 um 04:38 schrieb Tao, Yintian: > Ping... > > Hi Christian and Alex > > > Can you help review this? Thanks in advance. > > > Best Regards > Yintian Tao > > -Original Message- > From: amd-gfx On Behalf Of Yintian Tao > Sent: Thursday, May 16, 2019 8:11 PM > To: amd-gfx@lists.freedesktop.org > Cc: Liu, Monk ; Tao, Yintian > Subject: [PATCH] drm/amdgpu: don't read DF register for SRIOV > > [CAUTION: External Email] > > Under SRIOV, reading DF register has chance to lead to AER error in host > side, just skip reading it. > > Signed-off-by: Monk Liu > Signed-off-by: Yintian Tao > --- > drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > index a417763..b5bf9ed 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > @@ -837,7 +837,7 @@ static int gmc_v9_0_mc_init(struct amdgpu_device *adev) > > if (amdgpu_emu_mode != 1) > adev->gmc.vram_width = > amdgpu_atomfirmware_get_vram_width(adev); > - if (!adev->gmc.vram_width) { > + if (!adev->gmc.vram_width && !amdgpu_sriov_vf(adev)) { > /* hbm memory channel size */ > if (adev->flags & AMD_IS_APU) > chansize = 64; > -- > 2.7.4 > > ___ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
RE: [PATCH] drm/amdgpu: don't read DF register for SRIOV
Ping... Hi Christian and Alex Can you help review this? Thanks in advance. Best Regards Yintian Tao -Original Message- From: amd-gfx On Behalf Of Yintian Tao Sent: Thursday, May 16, 2019 8:11 PM To: amd-gfx@lists.freedesktop.org Cc: Liu, Monk ; Tao, Yintian Subject: [PATCH] drm/amdgpu: don't read DF register for SRIOV [CAUTION: External Email] Under SRIOV, reading DF register has chance to lead to AER error in host side, just skip reading it. Signed-off-by: Monk Liu Signed-off-by: Yintian Tao --- drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c index a417763..b5bf9ed 100644 --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c @@ -837,7 +837,7 @@ static int gmc_v9_0_mc_init(struct amdgpu_device *adev) if (amdgpu_emu_mode != 1) adev->gmc.vram_width = amdgpu_atomfirmware_get_vram_width(adev); - if (!adev->gmc.vram_width) { + if (!adev->gmc.vram_width && !amdgpu_sriov_vf(adev)) { /* hbm memory channel size */ if (adev->flags & AMD_IS_APU) chansize = 64; -- 2.7.4 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH] drm/amdgpu: don't read DF register for SRIOV
Under SRIOV, reading DF register has chance to lead to AER error in host side, just skip reading it. Signed-off-by: Monk Liu Signed-off-by: Yintian Tao --- drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c index a417763..b5bf9ed 100644 --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c @@ -837,7 +837,7 @@ static int gmc_v9_0_mc_init(struct amdgpu_device *adev) if (amdgpu_emu_mode != 1) adev->gmc.vram_width = amdgpu_atomfirmware_get_vram_width(adev); - if (!adev->gmc.vram_width) { + if (!adev->gmc.vram_width && !amdgpu_sriov_vf(adev)) { /* hbm memory channel size */ if (adev->flags & AMD_IS_APU) chansize = 64; -- 2.7.4 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx