diff options
| author | NeilBrown <[email protected]> | 2025-08-04 12:11:28 +0000 |
|---|---|---|
| committer | Amir Goldstein <[email protected]> | 2025-08-18 11:16:38 +0000 |
| commit | 5f1c8965e748c150d580a2ea8fbee1bd80d07a24 (patch) | |
| tree | 4f385461d955b312190cc524b92123f2eb2de850 /drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | |
| parent | Linux 6.17-rc1 (diff) | |
| download | kernel-5f1c8965e748c150d580a2ea8fbee1bd80d07a24.tar.gz kernel-5f1c8965e748c150d580a2ea8fbee1bd80d07a24.zip | |
ovl: use I_MUTEX_PARENT when locking parent in ovl_create_temp()
ovl_create_temp() treats "workdir" as a parent in which it creates an
object so it should use I_MUTEX_PARENT.
Prior to the commit identified below the lock was taken by the caller
which sometimes used I_MUTEX_PARENT and sometimes used I_MUTEX_NORMAL.
The use of I_MUTEX_NORMAL was incorrect but unfortunately copied into
ovl_create_temp().
Note to backporters: This patch only applies after the last Fixes given
below (post v6.16). To fix the bug in v6.7 and later the
inode_lock() call in ovl_copy_up_workdir() needs to nest using
I_MUTEX_PARENT.
Link: https://lore.kernel.org/all/[email protected]/
Cc: [email protected]
Reported-by: [email protected]
Tested-by: [email protected]
Fixes: c63e56a4a652 ("ovl: do not open/llseek lower file with upper sb_writers held")
Fixes: d2c995581c7c ("ovl: Call ovl_create_temp() without lock held.")
Signed-off-by: NeilBrown <[email protected]>
Signed-off-by: Amir Goldstein <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c')
0 files changed, 0 insertions, 0 deletions