[arch-commits] Commit in linux-lts/trunk (2 files)
Evangelos Foutras
foutrelis at archlinux.org
Fri May 22 04:03:18 UTC 2015
Date: Friday, May 22, 2015 @ 06:03:18
Author: foutrelis
Revision: 239654
upgpkg: linux-lts 3.14.43-2
Add proposed fix for data loss on md raid0 when discard is used (FS#45040).
Added:
linux-lts/trunk/md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch
Modified:
linux-lts/trunk/PKGBUILD
------------------------------------------------------------+
PKGBUILD | 7 +
md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch | 50 +++++++++++
2 files changed, 56 insertions(+), 1 deletion(-)
Modified: PKGBUILD
===================================================================
--- PKGBUILD 2015-05-22 04:03:07 UTC (rev 239653)
+++ PKGBUILD 2015-05-22 04:03:18 UTC (rev 239654)
@@ -5,7 +5,7 @@
pkgbase=linux-lts
_srcname=linux-3.14
pkgver=3.14.43
-pkgrel=1
+pkgrel=2
arch=('i686' 'x86_64')
url="http://www.kernel.org/"
license=('GPL2')
@@ -22,6 +22,7 @@
'0002-module-allow-multiple-calls-to-MODULE_DEVICE_TABLE-p.patch'
'0003-module-remove-MODULE_GENERIC_TABLE.patch'
'0006-genksyms-fix-typeof-handling.patch'
+ 'md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch'
'gcc5_buildfixes.diff'
)
# https://www.kernel.org/pub/linux/kernel/v3.x/sha256sums.asc
@@ -37,6 +38,7 @@
'52dec83a8805a8642d74d764494acda863e0aa23e3d249e80d4b457e20a3fd29'
'65d58f63215ee3c5f9c4fc6bce36fc5311a6c7dbdbe1ad29de40647b47ff9c0d'
'cf2e7a2d00787f754028e7459688c2755a406e632ce48b60952fa4ff7ed6f4b7'
+ 'bc83293e64653d60793708a0e277741f57c018f5ea3551a8aff3a220df917ceb'
'470d6d019d288dce02b4a9758a34ea71d41715663a19a164749212a470a131e7')
validpgpkeys=('ABAF11C65A2970B130ABE3C479BE3E4300411886' # Linus Torvalds <torvalds at linux-foundation.org>
'647F28654894E3BD457199BE38DBBDC86092693E' # Greg Kroah-Hartman (Linux kernel stable release signing key) <greg at kroah.com>
@@ -56,6 +58,9 @@
# add upstream patch
patch -p1 -i "${srcdir}/patch-${pkgver}"
+ # https://bugzilla.kernel.org/show_bug.cgi?id=98501
+ patch -Np1 -i "${srcdir}/md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch"
+
# buildfixes for gcc5
# https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/scsi/qla2xxx/qla_nx2.c?id=9493c2422cae272d6f1f567cbb424195defe4176
# https://lkml.org/lkml/2014/11/9/27
Added: md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch
===================================================================
--- md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch (rev 0)
+++ md-raid0-fix-restore-to-sector-variable-in-raid0_mak.patch 2015-05-22 04:03:18 UTC (rev 239654)
@@ -0,0 +1,50 @@
+From a81157768a00e8cf8a7b43b5ea5cac931262374f Mon Sep 17 00:00:00 2001
+From: Eric Work <work.eric at gmail.com>
+Date: Mon, 18 May 2015 23:26:23 -0700
+Subject: [PATCH] md/raid0: fix restore to sector variable in
+ raid0_make_request
+
+The variable "sector" in "raid0_make_request()" was improperly updated
+by a call to "sector_div()" which modifies its first argument in place.
+Commit 47d68979cc968535cb87f3e5f2e6a3533ea48fbd restored this variable
+after the call for later re-use. Unfortunetly the restore was done after
+the referenced variable "bio" was advanced. This lead to the original
+value and the restored value being different. Here we move this line to
+the proper place.
+
+One observed side effect of this bug was discarding a file though
+unlinking would cause an unrelated file's contents to be discarded.
+
+Signed-off-by: NeilBrown <neilb at suse.de>
+Fixes: 47d68979cc96 ("md/raid0: fix bug with chunksize not a power of 2.")
+Cc: stable at vger.kernel.org (any that received above backport)
+URL: https://bugzilla.kernel.org/show_bug.cgi?id=98501
+---
+ drivers/md/raid0.c | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c
+index 6a68ef5..efb654e 100644
+--- a/drivers/md/raid0.c
++++ b/drivers/md/raid0.c
+@@ -524,6 +524,9 @@ static void raid0_make_request(struct mddev *mddev, struct bio *bio)
+ ? (sector & (chunk_sects-1))
+ : sector_div(sector, chunk_sects));
+
++ /* Restore due to sector_div */
++ sector = bio->bi_iter.bi_sector;
++
+ if (sectors < bio_sectors(bio)) {
+ split = bio_split(bio, sectors, GFP_NOIO, fs_bio_set);
+ bio_chain(split, bio);
+@@ -531,7 +534,6 @@ static void raid0_make_request(struct mddev *mddev, struct bio *bio)
+ split = bio;
+ }
+
+- sector = bio->bi_iter.bi_sector;
+ zone = find_zone(mddev->private, §or);
+ tmp_dev = map_sector(mddev, zone, sector, §or);
+ split->bi_bdev = tmp_dev->bdev;
+--
+2.4.1
+
More information about the arch-commits
mailing list