From e9c8c834de60a9199a41664c06430e74d15f2973 Mon Sep 17 00:00:00 2001 From: Daniel Golle Date: Mon, 12 Apr 2021 21:40:35 +0100 Subject: [PATCH] kernel: fix deferred mtdblock registration Check if gendisk is already up instead of trying to figure out parent mtd device (which didn't work well). Reported-by: DENG Qingfang Reported-by: Koen Vandeputte Fixes: 2809d0000744 ("kernel: support FIT partition parser on mtdblock devices") Signed-off-by: Daniel Golle --- ...rans-call-add-disks-after-mtd-device.patch | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/target/linux/generic/hack-5.10/402-mtd-blktrans-call-add-disks-after-mtd-device.patch b/target/linux/generic/hack-5.10/402-mtd-blktrans-call-add-disks-after-mtd-device.patch index 77ecd9b72fc..c8c86ad255a 100644 --- a/target/linux/generic/hack-5.10/402-mtd-blktrans-call-add-disks-after-mtd-device.patch +++ b/target/linux/generic/hack-5.10/402-mtd-blktrans-call-add-disks-after-mtd-device.patch @@ -39,24 +39,19 @@ Signed-off-by: Daniel Golle return 0; error4: kfree(new->tag_set); -@@ -475,6 +468,32 @@ error1: +@@ -475,6 +468,27 @@ error1: return ret; } -+void register_mtd_blktrans_devs(struct mtd_info *mtd) ++void register_mtd_blktrans_devs(void) +{ + struct mtd_blktrans_ops *tr; + struct mtd_blktrans_dev *dev, *next; -+ struct mtd_info *top_mtd; + int ret; + + list_for_each_entry(tr, &blktrans_majors, list) { + list_for_each_entry_safe(dev, next, &tr->devs, list) { -+ top_mtd = dev->mtd; -+ while (top_mtd->parent) -+ top_mtd = top_mtd->parent; -+ -+ if (mtd->index != top_mtd->index) ++ if (dev->disk->flags & GENHD_FL_UP) + continue; + + device_add_disk(&dev->mtd->dev, dev->disk, NULL); @@ -86,18 +81,18 @@ Signed-off-by: Daniel Golle register_reboot_notifier(&mtd->reboot_notifier); } -+ register_mtd_blktrans_devs(mtd); ++ register_mtd_blktrans_devs(); + out: if (ret && device_is_registered(&mtd->dev)) del_mtd_device(mtd); --- a/include/linux/mtd/blktrans.h +++ b/include/linux/mtd/blktrans.h -@@ -76,6 +76,7 @@ extern int deregister_mtd_blktrans(struc +@@ -76,6 +76,6 @@ extern int deregister_mtd_blktrans(struc extern int add_mtd_blktrans_dev(struct mtd_blktrans_dev *dev); extern int del_mtd_blktrans_dev(struct mtd_blktrans_dev *dev); extern int mtd_blktrans_cease_background(struct mtd_blktrans_dev *dev); -+extern void register_mtd_blktrans_devs(struct mtd_info *mtd); - +- ++extern void register_mtd_blktrans_devs(void); #endif /* __MTD_TRANS_H__ */