Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Thu, Nov 10, 2016 at 01:08:56AM +0100, Rafael J. Wysocki wrote: > On Wed, Nov 2, 2016 at 6:07 AM, Brian Norris wrote: > > I can test this and send it in proper form if that looks preferable. > > It does to me as per the discussion at the LPC. > > Are you still going

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Thu, Nov 10, 2016 at 01:08:56AM +0100, Rafael J. Wysocki wrote: > On Wed, Nov 2, 2016 at 6:07 AM, Brian Norris wrote: > > I can test this and send it in proper form if that looks preferable. > > It does to me as per the discussion at the LPC. > > Are you still going to submit it? Yes, it

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Wed, Nov 2, 2016 at 6:07 AM, Brian Norris wrote: > + more genpd folks > > On Wed, Nov 02, 2016 at 04:51:08AM +0100, Rafael J. Wysocki wrote: >> On Tuesday, November 01, 2016 12:04:28 AM Dmitry Torokhov wrote: >> > On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Wed, Nov 2, 2016 at 6:07 AM, Brian Norris wrote: > + more genpd folks > > On Wed, Nov 02, 2016 at 04:51:08AM +0100, Rafael J. Wysocki wrote: >> On Tuesday, November 01, 2016 12:04:28 AM Dmitry Torokhov wrote: >> > On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki >> > wrote: >> > > On

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

+ more genpd folks On Wed, Nov 02, 2016 at 04:51:08AM +0100, Rafael J. Wysocki wrote: > On Tuesday, November 01, 2016 12:04:28 AM Dmitry Torokhov wrote: > > On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki > > wrote: > > > On Thursday, October 27, 2016 09:05:34 AM Brian

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

+ more genpd folks On Wed, Nov 02, 2016 at 04:51:08AM +0100, Rafael J. Wysocki wrote: > On Tuesday, November 01, 2016 12:04:28 AM Dmitry Torokhov wrote: > > On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki > > wrote: > > > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > > >>

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Tuesday, November 01, 2016 12:04:28 AM Dmitry Torokhov wrote: > On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki > wrote: > > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > >> Consider two devices, A and B, where B is a child of A, and B utilizes > >>

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Tuesday, November 01, 2016 12:04:28 AM Dmitry Torokhov wrote: > On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki > wrote: > > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > >> Consider two devices, A and B, where B is a child of A, and B utilizes > >> asynchronous suspend

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki wrote: > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: >> Consider two devices, A and B, where B is a child of A, and B utilizes >> asynchronous suspend (it does not matter whether A is sync or async). If >> B

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Mon, Oct 31, 2016 at 10:25 PM, Rafael J. Wysocki wrote: > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: >> Consider two devices, A and B, where B is a child of A, and B utilizes >> asynchronous suspend (it does not matter whether A is sync or async). If >> B fails to

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

Hi Rafael, On Tue, Nov 01, 2016 at 05:25:39AM +0100, Rafael J. Wysocki wrote: > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > > diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c > > index c58563581345..eaf6b53463a5 100644 > > --- a/drivers/base/power/main.c > >

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

Hi Rafael, On Tue, Nov 01, 2016 at 05:25:39AM +0100, Rafael J. Wysocki wrote: > On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > > diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c > > index c58563581345..eaf6b53463a5 100644 > > --- a/drivers/base/power/main.c > >

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > Consider two devices, A and B, where B is a child of A, and B utilizes > asynchronous suspend (it does not matter whether A is sync or async). If > B fails to suspend_noirq() or suspend_late(), or is interrupted by a > wakeup

Re: [PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

On Thursday, October 27, 2016 09:05:34 AM Brian Norris wrote: > Consider two devices, A and B, where B is a child of A, and B utilizes > asynchronous suspend (it does not matter whether A is sync or async). If > B fails to suspend_noirq() or suspend_late(), or is interrupted by a > wakeup

[PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

Consider two devices, A and B, where B is a child of A, and B utilizes asynchronous suspend (it does not matter whether A is sync or async). If B fails to suspend_noirq() or suspend_late(), or is interrupted by a wakeup (pm_wakeup_pending()), then it aborts and sets the async_error variable.

[PATCH v2 2/2] PM / sleep: don't suspend parent when async child suspend_{noirq,late} fails

Consider two devices, A and B, where B is a child of A, and B utilizes asynchronous suspend (it does not matter whether A is sync or async). If B fails to suspend_noirq() or suspend_late(), or is interrupted by a wakeup (pm_wakeup_pending()), then it aborts and sets the async_error variable.