All of lore.kernel.org
 help / color / mirror / Atom feed
From: konrad.wilk@oracle.com (Konrad Rzeszutek Wilk)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3] xen/arm: enable PV control for ARM
Date: Mon, 22 Jul 2013 16:00:20 -0400	[thread overview]
Message-ID: <20130722200020.GT30300@phenom.dumpdata.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1307221545030.4893@kaball.uk.xensource.com>

On Mon, Jul 22, 2013 at 03:46:34PM +0100, Stefano Stabellini wrote:
> On Mon, 22 Jul 2013, Mark Rutland wrote:
> > On Mon, Jul 22, 2013 at 03:09:35PM +0100, Stefano Stabellini wrote:
> > > On Mon, 22 Jul 2013, Mark Rutland wrote:
> > > > Hi,
> > > > 
> > > > [Adding Catalin to Cc]
> > > > 
> > > > On Mon, Jul 22, 2013 at 11:57:34AM +0100, Julien Grall wrote:
> > > > > Enable power management from the toolstack for ARM guest.
> > > > > 
> > > > > Signed-off-by: Julien Grall <julien.grall@linaro.org>
> > > > > 
> > > > > ---
> > > > >     Changes in v3:
> > > > >         - Fix condition to compile cpu_hotplug.o
> > > > >     Changes in v2:
> > > > >         - Don't compile xen/cpu_hotplug.o with ARM64
> > > > 
> > > > My arm64 cpu hotplug series is going to be dependent on not compiling
> > > > xen/cpu_hotplug.o. Currently I'm not able to compile an arm64 kernel
> > > > with my patches and Xen support.
> > > > 
> > > > How do you intend to push this? I'd like to ensure my cpu hotplug series
> > > > is able to go in for the next merge window.
> > > 
> > > This patch does exactly that: it makes sure cpu_hotplug is not compiled
> > > on ARM64.
> > 
> > Sorry, what I meant was that my series is going to be entirely dependent
> > on this patch or it'll break bisection. That'll make merging things a
> > little painful as I'll need a stable branch with this patch to base my
> > series upon.
> > 
> > Another option would be to split this patch into two parts, one enabling
> > power management and one fixing things up for HOTPLUG_CPU on arm64. That
> > way I could take the arm64 HOTPLUG_CPU fix in my series adding arm64
> > HOTPLUG_CPU support.
> > 
> > Would that be acceptable? Have I missed something?
> 
> As I am very sensitive to painful merging situations, I think we could
> split the patch in two and send upstream the fix (not compiling
> cpu_hotplug.o on ARM64) in 3.11-rc2 or rc3.  Konrad, what do you think?

Sure, prep a patch and send it my way.

WARNING: multiple messages have this Message-ID (diff)
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	Julien Grall <julien.grall@linaro.org>,
	"ian.campbell@citrix.com" <ian.campbell@citrix.com>,
	"patches@linaro.org" <patches@linaro.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Catalin Marinas <Catalin.Marinas@arm.com>
Subject: Re: [PATCH v3] xen/arm: enable PV control for ARM
Date: Mon, 22 Jul 2013 16:00:20 -0400	[thread overview]
Message-ID: <20130722200020.GT30300@phenom.dumpdata.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1307221545030.4893@kaball.uk.xensource.com>

On Mon, Jul 22, 2013 at 03:46:34PM +0100, Stefano Stabellini wrote:
> On Mon, 22 Jul 2013, Mark Rutland wrote:
> > On Mon, Jul 22, 2013 at 03:09:35PM +0100, Stefano Stabellini wrote:
> > > On Mon, 22 Jul 2013, Mark Rutland wrote:
> > > > Hi,
> > > > 
> > > > [Adding Catalin to Cc]
> > > > 
> > > > On Mon, Jul 22, 2013 at 11:57:34AM +0100, Julien Grall wrote:
> > > > > Enable power management from the toolstack for ARM guest.
> > > > > 
> > > > > Signed-off-by: Julien Grall <julien.grall@linaro.org>
> > > > > 
> > > > > ---
> > > > >     Changes in v3:
> > > > >         - Fix condition to compile cpu_hotplug.o
> > > > >     Changes in v2:
> > > > >         - Don't compile xen/cpu_hotplug.o with ARM64
> > > > 
> > > > My arm64 cpu hotplug series is going to be dependent on not compiling
> > > > xen/cpu_hotplug.o. Currently I'm not able to compile an arm64 kernel
> > > > with my patches and Xen support.
> > > > 
> > > > How do you intend to push this? I'd like to ensure my cpu hotplug series
> > > > is able to go in for the next merge window.
> > > 
> > > This patch does exactly that: it makes sure cpu_hotplug is not compiled
> > > on ARM64.
> > 
> > Sorry, what I meant was that my series is going to be entirely dependent
> > on this patch or it'll break bisection. That'll make merging things a
> > little painful as I'll need a stable branch with this patch to base my
> > series upon.
> > 
> > Another option would be to split this patch into two parts, one enabling
> > power management and one fixing things up for HOTPLUG_CPU on arm64. That
> > way I could take the arm64 HOTPLUG_CPU fix in my series adding arm64
> > HOTPLUG_CPU support.
> > 
> > Would that be acceptable? Have I missed something?
> 
> As I am very sensitive to painful merging situations, I think we could
> split the patch in two and send upstream the fix (not compiling
> cpu_hotplug.o on ARM64) in 3.11-rc2 or rc3.  Konrad, what do you think?

Sure, prep a patch and send it my way.

  parent reply	other threads:[~2013-07-22 20:00 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-22 10:57 [PATCH v3] xen/arm: enable PV control for ARM Julien Grall
2013-07-22 10:57 ` Julien Grall
2013-07-22 13:49 ` Mark Rutland
2013-07-22 13:49   ` Mark Rutland
2013-07-22 14:09   ` Stefano Stabellini
2013-07-22 14:09   ` Stefano Stabellini
2013-07-22 14:09     ` Stefano Stabellini
2013-07-22 14:39     ` Mark Rutland
2013-07-22 14:39       ` Mark Rutland
2013-07-22 14:46       ` Stefano Stabellini
2013-07-22 14:46       ` Stefano Stabellini
2013-07-22 14:46         ` Stefano Stabellini
2013-07-22 20:00         ` Konrad Rzeszutek Wilk
2013-07-22 20:00         ` Konrad Rzeszutek Wilk [this message]
2013-07-22 20:00           ` Konrad Rzeszutek Wilk
2013-07-22 14:39     ` Mark Rutland
2013-07-22 13:49 ` Mark Rutland
2013-07-22 14:09 ` Stefano Stabellini
2013-07-22 14:09 ` Stefano Stabellini
2013-07-22 14:09   ` Stefano Stabellini
  -- strict thread matches above, loose matches on Subject: below --
2013-07-22 10:57 Julien Grall

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130722200020.GT30300@phenom.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.