Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757806AbcLAShH (ORCPT ); Thu, 1 Dec 2016 13:37:07 -0500 Received: from relay3.sgi.com ([192.48.152.1]:49048 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755609AbcLAShG (ORCPT ); Thu, 1 Dec 2016 13:37:06 -0500 Date: Thu, 1 Dec 2016 12:37:03 -0600 From: Alex Thorlton To: Ingo Molnar Cc: Alex Thorlton , linux-kernel@vger.kernel.org, Russ Anderson , David Vrabel , Ingo Molnar , Juergen Gross , Thomas Gleixner , "H. Peter Anvin" , Denys Vlasenko , Boris Ostrovsky , x86@kernel.org, xen-devel@lists.xenproject.org Subject: Re: [PATCH 1/2] x86: Make E820_X_MAX unconditionally larger than E820MAX Message-ID: <20161201183703.GP67776@stormcage.americas.sgi.com> References: <1479168677-23633-1-git-send-email-athorlton@sgi.com> <1480476287-232878-1-git-send-email-athorlton@sgi.com> <1480476287-232878-2-git-send-email-athorlton@sgi.com> <20161130062148.GA11684@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161130062148.GA11684@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2351 Lines: 60 On Wed, Nov 30, 2016 at 07:21:48AM +0100, Ingo Molnar wrote: > > * Alex Thorlton 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 > > E820_X_MAX is always at least slightly larger than E820MAX. > > > > The real motivation behind this is actually to prevent some issues in > > the Xen kernel, where 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 > > Suggested-by: Ingo Molnar > > Cc: Russ Anderson > > Cc: David Vrabel > > Cc: Ingo Molnar > > Cc: Juergen Gross > > Cc: Thomas Gleixner > > Cc: "H. Peter Anvin" > > Cc: Denys Vlasenko > > Cc: Boris Ostrovsky > > Cc: x86@kernel.org > > Cc: xen-devel@lists.xenproject.org > > --- > > arch/x86/include/asm/e820.h | 10 ++++++---- > > 1 file changed, 6 insertions(+), 4 deletions(-) > > > > diff --git a/arch/x86/include/asm/e820.h b/arch/x86/include/asm/e820.h > > index 476b574..aa00d33 100644 > > --- a/arch/x86/include/asm/e820.h > > +++ b/arch/x86/include/asm/e820.h > > @@ -1,13 +1,15 @@ > > #ifndef _ASM_X86_E820_H > > #define _ASM_X86_E820_H > > > > -#ifdef CONFIG_EFI > > +/* > > + * We need to make sure that E820_X_MAX is defined > > + * before we include uapi/asm/e820.h > > + */ > > #include > > #define E820_X_MAX (E820MAX + 3 * MAX_NUMNODES) > > What we need an explanation for in the comment is what does this stand for (what > does the 'X' mean?), and what is the magic 3*MAX_NUMNODES about? I'm not actually certain why 3*MAX_NUMNODES was chosen as the max size for this table, but it was written by somebody here at SGI, so I'm sure I can find out :) As for the 'X,' I'm assuming that's meant to imply that this is the maxmimum size for the 'eXtended' table, but that could be made more clear in the comment as well. I'll come up with a better comment for this and submit a v3. Thanks, Ingo!