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=1794084754; b=3qOApJm3P3320cC3w7XO3e9Dtgm20Q7WzDRQKXneyX+iFUUsx7Xv3bTLOM4QM3+4f0av Kxg5t6/ZadQifQ1+gKGpVGzXT8yhZQM8WGgVskasFBumT+blrQL90IoAOarmTnfPY/BqM 6YzL8WzwU+k+jgT98tOXpsk7HRcpSlC25GFia1F9nIp7NFS7f8UrbcKD6lKHVPziQ1ZDX /eLu3f+vfkH0+H0r8w841boaJxjxdrRECliuaD77qarJsOBlBtP3zqPKxwFYB0ehTn9jX 0yAgyCeeg/mv0uxJhExNFs5J5YOw7oXtI7MnP3KeOtXK6PjNDnKzgjHQpMNfP/PUuqQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794084754; h=Received:DKIM-Signature:Message-ID:From:Subject:Reply-To:To:Cc: In-Reply-To:References:Content-Type:Content-Transfer-Encoding:Date; bh=5/yAbnSlT796zo6/LxI3k/ssH+kUZUnYtFmT5XnFCro=; b=oiAgaZPtloTXcV41iEpNygFygtTT+ojL6en4njalcjil9MHba1ldBtG7ohc4alzuOkaG Kuwr5USO0EpE3T2YOc7tkBDoIlMemPKEtj6omHfgPwA/3FS3YXt//m/ad0wGy4UEgfDGb 6PJLoEWfrwmPGjMnWInbEUzjyOX1Xi9Onzljzap/AZUhnVjIJd2MXm8kWNHu4b+BaNLQH QWrX+wYkCC4mEbv+PK2vqG6fZwFSMc2FKuUWSJ17HCapM4L5G0cuc+kQex1GmWt+p62Ji nsPzmEwkktr0H8/e0PSwqLLR4X+yBsdQuFtduv+/40rhnBqbzq2NHYfImwuI/JRnV2g== 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=fW9X6j0t; 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 5B117606A326; Sat, 7 Nov 2026 20:52:34 +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=fW9X6j0t Received: by smtp.kernel.org (Postfix) with ESMTPSA id 967C51F0089F; Sat, 7 Nov 2026 20:52:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794084754; bh=5/yAbnSlT796zo6/LxI3k/ssH+kUZUnYtFmT5XnFCro=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=fW9X6j0t4J83x/radFZsJ5F/wcRHLrWKavr+S7Vr2Tl6udtBGD4FTMEyo281qdzbT xuHuBJNvCDn8S6NOmEvftEBZ3yCw7rAg19u1q96kk0JHcwtxeevlKZQBz7iKcpalxl rEPJhacWGxLQlE7FLGspZn9DmCyjZ7SSDXwB1fbq5XnvOm6V/R356z5n0J9Ym+cVYe Lbi8CKzokYJlV6wuETANH9A2rBzisWEYyOJTL4E2GDADjQWUg6k+aiA7vLncA0mlNW 1s9RJJ41CUacldI2rQc/ielsN8oBmVxIhIAJ4bkS9d+Wxdsn1sTXW3UETxaKTBjG38 X8ZdsldmBVFKA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 56/58] mtd: rawnand: nandsim: retain chip state across 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-56-0181ef0cc4c8@gmail.com> References: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com> <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-56-0181ef0cc4c8@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Sat, 07 Nov 2026 20:52:29 +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 no new issues in this patch, but noticed 1 related pre-existing issue it does not introduce: - [High] mtd: rawnand: nandsim: shift-out-of-bounds in ns_init_module() [1] -- commit c6f7998578d1a1c3b52ebeff5e9664c6e727de3e Author: James Hilliard mtd: rawnand: nandsim: retain chip state across removal This commit disconnects the NAND chip before freeing simulated media and retains the chip metadata until the final MTD reference is dropped. > diff --git a/drivers/mtd/nand/raw/nandsim.c b/drivers/mtd/nand/raw/nandsi= m.c > index fe968037f75a2..39fefd83cd850 100644 > --- a/drivers/mtd/nand/raw/nandsim.c > +++ b/drivers/mtd/nand/raw/nandsim.c [ ... ] > @@ -2376,6 +2382,10 @@ static int __init ns_init_module(void) > ret =3D ns_parse_badblocks(ns, nsmtd); > if (ret) > goto free_ns_object; > + ret =3D nand_set_release(chip, ns_chip_release); > + if (ret) > + goto free_ns_object; > + retained =3D true; [Severity: High] This problem wasn't introduced by this patch, but can setting the overridesize module parameter to 31 or higher trigger a shift-out-of-bounds [1] earlier in ns_init_module()? drivers/mtd/nand/raw/nandsim.c:ns_init_module() { ... if (overridesize) { uint64_t new_size =3D (uint64_t)nsmtd->erasesize << overridesize; ... if (new_size >> overridesize !=3D nsmtd->erasesize) { NS_ERR("overridesize is too big\n"); ret =3D -EINVAL; goto cleanup_nand; } /* N.B. This relies on nand_scan not doing anything with the size before = we change it */ nsmtd->size =3D new_size; memorg->eraseblocks_per_lun =3D 1 << overridesize; ... } Because new_size is 64-bit, the bounds check passes for overridesize values up to 63, while "1 << overridesize" shifts a 32-bit signed int. An overridesize of 31 or more overflows or invokes undefined behavior, leaving eraseblocks_per_lun zero or bogus and leading to a divide-by-zero during MTD I/O operations. [1] https://sashiko.dev/#/bug/linux-85e29763-d721-4e3b-9253-7b65f9dada3a --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261107-submit-raw= nand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com?part=3D56 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 9B2D7606A32C for ; Sat, 7 Nov 2026 20:52:38 +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=fARJ0Pfh; 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=fW9X6j0t 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=yiNzSEOGKDRgoLavX/7/X1z0V3xKg2dW/lesJNXFwE4=; b=fARJ0Pfhr+CRnzSFTGYOZqP0fY pkdRWj6Mby7flLNWEz2VnyvbPqdUbBYhk3YinI7ZyVBhHA4QNhuCZ/ET625r741OyTX5rZli2z8Uh fX1fWlmUkZYqKH+7klDIYiRehnzJxuDYjnpwCtyuwMHCeUpXsLNovTklLE2jaOgyx6WjZqkFx8ZRw ptmQfg8uPnlWgOtkoQSUGqrBid3CzruLdguUGg4ltNHi2kfqDFGFdU+pBA33Ndvnx1hrF4h8RD0/n IMraPQEwlOCGyxVppRl+i9zcH+hat++NXoAK0hr8QQK6pQ/5M58EBOZ69ovAMi2lhUW9UFtzvrzzW 5ADKnvBg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnOv-0000000FPKm-3Od7; Sat, 07 Nov 2026 20:52:37 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnOu-0000000FPKS-1wUZ; Sat, 07 Nov 2026 20:52:36 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 4E2C241632; Sat, 7 Nov 2026 20:52:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 967C51F0089F; Sat, 7 Nov 2026 20:52:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794084754; bh=5/yAbnSlT796zo6/LxI3k/ssH+kUZUnYtFmT5XnFCro=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=fW9X6j0t4J83x/radFZsJ5F/wcRHLrWKavr+S7Vr2Tl6udtBGD4FTMEyo281qdzbT xuHuBJNvCDn8S6NOmEvftEBZ3yCw7rAg19u1q96kk0JHcwtxeevlKZQBz7iKcpalxl rEPJhacWGxLQlE7FLGspZn9DmCyjZ7SSDXwB1fbq5XnvOm6V/R356z5n0J9Ym+cVYe Lbi8CKzokYJlV6wuETANH9A2rBzisWEYyOJTL4E2GDADjQWUg6k+aiA7vLncA0mlNW 1s9RJJ41CUacldI2rQc/ielsN8oBmVxIhIAJ4bkS9d+Wxdsn1sTXW3UETxaKTBjG38 X8ZdsldmBVFKA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 56/58] mtd: rawnand: nandsim: retain chip state across 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-56-0181ef0cc4c8@gmail.com> References: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com> <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-56-0181ef0cc4c8@gmail.com> Date: Sat, 07 Nov 2026 20:52:29 +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 bm8gbmV3IGlzc3VlcwppbiB0aGlzIHBhdGNoLCBidXQgbm90aWNlZCAxIHJlbGF0ZWQgcHJlLWV4 aXN0aW5nIGlzc3VlIGl0IGRvZXMgbm90CmludHJvZHVjZToKLSBbSGlnaF0gbXRkOiByYXduYW5k OiBuYW5kc2ltOiBzaGlmdC1vdXQtb2YtYm91bmRzIGluIG5zX2luaXRfbW9kdWxlKCkKICBbMV0K LS0KCmNvbW1pdCBjNmY3OTk4NTc4ZDFhMWMzYjUyZWJlZmY1ZTk2NjRjNmU3MjdkZTNlCkF1dGhv cjogSmFtZXMgSGlsbGlhcmQgPGphbWVzLmhpbGxpYXJkMUBnbWFpbC5jb20+CgptdGQ6IHJhd25h bmQ6IG5hbmRzaW06IHJldGFpbiBjaGlwIHN0YXRlIGFjcm9zcyByZW1vdmFsCgpUaGlzIGNvbW1p dCBkaXNjb25uZWN0cyB0aGUgTkFORCBjaGlwIGJlZm9yZSBmcmVlaW5nIHNpbXVsYXRlZCBtZWRp YSBhbmQKcmV0YWlucyB0aGUgY2hpcCBtZXRhZGF0YSB1bnRpbCB0aGUgZmluYWwgTVREIHJlZmVy ZW5jZSBpcyBkcm9wcGVkLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9tdGQvbmFuZC9yYXcvbmFu ZHNpbS5jIGIvZHJpdmVycy9tdGQvbmFuZC9yYXcvbmFuZHNpbS5jCj4gaW5kZXggZmU5NjgwMzdm NzVhMi4uMzlmZWZkODNjZDg1MCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL210ZC9uYW5kL3Jhdy9u YW5kc2ltLmMKPiArKysgYi9kcml2ZXJzL210ZC9uYW5kL3Jhdy9uYW5kc2ltLmMKWyAuLi4gXQo+ IEBAIC0yMzc2LDYgKzIzODIsMTAgQEAgc3RhdGljIGludCBfX2luaXQgbnNfaW5pdF9tb2R1bGUo dm9pZCkKPiAgCXJldCA9IG5zX3BhcnNlX2JhZGJsb2NrcyhucywgbnNtdGQpOwo+ICAJaWYgKHJl dCkKPiAgCQlnb3RvIGZyZWVfbnNfb2JqZWN0Owo+ICsJcmV0ID0gbmFuZF9zZXRfcmVsZWFzZShj aGlwLCBuc19jaGlwX3JlbGVhc2UpOwo+ICsJaWYgKHJldCkKPiArCQlnb3RvIGZyZWVfbnNfb2Jq ZWN0Owo+ICsJcmV0YWluZWQgPSB0cnVlOwoKW1NldmVyaXR5OiBIaWdoXQpUaGlzIHByb2JsZW0g d2Fzbid0IGludHJvZHVjZWQgYnkgdGhpcyBwYXRjaCwgYnV0IGNhbiBzZXR0aW5nIHRoZQpvdmVy cmlkZXNpemUgbW9kdWxlIHBhcmFtZXRlciB0byAzMSBvciBoaWdoZXIgdHJpZ2dlciBhCnNoaWZ0 LW91dC1vZi1ib3VuZHMgWzFdIGVhcmxpZXIgaW4gbnNfaW5pdF9tb2R1bGUoKT8KCmRyaXZlcnMv bXRkL25hbmQvcmF3L25hbmRzaW0uYzpuc19pbml0X21vZHVsZSgpIHsKCS4uLgoJaWYgKG92ZXJy aWRlc2l6ZSkgewoJCXVpbnQ2NF90IG5ld19zaXplID0gKHVpbnQ2NF90KW5zbXRkLT5lcmFzZXNp emUgPDwgb3ZlcnJpZGVzaXplOwoJCS4uLgoJCWlmIChuZXdfc2l6ZSA+PiBvdmVycmlkZXNpemUg IT0gbnNtdGQtPmVyYXNlc2l6ZSkgewoJCQlOU19FUlIoIm92ZXJyaWRlc2l6ZSBpcyB0b28gYmln XG4iKTsKCQkJcmV0ID0gLUVJTlZBTDsKCQkJZ290byBjbGVhbnVwX25hbmQ7CgkJfQoKCQkvKiBO LkIuIFRoaXMgcmVsaWVzIG9uIG5hbmRfc2NhbiBub3QgZG9pbmcgYW55dGhpbmcgd2l0aCB0aGUg c2l6ZSBiZWZvcmUgd2UgY2hhbmdlIGl0ICovCgkJbnNtdGQtPnNpemUgPSBuZXdfc2l6ZTsKCQlt ZW1vcmctPmVyYXNlYmxvY2tzX3Blcl9sdW4gPSAxIDw8IG92ZXJyaWRlc2l6ZTsKCS4uLgp9CgpC ZWNhdXNlIG5ld19zaXplIGlzIDY0LWJpdCwgdGhlIGJvdW5kcyBjaGVjayBwYXNzZXMgZm9yIG92 ZXJyaWRlc2l6ZQp2YWx1ZXMgdXAgdG8gNjMsIHdoaWxlICIxIDw8IG92ZXJyaWRlc2l6ZSIgc2hp ZnRzIGEgMzItYml0IHNpZ25lZCBpbnQuCkFuIG92ZXJyaWRlc2l6ZSBvZiAzMSBvciBtb3JlIG92 ZXJmbG93cyBvciBpbnZva2VzIHVuZGVmaW5lZCBiZWhhdmlvciwKbGVhdmluZyBlcmFzZWJsb2Nr c19wZXJfbHVuIHplcm8gb3IgYm9ndXMgYW5kIGxlYWRpbmcgdG8gYSBkaXZpZGUtYnktemVybwpk dXJpbmcgTVREIEkvTyBvcGVyYXRpb25zLgoKWzFdIGh0dHBzOi8vc2FzaGlrby5kZXYvIy9idWcv bGludXgtODVlMjk3NjMtZDcyMS00ZTNiLTkyNTMtN2I2NWY5ZGFkYTNhCgotLSAKU2FzaGlrbyBB SSByZXZpZXcgwrcgaHR0cHM6Ly9zYXNoaWtvLmRldi8jL3BhdGNoc2V0LzIwMjYxMTA3LXN1Ym1p dC1yYXduYW5kLXJldGFpbmVkLWhhbmRsZS1saWZldGltZS12MS12MS0wLTAxODFlZjBjYzRjOEBn bWFpbC5jb20/cGFydD01NgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCkxpbnV4IE1URCBkaXNjdXNzaW9uIG1haWxpbmcgbGlzdApodHRwOi8v bGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LW10ZC8K 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 8759A606A326 for ; Sat, 7 Nov 2026 20:52:38 +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=RmgxLLem; 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=fW9X6j0t 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=4KvakSNHICuZhCq9/yLLxsxR8s4E3iUnH0XfCoggWhQ=; b=RmgxLLemgpiVz4GCyuZw8at1uq kf+vDfthMP/4hWIdN2wcORo0l5qb23L3DCyaTBLbID2Smcead2TnTR7xyZCSuyin6lCTY3VcY9TnL DirTy7nGCFZbW32UuYQUCvs4tTnBOHqBY7Xk9CgW6t6zyvp707EcIG8PhIANHTJI+9ckq6lK+wYtb wBvZKVOg0omR7/T+ouZnyetl/QHvGXNgQhKAc3RVXtvd4fWz9V0utpbiFmTrJYHw9+L97d6f+2zeO 9ZeHcj3QruRJueenrPKF1Cs3fvyIBZzE4ycFnjTKH3xmcF2Q8Ze2QsLN1W3GrbHrsdk+cg/k+K6VD YqW4IAQQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnOv-0000000FPKi-37n5; Sat, 07 Nov 2026 20:52:37 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPnOu-0000000FPKS-1wUZ; Sat, 07 Nov 2026 20:52:36 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 4E2C241632; Sat, 7 Nov 2026 20:52:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 967C51F0089F; Sat, 7 Nov 2026 20:52:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794084754; bh=5/yAbnSlT796zo6/LxI3k/ssH+kUZUnYtFmT5XnFCro=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=fW9X6j0t4J83x/radFZsJ5F/wcRHLrWKavr+S7Vr2Tl6udtBGD4FTMEyo281qdzbT xuHuBJNvCDn8S6NOmEvftEBZ3yCw7rAg19u1q96kk0JHcwtxeevlKZQBz7iKcpalxl rEPJhacWGxLQlE7FLGspZn9DmCyjZ7SSDXwB1fbq5XnvOm6V/R356z5n0J9Ym+cVYe Lbi8CKzokYJlV6wuETANH9A2rBzisWEYyOJTL4E2GDADjQWUg6k+aiA7vLncA0mlNW 1s9RJJ41CUacldI2rQc/ielsN8oBmVxIhIAJ4bkS9d+Wxdsn1sTXW3UETxaKTBjG38 X8ZdsldmBVFKA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 56/58] mtd: rawnand: nandsim: retain chip state across 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-56-0181ef0cc4c8@gmail.com> References: <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-0-0181ef0cc4c8@gmail.com> <20261107-submit-rawnand-retained-handle-lifetime-v1-v1-56-0181ef0cc4c8@gmail.com> Date: Sat, 07 Nov 2026 20:52:29 +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 bm8gbmV3IGlzc3VlcwppbiB0aGlzIHBhdGNoLCBidXQgbm90aWNlZCAxIHJlbGF0ZWQgcHJlLWV4 aXN0aW5nIGlzc3VlIGl0IGRvZXMgbm90CmludHJvZHVjZToKLSBbSGlnaF0gbXRkOiByYXduYW5k OiBuYW5kc2ltOiBzaGlmdC1vdXQtb2YtYm91bmRzIGluIG5zX2luaXRfbW9kdWxlKCkKICBbMV0K LS0KCmNvbW1pdCBjNmY3OTk4NTc4ZDFhMWMzYjUyZWJlZmY1ZTk2NjRjNmU3MjdkZTNlCkF1dGhv cjogSmFtZXMgSGlsbGlhcmQgPGphbWVzLmhpbGxpYXJkMUBnbWFpbC5jb20+CgptdGQ6IHJhd25h bmQ6IG5hbmRzaW06IHJldGFpbiBjaGlwIHN0YXRlIGFjcm9zcyByZW1vdmFsCgpUaGlzIGNvbW1p dCBkaXNjb25uZWN0cyB0aGUgTkFORCBjaGlwIGJlZm9yZSBmcmVlaW5nIHNpbXVsYXRlZCBtZWRp YSBhbmQKcmV0YWlucyB0aGUgY2hpcCBtZXRhZGF0YSB1bnRpbCB0aGUgZmluYWwgTVREIHJlZmVy ZW5jZSBpcyBkcm9wcGVkLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9tdGQvbmFuZC9yYXcvbmFu ZHNpbS5jIGIvZHJpdmVycy9tdGQvbmFuZC9yYXcvbmFuZHNpbS5jCj4gaW5kZXggZmU5NjgwMzdm NzVhMi4uMzlmZWZkODNjZDg1MCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL210ZC9uYW5kL3Jhdy9u YW5kc2ltLmMKPiArKysgYi9kcml2ZXJzL210ZC9uYW5kL3Jhdy9uYW5kc2ltLmMKWyAuLi4gXQo+ IEBAIC0yMzc2LDYgKzIzODIsMTAgQEAgc3RhdGljIGludCBfX2luaXQgbnNfaW5pdF9tb2R1bGUo dm9pZCkKPiAgCXJldCA9IG5zX3BhcnNlX2JhZGJsb2NrcyhucywgbnNtdGQpOwo+ICAJaWYgKHJl dCkKPiAgCQlnb3RvIGZyZWVfbnNfb2JqZWN0Owo+ICsJcmV0ID0gbmFuZF9zZXRfcmVsZWFzZShj aGlwLCBuc19jaGlwX3JlbGVhc2UpOwo+ICsJaWYgKHJldCkKPiArCQlnb3RvIGZyZWVfbnNfb2Jq ZWN0Owo+ICsJcmV0YWluZWQgPSB0cnVlOwoKW1NldmVyaXR5OiBIaWdoXQpUaGlzIHByb2JsZW0g d2Fzbid0IGludHJvZHVjZWQgYnkgdGhpcyBwYXRjaCwgYnV0IGNhbiBzZXR0aW5nIHRoZQpvdmVy cmlkZXNpemUgbW9kdWxlIHBhcmFtZXRlciB0byAzMSBvciBoaWdoZXIgdHJpZ2dlciBhCnNoaWZ0 LW91dC1vZi1ib3VuZHMgWzFdIGVhcmxpZXIgaW4gbnNfaW5pdF9tb2R1bGUoKT8KCmRyaXZlcnMv bXRkL25hbmQvcmF3L25hbmRzaW0uYzpuc19pbml0X21vZHVsZSgpIHsKCS4uLgoJaWYgKG92ZXJy aWRlc2l6ZSkgewoJCXVpbnQ2NF90IG5ld19zaXplID0gKHVpbnQ2NF90KW5zbXRkLT5lcmFzZXNp emUgPDwgb3ZlcnJpZGVzaXplOwoJCS4uLgoJCWlmIChuZXdfc2l6ZSA+PiBvdmVycmlkZXNpemUg IT0gbnNtdGQtPmVyYXNlc2l6ZSkgewoJCQlOU19FUlIoIm92ZXJyaWRlc2l6ZSBpcyB0b28gYmln XG4iKTsKCQkJcmV0ID0gLUVJTlZBTDsKCQkJZ290byBjbGVhbnVwX25hbmQ7CgkJfQoKCQkvKiBO LkIuIFRoaXMgcmVsaWVzIG9uIG5hbmRfc2NhbiBub3QgZG9pbmcgYW55dGhpbmcgd2l0aCB0aGUg c2l6ZSBiZWZvcmUgd2UgY2hhbmdlIGl0ICovCgkJbnNtdGQtPnNpemUgPSBuZXdfc2l6ZTsKCQlt ZW1vcmctPmVyYXNlYmxvY2tzX3Blcl9sdW4gPSAxIDw8IG92ZXJyaWRlc2l6ZTsKCS4uLgp9CgpC ZWNhdXNlIG5ld19zaXplIGlzIDY0LWJpdCwgdGhlIGJvdW5kcyBjaGVjayBwYXNzZXMgZm9yIG92 ZXJyaWRlc2l6ZQp2YWx1ZXMgdXAgdG8gNjMsIHdoaWxlICIxIDw8IG92ZXJyaWRlc2l6ZSIgc2hp ZnRzIGEgMzItYml0IHNpZ25lZCBpbnQuCkFuIG92ZXJyaWRlc2l6ZSBvZiAzMSBvciBtb3JlIG92 ZXJmbG93cyBvciBpbnZva2VzIHVuZGVmaW5lZCBiZWhhdmlvciwKbGVhdmluZyBlcmFzZWJsb2Nr c19wZXJfbHVuIHplcm8gb3IgYm9ndXMgYW5kIGxlYWRpbmcgdG8gYSBkaXZpZGUtYnktemVybwpk dXJpbmcgTVREIEkvTyBvcGVyYXRpb25zLgoKWzFdIGh0dHBzOi8vc2FzaGlrby5kZXYvIy9idWcv bGludXgtODVlMjk3NjMtZDcyMS00ZTNiLTkyNTMtN2I2NWY5ZGFkYTNhCgotLSAKU2FzaGlrbyBB SSByZXZpZXcgwrcgaHR0cHM6Ly9zYXNoaWtvLmRldi8jL3BhdGNoc2V0LzIwMjYxMTA3LXN1Ym1p dC1yYXduYW5kLXJldGFpbmVkLWhhbmRsZS1saWZldGltZS12MS12MS0wLTAxODFlZjBjYzRjOEBn bWFpbC5jb20/cGFydD01NgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX18KbGludXgtYW1sb2dpYyBtYWlsaW5nIGxpc3QKbGludXgtYW1sb2dpY0BsaXN0cy5p bmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8v bGludXgtYW1sb2dpYwo=