On 2023-11-15 2:01, Christian König wrote:
Felix that looks a bit fishy to me, can you take a look?
Why are we giving the xcc_mask as parameter here? IIRC the partition a
VM is used with is fixed because the page tables are created
individually for each partition.
Thanks,
Christian.
Am 12.11.23 um 05:45 schrieb Srinivasan Shanmugam:
Fixes the below:
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c:1373: warning: Function
parameter or member 'xcc_mask' not described in
'amdgpu_vm_flush_compute_tlb'
Cc: Felix Kuehling <Felix.Kuehling@xxxxxxx>
Cc: Christian König <christian.koenig@xxxxxxx>
Cc: Alex Deucher <alexander.deucher@xxxxxxx>
Cc: "Pan, Xinhui" <Xinhui.Pan@xxxxxxx>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index be084cbe7501..cd4970c2efc5 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -1360,7 +1360,7 @@ int amdgpu_vm_handle_moved(struct amdgpu_device
*adev,
* @adev: amdgpu_device pointer
* @vm: requested vm
* @flush_type: flush type
- *
+ * @xcc_mask: mask of XCCs for the XCCs available on ASIC.
This is not the mask of XCCs available on the ASIC. It's the mask of
XCCs that belong to the compute partition in need of a TLB flush.
Please leave the empty line between the arguments and the explanation below.
Regards,
Felix
* Flush TLB if needed for a compute VM.
*
* Returns:
