diff options
author | Coly Li <colyli@suse.de> | 2020-12-31 23:59:16 +0800 |
---|---|---|
committer | Coly Li <colyli@suse.de> | 2021-01-02 00:21:22 +0800 |
commit | 93e83d620d21411eb82fd3c4e95d7b73e79ce49d (patch) | |
tree | c763434082a7c1f561a9a58df9a6fa24d4822cb2 | |
parent | 86d708d996935edd2f53ada336c40c38cc7fdfd7 (diff) | |
download | bcache-tools-93e83d620d21411eb82fd3c4e95d7b73e79ce49d.tar.gz |
bcache-tools: check incompatible feature set
When bcache-tools is not updated with kernel driver (or on the versus),
we need to make sure the feature set conflict can be detected to avoid
unexpected data operation.
This patch checks whether there is unsupported {compatc, ro_compat,
incompat} features detected in detail_dev(). If there is, prints out
error message and return error to its caller.
Signed-off-by: Coly Li <colyli@suse.de>
-rw-r--r-- | lib.c | 27 |
1 files changed, 27 insertions, 0 deletions
@@ -442,6 +442,33 @@ int detail_dev(char *devname, struct bdev *bd, struct cdev *cd, int *type) goto Fail; } + /* Check for incompat feature set */ + if (sb.version >= BCACHE_SB_VERSION_BDEV_WITH_FEATURES || + sb.version >= BCACHE_SB_VERSION_CDEV_WITH_FEATURES) { + uint64_t features; + + features = sb.feature_compat & ~BCH_FEATURE_COMPAT_SUPP; + if (features) { + fprintf(stderr, + "Unsupported compatible feature found\n"); + goto Fail; + } + + features = sb.feature_ro_compat & ~BCH_FEATURE_RO_COMPAT_SUPP; + if (features) { + fprintf(stderr, + "Unsupported read-only compatible feature found\n"); + goto Fail; + } + + features = sb.feature_incompat & ~BCH_FEATURE_INCOMPAT_SUPP; + if (features) { + fprintf(stderr, + "Unsupported incompatible feature found\n"); + goto Fail; + } + } + *type = sb.version; if (sb.version == BCACHE_SB_VERSION_BDEV || sb.version == BCACHE_SB_VERSION_BDEV_WITH_OFFSET || |