[PATCH] drm/amdgpu: set correct vram_width for vega10 under sriov

[Tao, Yintian]

Hi  Alex

So sorry for my missing of you patch because the outlook on website didn’t show it.

Your patch seems cleaner and better. Can you help submit it ? Thanks in advance.

Reviewed-by: Yintian Tao<yttao at amd.com>


Can

From: Deucher, Alexander <Alexander.Deucher at amd.com>
Sent: Saturday, May 18, 2019 1:07 AM
To: Tao, Yintian <Yintian.Tao at amd.com>; Alex Deucher <alexdeucher at gmail.com>
Cc: amd-gfx at lists.freedesktop.org; Koenig, Christian <Christian.Koenig at amd.com>; Huang, Trigger <Trigger.Huang at amd.com>
Subject: Re: [PATCH] drm/amdgpu: set correct vram_width for vega10 under sriov

Did you see the patch I attached?

Alex
________________________________
From: Tao, Yintian
Sent: Friday, May 17, 2019 10:51 AM
To: Alex Deucher
Cc: amd-gfx at lists.freedesktop.org<mailto:amd-gfx at lists.freedesktop.org>; Koenig, Christian; Deucher, Alexander; Huang, Trigger
Subject: 答复: [PATCH] drm/amdgpu: set correct vram_width for vega10 under sriov


Hi Alex





Many thanks for your review. I will merge these two patches into one and submit again.





Best Regards

Yintian Tao

________________________________
发件人: Alex Deucher <alexdeucher at gmail.com<mailto:alexdeucher at gmail.com>>
发送时间: 2019年5月17日 22:34:30
收件人: Tao, Yintian
抄送: amd-gfx at lists.freedesktop.org<mailto:amd-gfx at lists.freedesktop.org>; Koenig, Christian; Deucher, Alexander; Huang, Trigger
主题: Re: [PATCH] drm/amdgpu: set correct vram_width for vega10 under sriov

[CAUTION: External Email]

How about combining these two patches into one?  This seems cleaner.

Alex

On Thu, May 16, 2019 at 10:39 PM Tao, Yintian <Yintian.Tao at amd.com<mailto:Yintian.Tao at amd.com>> wrote:
>
> Ping...
>
> Hi Christian and Alex
>
>
> Can you help review this? Thanks in advance.
>
>
> Best Regards
> Yintian Tao
>
> -----Original Message-----
> From: Yintian Tao <yttao at amd.com<mailto:yttao at amd.com>>
> Sent: Thursday, May 16, 2019 8:03 PM
> To: amd-gfx at lists.freedesktop.org<mailto:amd-gfx at lists.freedesktop.org>
> Cc: Tao, Yintian <Yintian.Tao at amd.com<mailto:Yintian.Tao at amd.com>>; Huang, Trigger <Trigger.Huang at amd.com<mailto:Trigger.Huang at amd.com>>
> Subject: [PATCH] drm/amdgpu: set correct vram_width for vega10 under sriov
>
> For Vega10 SR-IOV, vram_width can't be read from ATOM as RAVEN, and DF related registers is not readable, seems hardcord is the only way to set the correct vram_width
>
> Signed-off-by: Trigger Huang <Trigger.Huang at amd.com<mailto:Trigger.Huang at amd.com>>
> Signed-off-by: Yintian Tao <yttao at amd.com<mailto:yttao at amd.com>>
> ---
>  drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> index c221570..a417763 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> @@ -848,6 +848,13 @@ static int gmc_v9_0_mc_init(struct amdgpu_device *adev)
>                 adev->gmc.vram_width = numchan * chansize;
>         }
>
> +       /* For Vega10 SR-IOV, vram_width can't be read from ATOM as RAVEN,
> +        * and DF related registers is not readable, seems hardcord is the
> +        * only way to set the correct vram_width */
> +       if (amdgpu_sriov_vf(adev) && (adev->asic_type == CHIP_VEGA10)) {
> +               adev->gmc.vram_width = 2048;
> +       }
> +
>         /* size in MB on si */
>         adev->gmc.mc_vram_size =
>                 adev->nbio_funcs->get_memsize(adev) * 1024ULL * 1024ULL;
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org<mailto:amd-gfx at lists.freedesktop.org>
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20190520/d35c25fd/attachment-0001.html>