From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v3] xen: arm: Support <32MB frametables Date: Fri, 11 Sep 2015 15:10:25 +0100 Message-ID: <1441980625.3549.47.camel@citrix.com> References: <1440192637-15007-1-git-send-email-chris.brand@broadcom.com> <55DE5D89.9010308@citrix.com> <1441122232.26292.67.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1441122232.26292.67.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall , Chris Brand , xen-devel@lists.xen.org Cc: Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On Tue, 2015-09-01 at 16:43 +0100, Ian Campbell wrote: > On Wed, 2015-08-26 at 17:44 -0700, Julien Grall wrote: > > Hi Chris, > > > > On 21/08/2015 14:30, Chris Brand wrote: > > > setup_frametable_mappings() rounds frametable_size up to a multiple > > > of 32MB. This is wasteful on systems with less than 4GB of RAM, > > > although it does allow the "contig" bit to be set in the PTEs. > > > > > > Where the frametable is less than 32MB in size, instead round up > > > to a multiple of 2MB, not setting the "contig" bit in the PTEs. > > > > > > Signed-off-by: Chris Brand > > > > Reviewed-by: Julien Grall > > Acked-by: Ian Campbell > > Chris, please ping me if I haven't applied this within some reasonable > period after the tree opens for 4.7 development. Applied.