From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794084453; b=MB7rDDxlrKPKRCNlHfQKRHttpKvb6Ss04jS/PhE5RwV2nCdsDi7lsZS+9fH89VFGDmv5 VRK+sJhhOAEPdKMjKAE5qcQTx+greOeq/I+MjnSEZSyh6B/Zu7mNKqAZFm36Gj/2qfwAW hYTgbQSZWOJ8IaR08rWDY4AdaOEpnYuUd6ZY4qKNKTPu8tf4dP+OT6SUwM/2tJ57yeY0q Q0v4RFbLq12eGE+mgx8OfBOX78QfsFpn24n06HYBzTeCw5Q+fY9ybvww3IvBNE5FoEMMJ RlE5YvFYFFjjfeD4Zbdpgl6CLRmNVGr4YCPThVsbPQoFRY8nBmgDyejWwyELmfdVSAA== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794084453; h=Received:DKIM-Signature:Message-ID:From:Subject:Reply-To:To:Cc: In-Reply-To:References:Content-Type:Content-Transfer-Encoding:Date; bh=0KebwVLYB2D2L6loP8NHGLZcG3tV/DtfHEbIAoU8RMc=; b=Ru7ATrZFSWHHfCPswvcSYB1V57JxtSyaB488jNiqJUIssSryM98gxPTTmsPFGIk6/q6y JoGZuli2rZ/9RMq813I6dEkMmgZIt3cYysO110a232Awd5XTgK0qP5oJ4tpJljq4IcgLU kQTe0SFkDM2sBC1HhZuc+3NUeI4lbQfuVDC7fQ9ZWVMvqkPvMchgLCzbigdO3wiLtjJcj 2ZOIUMkVHkJCXsCsk5D2d2FP9z+ilI7tXy3fmeXAzpcdiaxewENoWXRIFsRyODUV3BKl0 6KSz5gilxA0BEcJpchQXcOakoulX6jlKpisBnw1H9/cbmUoMgvV6/DMTCXxC3pA1KnA== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=mYJoNCay; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A2EE7606A326; Sat, 7 Nov 2026 20:47:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=mYJoNCay Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5F67A1F0089F; Sat, 7 Nov 2026 20:47:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794084452; bh=0KebwVLYB2D2L6loP8NHGLZcG3tV/DtfHEbIAoU8RMc=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=mYJoNCay27RFXMdep+ySASAN5VNIZ5d5/8bbraSiMJjesxQ+flhHAeeLo/POilJH2 lIbg7gWtxKy6s4+kCBlSLvhOUVrHRg1Jq3BbQxeoK/1/2XADVNLSemos8qr1zxC7fr vGLU02y7h4gtXmMAnYBUOyIhoauo8XFi0M+ZYWH8Ng8lslGRXWOPjZM9/FyMbEMGCP TyUU1Ufy5fdqvIC2fOo28Ay1ouCYq+tdYUfbrQiaIGWnlvNwggEhVMfg9w+8ujfqn1 48P7XWdSOEoigw7Ixq74Oq7cFltc9I6ySeDd27idLc3E0480Tb16dYYi6WDbq6IDvu rlQdAb4eXbmAg== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 39/58] mtd: rawnand: qcom: retain open-handle state on removal Reply-To: sashiko-reviews@lists.linux.dev To: "James Hilliard" , "Miquel Raynal" , "Richard Weinberger" , "Vignesh Raghavendra" , "Brian Norris" , "Boris Brezillon" , "Arseniy Krasnov" , "Chen-Yu Tsai" , "Jernej Skrabec" , "Samuel Holland" , "Thomas Gleixner" , "Vitaly Wool" , "David Woodhouse" , "Russell King" , "David Woodhouse" , "Mike Rapoport" , "Maxime Coquelin" , "Alexandre Torgue" , "Christophe Kerello" , "Jason Roberts" , "Michal Simek" , "Boris Brezillon" , "Mason Yang" , "Ramuthevar Vadivel Murugan" , "Stefan Agner" , "Lucas Stach" , "Thierry Reding" , "Jonathan Hunter" , "Dmitry Osipenko" , "Matthias Brugger" , "AngeloGioacchino Del Regno" , "Jorge Ramirez-Ortiz" , "Heiko Stuebner" , "Yifeng Zhao" , "Nicolas Ferre" , "Alexandre Belloni" , "Claudiu Beznea" , "Keguang Zhang" , "Jacky Huang" , "Shan-Chun Hung" , "Frank Li" , "Sascha Hauer" , "Pengutronix Kernel Team" , "Fabio Estevam" , "Han Xu" , "Liang Yang" , "Neil Armstrong" , "Kevin Hilman" , "Jerome Brunet" , "Martin Blumenstingl" , "Manivannan Sadhasivam" , "Kamal Dasu" , "Broadcom internal kernel review list" , "Vladimir Zapolskiy" , "Piotr Wojtaszczyk" , "Paul Cercueil" , "Harvey Hunt" , "Maxim Levitsky" , "Michael Walle" , "Srinivas Kandagatla" Cc: "Tze Yee Ng" , linux-mtd@lists.infradead.org, linux-sunxi@lists.linux.dev, imx@lists.linux.dev, linux-amlogic@lists.infradead.org, Frank.Li@kernel.org, wsa+renesas@sang-engineering.com In-Reply-To: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-39-0181ef0cc4c8@gmail.com> References: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com> <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-39-0181ef0cc4c8@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Sat, 07 Nov 2026 20:47:28 +0000 X-Mailing-List: imx@lists.linux.dev List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list Thank you for your contribution! Sashiko AI review found 1 potential issue to consider: - [Medium] mtd: rawnand: qcom: broken error handling in qcom_probe_nand_devices() -- commit 0a9084cf5fec792ff4833fe6c2d5c8f9780c60a1 Author: James Hilliard mtd: rawnand: qcom: retain open-handle state on removal This commit ties the lifetime of qcom_nand_host and qcom_nand_controller to the MTD and NAND controller release callbacks, validates boot partitions before registering MTD devices, and adds cleanup of registered chips on probe failure and removal. > diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qco= m_nandc.c > index 2d98f79b492d6..f812de1d776fc 100644 > --- a/drivers/mtd/nand/raw/qcom_nandc.c > +++ b/drivers/mtd/nand/raw/qcom_nandc.c [ ... ] > @@ -2077,15 +2091,13 @@ static int qcom_probe_nand_devices(struct qcom_na= nd_controller *nandc) > int ret =3D -ENODEV; > =20 > for_each_available_child_of_node_scoped(dn, child) { > - host =3D devm_kzalloc(dev, sizeof(*host), GFP_KERNEL); > + host =3D kzalloc_obj(*host); > if (!host) > return -ENOMEM; > =20 > ret =3D qcom_nand_host_init_and_register(nandc, host, child); > - if (ret) { > - devm_kfree(dev, host); > + if (ret) > continue; > - } > =20 > list_add_tail(&host->node, &nandc->host_list); > } [Severity: Medium] Should qcom_probe_nand_devices() return ret instead of continuing when qcom_nand_host_init_and_register() fails? Continuing the loop means that if an earlier child node fails (for example with -EPROBE_DEFER or invalid boot partitions) and a later child succeeds, ret is overwritten with 0 and the partial probe is not unwound via err_setup. Worse, if the earlier child fails in qcom_nand_attach_chip() when qcom_alloc_bam_transaction() returns NULL after qcom_free_bam_transaction() freed nandc->bam_txn: drivers/mtd/nand/raw/qcom_nandc.c:qcom_nand_attach_chip() { ... if (nandc->props->supports_bam) qcom_free_bam_transaction(nandc); nandc->max_cwperpage =3D max_t(unsigned int, nandc->max_cwperpage, cwperpage); /* Now allocate the BAM transaction based on updated max_cwperpage */ if (nandc->props->supports_bam) { nandc->bam_txn =3D qcom_alloc_bam_transaction(nandc); if (!nandc->bam_txn) { dev_err(nandc->dev, "failed to allocate bam transaction\n"); return -ENOMEM; } } ... } continuing to the next child node calls nand_scan(), which runs qcom_clear_bam_transaction() and dereferences nandc->bam_txn while it is NULL. --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261107-submit-raw= nand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com?part=3D39 From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 390F2606A32E for ; Sat, 7 Nov 2026 20:47:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=INJeo8NW; dkim=fail reason="signature verification failed" (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=mYJoNCay DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Date:References: In-Reply-To:Cc:To:Subject:From:Message-ID:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Unsubscribe-Post:List-Owner; bh=MVJtUARtC2BjHkOIDMf5GxhXEHhwBZs2t4rgWMC1I4Y=; b=INJeo8NWzBUetS9wpFVhOduvT5 ied4tdG4qIorbH62ZDX+J/N80GqU77WQmUhiyHljrqds4pisyuW+T0mEsYxymc/eVz0iiDeubGhig 48FU3HEIG6y0gFqEFLNp9Ag6CyLHzINfzaKyMCPtNMIdSwjsvieYHWLNdkUX4EofMs+q8/3BLzzM7 564CZCWjAfkW/hdTWaUwN8e86R1yhQeJQPowq50o1h3VIsa5qlqQ4IHfzRySJjneFfmbzXBNZKlxR S1RCRFyXngFwck8Eiq06bfsQjfbiXr9fkxhLzu3FfQFQGzXZ6RirHAUvxlWcXB9f5q3v70sXQKi4o 10LCNrGQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnK3-0000000FOik-3KEH; Sat, 07 Nov 2026 20:47:35 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnK0-0000000FOiT-3umi; Sat, 07 Nov 2026 20:47:34 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 35883417C6; Sat, 7 Nov 2026 20:47:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5F67A1F0089F; Sat, 7 Nov 2026 20:47:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794084452; bh=0KebwVLYB2D2L6loP8NHGLZcG3tV/DtfHEbIAoU8RMc=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=mYJoNCay27RFXMdep+ySASAN5VNIZ5d5/8bbraSiMJjesxQ+flhHAeeLo/POilJH2 lIbg7gWtxKy6s4+kCBlSLvhOUVrHRg1Jq3BbQxeoK/1/2XADVNLSemos8qr1zxC7fr vGLU02y7h4gtXmMAnYBUOyIhoauo8XFi0M+ZYWH8Ng8lslGRXWOPjZM9/FyMbEMGCP TyUU1Ufy5fdqvIC2fOo28Ay1ouCYq+tdYUfbrQiaIGWnlvNwggEhVMfg9w+8ujfqn1 48P7XWdSOEoigw7Ixq74Oq7cFltc9I6ySeDd27idLc3E0480Tb16dYYi6WDbq6IDvu rlQdAb4eXbmAg== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 39/58] mtd: rawnand: qcom: retain open-handle state on removal To: "James Hilliard" , "Miquel Raynal" , "Richard Weinberger" , "Vignesh Raghavendra" , "Brian Norris" , "Boris Brezillon" , "Arseniy Krasnov" , "Chen-Yu Tsai" , "Jernej Skrabec" , "Samuel Holland" , "Thomas Gleixner" , "Vitaly Wool" , "David Woodhouse" , "Russell King" , "David Woodhouse" , "Mike Rapoport" , "Maxime Coquelin" , "Alexandre Torgue" , "Christophe Kerello" , "Jason Roberts" , "Michal Simek" , "Boris Brezillon" , "Mason Yang" , "Ramuthevar Vadivel Murugan" , "Stefan Agner" , "Lucas Stach" , "Thierry Reding" , "Jonathan Hunter" , "Dmitry Osipenko" , "Matthias Brugger" , "AngeloGioacchino Del Regno" , "Jorge Ramirez-Ortiz" , "Heiko Stuebner" , "Yifeng Zhao" , "Nicolas Ferre" , "Alexandre Belloni" , "Claudiu Beznea" , "Keguang Zhang" , "Jacky Huang" , "Shan-Chun Hung" , "Frank Li" , "Sascha Hauer" , "Pengutronix Kernel Team" , "Fabio Estevam" , "Han Xu" , "Liang Yang" , "Neil Armstrong" , "Kevin Hilman" , "Jerome Brunet" , "Martin Blumenstingl" , "Manivannan Sadhasivam" , "Kamal Dasu" , "Broadcom internal kernel review list" , "Vladimir Zapolskiy" , "Piotr Wojtaszczyk" , "Paul Cercueil" , "Harvey Hunt" , "Maxim Levitsky" , "Michael Walle" , "Srinivas Kandagatla" Cc: "Tze Yee Ng" , linux-mtd@lists.infradead.org, linux-sunxi@lists.linux.dev, imx@lists.linux.dev, linux-amlogic@lists.infradead.org, Frank.Li@kernel.org, wsa+renesas@sang-engineering.com In-Reply-To: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-39-0181ef0cc4c8@gmail.com> References: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com> <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-39-0181ef0cc4c8@gmail.com> Date: Sat, 07 Nov 2026 20:47:28 +0000 X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev X-Archived-By: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-mtd" X-Spam-Status: No, score=-8.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg MSBwb3RlbnRpYWwKaXNzdWUgdG8gY29uc2lkZXI6Ci0gW01lZGl1bV0gbXRkOiByYXduYW5kOiBx Y29tOiBicm9rZW4gZXJyb3IgaGFuZGxpbmcgaW4KICBxY29tX3Byb2JlX25hbmRfZGV2aWNlcygp Ci0tCgpjb21taXQgMGE5MDg0Y2Y1ZmVjNzkyZmY0ODMzZmU2YzJkNWM4Zjk3ODBjNjBhMQpBdXRo b3I6IEphbWVzIEhpbGxpYXJkIDxqYW1lcy5oaWxsaWFyZDFAZ21haWwuY29tPgoKbXRkOiByYXdu YW5kOiBxY29tOiByZXRhaW4gb3Blbi1oYW5kbGUgc3RhdGUgb24gcmVtb3ZhbAoKVGhpcyBjb21t aXQgdGllcyB0aGUgbGlmZXRpbWUgb2YgcWNvbV9uYW5kX2hvc3QgYW5kIHFjb21fbmFuZF9jb250 cm9sbGVyIHRvCnRoZSBNVEQgYW5kIE5BTkQgY29udHJvbGxlciByZWxlYXNlIGNhbGxiYWNrcywg dmFsaWRhdGVzIGJvb3QgcGFydGl0aW9ucwpiZWZvcmUgcmVnaXN0ZXJpbmcgTVREIGRldmljZXMs IGFuZCBhZGRzIGNsZWFudXAgb2YgcmVnaXN0ZXJlZCBjaGlwcyBvbgpwcm9iZSBmYWlsdXJlIGFu ZCByZW1vdmFsLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9tdGQvbmFuZC9yYXcvcWNvbV9uYW5k Yy5jIGIvZHJpdmVycy9tdGQvbmFuZC9yYXcvcWNvbV9uYW5kYy5jCj4gaW5kZXggMmQ5OGY3OWI0 OTJkNi4uZjgxMmRlMWQ3NzZmYyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL210ZC9uYW5kL3Jhdy9x Y29tX25hbmRjLmMKPiArKysgYi9kcml2ZXJzL210ZC9uYW5kL3Jhdy9xY29tX25hbmRjLmMKClsg Li4uIF0KCj4gQEAgLTIwNzcsMTUgKzIwOTEsMTMgQEAgc3RhdGljIGludCBxY29tX3Byb2JlX25h bmRfZGV2aWNlcyhzdHJ1Y3QgcWNvbV9uYW5kX2NvbnRyb2xsZXIgKm5hbmRjKQo+ICAJaW50IHJl dCA9IC1FTk9ERVY7Cj4gIAo+ICAJZm9yX2VhY2hfYXZhaWxhYmxlX2NoaWxkX29mX25vZGVfc2Nv cGVkKGRuLCBjaGlsZCkgewo+IC0JCWhvc3QgPSBkZXZtX2t6YWxsb2MoZGV2LCBzaXplb2YoKmhv c3QpLCBHRlBfS0VSTkVMKTsKPiArCQlob3N0ID0ga3phbGxvY19vYmooKmhvc3QpOwo+ICAJCWlm ICghaG9zdCkKPiAgCQkJcmV0dXJuIC1FTk9NRU07Cj4gIAo+ICAJCXJldCA9IHFjb21fbmFuZF9o b3N0X2luaXRfYW5kX3JlZ2lzdGVyKG5hbmRjLCBob3N0LCBjaGlsZCk7Cj4gLQkJaWYgKHJldCkg ewo+IC0JCQlkZXZtX2tmcmVlKGRldiwgaG9zdCk7Cj4gKwkJaWYgKHJldCkKPiAgCQkJY29udGlu dWU7Cj4gLQkJfQo+ICAKPiAgCQlsaXN0X2FkZF90YWlsKCZob3N0LT5ub2RlLCAmbmFuZGMtPmhv c3RfbGlzdCk7Cj4gIAl9CgpbU2V2ZXJpdHk6IE1lZGl1bV0KU2hvdWxkIHFjb21fcHJvYmVfbmFu ZF9kZXZpY2VzKCkgcmV0dXJuIHJldCBpbnN0ZWFkIG9mIGNvbnRpbnVpbmcgd2hlbgpxY29tX25h bmRfaG9zdF9pbml0X2FuZF9yZWdpc3RlcigpIGZhaWxzPwoKQ29udGludWluZyB0aGUgbG9vcCBt ZWFucyB0aGF0IGlmIGFuIGVhcmxpZXIgY2hpbGQgbm9kZSBmYWlscyAoZm9yIGV4YW1wbGUKd2l0 aCAtRVBST0JFX0RFRkVSIG9yIGludmFsaWQgYm9vdCBwYXJ0aXRpb25zKSBhbmQgYSBsYXRlciBj aGlsZCBzdWNjZWVkcywKcmV0IGlzIG92ZXJ3cml0dGVuIHdpdGggMCBhbmQgdGhlIHBhcnRpYWwg cHJvYmUgaXMgbm90IHVud291bmQgdmlhCmVycl9zZXR1cC4KCldvcnNlLCBpZiB0aGUgZWFybGll ciBjaGlsZCBmYWlscyBpbiBxY29tX25hbmRfYXR0YWNoX2NoaXAoKSB3aGVuCnFjb21fYWxsb2Nf YmFtX3RyYW5zYWN0aW9uKCkgcmV0dXJucyBOVUxMIGFmdGVyIHFjb21fZnJlZV9iYW1fdHJhbnNh Y3Rpb24oKQpmcmVlZCBuYW5kYy0+YmFtX3R4bjoKCmRyaXZlcnMvbXRkL25hbmQvcmF3L3Fjb21f bmFuZGMuYzpxY29tX25hbmRfYXR0YWNoX2NoaXAoKSB7CiAgICAuLi4KCWlmIChuYW5kYy0+cHJv cHMtPnN1cHBvcnRzX2JhbSkKCQlxY29tX2ZyZWVfYmFtX3RyYW5zYWN0aW9uKG5hbmRjKTsKCglu YW5kYy0+bWF4X2N3cGVycGFnZSA9IG1heF90KHVuc2lnbmVkIGludCwgbmFuZGMtPm1heF9jd3Bl cnBhZ2UsCgkJCQkgICAgIGN3cGVycGFnZSk7CgoJLyogTm93IGFsbG9jYXRlIHRoZSBCQU0gdHJh bnNhY3Rpb24gYmFzZWQgb24gdXBkYXRlZCBtYXhfY3dwZXJwYWdlICovCglpZiAobmFuZGMtPnBy b3BzLT5zdXBwb3J0c19iYW0pIHsKCQluYW5kYy0+YmFtX3R4biA9IHFjb21fYWxsb2NfYmFtX3Ry YW5zYWN0aW9uKG5hbmRjKTsKCQlpZiAoIW5hbmRjLT5iYW1fdHhuKSB7CgkJCWRldl9lcnIobmFu ZGMtPmRldiwKCQkJCSJmYWlsZWQgdG8gYWxsb2NhdGUgYmFtIHRyYW5zYWN0aW9uXG4iKTsKCQkJ cmV0dXJuIC1FTk9NRU07CgkJfQoJfQogICAgLi4uCn0KCmNvbnRpbnVpbmcgdG8gdGhlIG5leHQg Y2hpbGQgbm9kZSBjYWxscyBuYW5kX3NjYW4oKSwgd2hpY2ggcnVucwpxY29tX2NsZWFyX2JhbV90 cmFuc2FjdGlvbigpIGFuZCBkZXJlZmVyZW5jZXMgbmFuZGMtPmJhbV90eG4gd2hpbGUgaXQgaXMK TlVMTC4KCi0tIApTYXNoaWtvIEFJIHJldmlldyDCtyBodHRwczovL3Nhc2hpa28uZGV2LyMvcGF0 Y2hzZXQvMjAyNjExMDctc3VibWl0LXJhd25hbmQtcmV0YWluZWQtaGFuZGxlLWxpZmV0aW1lLXYx LXYxLTAtMDE4MWVmMGNjNGM4QGdtYWlsLmNvbT9wYXJ0PTM5CgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTGludXggTVREIGRpc2N1c3Npb24g bWFpbGluZyBsaXN0Cmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8v bGludXgtbXRkLwo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 316E9606A32E for ; Sat, 7 Nov 2026 20:47:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=0MSBYeW+; dkim=fail reason="signature verification failed" (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=mYJoNCay DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Date:References: In-Reply-To:Cc:To:Subject:From:Message-ID:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Unsubscribe-Post:List-Owner; bh=oKBnsW8nSS/T8T+N7j3kw3n3HQ3u+mZixi+UKD2WBRU=; b=0MSBYeW+ua9B6l2PSfD6MVCeCD EdT7z9K1MVN2P7kwSeNKfLq7UokDXNcAmxJcw0r//mLxpS3E8YUCGW9dxZPfk5YvjmlTKr+Z4Ww5I LOKimf7f2op0QXeztDDUVQ8c32LSNG9XBijzzIPIAQkwfALDNQv+Ja7apd66Y7LN0mbNqlUQEE1qi NQ5kT6J/8xaoJdUrcT4lV6RcYz95Do5QYettWAoHR4ClePUZ7jbe/H5L9PH1nq9D9KD34ldaeDtbW Cd8g3KmIGhffRunoVz2v9WVR5IMlZ1Gf3kVfoYqsHZMrG8M12rLkLq2ac1Bf8M0bihEIU3G2IstsW hwDC6sBA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnK3-0000000FOig-31xj; Sat, 07 Nov 2026 20:47:35 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnK0-0000000FOiT-3umi; Sat, 07 Nov 2026 20:47:34 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 35883417C6; Sat, 7 Nov 2026 20:47:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5F67A1F0089F; Sat, 7 Nov 2026 20:47:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794084452; bh=0KebwVLYB2D2L6loP8NHGLZcG3tV/DtfHEbIAoU8RMc=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=mYJoNCay27RFXMdep+ySASAN5VNIZ5d5/8bbraSiMJjesxQ+flhHAeeLo/POilJH2 lIbg7gWtxKy6s4+kCBlSLvhOUVrHRg1Jq3BbQxeoK/1/2XADVNLSemos8qr1zxC7fr vGLU02y7h4gtXmMAnYBUOyIhoauo8XFi0M+ZYWH8Ng8lslGRXWOPjZM9/FyMbEMGCP TyUU1Ufy5fdqvIC2fOo28Ay1ouCYq+tdYUfbrQiaIGWnlvNwggEhVMfg9w+8ujfqn1 48P7XWdSOEoigw7Ixq74Oq7cFltc9I6ySeDd27idLc3E0480Tb16dYYi6WDbq6IDvu rlQdAb4eXbmAg== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 39/58] mtd: rawnand: qcom: retain open-handle state on removal To: "James Hilliard" , "Miquel Raynal" , "Richard Weinberger" , "Vignesh Raghavendra" , "Brian Norris" , "Boris Brezillon" , "Arseniy Krasnov" , "Chen-Yu Tsai" , "Jernej Skrabec" , "Samuel Holland" , "Thomas Gleixner" , "Vitaly Wool" , "David Woodhouse" , "Russell King" , "David Woodhouse" , "Mike Rapoport" , "Maxime Coquelin" , "Alexandre Torgue" , "Christophe Kerello" , "Jason Roberts" , "Michal Simek" , "Boris Brezillon" , "Mason Yang" , "Ramuthevar Vadivel Murugan" , "Stefan Agner" , "Lucas Stach" , "Thierry Reding" , "Jonathan Hunter" , "Dmitry Osipenko" , "Matthias Brugger" , "AngeloGioacchino Del Regno" , "Jorge Ramirez-Ortiz" , "Heiko Stuebner" , "Yifeng Zhao" , "Nicolas Ferre" , "Alexandre Belloni" , "Claudiu Beznea" , "Keguang Zhang" , "Jacky Huang" , "Shan-Chun Hung" , "Frank Li" , "Sascha Hauer" , "Pengutronix Kernel Team" , "Fabio Estevam" , "Han Xu" , "Liang Yang" , "Neil Armstrong" , "Kevin Hilman" , "Jerome Brunet" , "Martin Blumenstingl" , "Manivannan Sadhasivam" , "Kamal Dasu" , "Broadcom internal kernel review list" , "Vladimir Zapolskiy" , "Piotr Wojtaszczyk" , "Paul Cercueil" , "Harvey Hunt" , "Maxim Levitsky" , "Michael Walle" , "Srinivas Kandagatla" Cc: "Tze Yee Ng" , linux-mtd@lists.infradead.org, linux-sunxi@lists.linux.dev, imx@lists.linux.dev, linux-amlogic@lists.infradead.org, Frank.Li@kernel.org, wsa+renesas@sang-engineering.com In-Reply-To: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-39-0181ef0cc4c8@gmail.com> References: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com> <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-39-0181ef0cc4c8@gmail.com> Date: Sat, 07 Nov 2026 20:47:28 +0000 X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev X-Archived-By: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-amlogic" X-Spam-Status: No, score=-8.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg MSBwb3RlbnRpYWwKaXNzdWUgdG8gY29uc2lkZXI6Ci0gW01lZGl1bV0gbXRkOiByYXduYW5kOiBx Y29tOiBicm9rZW4gZXJyb3IgaGFuZGxpbmcgaW4KICBxY29tX3Byb2JlX25hbmRfZGV2aWNlcygp Ci0tCgpjb21taXQgMGE5MDg0Y2Y1ZmVjNzkyZmY0ODMzZmU2YzJkNWM4Zjk3ODBjNjBhMQpBdXRo b3I6IEphbWVzIEhpbGxpYXJkIDxqYW1lcy5oaWxsaWFyZDFAZ21haWwuY29tPgoKbXRkOiByYXdu YW5kOiBxY29tOiByZXRhaW4gb3Blbi1oYW5kbGUgc3RhdGUgb24gcmVtb3ZhbAoKVGhpcyBjb21t aXQgdGllcyB0aGUgbGlmZXRpbWUgb2YgcWNvbV9uYW5kX2hvc3QgYW5kIHFjb21fbmFuZF9jb250 cm9sbGVyIHRvCnRoZSBNVEQgYW5kIE5BTkQgY29udHJvbGxlciByZWxlYXNlIGNhbGxiYWNrcywg dmFsaWRhdGVzIGJvb3QgcGFydGl0aW9ucwpiZWZvcmUgcmVnaXN0ZXJpbmcgTVREIGRldmljZXMs IGFuZCBhZGRzIGNsZWFudXAgb2YgcmVnaXN0ZXJlZCBjaGlwcyBvbgpwcm9iZSBmYWlsdXJlIGFu ZCByZW1vdmFsLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9tdGQvbmFuZC9yYXcvcWNvbV9uYW5k Yy5jIGIvZHJpdmVycy9tdGQvbmFuZC9yYXcvcWNvbV9uYW5kYy5jCj4gaW5kZXggMmQ5OGY3OWI0 OTJkNi4uZjgxMmRlMWQ3NzZmYyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL210ZC9uYW5kL3Jhdy9x Y29tX25hbmRjLmMKPiArKysgYi9kcml2ZXJzL210ZC9uYW5kL3Jhdy9xY29tX25hbmRjLmMKClsg Li4uIF0KCj4gQEAgLTIwNzcsMTUgKzIwOTEsMTMgQEAgc3RhdGljIGludCBxY29tX3Byb2JlX25h bmRfZGV2aWNlcyhzdHJ1Y3QgcWNvbV9uYW5kX2NvbnRyb2xsZXIgKm5hbmRjKQo+ICAJaW50IHJl dCA9IC1FTk9ERVY7Cj4gIAo+ICAJZm9yX2VhY2hfYXZhaWxhYmxlX2NoaWxkX29mX25vZGVfc2Nv cGVkKGRuLCBjaGlsZCkgewo+IC0JCWhvc3QgPSBkZXZtX2t6YWxsb2MoZGV2LCBzaXplb2YoKmhv c3QpLCBHRlBfS0VSTkVMKTsKPiArCQlob3N0ID0ga3phbGxvY19vYmooKmhvc3QpOwo+ICAJCWlm ICghaG9zdCkKPiAgCQkJcmV0dXJuIC1FTk9NRU07Cj4gIAo+ICAJCXJldCA9IHFjb21fbmFuZF9o b3N0X2luaXRfYW5kX3JlZ2lzdGVyKG5hbmRjLCBob3N0LCBjaGlsZCk7Cj4gLQkJaWYgKHJldCkg ewo+IC0JCQlkZXZtX2tmcmVlKGRldiwgaG9zdCk7Cj4gKwkJaWYgKHJldCkKPiAgCQkJY29udGlu dWU7Cj4gLQkJfQo+ICAKPiAgCQlsaXN0X2FkZF90YWlsKCZob3N0LT5ub2RlLCAmbmFuZGMtPmhv c3RfbGlzdCk7Cj4gIAl9CgpbU2V2ZXJpdHk6IE1lZGl1bV0KU2hvdWxkIHFjb21fcHJvYmVfbmFu ZF9kZXZpY2VzKCkgcmV0dXJuIHJldCBpbnN0ZWFkIG9mIGNvbnRpbnVpbmcgd2hlbgpxY29tX25h bmRfaG9zdF9pbml0X2FuZF9yZWdpc3RlcigpIGZhaWxzPwoKQ29udGludWluZyB0aGUgbG9vcCBt ZWFucyB0aGF0IGlmIGFuIGVhcmxpZXIgY2hpbGQgbm9kZSBmYWlscyAoZm9yIGV4YW1wbGUKd2l0 aCAtRVBST0JFX0RFRkVSIG9yIGludmFsaWQgYm9vdCBwYXJ0aXRpb25zKSBhbmQgYSBsYXRlciBj aGlsZCBzdWNjZWVkcywKcmV0IGlzIG92ZXJ3cml0dGVuIHdpdGggMCBhbmQgdGhlIHBhcnRpYWwg cHJvYmUgaXMgbm90IHVud291bmQgdmlhCmVycl9zZXR1cC4KCldvcnNlLCBpZiB0aGUgZWFybGll ciBjaGlsZCBmYWlscyBpbiBxY29tX25hbmRfYXR0YWNoX2NoaXAoKSB3aGVuCnFjb21fYWxsb2Nf YmFtX3RyYW5zYWN0aW9uKCkgcmV0dXJucyBOVUxMIGFmdGVyIHFjb21fZnJlZV9iYW1fdHJhbnNh Y3Rpb24oKQpmcmVlZCBuYW5kYy0+YmFtX3R4bjoKCmRyaXZlcnMvbXRkL25hbmQvcmF3L3Fjb21f bmFuZGMuYzpxY29tX25hbmRfYXR0YWNoX2NoaXAoKSB7CiAgICAuLi4KCWlmIChuYW5kYy0+cHJv cHMtPnN1cHBvcnRzX2JhbSkKCQlxY29tX2ZyZWVfYmFtX3RyYW5zYWN0aW9uKG5hbmRjKTsKCglu YW5kYy0+bWF4X2N3cGVycGFnZSA9IG1heF90KHVuc2lnbmVkIGludCwgbmFuZGMtPm1heF9jd3Bl cnBhZ2UsCgkJCQkgICAgIGN3cGVycGFnZSk7CgoJLyogTm93IGFsbG9jYXRlIHRoZSBCQU0gdHJh bnNhY3Rpb24gYmFzZWQgb24gdXBkYXRlZCBtYXhfY3dwZXJwYWdlICovCglpZiAobmFuZGMtPnBy b3BzLT5zdXBwb3J0c19iYW0pIHsKCQluYW5kYy0+YmFtX3R4biA9IHFjb21fYWxsb2NfYmFtX3Ry YW5zYWN0aW9uKG5hbmRjKTsKCQlpZiAoIW5hbmRjLT5iYW1fdHhuKSB7CgkJCWRldl9lcnIobmFu ZGMtPmRldiwKCQkJCSJmYWlsZWQgdG8gYWxsb2NhdGUgYmFtIHRyYW5zYWN0aW9uXG4iKTsKCQkJ cmV0dXJuIC1FTk9NRU07CgkJfQoJfQogICAgLi4uCn0KCmNvbnRpbnVpbmcgdG8gdGhlIG5leHQg Y2hpbGQgbm9kZSBjYWxscyBuYW5kX3NjYW4oKSwgd2hpY2ggcnVucwpxY29tX2NsZWFyX2JhbV90 cmFuc2FjdGlvbigpIGFuZCBkZXJlZmVyZW5jZXMgbmFuZGMtPmJhbV90eG4gd2hpbGUgaXQgaXMK TlVMTC4KCi0tIApTYXNoaWtvIEFJIHJldmlldyDCtyBodHRwczovL3Nhc2hpa28uZGV2LyMvcGF0 Y2hzZXQvMjAyNjExMDctc3VibWl0LXJhd25hbmQtcmV0YWluZWQtaGFuZGxlLWxpZmV0aW1lLXYx LXYxLTAtMDE4MWVmMGNjNGM4QGdtYWlsLmNvbT9wYXJ0PTM5CgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hbWxvZ2ljIG1haWxpbmcgbGlzdAps aW51eC1hbWxvZ2ljQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5v cmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hbWxvZ2ljCg==