diff options
author | Vishal Verma <vishal.l.verma@intel.com> | 2024-04-30 20:44:25 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-05-30 10:44:59 +0300 |
commit | 5c7c179e0bff4544b3cb24f52097be7ae68aa4a1 (patch) | |
tree | 31056e639c20ba4871bc7a2385563f9c5a5dc962 | |
parent | ed049a1dc3b921bcb7628c5e899cfb1bc30a5e39 (diff) | |
download | linux-5c7c179e0bff4544b3cb24f52097be7ae68aa4a1.tar.xz |
dax/bus.c: don't use down_write_killable for non-user processes
[ Upstream commit e39dbcfba714c4c2e924e96fc8fdde1080a5a737 ]
Change an instance of down_write_killable() to a simple down_write() where
there is no user process that might want to interrupt the operation.
Link: https://lkml.kernel.org/r/20240430-vv-dax_abi_fixes-v3-3-e3dcd755774c@intel.com
Fixes: c05ae9d85b47 ("dax/bus.c: replace driver-core lock usage by a local rwsem")
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
Reported-by: Dan Williams <dan.j.williams@intel.com>
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
Cc: Alison Schofield <alison.schofield@intel.com>
Cc: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r-- | drivers/dax/bus.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c index e2c7354ce328..0011a6e6a8f2 100644 --- a/drivers/dax/bus.c +++ b/drivers/dax/bus.c @@ -1540,12 +1540,8 @@ err_id: struct dev_dax *devm_create_dev_dax(struct dev_dax_data *data) { struct dev_dax *dev_dax; - int rc; - - rc = down_write_killable(&dax_region_rwsem); - if (rc) - return ERR_PTR(rc); + down_write(&dax_region_rwsem); dev_dax = __devm_create_dev_dax(data); up_write(&dax_region_rwsem); |