Re: [PATCH] drm/amdgpu/sriov move program of sq_config to PF

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Am 2020-04-08 um 9:27 a.m. schrieb Deucher, Alexander:

[AMD Public Use]


Acked-by: Alex Deucher <alexander.deucher@xxxxxxx>

From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of Jack Zhang <Jack.Zhang1@xxxxxxx>
Sent: Wednesday, April 8, 2020 6:33 AM
To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
Cc: Zhang, Jack (Jian) <Jack.Zhang1@xxxxxxx>
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.Zhang1@xxxxxxx>
---
 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@xxxxxxxxxxxxxxxxxxxxx
https://nam11.safelinks.protection.outlook.com/?url="">< br>

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://nam11.safelinks.protection.outlook.com/?url="">
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux