Subject: Re: [PATCH 2/2] xen/x86: Increase xen_e820_map to E820_X_MAX possible
 entries
To: Alex Thorlton <athorlton@sgi.com>, linux-kernel@vger.kernel.org
References: <1479168677-23633-1-git-send-email-athorlton@sgi.com>
 <1480960154-211541-1-git-send-email-athorlton@sgi.com>
 <1480960154-211541-3-git-send-email-athorlton@sgi.com>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>,
        Russ Anderson <rja@sgi.com>, David Vrabel <david.vrabel@citrix.com>,
        Ingo Molnar <mingo@redhat.com>, Thomas Gleixner <tglx@linutronix.de>,
        "H. Peter Anvin" <hpa@zytor.com>, Denys Vlasenko <dvlasenk@redhat.com>,
        x86@kernel.org, xen-devel@lists.xenproject.org
From: Juergen Gross <jgross@suse.com>
Message-ID: <70fc1f05-1a2e-53a1-91f9-30451f683eda@suse.com>
Date: Fri, 9 Dec 2016 11:12:54 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.5.1
MIME-Version: 1.0
In-Reply-To: <1480960154-211541-3-git-send-email-athorlton@sgi.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 878
Lines: 22

On 05/12/16 18:49, 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_memory_map) to attempt to return an e820 table with
> more than 128 entries.  This callback adds entries to the BIOS-provided
> e820 table to account for IOAPIC registers, which, on sufficiently large
> systems, can result in an e820 table that is too large to copy back into
> xen_e820_map.
> 
> This change simply increases the size of xen_e820_map to E820_X_MAX to
> ensure that there is enough room to store the entire e820 map returned
> from this callback.
> 
> Signed-off-by: Alex Thorlton <athorlton@sgi.com>
> Suggested-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Reviewed-by: Juergen Gross <jgross@suse.com>


Commited to xen/tip.git for-linus-4.10


Juergen