meta-agl-bsp: meta-sancloud: Revert problematic kernel commit 20/27120/2
authorPaul Barker <paul.barker@sancloud.com>
Mon, 31 Jan 2022 10:54:55 +0000 (10:54 +0000)
committerJan-Simon Moeller <jsmoeller@linuxfoundation.org>
Mon, 31 Jan 2022 17:44:47 +0000 (17:44 +0000)
Linux 5.10.y commit 4b21d4e820bb causes a backtrace to be printed on
boot:

    [    3.244915] ------------[ cut here ]------------
    [    3.249608] WARNING: CPU: 0 PID: 1 at block/blk-mq.c:3045 blk_mq_release+0x78/0xe8
    [    3.257337] Modules linked in:
    [    3.260422] CPU: 0 PID: 1 Comm: swapper Not tainted 5.10.65-sc1 #1
    [    3.266670] Hardware name: Generic AM33XX (Flattened Device Tree)
    [    3.272822] [<c010e880>] (unwind_backtrace) from [<c010b2bc>] (show_stack+0x10/0x14)
    [    3.280681] [<c010b2bc>] (show_stack) from [<c0129160>] (__warn+0xec/0x104)
    [    3.287740] [<c0129160>] (__warn) from [<c0b16b34>] (warn_slowpath_fmt+0x64/0xc8)
    [    3.295302] [<c0b16b34>] (warn_slowpath_fmt) from [<c04ef948>] (blk_mq_release+0x78/0xe8)
    [    3.303580] [<c04ef948>] (blk_mq_release) from [<c04e20f4>] (blk_release_queue+0xb0/0x120)
    [    3.311889] [<c04e20f4>] (blk_release_queue) from [<c0573b84>] (kobject_put+0x90/0xfc)
    [    3.319900] [<c0573b84>] (kobject_put) from [<c04f4eb8>] (disk_release+0xa0/0xd8)
    [    3.327441] [<c04f4eb8>] (disk_release) from [<c06f8c30>] (device_release+0x30/0xa0)
    [    3.335232] [<c06f8c30>] (device_release) from [<c0573b84>] (kobject_put+0x90/0xfc)
    [    3.342929] [<c0573b84>] (kobject_put) from [<c072c474>] (nbd_dev_add+0x20c/0x260)
    [    3.350551] [<c072c474>] (nbd_dev_add) from [<c112aa0c>] (nbd_init+0xfc/0x11c)
    [    3.357824] [<c112aa0c>] (nbd_init) from [<c0101ef0>] (do_one_initcall+0x44/0x26c)
    [    3.365445] [<c0101ef0>] (do_one_initcall) from [<c11010b8>] (kernel_init_freeable+0x1c4/0x224)
    [    3.374196] [<c11010b8>] (kernel_init_freeable) from [<c0b23ad0>] (kernel_init+0x8/0x120)
    [    3.382411] [<c0b23ad0>] (kernel_init) from [<c0100148>] (ret_from_fork+0x14/0x2c)
    [    3.390023] Exception stack(0xc18dffb0 to 0xc18dfff8)
    [    3.395106] ffa0:                                     00000000 00000000 00000000 00000000
    [    3.403329] ffc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
    [    3.411542] ffe0: 00000000 00000000 00000000 00000000 00000013 00000000
    [    3.418203] ---[ end trace de955d2022fa8082 ]---

This commit was reverted in Linux 5.10.66 but the Sancloud kernel tree
is based on Linux 5.10.65. For now we can simply carry the patch to
revert this change in meta-agl-bsp.

Bug-AGL: SPEC-4156
Signed-off-by: Paul Barker <paul.barker@sancloud.com>
Change-Id: I448037f3273c6ee249facb14c42fe93fbfd0c27b
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/27120
Tested-by: Jenkins Job builder account
ci-image-build: Jenkins Job builder account
ci-image-boot-test: Jenkins Job builder account
Reviewed-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
meta-agl-bsp/meta-sancloud/recipes-kernel/linux/linux-bbe/0001-Revert-block-nbd-add-sanity-check-for-first_minor.patch [new file with mode: 0644]
meta-agl-bsp/meta-sancloud/recipes-kernel/linux/linux-bbe_%.bbappend

diff --git a/meta-agl-bsp/meta-sancloud/recipes-kernel/linux/linux-bbe/0001-Revert-block-nbd-add-sanity-check-for-first_minor.patch b/meta-agl-bsp/meta-sancloud/recipes-kernel/linux/linux-bbe/0001-Revert-block-nbd-add-sanity-check-for-first_minor.patch
new file mode 100644 (file)
index 0000000..3b92bc3
--- /dev/null
@@ -0,0 +1,47 @@
+From 36b3d70d866d6781db4ab4c575b8bdab79244c7f Mon Sep 17 00:00:00 2001
+From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+Date: Thu, 16 Sep 2021 10:17:11 +0200
+Subject: [PATCH] Revert "block: nbd: add sanity check for first_minor"
+
+This reverts commit 4b21d4e820bb9a1415ec76dfe565e4c5937337dd which is
+commit b1a811633f7321cf1ae2bb76a66805b7720e44c9 upstream.
+
+The backport of this is reported to be causing some problems, so revert
+this for now until they are worked out.
+
+Link: https://lore.kernel.org/r/CACPK8XfUWoOHr-0RwRoYoskia4fbAbZ7DYf5wWBnv6qUnGq18w@mail.gmail.com
+Reported-by: Joel Stanley <joel@jms.id.au>
+Cc: Christoph Hellwig <hch@lst.de>
+Cc: Pavel Skripkin <paskripkin@gmail.com>
+Cc: Jens Axboe <axboe@kernel.dk>
+Cc: Sasha Levin <sashal@kernel.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ drivers/block/nbd.c | 10 ----------
+ 1 file changed, 10 deletions(-)
+
+diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
+index 98274ba0701d..59c452fff835 100644
+--- a/drivers/block/nbd.c
++++ b/drivers/block/nbd.c
+@@ -1759,17 +1759,7 @@ static int nbd_dev_add(int index)
+       refcount_set(&nbd->refs, 1);
+       INIT_LIST_HEAD(&nbd->list);
+       disk->major = NBD_MAJOR;
+-
+-      /* Too big first_minor can cause duplicate creation of
+-       * sysfs files/links, since first_minor will be truncated to
+-       * byte in __device_add_disk().
+-       */
+       disk->first_minor = index << part_shift;
+-      if (disk->first_minor > 0xff) {
+-              err = -EINVAL;
+-              goto out_free_idr;
+-      }
+-
+       disk->fops = &nbd_fops;
+       disk->private_data = nbd;
+       sprintf(disk->disk_name, "nbd%d", index);
+-- 
+2.34.1
+
index b3709a2..d5e19ef 100644 (file)
@@ -3,3 +3,5 @@ require recipes-kernel/linux/linux-agl.inc
 FILESEXTRAPATHS:prepend := "${THISDIR}/${BPN}:"
 
 AGL_KCONFIG_FRAGMENTS += "cma-256.cfg"
+
+SRC_URI += "file://0001-Revert-block-nbd-add-sanity-check-for-first_minor.patch"