All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Vrabel <david.vrabel@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Daniel Kiper <daniel.kiper@oracle.com>,
	"kexec@lists.infradead.org" <kexec@lists.infradead.org>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [Xen-devel] [PATCH 5/8] kexec: extend hypercall with improved load/unload ops
Date: Fri, 22 Feb 2013 11:54:52 +0000	[thread overview]
Message-ID: <51275C8C.80506@citrix.com> (raw)
In-Reply-To: <51273D9D02000078000C03D4@nat28.tlf.novell.com>

On 22/02/13 08:42, Jan Beulich wrote:
>>>> On 21.02.13 at 18:48, David Vrabel <david.vrabel@citrix.com> wrote:
>> Crash images are copied directly into the crash region on load.
>> Default images are copied into Xen heap pages and a list of source and
>> destination machine addresses is created.  This is list is used in
>> kexec_reloc() to relocate the image to its destination.
> 
> Did you carefully consider the implications of using Xen heap pages
> here as opposed to domain heap ones? On huge systems, this may
> prevent kexec from working, as you're not just trying to allocate a
> handful of pages. IOW, is the less complex code really worth the
> increased likelihood of a failure here?

I wouldn't say carefully considered... I thought about using dom heap
briefly and took the lazy route.

I take your point though and will change it to use the dom heap.  Is
there a way to verify that all the map/unmaps are correctly done and it
isn't just working by chance?  Some sort of debug option?

David

_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

  reply	other threads:[~2013-02-22 11:54 UTC|newest]

