Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-13 Thread Grygorii Strashko
On 10/10/2015 12:16 AM, Rafael J. Wysocki wrote: On Friday, October 09, 2015 09:38:13 AM Grygorii Strashko wrote: On 10/08/2015 03:53 PM, Alan Stern wrote: On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: @@ -391,6 +391,10 @@ int driver_probe_done(void) */ void wait_for_device_probe(void)

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-13 Thread Grygorii Strashko
On 10/10/2015 12:16 AM, Rafael J. Wysocki wrote: On Friday, October 09, 2015 09:38:13 AM Grygorii Strashko wrote: On 10/08/2015 03:53 PM, Alan Stern wrote: On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: @@ -391,6 +391,10 @@ int driver_probe_done(void) */ void wait_for_device_probe(void)

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-09 Thread Rafael J. Wysocki
On Friday, October 09, 2015 09:38:13 AM Grygorii Strashko wrote: > On 10/08/2015 03:53 PM, Alan Stern wrote: > > On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: > > > >>> @@ -391,6 +391,10 @@ int driver_probe_done(void) > >>>*/ > >>> void wait_for_device_probe(void) > >>> { > >>> + /* wait

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-09 Thread Grygorii Strashko
On 10/08/2015 03:53 PM, Alan Stern wrote: > On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: > >>> @@ -391,6 +391,10 @@ int driver_probe_done(void) >>>*/ >>> void wait_for_device_probe(void) >>> { >>> + /* wait for the deferred probe workqueue to finish */ >>> + if

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-09 Thread Rafael J. Wysocki
On Friday, October 09, 2015 09:38:13 AM Grygorii Strashko wrote: > On 10/08/2015 03:53 PM, Alan Stern wrote: > > On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: > > > >>> @@ -391,6 +391,10 @@ int driver_probe_done(void) > >>>*/ > >>> void wait_for_device_probe(void) > >>> { > >>> + /* wait

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-09 Thread Grygorii Strashko
On 10/08/2015 03:53 PM, Alan Stern wrote: > On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: > >>> @@ -391,6 +391,10 @@ int driver_probe_done(void) >>>*/ >>> void wait_for_device_probe(void) >>> { >>> + /* wait for the deferred probe workqueue to finish */ >>> + if

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-08 Thread Alan Stern
On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: > > @@ -391,6 +391,10 @@ int driver_probe_done(void) > > */ > > void wait_for_device_probe(void) > > { > > + /* wait for the deferred probe workqueue to finish */ > > + if (driver_deferred_probe_enable) > > +

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-08 Thread Rafael J. Wysocki
On Thursday, October 08, 2015 11:57:06 AM Grygorii Strashko wrote: > Now wait_for_device_probe() waits for currently executing probes to finish, > but it doesn't take into account deferred probing mechanism. As result, > nothing prevents deferred probe workqueue to continue probing devices right >

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-08 Thread Rafael J. Wysocki
On Thursday, October 08, 2015 11:57:06 AM Grygorii Strashko wrote: > Now wait_for_device_probe() waits for currently executing probes to finish, > but it doesn't take into account deferred probing mechanism. As result, > nothing prevents deferred probe workqueue to continue probing devices right >

Re: [PATCH 1/2] PM / sleep: ensure deferred probe workqueue is finished in wait_for_device_probe

2015-10-08 Thread Alan Stern
On Thu, 8 Oct 2015, Rafael J. Wysocki wrote: > > @@ -391,6 +391,10 @@ int driver_probe_done(void) > > */ > > void wait_for_device_probe(void) > > { > > + /* wait for the deferred probe workqueue to finish */ > > + if (driver_deferred_probe_enable) > > +