Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1c8f805a authored by Chunming Zhou's avatar Chunming Zhou Committed by Alex Deucher
Browse files

drm/amdgpu: fix unnecessary wake up 



decrease CPU extra overhead.

Signed-off-by: default avatarChunming Zhou <david1.zhou@amd.com>
Reviewed-by: default avatarJammy Zhou <Jammy.Zhou@amd.com>
Reviewed-by: default avatarChristian K?nig <christian.koenig@amd.com>
parent 6d1d0ef7

drivers/gpu/drm/amd/scheduler/gpu_scheduler.c

+6 −4
Original line number Diff line number Diff line
@@ -158,7 +158,7 @@ select_context(struct amd_gpu_scheduler *sched) 
	if (sched->current_entity && (sched->current_entity != tmp))

		wake_entity = sched->current_entity;

	sched->current_entity = tmp;

	if (wake_entity)

	if (wake_entity && wake_entity->need_wakeup)

		wake_up(&wake_entity->wait_queue);

	return tmp;

}
@@ -195,6 +195,7 @@ int amd_sched_entity_init(struct amd_gpu_scheduler *sched, 
	entity->fence_context = fence_context_alloc(1);

	snprintf(name, sizeof(name), "c_entity[%llu]", entity->fence_context);

	memcpy(entity->name, name, 20);

	entity->need_wakeup = false;

	if(kfifo_alloc(&entity->job_queue,

		       jobs * sizeof(void *),

		       GFP_KERNEL))
@@ -257,7 +258,7 @@ int amd_sched_entity_fini(struct amd_gpu_scheduler *sched, 


	if (!is_context_entity_initialized(sched, entity))

		return 0;



	entity->need_wakeup = true;

	/**

	 * The client will not queue more IBs during this fini, consume existing

	 * queued IBs
@@ -323,7 +324,8 @@ int amd_sched_push_job(struct amd_gpu_scheduler *sched, 
		*/

		schedule();

	}



	/* first job wake up scheduler */

	if ((kfifo_len(&c_entity->job_queue) / sizeof(void *)) == 1)

		wake_up_interruptible(&sched->wait_queue);

	return 0;

}

drivers/gpu/drm/amd/scheduler/gpu_scheduler.h

+1 −0
Original line number Diff line number Diff line
@@ -54,6 +54,7 @@ struct amd_sched_entity { 
	bool                            is_pending;

	uint64_t                        fence_context;

	char                            name[20];

	bool                            need_wakeup;

};



/**