Skip to content

Commit 80101db

Browse files
committed
dax/bus.c: don't use down_write_killable for non-user processes
JIRA: https://issues.redhat.com/browse/RHEL-23824 commit e39dbcf Author: Vishal Verma <vishal.l.verma@intel.com> Date: Tue Apr 30 11:44:25 2024 -0600 dax/bus.c: don't use down_write_killable for non-user processes 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: c05ae9d ("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: Jeff Moyer <jmoyer@redhat.com>
1 parent b2fa712 commit 80101db

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

drivers/dax/bus.c

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1545,12 +1545,8 @@ static struct dev_dax *__devm_create_dev_dax(struct dev_dax_data *data)
15451545
struct dev_dax *devm_create_dev_dax(struct dev_dax_data *data)
15461546
{
15471547
struct dev_dax *dev_dax;
1548-
int rc;
1549-
1550-
rc = down_write_killable(&dax_region_rwsem);
1551-
if (rc)
1552-
return ERR_PTR(rc);
15531548

1549+
down_write(&dax_region_rwsem);
15541550
dev_dax = __devm_create_dev_dax(data);
15551551
up_write(&dax_region_rwsem);
15561552

0 commit comments

Comments
 (0)