* async probe: boot crash with v5.10.188 on STM32MP157
[not found] ` <AS8PR10MB4712C2960C439EF76E25DA98EEF7A@AS8PR10MB4712.EURPRD10.PROD.OUTLOOK.COM>
@ 2023-09-18 12:26 ` Antonio Borneo
0 siblings, 0 replies; only message in thread
From: Antonio Borneo @ 2023-09-18 12:26 UTC (permalink / raw)
To: Saravana Kannan; +Cc: Linus Walleij, linux-kernel, Alexandre TORGUE
Hi Saravana,
while devlink and async probe looks ok in v6.6-rc1, I get randomly boot
crash on STM32MP157 with kernel v5.10.188 after the upstream commit
3108eb2e8aa7 ("mmc: mmci: Set PROBE_PREFER_ASYNCHRONOUS") got backported.
In v6.6-rc1, really_probe() checks for probe dependencies with
device_links_check_suppliers(). In older kernels this part is missing.
In the specific case of STM32MP157, the driver mmci can get async-probed
while the probe of pinctrl/stm32/pinctrl-stm32.c has already registered
the pinctrl but has not added any gpiochip yet.
This causes pinmux_ops::set_mux() to fail and thus mmci probe fails too.
Boot log contains:
[2.205476] stm32mp157-pinctrl soc:pin-controller@50002000: No gpio range defined.
[2.205515] mmci-pl18x 58005000.sdmmc: Error applying setting, reverse things back
[2.205828] mmci-pl18x: probe of 58005000.sdmmc failed with error -22
[2.210434] stm32mp157-pinctrl soc:pin-controller@50002000: GPIOA bank added
From a quick code check, also the LTS v5.15 and v6.1 looks impacted, but
I have not tested them, so far, due to the difficulties to replicate the
crash.
Is there any mandatory upstream patch that got lost in the backport process?
Or is there something incorrect to be fixed in pinctrl-stm32 driver?
What are the possible workaround I can use? Would kernel flag fw_devlink=off
work reliably for all the mentioned kernel versions?
Regards,
Antonio
^ permalink raw reply [flat|nested] only message in thread