lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <c2cdfba1-afcc-4a77-8890-7da49c4b73c2@pengutronix.de>
Date: Mon, 14 Oct 2024 14:43:39 +0200
From: Stefan Kerkmann <s.kerkmann@...gutronix.de>
To: regressions@...ts.linux.dev
Cc: Dhruva Gole <d-gole@...com>, Yoshitaka Ikeda <ikeda@...int.co.jp>,
 Mark Brown <broonie@...nel.org>, linux-spi@...r.kernel.org,
 linux-kernel@...r.kernel.org, kernel-team@...gutronix.de
Subject: [REGRESSION] spi: cadence-quadspi: STIG mode results in timeouts for
 Micron MT25QL01 flash

Hello,

I have run into the same regression when probing a Micron MT25QL01 SPI-NOR flash
on a Intel CycloneV platform (socfpga) as Yoshitaka Ikeda[1]. The regression
manifests in read timeouts. Bisecting the issue tracked it down to commit
"d403fb6e76bf8 spi: cadence-quadspi: use STIG mode for small reads". Reverting
the commit resolves the issue on v6.12-rc3. There are no custom patches applied,
except for the debugging output mentioned in the linked thread.

The good case is as follows:

```
[    1.063171] **********spi_mem_op dump**************
[    1.063183] addr: nbytes:0x0 , buswidth 0x0, dtr 0x0, val 0x0
[    1.068093] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x9F
[    1.073837] data: nbytes:0x6 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.079823] ***************************************
[    1.086068] **********spi_mem_op dump**************
[    1.090931] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x0
[    1.095815] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    1.101543] data: nbytes:0x10 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.107547] ***************************************
[    1.113835] **********spi_mem_op dump**************
[    1.118695] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x10
[    1.123573] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    1.129395] data: nbytes:0x8 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.135392] ***************************************
[    1.141592] **********spi_mem_op dump**************
[    1.146471] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x0
[    1.151335] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    1.157072] data: nbytes:0x88 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.163058] ***************************************
[    1.169341] **********spi_mem_op dump**************
[    1.174219] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x30
[    1.179082] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    1.184904] data: nbytes:0x40 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.190890] ***************************************
[    1.197183] **********spi_mem_op dump**************
[    1.202045] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x80
[    1.206925] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    1.212740] data: nbytes:0x8 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.218736] ***************************************
[    1.224916] **********spi_mem_op dump**************
[    1.229776] addr: nbytes:0x0 , buswidth 0x0, dtr 0x0, val 0x0
[    1.234649] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x6
[    1.240376] data: nbytes:0x0 , buswidth 0x0, dtr 0x0, data dir 0x0
[    1.246283] ***************************************
[    1.252443] **********spi_mem_op dump**************
[    1.257314] addr: nbytes:0x0 , buswidth 0x0, dtr 0x0, val 0x0
[    1.262176] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0xB7
[    1.267917] data: nbytes:0x0 , buswidth 0x0, dtr 0x0, data dir 0x0
[    1.273914] ***************************************
[    1.280074] **********spi_mem_op dump**************
[    1.284946] addr: nbytes:0x0 , buswidth 0x0, dtr 0x0, val 0x0
[    1.289809] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x4
[    1.295544] data: nbytes:0x0 , buswidth 0x0, dtr 0x0, data dir 0x0
[    1.301444] ***************************************
[    1.307694] 5 fixed-partitions partitions found on MTD device ff705000.spi.0
[    1.319593] Creating 5 MTD partitions on "ff705000.spi.0":
[    1.325080] 0x000000000000-0x000000040000 : "preloader"
[    1.331825] 0x000000040000-0x0000000c0000 : "bootloader"
[    1.338645] 0x0000000c0000-0x0000000e0000 : "barebox-environment"
[    1.346299] 0x0000000e0000-0x000000110000 : "state-storage"
[    1.353319] 0x000000110000-0x000008000000 : "ubi"
```

With the STIG short read optimization enabled, the read timeouts occur:

```
[    0.931469] **********spi_mem_op dump**************
[    0.931482] addr: nbytes:0x0 , buswidth 0x0, dtr 0x0, val 0x0
[    0.936398] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x9F
[    0.942129] data: nbytes:0x6 , buswidth 0x1, dtr 0x0, data dir 0x1
[    0.948132] ***************************************
[    0.954369] **********spi_mem_op dump**************
[    0.959233] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x0
[    0.964117] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    0.969845] data: nbytes:0x10 , buswidth 0x1, dtr 0x0, data dir 0x1
[    0.975844] ***************************************
[    1.482104] cadence-qspi ff705000.spi: Flash command execution timed out.
[    1.493754] **********spi_mem_op dump**************
[    1.493759] addr: nbytes:0x3 , buswidth 0x1, dtr 0x0, val 0x10
[    1.498623] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x5A
[    1.504451] data: nbytes:0x8 , buswidth 0x1, dtr 0x0, data dir 0x1
[    1.510438] ***************************************
[    1.516611] spi-nor spi0.0: operation failed with -110
[    2.026639] cadence-qspi ff705000.spi: Flash command execution timed out.
[    2.033430] **********spi_mem_op dump**************
[    2.033437] addr: nbytes:0x0 , buswidth 0x0, dtr 0x0, val 0x0
[    2.038300] cmd: nbytes:0x1 , buswidth 0x1, dtr 0x0, opcode 0x6
[    2.044041] data: nbytes:0x0 , buswidth 0x0, dtr 0x0, data dir 0x0
[    2.049942] ***************************************
[    2.056112] spi-nor spi0.0: operation failed with -110
[    2.066110] spi-nor spi0.0: probe with driver spi-nor failed with error -110
```

The DT node for the flash is as follows:

```
&qspi {
	status = "okay";

	flash0: flash@0 {
		#address-cells = <1>;
		#size-cells = <1>;
		/* Micron MT25QL01 */
		compatible = "n25q00", "jedec,spi-nor";
		reg = <0>;	/* chip select */
		spi-max-frequency = <100000000>;
		m25p,fast-read;
		cdns,page-size = <256>;
		cdns,block-size = <16>;   /* 2^16, 64KB */
		cdns,read-delay = <4>;    /* delay value in read data capture register */
		cdns,tshsl-ns = <50>;
		cdns,tsd2d-ns = <50>;
		cdns,tchsh-ns = <4>;
		cdns,tslch-ns = <4>;
	};
};
```

Regards,
Stefan Kerkmann

[1]:
https://lore.kernel.org/lkml/OSZPR01MB70048CE259A3D63C4179199A8B659@OSZPR01MB7004.jpnprd01.prod.outlook.com/#t

#regzbot introduced: d403fb6e76bf854ef0f7d84e797e51b9494788e0

-- 
Pengutronix e.K.                       | Stefan Kerkmann             |
Steuerwalder Str. 21                   | https://www.pengutronix.de/ |
31137 Hildesheim, Germany              | Phone: +49-5121-206917-128  |
Amtsgericht Hildesheim, HRA 2686       | Fax:   +49-5121-206917-9    |


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