diff options
author | Mike Snitzer <snitzer@redhat.com> | 2012-12-21 15:23:32 -0500 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2012-12-21 15:23:32 -0500 |
commit | 018debea8de9b8f17a9637e07c98c61517eb2a6b (patch) | |
tree | 80fa9a7e9b79e09e31081db699d1ad449cb7f46a /drivers/md | |
parent | e3cbf94513c21516fbb44561857b155d1c599625 (diff) |
dm thin: emit ignore_discard in status when discards disabled
If "ignore_discard" is specified when creating the thin pool device then
discard support is disabled for that device. The pool device's status
should reflect this fact rather than stating "no_discard_passdown"
(which implies discards are enabled but passdown is disabled).
Reported-by: Zdenek Kabelac <zkabelac@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Diffstat (limited to 'drivers/md')
-rw-r--r-- | drivers/md/dm-thin.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/md/dm-thin.c b/drivers/md/dm-thin.c index aeb9e20f5978..063fe04e60e9 100644 --- a/drivers/md/dm-thin.c +++ b/drivers/md/dm-thin.c | |||
@@ -2391,7 +2391,9 @@ static int pool_status(struct dm_target *ti, status_type_t type, | |||
2391 | else | 2391 | else |
2392 | DMEMIT("rw "); | 2392 | DMEMIT("rw "); |
2393 | 2393 | ||
2394 | if (pool->pf.discard_enabled && pool->pf.discard_passdown) | 2394 | if (!pool->pf.discard_enabled) |
2395 | DMEMIT("ignore_discard"); | ||
2396 | else if (pool->pf.discard_passdown) | ||
2395 | DMEMIT("discard_passdown"); | 2397 | DMEMIT("discard_passdown"); |
2396 | else | 2398 | else |
2397 | DMEMIT("no_discard_passdown"); | 2399 | DMEMIT("no_discard_passdown"); |
@@ -2487,7 +2489,7 @@ static struct target_type pool_target = { | |||
2487 | .name = "thin-pool", | 2489 | .name = "thin-pool", |
2488 | .features = DM_TARGET_SINGLETON | DM_TARGET_ALWAYS_WRITEABLE | | 2490 | .features = DM_TARGET_SINGLETON | DM_TARGET_ALWAYS_WRITEABLE | |
2489 | DM_TARGET_IMMUTABLE, | 2491 | DM_TARGET_IMMUTABLE, |
2490 | .version = {1, 5, 0}, | 2492 | .version = {1, 6, 0}, |
2491 | .module = THIS_MODULE, | 2493 | .module = THIS_MODULE, |
2492 | .ctr = pool_ctr, | 2494 | .ctr = pool_ctr, |
2493 | .dtr = pool_dtr, | 2495 | .dtr = pool_dtr, |