Messages in this thread | | | Date | Fri, 30 Jun 2023 15:34:36 +0200 | From | Wolfram Sang <> | Subject | Re: [PATCH v2] mmc: inline the first mmc_scan() on mmc_start_host() |
| |
> Note that, even if the offending commit below triggers this problem > 100% of the cases (as the probe path has now becomes synchronous), > there was a potential risk even before. Previously, mmc_add_host() > ended up punting a work - and if that work ended up sending a request > to the host driver, *before* the irq-handler would be ready, we would > hit the similar problem. I bet adding an msleep(1000) immediately > after mmc_add_host() in tmio_mmc_host_probe(), would then trigger this > problem too. :-) > > That said, I am going to revert the offending commit to fix these > problems, for now. Then I will try to help out and fixup the relevant > host drivers - and when that is done, we can give this whole thing a > new try.
I'll work on the TMIO/SDHI driver next week. Thanks for the input!
[unhandled content-type:application/pgp-signature]
| |