Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-29 Thread Niklas Cassel via iommu
On Wed, May 25, 2022 at 12:49:00PM -0700, Saravana Kannan wrote: > On Wed, May 25, 2022 at 12:12 AM Sebastian Andrzej Siewior > wrote: > > > > On 2022-05-24 10:46:49 [-0700], Saravana Kannan wrote: > > > > Removing probe_timeout_waitqueue (as suggested) or setting the timeout > > > > to 0 avoids t

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-29 Thread Saravana Kannan via iommu
On Sun, May 29, 2022 at 1:34 AM 'Niklas Cassel' via kernel-team wrote: > > On Wed, May 25, 2022 at 12:49:00PM -0700, Saravana Kannan wrote: > > On Wed, May 25, 2022 at 12:12 AM Sebastian Andrzej Siewior > > wrote: > > > > > > On 2022-05-24 10:46:49 [-0700], Saravana Kannan wrote: > > > > > Removi

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-25 Thread Saravana Kannan via iommu
On Wed, May 25, 2022 at 12:12 AM Sebastian Andrzej Siewior wrote: > > On 2022-05-24 10:46:49 [-0700], Saravana Kannan wrote: > > > Removing probe_timeout_waitqueue (as suggested) or setting the timeout > > > to 0 avoids the delay. > > > > In your case, I think it might be working as intended? Curi

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-25 Thread Sebastian Andrzej Siewior
On 2022-05-24 10:46:49 [-0700], Saravana Kannan wrote: > > Removing probe_timeout_waitqueue (as suggested) or setting the timeout > > to 0 avoids the delay. > > In your case, I think it might be working as intended? Curious, what > was the call stack in your case where it was blocked? Why is then

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-24 Thread Saravana Kannan via iommu
On Tue, May 24, 2022 at 9:41 AM Sebastian Andrzej Siewior wrote: > > On 2022-05-23 20:43:06 [-0700], Saravana Kannan wrote: > … > > Thanks for all the help. I think I know what's going on. > > I, too got here because my boot recently was extended by 10 seconds and > bisected to that commit in ques

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-24 Thread Sebastian Andrzej Siewior
On 2022-05-23 20:43:06 [-0700], Saravana Kannan wrote: … > Thanks for all the help. I think I know what's going on. I, too got here because my boot recently was extended by 10 seconds and bisected to that commit in question. > If you revert the following commit, then you'll see that your device >

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-23 Thread Saravana Kannan via iommu
On Mon, May 23, 2022 at 3:14 PM Nathan Chancellor wrote: > > On Mon, May 23, 2022 at 01:04:03PM -0700, Saravana Kannan wrote: > > On Mon, May 23, 2022 at 8:17 AM Nathan Chancellor wrote: > > > > > > On Fri, May 20, 2022 at 05:15:55PM -0700, Saravana Kannan wrote: > > > > On Fri, May 20, 2022 at 5

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-23 Thread Nathan Chancellor
On Mon, May 23, 2022 at 01:04:03PM -0700, Saravana Kannan wrote: > On Mon, May 23, 2022 at 8:17 AM Nathan Chancellor wrote: > > > > On Fri, May 20, 2022 at 05:15:55PM -0700, Saravana Kannan wrote: > > > On Fri, May 20, 2022 at 5:04 PM Nathan Chancellor > > > wrote: > > > > > > > > On Fri, May 20

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-23 Thread Saravana Kannan via iommu
On Mon, May 23, 2022 at 8:17 AM Nathan Chancellor wrote: > > On Fri, May 20, 2022 at 05:15:55PM -0700, Saravana Kannan wrote: > > On Fri, May 20, 2022 at 5:04 PM Nathan Chancellor wrote: > > > > > > On Fri, May 20, 2022 at 04:49:48PM -0700, Saravana Kannan wrote: > > > > On Fri, May 20, 2022 at 4

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-23 Thread Nathan Chancellor
On Fri, May 20, 2022 at 05:15:55PM -0700, Saravana Kannan wrote: > On Fri, May 20, 2022 at 5:04 PM Nathan Chancellor wrote: > > > > On Fri, May 20, 2022 at 04:49:48PM -0700, Saravana Kannan wrote: > > > On Fri, May 20, 2022 at 4:30 PM Nathan Chancellor > > > wrote: > > > > > > > > Hi Saravana, >

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-20 Thread Saravana Kannan via iommu
On Fri, May 20, 2022 at 5:04 PM Nathan Chancellor wrote: > > On Fri, May 20, 2022 at 04:49:48PM -0700, Saravana Kannan wrote: > > On Fri, May 20, 2022 at 4:30 PM Nathan Chancellor wrote: > > > > > > Hi Saravana, > > > > > > On Fri, Apr 29, 2022 at 03:09:32PM -0700, Saravana Kannan wrote: > > > >

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-20 Thread Nathan Chancellor
On Fri, May 20, 2022 at 04:49:48PM -0700, Saravana Kannan wrote: > On Fri, May 20, 2022 at 4:30 PM Nathan Chancellor wrote: > > > > Hi Saravana, > > > > On Fri, Apr 29, 2022 at 03:09:32PM -0700, Saravana Kannan wrote: > > > The deferred probe timer that's used for this currently starts at > > > la

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-20 Thread Saravana Kannan via iommu
On Fri, May 20, 2022 at 4:30 PM Nathan Chancellor wrote: > > Hi Saravana, > > On Fri, Apr 29, 2022 at 03:09:32PM -0700, Saravana Kannan wrote: > > The deferred probe timer that's used for this currently starts at > > late_initcall and runs for driver_deferred_probe_timeout seconds. The > > assumpt

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-20 Thread Nathan Chancellor
Hi Saravana, On Fri, Apr 29, 2022 at 03:09:32PM -0700, Saravana Kannan wrote: > The deferred probe timer that's used for this currently starts at > late_initcall and runs for driver_deferred_probe_timeout seconds. The > assumption being that all available drivers would be loaded and > registered b

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-16 Thread Rob Herring
On Fri, May 13, 2022 at 12:26 PM Saravana Kannan wrote: > > On Fri, May 13, 2022 at 6:58 AM Rob Herring wrote: > > > > On Fri, Apr 29, 2022 at 5:09 PM Saravana Kannan > > wrote: > > > > > > The deferred probe timer that's used for this currently starts at > > > late_initcall and runs for driver

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-13 Thread Saravana Kannan via iommu
On Fri, May 13, 2022 at 6:58 AM Rob Herring wrote: > > On Fri, Apr 29, 2022 at 5:09 PM Saravana Kannan wrote: > > > > The deferred probe timer that's used for this currently starts at > > late_initcall and runs for driver_deferred_probe_timeout seconds. The > > assumption being that all available

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-13 Thread Rob Herring
On Fri, Apr 29, 2022 at 5:09 PM Saravana Kannan wrote: > > The deferred probe timer that's used for this currently starts at > late_initcall and runs for driver_deferred_probe_timeout seconds. The > assumption being that all available drivers would be loaded and > registered before the timer expir

Re: [PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-05-13 Thread Mark Brown
On Fri, Apr 29, 2022 at 03:09:32PM -0700, Saravana Kannan wrote: > The deferred probe timer that's used for this currently starts at > late_initcall and runs for driver_deferred_probe_timeout seconds. The > assumption being that all available drivers would be loaded and > registered before the time

[PATCH v1] driver core: Extend deferred probe timeout on driver registration

2022-04-29 Thread Saravana Kannan via iommu
The deferred probe timer that's used for this currently starts at late_initcall and runs for driver_deferred_probe_timeout seconds. The assumption being that all available drivers would be loaded and registered before the timer expires. This means, the driver_deferred_probe_timeout has to be pretty