Am 2020-04-08 um 9:27 a.m. schrieb Deucher, Alexander:
>
> [AMD Public Use]
>
>
> Acked-by: Alex Deucher <alexander.deuc...@amd.com>
> ------------------------------------------------------------------------
> *From:* amd-gfx <amd-gfx-boun...@lists.freedesktop.org> on behalf of
> Jack Zhang <jack.zha...@amd.com>
> *Sent:* Wednesday, April 8, 2020 6:33 AM
> *To:* amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>
> *Cc:* Zhang, Jack (Jian) <jack.zha...@amd.com>
> *Subject:* [PATCH] drm/amdgpu/sriov move program of sq_config to PF
>  
> Since SQ_CONFIG will influence performance behavior, it's
> unsafe to program in VF. Therefore, program of SQ_CONFIG
> need to be moved to PF under siov. Skip it here in guest driver.

We'll need to change this register when the debugger enables address
watch. If this setting is global across VFs, it will be a problem. Can
this be saved and restored on world-switch so that each VF has its own
setting? Then you could allow changing it in the VF.


Otherwise we will not be able to enable the address watch feature in the
debugger under virtualization.


Regards,
  Felix


>
> Signed-off-by: Jack Zhang <jack.zha...@amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> index 2d56b067..9bb89f2 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> @@ -2575,7 +2575,8 @@ static void gfx_v9_0_constants_init(struct
> amdgpu_device *adev)
>  
>          gfx_v9_0_init_compute_vmid(adev);
>          gfx_v9_0_init_gds_vmid(adev);
> -       gfx_v9_0_init_sq_config(adev);
> +       if (!amdgpu_sriov_vf(adev))
> +               gfx_v9_0_init_sq_config(adev);
>  }
>  
>  static void gfx_v9_0_wait_for_rlc_serdes(struct amdgpu_device *adev)
> -- 
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&amp;data=02%7C01%7Calexander.deucher%40amd.com%7C38604ab03f8142a547f808d7dba854d4%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637219388971131150&amp;sdata=gBSa%2BchPmFMSaTgAq8z5Sabs3G6ARuHN8wAPSi%2FoJqA%3D&amp;reserved=0
> <https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Cfelix.kuehling%40amd.com%7Cda6c5ebaaf6c4f1619f108d7dbc08d98%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637219493056058157&sdata=MAhOm9usqEEnTCO1XMxteWqFHMnlOeDG0V93sA%2BhE%2Bo%3D&reserved=0>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&amp;data=02%7C01%7Cfelix.kuehling%40amd.com%7Cda6c5ebaaf6c4f1619f108d7dbc08d98%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637219493056078139&amp;sdata=HCjruFYRNoF4bWVTCgwuxqk0h3dnxP%2FJBsq6%2Flr%2FFXw%3D&amp;reserved=0
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to