3.6.11.4 stable review patch. If anyone has any objections, please let me know.
------------------ From: Shaohua Li <[email protected]> [ Upstream commit 32f9f570d04461a41bdcd5c1d93b41ebc5ce182a ] In SSD/hard disk hybid storage, discard request should be ignored for hard disk. We used to be doing this way, but the unplug path forgets it. This is suitable for stable tree since v3.6. Cc: [email protected] Reported-and-tested-by: Markus <[email protected]> Signed-off-by: Shaohua Li <[email protected]> Signed-off-by: NeilBrown <[email protected]> Signed-off-by: Steven Rostedt <[email protected]> --- drivers/md/raid1.c | 7 ++++++- drivers/md/raid10.c | 6 ++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index 7077dcf..36c6003 100644 --- a/drivers/md/raid1.c +++ b/drivers/md/raid1.c @@ -976,7 +976,12 @@ static void raid1_unplug(struct blk_plug_cb *cb, bool from_schedule) while (bio) { /* submit pending writes */ struct bio *next = bio->bi_next; bio->bi_next = NULL; - generic_make_request(bio); + if (unlikely((bio->bi_rw & REQ_DISCARD) && + !blk_queue_discard(bdev_get_queue(bio->bi_bdev)))) + /* Just ignore it */ + bio_endio(bio, 0); + else + generic_make_request(bio); bio = next; } kfree(plug); diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c index c52d893..bec9db8 100644 --- a/drivers/md/raid10.c +++ b/drivers/md/raid10.c @@ -912,6 +912,12 @@ static void flush_pending_writes(struct r10conf *conf) struct bio *next = bio->bi_next; bio->bi_next = NULL; generic_make_request(bio); + if (unlikely((bio->bi_rw & REQ_DISCARD) && + !blk_queue_discard(bdev_get_queue(bio->bi_bdev)))) + /* Just ignore it */ + bio_endio(bio, 0); + else + generic_make_request(bio); bio = next; } } else -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
