diff options
| author | Liu Bo <bo.li.liu@oracle.com> | 2017-11-03 18:24:44 +0100 |
|---|---|---|
| committer | Shaohua Li <shli@fb.com> | 2017-11-03 19:29:50 +0100 |
| commit | 39b4954c0a1556f8f7f1fdcf59a227117fcd8a0b (patch) | |
| tree | 91ea4eb6e8b4cbe512ddade95f43651d3a873d34 /block | |
| parent | md: don't check MD_SB_CHANGE_CLEAN in md_allow_write (diff) | |
| download | linux-39b4954c0a1556f8f7f1fdcf59a227117fcd8a0b.tar.xz linux-39b4954c0a1556f8f7f1fdcf59a227117fcd8a0b.zip | |
badblocks: fix wrong return value in badblocks_set if badblocks are disabled
MD's rdev_set_badblocks() expects that badblocks_set() returns 1 if
badblocks are disabled, otherwise, rdev_set_badblocks() will record
superblock changes and return success in that case and md will fail to
report an IO error which it should.
This bug has existed since badblocks were introduced in commit
9e0e252a048b ("badblocks: Add core badblock management code").
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Acked-by: Guoqing Jiang <gqjiang@suse.com>
Signed-off-by: Shaohua Li <shli@fb.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/badblocks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/badblocks.c b/block/badblocks.c index 43c71166e1e2..91f7bcf979d3 100644 --- a/block/badblocks.c +++ b/block/badblocks.c @@ -178,7 +178,7 @@ int badblocks_set(struct badblocks *bb, sector_t s, int sectors, if (bb->shift < 0) /* badblocks are disabled */ - return 0; + return 1; if (bb->shift) { /* round the start down, and the end up */ |