drm/amdkfd: Fix wrong usage of INIT_WORK()
[ Upstream commit21cae8debc] In kfd_procfs_show(), the sdma_activity_work_handler is a local variable and the sdma_activity_work_handler.sdma_activity_work should initialize with INIT_WORK_ONSTACK() instead of INIT_WORK(). Fixes:32cb59f313("drm/amdkfd: Track SDMA utilization per process") Signed-off-by: Yuan Can <yuancan@huawei.com> Signed-off-by: Felix Kuehling <felix.kuehling@amd.com> Reviewed-by: Felix Kuehling <felix.kuehling@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
4a1db2a301
commit
fa6813b110
@@ -341,8 +341,8 @@ static ssize_t kfd_procfs_show(struct kobject *kobj, struct attribute *attr,
|
||||
attr_sdma);
|
||||
struct kfd_sdma_activity_handler_workarea sdma_activity_work_handler;
|
||||
|
||||
INIT_WORK(&sdma_activity_work_handler.sdma_activity_work,
|
||||
kfd_sdma_activity_worker);
|
||||
INIT_WORK_ONSTACK(&sdma_activity_work_handler.sdma_activity_work,
|
||||
kfd_sdma_activity_worker);
|
||||
|
||||
sdma_activity_work_handler.pdd = pdd;
|
||||
sdma_activity_work_handler.sdma_activity_counter = 0;
|
||||
@@ -350,6 +350,7 @@ static ssize_t kfd_procfs_show(struct kobject *kobj, struct attribute *attr,
|
||||
schedule_work(&sdma_activity_work_handler.sdma_activity_work);
|
||||
|
||||
flush_work(&sdma_activity_work_handler.sdma_activity_work);
|
||||
destroy_work_on_stack(&sdma_activity_work_handler.sdma_activity_work);
|
||||
|
||||
return snprintf(buffer, PAGE_SIZE, "%llu\n",
|
||||
(sdma_activity_work_handler.sdma_activity_counter)/
|
||||
|
||||
Reference in New Issue
Block a user