All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Williamson <alex.williamson@hp.com>
To: Simon Horman <horms@verge.net.au>
Cc: xen-devel@lists.xensource.com,
	Ian Campbell <Ian.Campbell@citrix.com>,
	xen-ia64-devel@lists.xensource.com
Subject: Re: [patch 0/3] xen: Extend xen kexec hypercall to return additional regions
Date: Wed, 27 Feb 2008 12:55:52 -0700	[thread overview]
Message-ID: <1204142152.6798.32.camel@lappy> (raw)
In-Reply-To: <20080227070105.949799153@vergenet.net>

Hi Simon,

   I'm not seeing how this doesn't break the x86 COMPAT/CONFIG_COMPAT
code paths.  Where does kexec_get_range_internal() get defined for
COMPAT in [1/3], and where is machine_kexec_get_xen() defined for
CONFIG_COMPAT in [2/3]?  I'm fine with the ia64 parts if Ian/Keir want
to check them into the main tree, but there is some mixed indenting in
[3/3].  Thanks,

	Alex

On Wed, 2008-02-27 at 16:01 +0900, Simon Horman wrote:
> Hi,
> 
> this series starts off by reworking the hypercall a bit to
> allow it to have architecture-specific code under xen/arch/
> 
> It then extends the hypercall for some extra regions that
> are needed for xen ia64.
> 
> There are generic and ia64 specific patches in this series.
> I wanted to post them together as they don't make much sense
> without each other.
> 
> There are related xen-linux patches that I will send as a separate series.
> There are related kexec-tools patches which I have posted to
> the kexec mailing list and intend to merge.
>   http://lists.infradead.org/pipermail/kexec/2008-February/001348.html
> 
> 
> The end-game here is to make sure that all the reserved regions
> show up in /proc/iomem_machine on ia64. This currently does not happen.
> And manifests as kexec only being able to be performed once as
> the boot parameter ends up being overwritten in relocate_kernel before
> purgatory is reached.
> 
>   Xen--kexec-->Xen--kexec [hang in purgatory!]
> 
> 
> The EFI-RID patches for ia64 xen kexec are also needed for kexec
> to function on ia64. However they touch different code paths and can
> be merged separately.
> 
-- 
Alex Williamson                             HP Open Source & Linux Org.

  parent reply	other threads:[~2008-02-27 19:55 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-27  7:01 [patch 0/3] xen: Extend xen kexec hypercall to return additional regions Simon Horman
2008-02-27  7:01 ` [patch 1/3] kexec: limit scope of the use of compat_kexec_range_t Simon Horman
2008-02-27  7:01 ` [patch 2/3] kexec: Add machine_kexec_get() Simon Horman
2008-02-27  7:01 ` [patch 3/3] kexec: extend hypercall for efi memory map, boot parameter and xenheap regions Simon Horman
2008-02-28  7:50   ` [Xen-ia64-devel] " Simon Horman
2008-02-27 19:55 ` Alex Williamson [this message]
2008-02-28  3:35   ` [patch 0/3] xen: Extend xen kexec hypercall to return additional regions Simon Horman
2008-02-28  8:55     ` Keir Fraser
2008-02-29  0:54       ` Simon Horman
2008-02-28 15:36     ` Alex Williamson

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=1204142152.6798.32.camel@lappy \
    --to=alex.williamson@hp.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=horms@verge.net.au \
    --cc=xen-devel@lists.xensource.com \
    --cc=xen-ia64-devel@lists.xensource.com \
    /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.