Thread overview: 106+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-21 17:48 [PATCH 0/8] kexec: extended kexec hypercall for use with pv-ops kernels David Vrabel
2013-02-21 17:48 ` [PATCH 1/8] x86: give FIX_EFI_MPF its own fixmap entry David Vrabel
2013-02-21 17:48 ` David Vrabel
2013-02-21 17:48 ` [PATCH 2/8] xen: make GUEST_HANDLE_64() and uint64_aligned_t available everywhere David Vrabel
2013-02-21 17:48 ` David Vrabel
2013-02-21 17:48 ` [PATCH 3/8] kexec: add public interface for improved load/unload sub-ops David Vrabel
2013-02-21 17:48 ` David Vrabel
2013-02-21 22:29   ` Daniel Kiper
2013-02-21 22:29   ` Daniel Kiper
2013-02-22 11:49     ` David Vrabel
2013-02-22 11:49     ` David Vrabel
2013-02-22  8:33   ` Jan Beulich
2013-02-22  8:33   ` [Xen-devel] " Jan Beulich
2013-02-22 11:50     ` David Vrabel
2013-02-22 11:50     ` [Xen-devel] " David Vrabel
2013-02-22 13:09       ` Jan Beulich
2013-02-22 13:09       ` [Xen-devel] " Jan Beulich
2013-03-08 10:50   ` Daniel Kiper
2013-03-08 10:50   ` Daniel Kiper
2013-03-08 11:52     ` David Vrabel
2013-03-08 12:28       ` Daniel Kiper
2013-03-08 12:36         ` [Xen-devel] " Jan Beulich
2013-03-08 15:34           ` Daniel Kiper
2013-03-08 15:34           ` Daniel Kiper
2013-03-08 12:36         ` Jan Beulich
2013-03-08 12:28       ` Daniel Kiper
2013-03-08 11:52     ` David Vrabel
2013-02-21 17:48 ` [PATCH 4/8] kexec: add infrastructure for handling kexec images David Vrabel
2013-03-08 11:37   ` Daniel Kiper
2013-03-08 11:42     ` David Vrabel
2013-03-08 11:58       ` Daniel Kiper
2013-03-08 11:58       ` Daniel Kiper
2013-03-08 11:42     ` David Vrabel
2013-03-08 11:37   ` Daniel Kiper
2013-02-21 17:48 ` David Vrabel
2013-02-21 17:48 ` [PATCH 5/8] kexec: extend hypercall with improved load/unload ops David Vrabel
2013-02-21 22:41   ` Daniel Kiper
2013-02-21 22:41   ` Daniel Kiper
2013-02-22  8:42   ` Jan Beulich
2013-02-22  8:42   ` [Xen-devel] " Jan Beulich
2013-02-22 11:54     ` David Vrabel [this message]
2013-02-22 13:11       ` Jan Beulich
2013-02-22 13:11       ` Jan Beulich
2013-02-22 11:54     ` David Vrabel
2013-03-08 11:23   ` Daniel Kiper
2013-03-08 11:23   ` Daniel Kiper
2013-03-08 11:40     ` David Vrabel
2013-03-08 11:40     ` David Vrabel
2013-03-08 12:21       ` Daniel Kiper
2013-03-08 12:21       ` Daniel Kiper
2013-03-08 14:01         ` David Vrabel
2013-03-08 14:01         ` David Vrabel
2013-03-08 15:23           ` Daniel Kiper
2013-03-08 15:23             ` Daniel Kiper
2013-03-08 17:29             ` [Xen-devel] " Andrew Cooper
2013-03-08 17:29               ` Andrew Cooper
2013-03-08 21:45               ` Daniel Kiper
2013-03-08 21:45               ` [Xen-devel] " Daniel Kiper
2013-03-08 23:38                 ` Andrew Cooper
2013-03-08 23:38                 ` [Xen-devel] " Andrew Cooper
2013-03-11 11:17                   ` Daniel Kiper
2013-03-11 11:17                   ` [Xen-devel] " Daniel Kiper
2013-03-11 13:21                     ` David Vrabel
2013-03-11 13:21                     ` [Xen-devel] " David Vrabel
2013-03-11 13:30                       ` Daniel Kiper
2013-03-11 13:43                         ` David Vrabel
2013-03-11 14:13                           ` Daniel Kiper
2013-03-11 14:27                             ` Andrew Cooper
2013-03-11 20:45                               ` Daniel Kiper
2013-03-11 20:45                               ` [Xen-devel] " Daniel Kiper
2013-03-11 21:18                                 ` Andrew Cooper
2013-03-11 21:18                                 ` [Xen-devel] " Andrew Cooper
2013-03-12 11:17                                   ` Daniel Kiper
2013-03-12 11:17                                   ` [Xen-devel] " Daniel Kiper
2013-03-11 14:27                             ` Andrew Cooper
2013-03-11 14:13                           ` Daniel Kiper
2013-03-11 13:43                         ` David Vrabel
2013-03-11 13:30                       ` Daniel Kiper
2013-03-12 11:36   ` Daniel Kiper
2013-03-12 11:36   ` Daniel Kiper
2013-02-21 17:48 ` David Vrabel
2013-02-21 17:48 ` [PATCH 6/8] xen: kexec crash image when dom0 crashes David Vrabel
2013-02-21 17:48 ` David Vrabel
2013-02-21 17:48 ` [PATCH 7/8] libxc: add hypercall buffer arrays David Vrabel
2013-02-21 17:48 ` David Vrabel
2013-03-06 14:25   ` Ian Jackson
2013-03-06 14:25   ` [Xen-devel] " Ian Jackson
2013-03-07  2:44   ` Ian Campbell
2013-03-07  2:44   ` [Xen-devel] " Ian Campbell
2013-02-21 17:48 ` [PATCH 8/8] libxc: add API for kexec hypercall David Vrabel
2013-03-07  2:46   ` [Xen-devel] " Ian Campbell
2013-03-07  2:46   ` Ian Campbell
2013-02-21 17:48 ` David Vrabel
2013-02-21 22:47 ` [PATCH 0/8] kexec: extended kexec hypercall for use with pv-ops kernels Daniel Kiper
2013-02-21 22:47 ` Daniel Kiper
2013-02-22  8:17 ` Jan Beulich
2013-02-22  8:17 ` [Xen-devel] " Jan Beulich
2013-02-22 11:56   ` David Vrabel
2013-02-22 11:56   ` [Xen-devel] " David Vrabel
2013-02-26 13:58 ` Don Slutz
2013-02-26 13:58 ` [Xen-devel] " Don Slutz
2013-03-05 11:04 ` David Vrabel
2013-03-05 11:04 ` [Xen-devel] " David Vrabel
  -- strict thread matches above, loose matches on Subject: below --
2013-04-16 17:13 [PATCHv4 0/8] kexec: extend " David Vrabel
2013-04-16 17:13 ` [PATCH 5/8] kexec: extend hypercall with improved load/unload ops David Vrabel
2013-04-17  8:55   ` [Xen-devel] " Jan Beulich
2013-04-17 10:11     ` David Vrabel
2013-04-17 10:20       ` Jan Beulich

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=51275C8C.80506@citrix.com \
    --to=david.vrabel@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=daniel.kiper@oracle.com \
    --cc=kexec@lists.infradead.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.