From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 89E97D59F40 for ; Wed, 6 Nov 2024 15:36:16 +0000 (UTC) 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:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=OqSAYj88/d1k3HUGfIqiCsE4HRsvfiJJ7CSzt3kxXDc=; b=oU5xtZx0e6CP2t KtuwnsUvoRR/L8vB6rYI8X5Ddj7Z0QqtvOfNjxQYQ+dsIbjEFCloFfB6GCUVnM5St1Ubrq9pU7g5q rTxc61IhO9AM8ft3Kd54dhp76Jy/Ifv/bXoARKSgypgIn9Q3V3veYyuBvyWu3n6t7xD2k6maHAVPg iMgAUgbggmKqeFlGaIxHqjnfIF8LNv0GTdH8tUf326FWro8d9W8yJtSVnnapYwNUtDXMjR1iSAeOj FQ5+WndISxWIuIikA8VvUemSRArylFdz5COaT+3wPgd++QacbmMzcFa+YmINdfYqF/lsZ8t3RpjLd noTRXBU/Knl5KCbxXlHw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t8i4p-00000003qHF-2VQm; Wed, 06 Nov 2024 15:36:11 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t8i4P-00000003qBk-3pfv for linux-mtd@lists.infradead.org; Wed, 06 Nov 2024 15:35:48 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id A297B240007; Wed, 6 Nov 2024 15:35:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1730907340; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=u0O1DZej7ntn5vlTTkeOAaf/z20TWgE994IzTCP97PI=; b=fK/QW1FCogdbqrToYB8N3GWm4ZqL1Zprc0NhwciUGNy+i+X97EREKODC6FfqKFJBmRoU30 u36cU4dfvTnv/uHDR8xke1mBANmSr7IDVVIR39I8AoTZhGkzuq5LptP7ovk58yDtaZddqH TIV2a0IbXGSg4mWVb5qT85gP95qAW9s/KdaNt2hqmW6axv9Szhr4srC8M+kHqrjr09xUiP Opvivl4D3bdn7CM4qNtcB47gL6CD/TX8EY+GL/xBmpQOfCguWnzP0X/N8uKqJAz3jqryL2 EJYRwHNVhc3+ImBs1PU3YH20OhE5X2zVbLH8HfWMZqKBcM13KBci/mX1Dep2JQ== From: Miquel Raynal To: Chris Packham Cc: "linux-mtd@lists.infradead.org" , "linux-spi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: ubifs_recover_master_node: failed to recover master node In-Reply-To: <7eaf332e-9439-4d4c-a2ea-d963e41f44f2@alliedtelesis.co.nz> (Chris Packham's message of "Tue, 29 Oct 2024 13:37:31 +1300") References: <7eaf332e-9439-4d4c-a2ea-d963e41f44f2@alliedtelesis.co.nz> User-Agent: mu4e 1.12.1; emacs 29.4 Date: Wed, 06 Nov 2024 16:35:39 +0100 Message-ID: <87jzdgjrxw.fsf@bootlin.com> MIME-Version: 1.0 X-GND-Sasl: miquel.raynal@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241106_073547_466179_FC22F1FA X-CRM114-Status: GOOD ( 15.12 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org CkhpIENocmlzLAoKT24gMjkvMTAvMjAyNCBhdCAxMzozNzozMSArMTMsIENocmlzIFBhY2toYW0g PGNocmlzLnBhY2toYW1AYWxsaWVkdGVsZXNpcy5jby5uej4gd3JvdGU6Cgo+IEhpLAo+Cj4gSSBy ZWNlbnRseSBhZGRlZCBzdXBwb3J0IGZvciB0aGUgU1BJLU5BTkQgY29udHJvbGxlciBvbiB0aGUg UlRMOTMwMkMgU29DWzFdLiBJIGRpZCBtb3N0IG9mIHRoZSB3b3JrIGFnYWluc3QgTGludXggNi4x MQo+IGFuZCBpdCdzIHdvcmtpbmcgZmluZSB0aGVyZS4gSSByZWNlbnRseSByZWJhc2VkIGFnYWlu c3QgdGhlIHRpcCBvZiBMaW51cydzIHRyZWUgKDYuMTItcmM1KSBhbmQgZm91bmQgSSB3YXMgZ2V0 dGluZyB1Ymlmcwo+IGVycm9ycyB3aGVuIG1vdW50aW5nOgo+Cj4gWyAgICAxLjI1NTE5MV0gc3Bp LW5hbmQgc3BpMS4wOiBNYWNyb25peCBTUEkgTkFORCB3YXMgZm91bmQuCj4gWyAgICAxLjI2MTI4 M10gc3BpLW5hbmQgc3BpMS4wOiAyNTYgTWlCLCBibG9jayBzaXplOiAxMjggS2lCLCBwYWdlIHNp emU6IDIwNDgsIE9PQiBzaXplOiA2NAo+IFsgICAgMS4yNzExMzRdIDIgZml4ZWQtcGFydGl0aW9u cyBwYXJ0aXRpb25zIGZvdW5kIG9uIE1URCBkZXZpY2Ugc3BpMS4wCj4gWyAgICAxLjI3ODI0N10g Q3JlYXRpbmcgMiBNVEQgcGFydGl0aW9ucyBvbiAic3BpMS4wIjoKPiBbICAgIDEuMjgzNjMxXSAw eDAwMDAwMDAwMDAwMC0weDAwMDAwZjAwMDAwMCA6ICJ1c2VyIgo+IFsgICAyMC40ODExMDhdIDB4 MDAwMDBmMDAwMDAwLTB4MDAwMDEwMDAwMDAwIDogIlJlc2VydmVkIgo+IFsgICA3Mi4yNDAzNDdd IHViaTA6IHNjYW5uaW5nIGlzIGZpbmlzaGVkCj4gWyAgIDcyLjI3MDU3N10gdWJpMDogYXR0YWNo ZWQgbXRkMyAobmFtZSAidXNlciIsIHNpemUgMjQwIE1pQikKPiBbICAgNzIuMjc2ODE1XSB1Ymkw OiBQRUIgc2l6ZTogMTMxMDcyIGJ5dGVzICgxMjggS2lCKSwgTEVCIHNpemU6IDEyNjk3NiBieXRl cwo+IFsgICA3Mi4yODQ1MzddIHViaTA6IG1pbi4vbWF4LiBJL08gdW5pdCBzaXplczogMjA0OC8y MDQ4LCBzdWItcGFnZSBzaXplIDIwNDgKPiBbICAgNzIuMjkyMTMyXSB1YmkwOiBWSUQgaGVhZGVy IG9mZnNldDogMjA0OCAoYWxpZ25lZCAyMDQ4KSwgZGF0YSBvZmZzZXQ6IDQwOTYKPiBbICAgNzIu Mjk5ODg1XSB1YmkwOiBnb29kIFBFQnM6IDE5MjAsIGJhZCBQRUJzOiAwLCBjb3JydXB0ZWQgUEVC czogMAo+IFsgICA3Mi4zMDY2ODldIHViaTA6IHVzZXIgdm9sdW1lOiAxLCBpbnRlcm5hbCB2b2x1 bWVzOiAxLCBtYXguIHZvbHVtZXMgY291bnQ6IDEyOAo+IFsgICA3Mi4zMTQ3NDddIHViaTA6IG1h eC9tZWFuIGVyYXNlIGNvdW50ZXI6IDEvMCwgV0wgdGhyZXNob2xkOiA0MDk2LCBpbWFnZSBzZXF1 ZW5jZSBudW1iZXI6IDI1MjY0MjIzMAo+IFsgICA3Mi4zMjQ4NTBdIHViaTA6IGF2YWlsYWJsZSBQ RUJzOiAwLCB0b3RhbCByZXNlcnZlZCBQRUJzOiAxOTIwLCBQRUJzIHJlc2VydmVkIGZvciBiYWQg UEVCIGhhbmRsaW5nOiA0MAo+IFsgICA3Mi4zNzAxMjNdIHViaTA6IGJhY2tncm91bmQgdGhyZWFk ICJ1YmlfYmd0MGQiIHN0YXJ0ZWQsIFBJRCAxNDEKPiBbICAgNzIuNDcwNzQwXSBVQklGUyAodWJp MDowKTogTW91bnRpbmcgaW4gdW5hdXRoZW50aWNhdGVkIG1vZGUKPiBbICAgNzIuNDkwMjQ2XSBV QklGUyAodWJpMDowKTogYmFja2dyb3VuZCB0aHJlYWQgInViaWZzX2JndDBfMCIgc3RhcnRlZCwg UElEIDE0NAo+IFsgICA3Mi41MjgyNzJdIFVCSUZTIGVycm9yICh1YmkwOjAgcGlkIDE0Myk6IHVi aWZzX3JlY292ZXJfbWFzdGVyX25vZGU6IGZhaWxlZCB0byByZWNvdmVyIG1hc3RlciBub2RlCj4g WyAgIDcyLjU1MDEyMl0gVUJJRlMgKHViaTA6MCk6IGJhY2tncm91bmQgdGhyZWFkICJ1Ymlmc19i Z3QwXzAiIHN0b3BzCj4gWyAgIDcyLjcxMDcyMF0gVUJJRlMgKHViaTA6MCk6IE1vdW50aW5nIGlu IHVuYXV0aGVudGljYXRlZCBtb2RlCj4gWyAgIDcyLjcxNzQ0N10gVUJJRlMgKHViaTA6MCk6IGJh Y2tncm91bmQgdGhyZWFkICJ1Ymlmc19iZ3QwXzAiIHN0YXJ0ZWQsIFBJRCAxNDkKPiBbICAgNzIu Nzc3NjAyXSBVQklGUyBlcnJvciAodWJpMDowIHBpZCAxNDgpOiB1Ymlmc19yZWNvdmVyX21hc3Rl cl9ub2RlOiBmYWlsZWQgdG8gcmVjb3ZlciBtYXN0ZXIgbm9kZQo+IFsgICA3Mi43ODc3OTJdIFVC SUZTICh1YmkwOjApOiBiYWNrZ3JvdW5kIHRocmVhZCAidWJpZnNfYmd0MF8wIiBzdG9wcwo+Cj4g RnVsbCBkbWVzZyBvdXRwdXQgaXMgYXRbMl0KPgo+IGdpdCBiaXNlY3QgbGVhZCBtZSB0byBjb21t aXQgMTE4MTM4NTc4NjRmICgibXRkOiBzcGktbmFuZDogbWFjcm9uaXg6IENvbnRpbnVvdXMgcmVh ZCBzdXBwb3J0IikuIFJldmVydGluZyB0aGUgYmxhbWVkCj4gY29tbWl0IGZyb20gNi4xMi1yYzUg c2VlbXMgdG8gYXZvaWQgdGhlIHByb2JsZW0uIFRoZSBmbGFzaCBjaGlwIG9uIG15IGJvYXJkIGlz IGEgTVgzMExGMkcyOEFELVRJLiBJJ20gbm90IHN1cmUgaWYgdGhlcmUKPiBpcyBhIHByb2JsZW0g d2l0aCAxMTgxMzg1Nzg2NGYgb3Igd2l0aCBteSBzcGktbWVtIGRyaXZlciB0aGF0IGlzCj4gZXhw b3NlZCBhZnRlciBzdXBwb3J0IGZvciBjb250aW51b3VzIHJlYWQgaXMgZW5hYmxlZC4KCkNyYXAu IEkgaGFkIGEgbG9vaywgYW5kIFRCSCBJIGRvbid0IGtub3cuIFRoZSBvbmx5IHRoaW5nIEkgc2Vl IGluIHlvdXIKZHJpdmVyIG1pZ2h0IGJlIHRoZSBETUEgdnMgUElPIGNob2ljZS4gQ291bGQgeW91 IHRyeSB0byBhbHdheXMgcmV0dXJuCmZhbHNlIGZyb20gcnRsX3NuYW5kX2RtYV9vcCgpPwoKSG93 ZXZlciB5b3Ugc2F5IHlvdSdyZSB1c2luZyBhbiBNWDMwKiBkZXZpY2UsIHRoaXMgaXMgYSByYXcg TkFORCBjaGlwLApTUEktTkFORCBjaGlwcyBhcmUgSSBiZWxpZXZlIHN0YXJ0aW5nIHdpdGggTVgz NSogaW4gdGhlaXIgSURzLCBubz8KClRoYW5rcywKTWlxdcOobAoKX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkxpbnV4IE1URCBkaXNjdXNzaW9u IG1haWxpbmcgbGlzdApodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2xpbnV4LW10ZC8K From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6F10A16CD29; Wed, 6 Nov 2024 15:35:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.193 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730907345; cv=none; b=quZ4oykibC9amb1CfnHdgYIVdLm9FWdZpcp53ObjToXivT8Jq8iTXM371Y7lmRcWtEx7/LFjpv13D14qbHSXKLg+phuqVlRt034O9HHAYfvhewP9F3e2khxJDxdTn+KHkbd9bk7B1iH+3GVQeJcAmMyekjUuUH0v/3dtkMcmVfQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730907345; c=relaxed/simple; bh=7qHzpB3TG+C8tGAFsrcXsTV+YPrG0APq81/k9Dc8bHw=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=jwH98cim8WdUuGfsXvFyjq717P4zfdcQTwvfPEXtS35Idd57Otr4djQQtzKTnL51u8W/a0uSaq6LnizaSzTOvBvfYsZjgQCtD4i6mUQV5XHmA4h0JLokrl5DBsv8jLxMCqBFnzEgiQDhu6J9Vd+89otAKd21dzAlpVbI8zlFW3o= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=fK/QW1FC; arc=none smtp.client-ip=217.70.183.193 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="fK/QW1FC" Received: by mail.gandi.net (Postfix) with ESMTPSA id A297B240007; Wed, 6 Nov 2024 15:35:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1730907340; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=u0O1DZej7ntn5vlTTkeOAaf/z20TWgE994IzTCP97PI=; b=fK/QW1FCogdbqrToYB8N3GWm4ZqL1Zprc0NhwciUGNy+i+X97EREKODC6FfqKFJBmRoU30 u36cU4dfvTnv/uHDR8xke1mBANmSr7IDVVIR39I8AoTZhGkzuq5LptP7ovk58yDtaZddqH TIV2a0IbXGSg4mWVb5qT85gP95qAW9s/KdaNt2hqmW6axv9Szhr4srC8M+kHqrjr09xUiP Opvivl4D3bdn7CM4qNtcB47gL6CD/TX8EY+GL/xBmpQOfCguWnzP0X/N8uKqJAz3jqryL2 EJYRwHNVhc3+ImBs1PU3YH20OhE5X2zVbLH8HfWMZqKBcM13KBci/mX1Dep2JQ== From: Miquel Raynal To: Chris Packham Cc: "linux-mtd@lists.infradead.org" , "linux-spi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: ubifs_recover_master_node: failed to recover master node In-Reply-To: <7eaf332e-9439-4d4c-a2ea-d963e41f44f2@alliedtelesis.co.nz> (Chris Packham's message of "Tue, 29 Oct 2024 13:37:31 +1300") References: <7eaf332e-9439-4d4c-a2ea-d963e41f44f2@alliedtelesis.co.nz> User-Agent: mu4e 1.12.1; emacs 29.4 Date: Wed, 06 Nov 2024 16:35:39 +0100 Message-ID: <87jzdgjrxw.fsf@bootlin.com> Precedence: bulk X-Mailing-List: linux-spi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: miquel.raynal@bootlin.com Hi Chris, On 29/10/2024 at 13:37:31 +13, Chris Packham wrote: > Hi, > > I recently added support for the SPI-NAND controller on the RTL9302C SoC[= 1]. I did most of the work against Linux 6.11 > and it's working fine there. I recently rebased against the tip of Linus'= s tree (6.12-rc5) and found I was getting ubifs > errors when mounting: > > [ 1.255191] spi-nand spi1.0: Macronix SPI NAND was found. > [ 1.261283] spi-nand spi1.0: 256 MiB, block size: 128 KiB, page size: = 2048, OOB size: 64 > [ 1.271134] 2 fixed-partitions partitions found on MTD device spi1.0 > [ 1.278247] Creating 2 MTD partitions on "spi1.0": > [ 1.283631] 0x000000000000-0x00000f000000 : "user" > [ 20.481108] 0x00000f000000-0x000010000000 : "Reserved" > [ 72.240347] ubi0: scanning is finished > [ 72.270577] ubi0: attached mtd3 (name "user", size 240 MiB) > [ 72.276815] ubi0: PEB size: 131072 bytes (128 KiB), LEB size: 126976 b= ytes > [ 72.284537] ubi0: min./max. I/O unit sizes: 2048/2048, sub-page size 2= 048 > [ 72.292132] ubi0: VID header offset: 2048 (aligned 2048), data offset:= 4096 > [ 72.299885] ubi0: good PEBs: 1920, bad PEBs: 0, corrupted PEBs: 0 > [ 72.306689] ubi0: user volume: 1, internal volumes: 1, max. volumes co= unt: 128 > [ 72.314747] ubi0: max/mean erase counter: 1/0, WL threshold: 4096, ima= ge sequence number: 252642230 > [ 72.324850] ubi0: available PEBs: 0, total reserved PEBs: 1920, PEBs r= eserved for bad PEB handling: 40 > [ 72.370123] ubi0: background thread "ubi_bgt0d" started, PID 141 > [ 72.470740] UBIFS (ubi0:0): Mounting in unauthenticated mode > [ 72.490246] UBIFS (ubi0:0): background thread "ubifs_bgt0_0" started, = PID 144 > [ 72.528272] UBIFS error (ubi0:0 pid 143): ubifs_recover_master_node: f= ailed to recover master node > [ 72.550122] UBIFS (ubi0:0): background thread "ubifs_bgt0_0" stops > [ 72.710720] UBIFS (ubi0:0): Mounting in unauthenticated mode > [ 72.717447] UBIFS (ubi0:0): background thread "ubifs_bgt0_0" started, = PID 149 > [ 72.777602] UBIFS error (ubi0:0 pid 148): ubifs_recover_master_node: f= ailed to recover master node > [ 72.787792] UBIFS (ubi0:0): background thread "ubifs_bgt0_0" stops > > Full dmesg output is at[2] > > git bisect lead me to commit 11813857864f ("mtd: spi-nand: macronix: Cont= inuous read support"). Reverting the blamed > commit from 6.12-rc5 seems to avoid the problem. The flash chip on my boa= rd is a MX30LF2G28AD-TI. I'm not sure if there > is a problem with 11813857864f or with my spi-mem driver that is > exposed after support for continuous read is enabled. Crap. I had a look, and TBH I don't know. The only thing I see in your driver might be the DMA vs PIO choice. Could you try to always return false from rtl_snand_dma_op()? However you say you're using an MX30* device, this is a raw NAND chip, SPI-NAND chips are I believe starting with MX35* in their IDs, no? Thanks, Miqu=C3=A8l