Commit b6890efb authored by Victor Zhao's avatar Victor Zhao Committed by Alex Deucher
Browse files

drm/amdkfd: remove extra use of volatile 



as the adding of mb() should be sufficient in function unmap_queues_cpsch,
remove the add of volatile type as recommended

Signed-off-by: default avatarVictor Zhao <Victor.Zhao@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3d0ffc64

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

+1 −1
Original line number Diff line number Diff line
@@ -2048,7 +2048,7 @@ int amdkfd_fence_wait_timeout(struct device_queue_manager *dqm, 
{

	unsigned long end_jiffies = msecs_to_jiffies(timeout_ms) + jiffies;

	struct device *dev = dqm->dev->adev->dev;

	volatile uint64_t *fence_addr = dqm->fence_addr;

	uint64_t *fence_addr = dqm->fence_addr;



	while (*fence_addr != fence_value) {

		/* Fatal err detected, this response won't come */

drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h

+1 −1
Original line number Diff line number Diff line
@@ -260,7 +260,7 @@ struct device_queue_manager { 
	uint16_t		vmid_pasid[VMID_NUM];

	uint64_t		pipelines_addr;

	uint64_t		fence_gpu_addr;

	volatile uint64_t	*fence_addr;

	uint64_t		*fence_addr;

	struct kfd_mem_obj	*fence_mem;

	bool			active_runlist;

	int			sched_policy;