Re: [PATCH v2 0/9] KFD user queue validation

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

 



Am 18.07.24 um 23:12 schrieb Felix Kuehling:
On 2024-07-18 17:05, Philip Yang wrote:
This patch series do additional queue buffers validation in the queue
creation IOCTLS, fail the queue creation if buffers not mapped on the GPU
with the expected size.

Ensure queue buffers residency by tracking the GPUVM virtual addresses
for queue buffers to return error if the user tries to free and unmap them
when the qeueu is active, or evict the queue if SVM memory is unmapped and
freed from CPU.

Patch 1-2 is prepration work and general fix.

v2:
  - patch 3/9, keep wptr_bo_gart in struct queue
The series is
Reviewed-by: Felix Kuehling <felix.kuehling@xxxxxxx>

I only skimmed over it and will probably find something to complain on later.

But we need to get this out of the door, so feel free to add Acked-by: Christian König <christian.koenig@xxxxxxx> to the series for now.

Thanks,
Christian.


Philip Yang (9):
   drm/amdkfd: kfd_bo_mapped_dev support partition
   drm/amdkfd: amdkfd_free_gtt_mem clear the correct pointer
   drm/amdkfd: Refactor queue wptr_bo GART mapping
   drm/amdkfd: Validate user queue buffers
   drm/amdkfd: Ensure user queue buffers residency
   drm/amdkfd: Validate user queue svm memory residency
   drm/amdkfd: Validate user queue update
   drm/amdkfd: Store queue cwsr area size to node properties
   drm/amdkfd: Validate queue cwsr area and eop buffer size

  drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c    |  14 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h    |   6 +-
  .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c  |  24 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_object.h    |   6 +
  drivers/gpu/drm/amd/amdkfd/kfd_chardev.c      |  61 +---
  drivers/gpu/drm/amd/amdkfd/kfd_device.c       |   4 +-
  .../drm/amd/amdkfd/kfd_device_queue_manager.c |   8 +-
  drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager.c  |   2 +-
  drivers/gpu/drm/amd/amdkfd/kfd_priv.h         |  19 +-
  drivers/gpu/drm/amd/amdkfd/kfd_process.c      |   2 +-
  .../amd/amdkfd/kfd_process_queue_manager.c    |  79 +++-
  drivers/gpu/drm/amd/amdkfd/kfd_queue.c        | 336 ++++++++++++++++++
  drivers/gpu/drm/amd/amdkfd/kfd_svm.c          |  12 +
  drivers/gpu/drm/amd/amdkfd/kfd_svm.h          |   1 +
  drivers/gpu/drm/amd/amdkfd/kfd_topology.c     |   2 +
  drivers/gpu/drm/amd/amdkfd/kfd_topology.h     |   4 +
  16 files changed, 489 insertions(+), 91 deletions(-)





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

  Powered by Linux