kernel: backport mtd partition address fix sent upstream
authorRafał Miłecki <rafal@milecki.pl>
Mon, 9 Mar 2020 10:46:20 +0000 (11:46 +0100)
committerRafał Miłecki <rafal@milecki.pl>
Mon, 9 Mar 2020 10:49:42 +0000 (11:49 +0100)
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
target/linux/generic/backport-4.14/410-mtd-fix-calculating-partition-end-address.patch [new file with mode: 0644]
target/linux/generic/backport-4.19/410-mtd-fix-calculating-partition-end-address.patch [new file with mode: 0644]
target/linux/generic/backport-5.4/410-mtd-fix-calculating-partition-end-address.patch [new file with mode: 0644]
target/linux/generic/pending-4.14/411-mtd-partial_eraseblock_write.patch
target/linux/generic/pending-4.19/411-mtd-partial_eraseblock_write.patch
target/linux/generic/pending-5.4/411-mtd-partial_eraseblock_write.patch

diff --git a/target/linux/generic/backport-4.14/410-mtd-fix-calculating-partition-end-address.patch b/target/linux/generic/backport-4.14/410-mtd-fix-calculating-partition-end-address.patch
new file mode 100644 (file)
index 0000000..ec25eef
--- /dev/null
@@ -0,0 +1,28 @@
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
+Date: Mon, 9 Mar 2020 08:30:19 +0100
+Subject: [PATCH] mtd: fix calculating partition end address
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This fixes check for partitions that don't start at beginning of their
+parents. Missing partition's offset in formula could result in forcing
+read-only incorrectly.
+
+Fixes: 6750f61a13a0 ("mtd: improve calculating partition boundaries when checking for alignment")
+Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
+---
+ drivers/mtd/mtdpart.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/drivers/mtd/mtdpart.c
++++ b/drivers/mtd/mtdpart.c
+@@ -582,7 +582,7 @@ static struct mtd_part *allocate_partiti
+                       part->name);
+       }
+-      tmp = part_absolute_offset(parent) + slave->mtd.size;
++      tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
+       remainder = do_div(tmp, wr_alignment);
+       if ((slave->mtd.flags & MTD_WRITEABLE) && remainder) {
+               slave->mtd.flags &= ~MTD_WRITEABLE;
diff --git a/target/linux/generic/backport-4.19/410-mtd-fix-calculating-partition-end-address.patch b/target/linux/generic/backport-4.19/410-mtd-fix-calculating-partition-end-address.patch
new file mode 100644 (file)
index 0000000..4027bfd
--- /dev/null
@@ -0,0 +1,28 @@
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
+Date: Mon, 9 Mar 2020 08:30:19 +0100
+Subject: [PATCH] mtd: fix calculating partition end address
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This fixes check for partitions that don't start at beginning of their
+parents. Missing partition's offset in formula could result in forcing
+read-only incorrectly.
+
+Fixes: 6750f61a13a0 ("mtd: improve calculating partition boundaries when checking for alignment")
+Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
+---
+ drivers/mtd/mtdpart.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/drivers/mtd/mtdpart.c
++++ b/drivers/mtd/mtdpart.c
+@@ -538,7 +538,7 @@ static struct mtd_part *allocate_partiti
+                       part->name);
+       }
+-      tmp = part_absolute_offset(parent) + slave->mtd.size;
++      tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
+       remainder = do_div(tmp, wr_alignment);
+       if ((slave->mtd.flags & MTD_WRITEABLE) && remainder) {
+               slave->mtd.flags &= ~MTD_WRITEABLE;
diff --git a/target/linux/generic/backport-5.4/410-mtd-fix-calculating-partition-end-address.patch b/target/linux/generic/backport-5.4/410-mtd-fix-calculating-partition-end-address.patch
new file mode 100644 (file)
index 0000000..1eae015
--- /dev/null
@@ -0,0 +1,28 @@
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
+Date: Mon, 9 Mar 2020 08:30:19 +0100
+Subject: [PATCH] mtd: fix calculating partition end address
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This fixes check for partitions that don't start at beginning of their
+parents. Missing partition's offset in formula could result in forcing
+read-only incorrectly.
+
+Fixes: 6750f61a13a0 ("mtd: improve calculating partition boundaries when checking for alignment")
+Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
+---
+ drivers/mtd/mtdpart.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/drivers/mtd/mtdpart.c
++++ b/drivers/mtd/mtdpart.c
+@@ -524,7 +524,7 @@ static struct mtd_part *allocate_partiti
+                       part->name);
+       }
+-      tmp = part_absolute_offset(parent) + slave->mtd.size;
++      tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
+       remainder = do_div(tmp, wr_alignment);
+       if ((slave->mtd.flags & MTD_WRITEABLE) && remainder) {
+               slave->mtd.flags &= ~MTD_WRITEABLE;
index 3b862dae61d851f0f7b5709084aa4c021adc5716..52f1cb5abff1401968efae0730d24b472a15dc66 100644 (file)
@@ -123,8 +123,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
 +                      slave->mtd.erasesize = slave->mtd.size;
        }
  
--      tmp = part_absolute_offset(parent) + slave->mtd.size;
-+      tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
+       tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
        remainder = do_div(tmp, wr_alignment);
        if ((slave->mtd.flags & MTD_WRITEABLE) && remainder) {
 -              slave->mtd.flags &= ~MTD_WRITEABLE;
index bf9822fda83c6af4285df43e767f06d73e645a20..f3a314ae02e16def6ffaace8593ef89390ec703f 100644 (file)
@@ -114,8 +114,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
 +                      slave->mtd.erasesize = slave->mtd.size;
        }
  
--      tmp = part_absolute_offset(parent) + slave->mtd.size;
-+      tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
+       tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
        remainder = do_div(tmp, wr_alignment);
        if ((slave->mtd.flags & MTD_WRITEABLE) && remainder) {
 -              slave->mtd.flags &= ~MTD_WRITEABLE;
index f2a187df30f114be673f5f4ee2bfa60290800bc6..b46c3f5ed42e1ff342c5778ab77ea4da8ed000f0 100644 (file)
@@ -114,8 +114,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
 +                      slave->mtd.erasesize = slave->mtd.size;
        }
  
--      tmp = part_absolute_offset(parent) + slave->mtd.size;
-+      tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
+       tmp = part_absolute_offset(parent) + slave->offset + slave->mtd.size;
        remainder = do_div(tmp, wr_alignment);
        if ((slave->mtd.flags & MTD_WRITEABLE) && remainder) {
 -              slave->mtd.flags &= ~MTD_WRITEABLE;