* [PATCH] drm/amdkfd: remove extra use of volatile
@ 2024-10-22 13:53 Victor Zhao
2024-10-22 14:05 ` Alex Deucher
2024-10-22 14:05 ` Christian König
0 siblings, 2 replies; 3+ messages in thread
From: Victor Zhao @ 2024-10-22 13:53 UTC (permalink / raw)
To: amd-gfx; +Cc: philip.yang, Victor Zhao
as the adding of mb() should be sufficient in function unmap_queues_cpsch,
remove the add of volatile type as recommended
Signed-off-by: Victor Zhao <Victor.Zhao@amd.com>
---
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 2 +-
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
index 5a318376203c..38c19dc8311d 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
@@ -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 */
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
index bddb169bb301..09ab36f8e8c6 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
@@ -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;
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/amdkfd: remove extra use of volatile
2024-10-22 13:53 [PATCH] drm/amdkfd: remove extra use of volatile Victor Zhao
@ 2024-10-22 14:05 ` Alex Deucher
2024-10-22 14:05 ` Christian König
1 sibling, 0 replies; 3+ messages in thread
From: Alex Deucher @ 2024-10-22 14:05 UTC (permalink / raw)
To: Victor Zhao; +Cc: amd-gfx, philip.yang
On Tue, Oct 22, 2024 at 10:03 AM Victor Zhao <Victor.Zhao@amd.com> wrote:
>
> as the adding of mb() should be sufficient in function unmap_queues_cpsch,
> remove the add of volatile type as recommended
>
> Signed-off-by: Victor Zhao <Victor.Zhao@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 2 +-
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> index 5a318376203c..38c19dc8311d 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> @@ -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 */
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> index bddb169bb301..09ab36f8e8c6 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> @@ -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;
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/amdkfd: remove extra use of volatile
2024-10-22 13:53 [PATCH] drm/amdkfd: remove extra use of volatile Victor Zhao
2024-10-22 14:05 ` Alex Deucher
@ 2024-10-22 14:05 ` Christian König
1 sibling, 0 replies; 3+ messages in thread
From: Christian König @ 2024-10-22 14:05 UTC (permalink / raw)
To: Victor Zhao, amd-gfx; +Cc: philip.yang
Am 22.10.24 um 15:53 schrieb Victor Zhao:
> as the adding of mb() should be sufficient in function unmap_queues_cpsch,
> remove the add of volatile type as recommended
>
> Signed-off-by: Victor Zhao <Victor.Zhao@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 2 +-
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> index 5a318376203c..38c19dc8311d 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> @@ -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 */
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> index bddb169bb301..09ab36f8e8c6 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> @@ -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;
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-10-22 14:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-22 13:53 [PATCH] drm/amdkfd: remove extra use of volatile Victor Zhao
2024-10-22 14:05 ` Alex Deucher
2024-10-22 14:05 ` Christian König
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox