diff options
author | Stefan Bader <stefan.bader@canonical.com> | 2015-11-29 21:44:49 -0500 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2015-12-30 22:23:14 -0500 |
commit | 8d16ce540c94c9d366eb36fc91b7154d92d6397b (patch) | |
tree | 0fae348076b0dcd745a2ee2cadaba965bd0587e4 /drivers/md/bcache | |
parent | d7076f21629f8f329bca4a44dc408d94670f49e2 (diff) |
bcache: prevent crash on changing writeback_running
Added a safeguard in the shutdown case. At least while not being
attached it is also possible to trigger a kernel bug by writing into
writeback_running. This change adds the same check before trying to
wake up the thread for that case.
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
Cc: Kent Overstreet <kent.overstreet@gmail.com>
Cc: stable@vger.kernel.org
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'drivers/md/bcache')
-rw-r--r-- | drivers/md/bcache/writeback.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/md/bcache/writeback.h b/drivers/md/bcache/writeback.h index 0a9dab187b79..073a042aed24 100644 --- a/drivers/md/bcache/writeback.h +++ b/drivers/md/bcache/writeback.h | |||
@@ -63,7 +63,8 @@ static inline bool should_writeback(struct cached_dev *dc, struct bio *bio, | |||
63 | 63 | ||
64 | static inline void bch_writeback_queue(struct cached_dev *dc) | 64 | static inline void bch_writeback_queue(struct cached_dev *dc) |
65 | { | 65 | { |
66 | wake_up_process(dc->writeback_thread); | 66 | if (!IS_ERR_OR_NULL(dc->writeback_thread)) |
67 | wake_up_process(dc->writeback_thread); | ||
67 | } | 68 | } |
68 | 69 | ||
69 | static inline void bch_writeback_add(struct cached_dev *dc) | 70 | static inline void bch_writeback_add(struct cached_dev *dc) |