dm, dax: Make sure dm_dax_flush() is called if device supports it
Subsystems (4)
PATCH v2 1/1 dm, dax: Make sure dm_dax_flush() is called if device supports it
Right now, dm_dax_flush() is not being called and I think it is not being called becuase DAXDEV_WRITE_CACHE is not set on dm dax device.
If underlying dax device supports write cache, set DAXDEV_WRITE_CACHE on dm dax device. This will get dm_dax_flush() being called.
--- drivers/dax/super.c | 6 ++++++ drivers/md/dm-table.c | 35 +++++++++++++++++++++++++++++++++++ include/linux/dax.h | 1 + 3 files changed, 42 insertions(+)
drivers/dax/super.c +6 -0
@@ -278,6 +278,12 @@ void dax_write_cache(struct dax_device *dax_dev, bool wc) }EXPORT_SYMBOL_GPL (dax_write_cache); +bool dax_write_cache_enabled (struct dax_device *dax_dev) +{ +return test_bit (DAXDEV_WRITE_CACHE , &dax_dev->flags); +} +EXPORT_SYMBOL_GPL (dax_write_cache_enabled); +bool dax_alive (struct dax_device *dax_dev) {lockdep_assert_held (&dax_srcu);
drivers/md/dm-table.c +35 -0
@@ -20,6 +20,7 @@#include <linux/atomic.h> #include <linux/blk-mq.h> #include <linux/mount.h> +#include <linux/dax.h> #define DM_MSG_PREFIX "table"
@@ -1630,6 +1631,37 @@ static bool dm_table_supports_flush(struct dm_table *t, unsigned long flush)return false; } +static int device_dax_write_cache_enabled (struct dm_target *ti, +struct dm_dev *dev,sector_t start, +sector_t len,void *data) +{ +struct dax_device *dax_dev = dev->dax_dev; + +if (!dax_dev) +return false; + +if (dax_write_cache_enabled (dax_dev)) +return true; +return false; +} + +static int dm_table_supports_dax_write_cache (struct dm_table *t) +{ +struct dm_target *ti; +unsigned i; + +for (i =0 ; i <dm_table_get_num_targets (t); i++) { + ti =dm_table_get_target (t, i); + +if (ti->type->iterate_devices && + ti->type->iterate_devices (ti, + device_dax_write_cache_enabled,NULL )) +return true; + } + +return false; +} +static int device_is_nonrot (struct dm_target *ti,struct dm_dev *dev,sector_t start,sector_t len,void *data) {
@@ -1785,6 +1817,9 @@ void dm_table_set_restrictions(struct dm_table *t, struct request_queue *q, }blk_queue_write_cache (q, wc, fua); +if (dm_table_supports_dax_write_cache (t)) +dax_write_cache (t->md->dax_dev, true); +/* Ensure that all underlying devices are non-rotational. */ if (dm_table_all_devices_attribute (t, device_is_nonrot))queue_flag_set_unlocked (QUEUE_FLAG_NONROT , q);
include/linux/dax.h +1 -0
@@ -87,6 +87,7 @@ size_t dax_copy_from_iter(struct dax_device *dax_dev, pgoff_t pgoff, void *addr,void dax_flush (struct dax_device *dax_dev,pgoff_t pgoff,void *addr,size_t size);void dax_write_cache (struct dax_device *dax_dev,bool wc); +bool dax_write_cache_enabled (struct dax_device *dax_dev);/* * We use lowest available bit in exceptional entry for locking, one bit for
Signature · 6 lines
2.5.5 _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
A message that is not in the archive (sent off-list or not yet received): [email protected]
Quoted 8 lines · Vivek Goyal
Right now,
dm_dax_flush()is not being called and I think it is not being called becuaseDAXDEV_WRITE_CACHEis not set on dm dax device.If underlying dax device supports write cache, set
DAXDEV_WRITE_CACHEon dm dax device. This will getdm_dax_flush()being called.Acked-by Dan Williams <[email protected]>Signed-off-by Vivek Goyal <[email protected]>
I tweaked the header a bit and staged for 4.13-rc3 here:
https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=for-4.13/dm&id=325ab51b81ca2a84ffd434652273eb5d8a8a5750
Thanks!
_______________________________________________
Linux-nvdimm mailing list
[email protected]
https://lists.01.org/mailman/listinfo/linux-nvdimm