Commit afaf4250 authored by Yury Norov's avatar Yury Norov Committed by Alex Deucher
Browse files

drm/amdkfd: simplify svm_range_unmap_from_gpus() 



The function calls bitmap_or() followed by for_each_set_bit().
Switch it to the dedicated for_each_or_bit() and drop the temporary
bitmap.

Signed-off-by: default avatarYury Norov <ynorov@nvidia.com>
Signed-off-by: default avatarFelix Kuehling <felix.kuehling@amd.com>
Reviewed-by: default avatarFelix Kuehling <felix.kuehling@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 532e2c87

drivers/gpu/drm/amd/amdkfd/kfd_svm.c

+1 −4
Original line number Diff line number Diff line
@@ -1372,7 +1372,6 @@ static int 
svm_range_unmap_from_gpus(struct svm_range *prange, unsigned long start,

			  unsigned long last, uint32_t trigger)

{

	DECLARE_BITMAP(bitmap, MAX_GPU_INSTANCE);

	struct kfd_process_device *pdd;

	struct dma_fence *fence = NULL;

	struct kfd_process *p;
@@ -1390,11 +1389,9 @@ svm_range_unmap_from_gpus(struct svm_range *prange, unsigned long start, 
		prange->mapped_to_gpu = false;

	}



	bitmap_or(bitmap, prange->bitmap_access, prange->bitmap_aip,

		  MAX_GPU_INSTANCE);

	p = container_of(prange->svms, struct kfd_process, svms);



	for_each_set_bit(gpuidx, bitmap, MAX_GPU_INSTANCE) {

	for_each_or_bit(gpuidx, prange->bitmap_access, prange->bitmap_aip, MAX_GPU_INSTANCE) {

		pr_debug("unmap from gpu idx 0x%x\n", gpuidx);

		pdd = kfd_process_device_from_gpuidx(p, gpuidx);

		if (!pdd) {