@@ -2459,7 +2459,7 @@ static int cache_create(struct cache_args *ca, struct cache **result)
ti->flush_supported = true;
ti->num_discard_bios = 1;
- ti->discards_supported = true;
+ ti->discard_support = DM_DISCARD_IGN_DEVS;
ti->per_io_data_size = sizeof(struct per_bio_data);
@@ -1938,7 +1938,7 @@ static int clone_ctr(struct dm_target *ti, unsigned int argc, char **argv)
ti->flush_supported = true;
/* Enable discards */
- ti->discards_supported = true;
+ ti->discard_support = DM_DISCARD_IGN_DEVS;
ti->num_discard_bios = 1;
ti->private = clone;
@@ -593,7 +593,7 @@ static int log_writes_ctr(struct dm_target *ti, unsigned int argc, char **argv)
ti->num_flush_bios = 1;
ti->flush_supported = true;
ti->num_discard_bios = 1;
- ti->discards_supported = true;
+ ti->discard_support = DM_DISCARD_IGN_DEVS;
ti->per_io_data_size = sizeof(struct per_bio_data);
ti->private = lc;
return 0;
@@ -789,8 +789,8 @@ int dm_table_add_target(struct dm_table *t, const char *type,
t->highs[t->num_targets++] = tgt->begin + tgt->len - 1;
- if (!tgt->num_discard_bios && tgt->discards_supported)
- DMWARN("%s: %s: ignoring discards_supported because num_discard_bios is zero.",
+ if (!tgt->num_discard_bios && (tgt->discard_support != DM_DISCARD_ALL_DEVS))
+ DMWARN("%s: %s: ignoring discard support because num_discard_bios is zero.",
dm_device_name(t->md), type);
return 0;
@@ -1786,6 +1786,14 @@ static bool dm_table_supports_write_zeroes(struct dm_table *t)
return true;
}
+static int device_discard_capable(struct dm_target *ti, struct dm_dev *dev,
+ sector_t start, sector_t len, void *data)
+{
+ struct request_queue *q = bdev_get_queue(dev->bdev);
+
+ return q && blk_queue_discard(q);
+}
+
static int device_not_discard_capable(struct dm_target *ti, struct dm_dev *dev,
sector_t start, sector_t len, void *data)
{
@@ -1805,15 +1813,17 @@ static bool dm_table_supports_discards(struct dm_table *t)
if (!ti->num_discard_bios)
return false;
- /*
- * Either the target provides discard support (as implied by setting
- * 'discards_supported') or it relies on _all_ data devices having
- * discard support.
- */
- if (!ti->discards_supported &&
- (!ti->type->iterate_devices ||
- ti->type->iterate_devices(ti, device_not_discard_capable, NULL)))
- return false;
+ if (ti->discard_support == DM_DISCARD_IGN_DEVS) {
+ continue;
+ } else if (ti->discard_support == DM_DISCARD_ANY_DEVS) {
+ if (!ti->type->iterate_devices ||
+ !ti->type->iterate_devices(ti, device_discard_capable, NULL))
+ return false;
+ } else {
+ if (!ti->type->iterate_devices ||
+ ti->type->iterate_devices(ti, device_not_discard_capable, NULL))
+ return false;
+ }
}
return true;
@@ -3402,11 +3402,11 @@ static int pool_ctr(struct dm_target *ti, unsigned argc, char **argv)
ti->num_discard_bios = 1;
/*
- * Setting 'discards_supported' circumvents the normal
- * stacking of discard limits (this keeps the pool and
+ * Setting this circumvents the normal stacking
+ * of discard limits (this keeps the pool and
* thin devices' discard limits consistent).
*/
- ti->discards_supported = true;
+ ti->discard_support = DM_DISCARD_IGN_DEVS;
}
ti->private = pt;
@@ -4266,7 +4266,7 @@ static int thin_ctr(struct dm_target *ti, unsigned argc, char **argv)
/* In case the pool supports discards, pass them on. */
if (tc->pool->pf.discard_enabled) {
- ti->discards_supported = true;
+ ti->discard_support = DM_DISCARD_IGN_DEVS;
ti->num_discard_bios = 1;
}
@@ -796,7 +796,7 @@ static int dmz_ctr(struct dm_target *ti, unsigned int argc, char **argv)
ti->num_write_zeroes_bios = 1;
ti->per_io_data_size = sizeof(struct dmz_bioctx);
ti->flush_supported = true;
- ti->discards_supported = true;
+ ti->discard_support = DM_DISCARD_IGN_DEVS;
/* The exposed capacity is the number of chunks that can be mapped */
ti->len = (sector_t)dmz_nr_chunks(dmz->metadata) << dev->zone_nr_sectors_shift;
@@ -252,6 +252,12 @@ struct target_type {
#define DM_TARGET_ZONED_HM 0x00000040
#define dm_target_supports_zoned_hm(type) ((type)->features & DM_TARGET_ZONED_HM)
+enum dm_discard_mode {
+ DM_DISCARD_ALL_DEVS = 0,
+ DM_DISCARD_ANY_DEVS = 1,
+ DM_DISCARD_IGN_DEVS = 2,
+};
+
struct dm_target {
struct dm_table *table;
struct target_type *type;
@@ -315,11 +321,8 @@ struct dm_target {
*/
bool flush_supported:1;
- /*
- * Set if this target needs to receive discards regardless of
- * whether or not its underlying devices have support.
- */
- bool discards_supported:1;
+ /* Use to specify the discard support of this target */
+ enum dm_discard_mode discard_support;
};
/* Each target can link one of these into the table */
@@ -272,9 +272,9 @@ enum {
#define DM_DEV_SET_GEOMETRY _IOWR(DM_IOCTL, DM_DEV_SET_GEOMETRY_CMD, struct dm_ioctl)
#define DM_VERSION_MAJOR 4
-#define DM_VERSION_MINOR 42
+#define DM_VERSION_MINOR 43
#define DM_VERSION_PATCHLEVEL 0
-#define DM_VERSION_EXTRA "-ioctl (2020-02-27)"
+#define DM_VERSION_EXTRA "-ioctl (2020-04-14)"
/* Status bits */
#define DM_READONLY_FLAG (1 << 0) /* In/Out */
Rename the discards_supported variable to discard_support since the discard support isn't just turned on or off and there are now three types of discard support for dm targets. Switch discard_support to using an enum so that the other types of discard support (all devices or ignore devices) can be accommodated. This is needed so that dm raid and dm raid1 can support discard for arrays of devices of varying discard support where this is safe. Signed-off-by: Paul Wise <pabs3@bonedaddy.net> --- drivers/md/dm-cache-target.c | 2 +- drivers/md/dm-clone-target.c | 2 +- drivers/md/dm-log-writes.c | 2 +- drivers/md/dm-table.c | 32 +++++++++++++++++++++----------- drivers/md/dm-thin.c | 8 ++++---- drivers/md/dm-zoned-target.c | 2 +- include/linux/device-mapper.h | 13 ++++++++----- include/uapi/linux/dm-ioctl.h | 4 ++-- 8 files changed, 39 insertions(+), 26 deletions(-)