All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Julien Grall <julien.grall@citrix.com>
Cc: Iurii Konovalenko <iurii.konovalenko@globallogic.com>,
	Julien Grall <julien.grall@linaro.org>,
	tim@xen.org, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [PATCH v1 1/2] arm: Add ability to relocate Xen in over 4GB space
Date: Tue, 14 Apr 2015 12:00:08 +0100	[thread overview]
Message-ID: <1429009208.15516.28.camel@citrix.com> (raw)
In-Reply-To: <5527DD64.5040206@citrix.com>

On Fri, 2015-04-10 at 15:25 +0100, Julien Grall wrote:
> On 10/04/15 14:58, Iurii Konovalenko wrote:
> > Hi, Julien!
> 
> Hi Iurii,
> 
> > On Wed, Apr 8, 2015 at 7:05 PM, Julien Grall <julien.grall@citrix.com> wrote:
> > 
> >> The virtualization extension requires LPAE. Any reason to make this
> >> optional?
> > I agree with you - there is no real reason to make it optional. I will
> > rework patch
> > to include functionality without any flags by default.
> 
> Before doing a such change, I'd like the point of view of Stefano or Ian
> on this patch.

Such things shouldn't be compile time optional. I don't think there's
any reason why it can't be done for every arm32 platform.

Normally I'd ask for a command line option which can overrides (i.e.
disable) this behaviour -- just in case. But I suspect that the relevant
code runs too early?

(I'll read the rest of this thread as I catch up with my email backlog).

Ian

  reply	other threads:[~2015-04-14 11:00 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-08 12:36 [PATCH v1 0/2] relocate Xen in over 4GB space for arm32 Iurii Konovalenko
2015-04-08 12:36 ` [PATCH v1 1/2] arm: Add ability to relocate Xen in over 4GB space Iurii Konovalenko
2015-04-08 16:05   ` Julien Grall
2015-04-08 17:24     ` Andrii Anisov
2015-04-08 17:25       ` Andrii Anisov
2015-05-08 16:02       ` Ian Campbell
2015-04-10 13:58     ` Iurii Konovalenko
2015-04-10 14:25       ` Julien Grall
2015-04-14 11:00         ` Ian Campbell [this message]
2015-04-10 14:38     ` Andrii Anisov
2015-04-10 14:49       ` Julien Grall
2015-04-16 12:59         ` Ian Campbell
2015-04-15 16:29     ` Ian Campbell
2015-04-15 16:41   ` Ian Campbell
2015-04-08 12:36 ` [PATCH v1 2/2] arm: skip verifying memory continuity Iurii Konovalenko
2015-04-08 16:20   ` 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=1429009208.15516.28.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=iurii.konovalenko@globallogic.com \
    --cc=julien.grall@citrix.com \
    --cc=julien.grall@linaro.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.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.