Re: [Xen-devel] [PATCH 0/3] xen: support of large memory maps

2017-03-22 Thread Alex Thorlton
On Tue, Mar 21, 2017 at 02:10:20PM +0100, Juergen Gross wrote: > This patch series is the first part for adding support of large EFI > memory maps (> the current limit of 128 entries) while reducing > trampoline size. > > I'm not posting the final patch for making the trampoline size > reduction

[Xen-devel] Supporting systems with large E820 maps

2017-03-20 Thread Alex Thorlton
Hey everyone, Recently, I've been working with Boris Ostrovsky to get Xen running on some of our larger systems, and we've run into a few problems with the amount of space that Xen sets aside for the E820 map. The first problem that I hit was that E820MAX is far too small, at 128 entries, for

[Xen-devel] [PATCH 1/2] x86: Make E820_X_MAX unconditionally larger than E820MAX

2016-12-05 Thread Alex Thorlton
the XENMEM_machine_memory_map hypercall can produce an e820 map larger than 128 entries, even on systems where the original e820 table was quite a bit smaller than that, depending on how many IOAPICs are installed on the system. Signed-off-by: Alex Thorlton <athorl...@sgi.com> Suggested-by: Ingo

[Xen-devel] [RFC PATCH v3] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-12-05 Thread Alex Thorlton
/comments! Alex Thorlton (2): x86: Make E820_X_MAX unconditionally larger than E820MAX xen/x86: Increase xen_e820_map to E820_X_MAX possible entries arch/x86/include/asm/e820.h | 12 arch/x86/xen/setup.c| 6 +++--- 2 files changed, 11 insertions(+), 7 deletions

[Xen-devel] [PATCH 2/2] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-12-05 Thread Alex Thorlton
this callback. Signed-off-by: Alex Thorlton <athorl...@sgi.com> Suggested-by: Boris Ostrovsky <boris.ostrov...@oracle.com> Reviewed-by: Juergen Gross <jgr...@suse.com> Cc: Russ Anderson <r...@sgi.com> Cc: David Vrabel <david.vra...@citrix.com> Cc: Ingo Molnar <mi...@r

Re: [Xen-devel] [PATCH 1/2] x86: Make E820_X_MAX unconditionally larger than E820MAX

2016-12-01 Thread Alex Thorlton
On Wed, Nov 30, 2016 at 07:21:48AM +0100, Ingo Molnar wrote: > > * Alex Thorlton <athorl...@sgi.com> wrote: > > > It's really not necessary to limit E820_X_MAX to 128 in the non-EFI > > case. This commit drops E820_X_MAX's dependency on CONFIG_EFI, so that > &

[Xen-devel] [PATCH 2/2] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-29 Thread Alex Thorlton
this callback. Signed-off-by: Alex Thorlton <athorl...@sgi.com> Suggested-by: Boris Ostrovsky <boris.ostrov...@oracle.com> Cc: Russ Anderson <r...@sgi.com> Cc: David Vrabel <david.vra...@citrix.com> Cc: Ingo Molnar <mi...@redhat.com> Cc: Juergen Gross <jgr...@suse.com> Cc:

[Xen-devel] [PATCH 1/2] x86: Make E820_X_MAX unconditionally larger than E820MAX

2016-11-29 Thread Alex Thorlton
the XENMEM_machine_memory_map hypercall can produce an e820 map larger than 128 entries, even on systems where the original e820 table was quite a bit smaller than that, depending on how many IOAPICs are installed on the system. Signed-off-by: Alex Thorlton <athorl...@sgi.com> Suggested-by: Ingo

[Xen-devel] [RFC PATCH v2] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-29 Thread Alex Thorlton
that others can give! - Alex Alex Thorlton (2): x86: Make E820_X_MAX unconditionally larger than E820MAX xen/x86: Increase xen_e820_map to E820_X_MAX possible entries arch/x86/include/asm/e820.h | 8 +++- arch/x86/xen/setup.c| 6 +++--- 2 files changed, 6 insertions(+), 8 deletions

Re: [Xen-devel] ACPI fixmap overflow

2016-11-17 Thread Alex Thorlton
On Thu, Nov 17, 2016 at 10:01:15AM -0500, Boris Ostrovsky wrote: > On 11/16/2016 05:08 AM, Jan Beulich wrote: > On 15.11.16 at 21:51, wrote: > >> On 11/15/2016 03:44 PM, Andrew Cooper wrote: > >>> On 15/11/2016 20:39, Boris Ostrovsky wrote: > On 11/15/2016

Re: [Xen-devel] [PATCH] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-16 Thread Alex Thorlton
On Wed, Nov 16, 2016 at 07:09:01AM +0100, Juergen Gross wrote: > On 15/11/16 01:11, Alex Thorlton wrote: > > On systems with sufficiently large e820 tables, and several IOAPICs, it > > is possible for the XENMEM_machine_memory_map callback (and its > > counterpart, XENMEM_m

Re: [Xen-devel] [PATCH] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-16 Thread Alex Thorlton
On Tue, Nov 15, 2016 at 07:30:35AM +0100, Ingo Molnar wrote: > > * Alex Thorlton <athorl...@sgi.com> wrote: > > > On systems with sufficiently large e820 tables, and several IOAPICs, it > > is possible for the XENMEM_machine_memory_map callback (and its > &g

Re: [Xen-devel] [RFC PATCH] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-16 Thread Alex Thorlton
On Wed, Nov 16, 2016 at 05:42:11PM +0100, Juergen Gross wrote: > On 15/11/16 08:15, Jan Beulich wrote: > >>>> On 15.11.16 at 07:33, <jgr...@suse.com> wrote: > >> On 15/11/16 01:11, Alex Thorlton wrote: > >>> Hey everyone, > >>> >

Re: [Xen-devel] [RFC PATCH] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-15 Thread Alex Thorlton
On Tue, Nov 15, 2016 at 10:55:49AM +0100, Juergen Gross wrote: > I'd go with the new error code. What about E2BIG or ENOSPC? > > I think the hypervisor should fill in the number of entries required > in this case. > > In case nobody objects I can post patches for this purpose (both Xen > and

[Xen-devel] [RFC PATCH] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-14 Thread Alex Thorlton
, there isn't exactly a "more appropriate" macro/variable to use, so this may not really be an issue. Any input on the patch, or the questions I've raised above is greatly appreciated! - Alex Alex Thorlton (1): xen/x86: Increase xen_e820_map to E820_X_MAX possible entries arch/x86/x

[Xen-devel] [PATCH] xen/x86: Increase xen_e820_map to E820_X_MAX possible entries

2016-11-14 Thread Alex Thorlton
this callback. Signed-off-by: Alex Thorlton <athorl...@sgi.com> Suggested-by: Boris Ostrovsky <boris.ostrov...@oracle.com> Cc: Russ Anderson <r...@sgi.com> Cc: David Vrabel <david.vra...@citrix.com> Cc: Juergen Gross <jgr...@suse.com> Cc: Thomas Gleixner <t...@linutronix.de&g