diff options
author | Theodore Ts'o <tytso@mit.edu> | 2017-04-30 20:08:05 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2017-04-30 20:08:05 -0400 |
commit | 72d622b42258a0ed0b6e8c0f40d7628de935d058 (patch) | |
tree | 02103c9900f34edfa55341f75f470328e67368fd | |
parent | dddbd6ac8f25b2d61e3e1c7134e3b3e1b9cd8670 (diff) |
ext4: replace BUG_ON with WARN_ONCE in ext4_end_bio()
Add fallback code and a WARN_ONCE() call instead of a BUG_ON() in
the ext4_end_bio() function.
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-rw-r--r-- | fs/ext4/page-io.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/fs/ext4/page-io.c b/fs/ext4/page-io.c index 208241b06662..1a82138ba739 100644 --- a/fs/ext4/page-io.c +++ b/fs/ext4/page-io.c | |||
@@ -297,8 +297,17 @@ static void ext4_end_bio(struct bio *bio) | |||
297 | { | 297 | { |
298 | ext4_io_end_t *io_end = bio->bi_private; | 298 | ext4_io_end_t *io_end = bio->bi_private; |
299 | sector_t bi_sector = bio->bi_iter.bi_sector; | 299 | sector_t bi_sector = bio->bi_iter.bi_sector; |
300 | char b[BDEVNAME_SIZE]; | ||
300 | 301 | ||
301 | BUG_ON(!io_end); | 302 | if (WARN_ONCE(!io_end, "io_end is NULL: %s: sector %Lu len %u err %d\n", |
303 | bdevname(bio->bi_bdev, b), | ||
304 | (long long) bio->bi_iter.bi_sector, | ||
305 | (unsigned) bio_sectors(bio), | ||
306 | bio->bi_error)) { | ||
307 | ext4_finish_bio(bio); | ||
308 | bio_put(bio); | ||
309 | return; | ||
310 | } | ||
302 | bio->bi_end_io = NULL; | 311 | bio->bi_end_io = NULL; |
303 | 312 | ||
304 | if (bio->bi_error) { | 313 | if (bio->bi_error) { |