From: "Darrick J. Wong" <[email protected]>
Subject: mm/bounce.c: fix a regression where MS_SNAP_STABLE (stable pages 
snapshotting) was ignored

The "force" parameter in __blk_queue_bounce was being ignored, which means
that stable page snapshots are not always happening (on ext3).  This of
course leads to DIF disks reporting checksum errors, so fix this
regression.

The regression was introduced in 6bc454d1 ("bounce: Refactor
__blk_queue_bounce to not use bi_io_vec")

Reported-by: Mel Gorman <[email protected]>
Signed-off-by: Darrick J. Wong <[email protected]>
Cc: Kent Overstreet <[email protected]>
Cc: <[email protected]>    [3.10+]
Signed-off-by: Andrew Morton <[email protected]>
---

 mm/bounce.c |    2 ++
 1 file changed, 2 insertions(+)

diff -puN 
mm/bounce.c~mm-bouncec-wix-a-regression-where-ms_snap_stable-stable-pages-snapshotting-was-ignored
 mm/bounce.c
--- 
a/mm/bounce.c~mm-bouncec-wix-a-regression-where-ms_snap_stable-stable-pages-snapshotting-was-ignored
+++ a/mm/bounce.c
@@ -204,6 +204,8 @@ static void __blk_queue_bounce(struct re
        struct bio_vec *to, *from;
        unsigned i;
 
+       if (force)
+               goto bounce;
        bio_for_each_segment(from, *bio_orig, i)
                if (page_to_pfn(from->bv_page) > queue_bounce_pfn(q))
                        goto bounce;
_
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to