mirror of
				git://git.openwrt.org/openwrt/openwrt.git
				synced 2025-11-03 14:34:27 -05:00 
			
		
		
		
	Automatically refreshed: ath79/patches-5.10/0032-MIPS-ath79-sanitize-symbols.patch bcm63xx/patches-5.10/322-MIPS-BCM63XX-switch-to-IRQ_DOMAIN.patch bcm63xx/patches-5.10/434-nand-brcmnand-fix-OOB-R-W-with-Hamming-ECC.patch bmips/patches-5.10/001-v5.11-mips-bmips-select-ARCH_HAS_RESET_CONTROLLER.patch bmips/patches-5.10/041-v5.13-mtd-rawnand-brcmnand-fix-OOB-R-W-with-Hamming-ECC.patch bmips/patches-5.10/202-mips-bmips-disable-ARCH_HAS_SYNC_DMA_FOR_CPU_ALL.patch bmips/patches-5.10/600-mips-bmips-add-pci-support.patch generic/backport-5.10/103-v5.13-MIPS-select-CPU_MIPS64-for-remaining-MIPS64-CPUs.patch generic/hack-5.10/301-mips_image_cmdline_hack.patch generic/hack-5.10/402-mtd-blktrans-call-add-disks-after-mtd-device.patch generic/hack-5.10/902-debloat_proc.patch generic/pending-5.10/300-mips_expose_boot_raw.patch generic/pending-5.10/495-mtd-core-add-get_mtd_device_by_node.patch generic/pending-5.10/630-packet_socket_type.patch ipq806x/patches-5.10/0072-add-ipq806x-with-no-clocks.patch ipq806x/patches-5.10/099-1-mtd-nand-raw-qcom_nandc-add-boot_layout_mode-support.patch lantiq/patches-5.10/0001-MIPS-lantiq-add-pcie-driver.patch lantiq/patches-5.10/0023-NET-PHY-add-led-support-for-intel-xway.patch lantiq/patches-5.10/0152-lantiq-VPE.patch Deleted (reverse-appliable): bmips/patches-5.10/052-v5.13-gpio-guard-gpiochip_irqchip_add_domain-with-GPIOLIB_.patch generic/backport-5.10/499-v5.13-mtd-don-t-lock-when-recursively-deleting-partitions.patch Deleted (alternative upstream fix): ramips/patches-5.10/330-fix-pci-init-mt7620.patch Signed-off-by: Rui Salvaterra <rsalvaterra@gmail.com>
		
			
				
	
	
		
			99 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			99 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
From 0bccc3722bdd88e8ae995e77ef9f7b77ee4cbdee Mon Sep 17 00:00:00 2001
 | 
						|
From: Daniel Golle <daniel@makrotopia.org>
 | 
						|
Date: Wed, 7 Apr 2021 22:45:54 +0100
 | 
						|
Subject: [PATCH 2/2] mtd: blktrans: call add disks after mtd device
 | 
						|
To: linux-mtd@lists.infradead.org
 | 
						|
Cc: Vignesh Raghavendra <vigneshr@ti.com>,
 | 
						|
    Richard Weinberger <richard@nod.at>,
 | 
						|
    Miquel Raynal <miquel.raynal@bootlin.com>,
 | 
						|
    David Woodhouse <dwmw2@infradead.org>
 | 
						|
 | 
						|
Calling device_add_disk while holding mtd_table_mutex leads
 | 
						|
to deadlock in case part_bits!=0 as block partition parsers
 | 
						|
will try to open the newly created disks, trying to acquire
 | 
						|
mutex once again.
 | 
						|
Move device_add_disk to additional function called after
 | 
						|
add partitions of an MTD device have been added and locks
 | 
						|
have been released.
 | 
						|
 | 
						|
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
 | 
						|
---
 | 
						|
 drivers/mtd/mtd_blkdevs.c    | 33 ++++++++++++++++++++++++++-------
 | 
						|
 drivers/mtd/mtdcore.c        |  3 +++
 | 
						|
 include/linux/mtd/blktrans.h |  1 +
 | 
						|
 3 files changed, 30 insertions(+), 7 deletions(-)
 | 
						|
 | 
						|
--- a/drivers/mtd/mtd_blkdevs.c
 | 
						|
+++ b/drivers/mtd/mtd_blkdevs.c
 | 
						|
@@ -457,13 +457,6 @@ int add_mtd_blktrans_dev(struct mtd_blkt
 | 
						|
 	if (new->readonly)
 | 
						|
 		set_disk_ro(gd, 1);
 | 
						|
 
 | 
						|
-	device_add_disk(&new->mtd->dev, gd, NULL);
 | 
						|
-
 | 
						|
-	if (new->disk_attributes) {
 | 
						|
-		ret = sysfs_create_group(&disk_to_dev(gd)->kobj,
 | 
						|
-					new->disk_attributes);
 | 
						|
-		WARN_ON(ret);
 | 
						|
-	}
 | 
						|
 	return 0;
 | 
						|
 error4:
 | 
						|
 	kfree(new->tag_set);
 | 
						|
@@ -475,6 +468,27 @@ error1:
 | 
						|
 	return ret;
 | 
						|
 }
 | 
						|
 
 | 
						|
+void register_mtd_blktrans_devs(void)
 | 
						|
+{
 | 
						|
+	struct mtd_blktrans_ops *tr;
 | 
						|
+	struct mtd_blktrans_dev *dev, *next;
 | 
						|
+	int ret;
 | 
						|
+
 | 
						|
+	list_for_each_entry(tr, &blktrans_majors, list) {
 | 
						|
+		list_for_each_entry_safe(dev, next, &tr->devs, list) {
 | 
						|
+			if (dev->disk->flags & GENHD_FL_UP)
 | 
						|
+				continue;
 | 
						|
+
 | 
						|
+			device_add_disk(&dev->mtd->dev, dev->disk, NULL);
 | 
						|
+			if (dev->disk_attributes) {
 | 
						|
+				ret = sysfs_create_group(&disk_to_dev(dev->disk)->kobj,
 | 
						|
+							dev->disk_attributes);
 | 
						|
+				WARN_ON(ret);
 | 
						|
+			}
 | 
						|
+		}
 | 
						|
+	}
 | 
						|
+}
 | 
						|
+
 | 
						|
 int del_mtd_blktrans_dev(struct mtd_blktrans_dev *old)
 | 
						|
 {
 | 
						|
 	unsigned long flags;
 | 
						|
--- a/drivers/mtd/mtdcore.c
 | 
						|
+++ b/drivers/mtd/mtdcore.c
 | 
						|
@@ -31,6 +31,7 @@
 | 
						|
 
 | 
						|
 #include <linux/mtd/mtd.h>
 | 
						|
 #include <linux/mtd/partitions.h>
 | 
						|
+#include <linux/mtd/blktrans.h>
 | 
						|
 
 | 
						|
 #include "mtdcore.h"
 | 
						|
 
 | 
						|
@@ -851,6 +852,8 @@ int mtd_device_parse_register(struct mtd
 | 
						|
 		register_reboot_notifier(&mtd->reboot_notifier);
 | 
						|
 	}
 | 
						|
 
 | 
						|
+	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,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(void);
 | 
						|
 
 | 
						|
 #endif /* __MTD_TRANS_H__ */
 |