| Message ID | 20220526081550.1089805-1-saravanak@google.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | deferred_probe_timeout logic clean up | expand |
On 2022-05-26 01:15:39 [-0700], Saravana Kannan wrote: > Yoshihiro/Geert, Hi Saravana, > If you can test this patch series and confirm that the NFS root case > works, I'd really appreciate that. The two patches you sent earlier, plus this series, plus diff --git a/drivers/base/core.c b/drivers/base/core.c index 7ff7fbb006431..829d9b1f7403f 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c @@ -1697,8 +1697,6 @@ static int fw_devlink_may_probe(struct device *dev, void *data) */ void __init fw_devlink_unblock_may_probe(void) { - struct device_link *link, *ln; - if (!fw_devlink_flags || fw_devlink_is_permissive()) return; and it compiles + boots without a delay. Sebastian
Hi Saravana, On Thu, May 26, 2022 at 10:15 AM Saravana Kannan <saravanak@google.com> wrote: > This series is based on linux-next + these 2 small patches applies on top: > https://lore.kernel.org/lkml/20220526034609.480766-1-saravanak@google.com/ > > A lot of the deferred_probe_timeout logic is redundant with > fw_devlink=on. Also, enabling deferred_probe_timeout by default breaks > a few cases. > > This series tries to delete the redundant logic, simplify the frameworks > that use driver_deferred_probe_check_state(), enable > deferred_probe_timeout=10 by default, and fixes the nfsroot failure > case. > > Patches 1 to 3 are fairly straightforward and can probably be applied > right away. > > Patches 4 to 9 are related and are the complicated bits of this series. > > Patch 8 is where someone with more knowledge of the IP auto config code > can help rewrite the patch to limit the scope of the workaround by > running the work around only if IP auto config fails the first time > around. But it's also something that can be optimized in the future > because it's already limited to the case where IP auto config is enabled > using the kernel commandline. Thanks for your series! > Yoshihiro/Geert, > > If you can test this patch series and confirm that the NFS root case > works, I'd really appreciate that. On Salvator-XS, Micrel KSZ9031 Gigabit PHY probe is no longer delayed by 9s after applying the two earlier patches, and the same is true after applying this series on top. Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> I will do testing on more boards, but that may take a while, as we're in the middle of the merge window. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
On Mon, May 30, 2022 at 1:49 AM Sebastian Andrzej Siewior <bigeasy@linutronix.de> wrote: > > On 2022-05-26 01:15:39 [-0700], Saravana Kannan wrote: > > Yoshihiro/Geert, > Hi Saravana, > > > If you can test this patch series and confirm that the NFS root case > > works, I'd really appreciate that. > > The two patches you sent earlier, plus this series, plus > > diff --git a/drivers/base/core.c b/drivers/base/core.c > index 7ff7fbb006431..829d9b1f7403f 100644 > --- a/drivers/base/core.c > +++ b/drivers/base/core.c > @@ -1697,8 +1697,6 @@ static int fw_devlink_may_probe(struct device *dev, void *data) > */ > void __init fw_devlink_unblock_may_probe(void) > { > - struct device_link *link, *ln; > - > if (!fw_devlink_flags || fw_devlink_is_permissive()) > return; > > and it compiles + boots without a delay. Thanks for testing! I missed your reply until now and I ended up sending out a v2 a few days back. That's a much better series IMO because it only temporarily ignores dependencies and only when NFS root mounting might not work. If you can test that too (and it shouldn't have the useless variable), I'd appreciate it. -Saravana
On Mon, May 30, 2022 at 2:38 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Saravana, > > On Thu, May 26, 2022 at 10:15 AM Saravana Kannan <saravanak@google.com> wrote: > > This series is based on linux-next + these 2 small patches applies on top: > > https://lore.kernel.org/lkml/20220526034609.480766-1-saravanak@google.com/ > > > > A lot of the deferred_probe_timeout logic is redundant with > > fw_devlink=on. Also, enabling deferred_probe_timeout by default breaks > > a few cases. > > > > This series tries to delete the redundant logic, simplify the frameworks > > that use driver_deferred_probe_check_state(), enable > > deferred_probe_timeout=10 by default, and fixes the nfsroot failure > > case. > > > > Patches 1 to 3 are fairly straightforward and can probably be applied > > right away. > > > > Patches 4 to 9 are related and are the complicated bits of this series. > > > > Patch 8 is where someone with more knowledge of the IP auto config code > > can help rewrite the patch to limit the scope of the workaround by > > running the work around only if IP auto config fails the first time > > around. But it's also something that can be optimized in the future > > because it's already limited to the case where IP auto config is enabled > > using the kernel commandline. > > Thanks for your series! > > > Yoshihiro/Geert, > > > > If you can test this patch series and confirm that the NFS root case > > works, I'd really appreciate that. > > On Salvator-XS, Micrel KSZ9031 Gigabit PHY probe is no longer delayed > by 9s after applying the two earlier patches, and the same is true > after applying this series on top. > Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> > > I will do testing on more boards, but that may take a while, as we're > in the middle of the merge window. Thanks for testing. I missed your email until now. I sent out a v2 series a few days back and that's a much better solution than v1. If you can test that series instead, that'd be nice. -Saravana