Skip to content

Commit e39dbcf

Browse files
stellarhopperakpm00
authored andcommitted
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>
1 parent 6f6544f commit e39dbcf

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
@@ -1540,12 +1540,8 @@ static struct dev_dax *__devm_create_dev_dax(struct dev_dax_data *data)
15401540
struct dev_dax *devm_create_dev_dax(struct dev_dax_data *data)
15411541
{
15421542
struct dev_dax *dev_dax;
1543-
int rc;
1544-
1545-
rc = down_write_killable(&dax_region_rwsem);
1546-
if (rc)
1547-
return ERR_PTR(rc);
15481543

1544+
down_write(&dax_region_rwsem);
15491545
dev_dax = __devm_create_dev_dax(data);
15501546
up_write(&dax_region_rwsem);
15511547

0 commit comments

Comments
 (0)