dax/bus.c: don't use down_write_killable for non-user processes
authorVishal Verma <vishal.l.verma@intel.com>
Tue, 30 Apr 2024 17:44:25 +0000 (11:44 -0600)
committerAndrew Morton <akpm@linux-foundation.org>
Tue, 7 May 2024 17:37:01 +0000 (10:37 -0700)
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>
drivers/dax/bus.c

index e2c7354ce3281b307c9ff36b9a2944b11f53f7ad..0011a6e6a8f2a629e1aa355527cdd9f0c98d337f 100644 (file)
@@ -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);