On Tue, Feb 27, 2024 at 5:27 AM Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > 5.4-stable review patch. If anyone has any objections, please let me know. > > ------------------ > > From: John Stultz <john.stultz@xxxxxxxxxx> > > [ Upstream commit e2cec7d6853712295cef5377762165a489b2957f ] > > When using modules, its common for the modules not to be loaded > until quite late by userland. With the current code, > driver_deferred_probe_check_state() will stop returning > EPROBE_DEFER after late_initcall, which can cause module > dependency resolution to fail after that. > > So allow a longer window of 30 seconds (picked somewhat > arbitrarily, but influenced by the similar regulator core > timeout value) in the case where modules are enabled. > > Cc: linux-pm@xxxxxxxxxxxxxxx > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> > Cc: Thierry Reding <treding@xxxxxxxxxx> > Cc: Mark Brown <broonie@xxxxxxxxxx> > Cc: Liam Girdwood <lgirdwood@xxxxxxxxx> > Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > Cc: Saravana Kannan <saravanak@xxxxxxxxxx> > Cc: Todd Kjos <tkjos@xxxxxxxxxx> > Cc: Len Brown <len.brown@xxxxxxxxx> > Cc: Pavel Machek <pavel@xxxxxx> > Cc: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > Cc: Kevin Hilman <khilman@xxxxxxxxxx> > Cc: "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx> > Cc: Rob Herring <robh@xxxxxxxxxx> > Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > Signed-off-by: John Stultz <john.stultz@xxxxxxxxxx> > Link: https://lore.kernel.org/r/20200225050828.56458-3-john.stultz@xxxxxxxxxx > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx> > --- > drivers/base/dd.c | 9 +++++++++ > 1 file changed, 9 insertions(+) This change ended up being reverted upstream in ce68929f07de Is there some specific reason it got selected to be pulled into -stable? thanks -john