From mboxrd@z Thu Jan  1 00:00:00 1970
From: Greg KH <greg@kroah.com>
Subject: Re: [PATCH 4/6 v2] PM: Limit race conditions between
 runtime PM and system sleep (v2)
Date: Wed, 2 Nov 2011 14:17:08 -0700
Message-ID: <20111102211708.GA19025@kroah.com>
References: <201106260054.20578.rjw@sisk.pl> <201106292329.04163.rjw@sisk.pl>
	<201106292334.24518.rjw@sisk.pl>
	<CAKnu2Mru4c-2LBCZ_k3ZA+v47u0+um8igYU7EWt8=qOTVgcUjA@mail.gmail.com>
	<20111102191432.GC29355@kroah.com>
	<CACRpkdY8zAw_pdwn2yUEJPdfYPS+VCcLrpucZzhCyNsVPYehdg@mail.gmail.com>
	<20111102201101.GA5581@kroah.com>
	<CACRpkdafpsgUi-0K5krr2_Hp+8GzCG8ZPu-HVNOK0_7uwURxdg@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <linux-pm-bounces@lists.linux-foundation.org>
Content-Disposition: inline
In-Reply-To: <CACRpkdafpsgUi-0K5krr2_Hp+8GzCG8ZPu-HVNOK0_7uwURxdg@mail.gmail.com>
List-Unsubscribe: <https://lists.linuxfoundation.org/mailman/options/linux-pm>,
	<mailto:linux-pm-request@lists.linux-foundation.org?subject=unsubscribe>
List-Archive: <http://lists.linuxfoundation.org/pipermail/linux-pm/>
List-Post: <mailto:linux-pm@lists.linux-foundation.org>
List-Help: <mailto:linux-pm-request@lists.linux-foundation.org?subject=help>
List-Subscribe: <https://lists.linuxfoundation.org/mailman/listinfo/linux-pm>,
	<mailto:linux-pm-request@lists.linux-foundation.org?subject=subscribe>
Sender: linux-pm-bounces@lists.linux-foundation.org
Errors-To: linux-pm-bounces@lists.linux-foundation.org
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Kevin Hilman <khilman@ti.com>, linux-scsi@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>, Jesse Barnes <jbarnes@virtuousgeek.org>, Ulf Hansson <Ulf.Hansson@stericsson.com>, Tejun Heo <tj@kernel.org>, Linux PM mailing list <linux-pm@lists.linux-foundation.org>, stable@kernel.org
List-Id: linux-pm@vger.kernel.org

On Wed, Nov 02, 2011 at 10:00:39PM +0100, Linus Walleij wrote:
> On Wed, Nov 2, 2011 at 9:11 PM, Greg KH <greg@kroah.com> wrote:
> > On Wed, Nov 02, 2011 at 09:06:02PM +0100, Linus Walleij wrote:
> >> On Wed, Nov 2, 2011 at 8:14 PM, Greg KH <greg@kroah.com> wrote:
> >> > So, what specific git commits do you want to see in the 3.0-stable
> >> > tree, and in what order should they be applied in?
> >>
> >> So to my untrained eye it looks like it should be applied like this (t=
op to
> >> bottom) using the reverse commit order from the mainline kernel:
> >>
> >> e529192 PM: Introduce generic "noirq" callback routines for subsystems=
 (v2)
> >> 455716e PM / Runtime: Update documentation of interactions with system=
 sleep
> >> 1e2ef05 PM: Limit race conditions between runtime PM and system sleep =
(v2)
> >> f3393b6 PM / Runtime: Add new helper function: pm_runtime_status_suspe=
nded()
> >>
> >> So (2) documents the problem, (3) fixes it, whereas (1) and (4) makes
> >> it possible
> >> to write proper _noirq() code that does not race, IIRC.
> >
> > But, after this, it's just adding new infrastructure that drivers will
> > then be able to use. =A0As I'm not adding new drivers to 3.0, there will
> > not be any users of this code, so why add it in the first place? =A0It
> > doesn't look like this follows the rules of the stable kernel tree at
> > all, does it?
> =

> We'll have to get some feedback from Rafael, but the way I percieved
> it, the patch (2) fixes a very real race, then fixing that makes it
> necessary for drivers using runtime_pm to do things differently to
> be on the safe side. But none were really fixed when first merging
> it.
> =

> So on second thought you probably only want patch (2)
> "PM: Limit race conditions between runtime PM and system sleep (v2)"
> for stable.

Ok, Rafael, any objection to me applying just this one?

Linus, have you tested the 3.0 kernel with just this patch applied?  If
not, please do so and let me know.  I'll wait for that before doing
anything.

thanks,

greg k-h