From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH 2/2] xen/arm: grant-table: Increased the initial number of grant frame to 4 Date: Thu, 15 Jan 2015 15:13:10 +0000 Message-ID: <1421334790.5437.19.camel@citrix.com> References: <1421173041-29870-1-git-send-email-julien.grall@linaro.org> <1421173041-29870-3-git-send-email-julien.grall@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YBm7r-0004vX-D1 for xen-devel@lists.xenproject.org; Thu, 15 Jan 2015 15:14:23 +0000 In-Reply-To: <1421173041-29870-3-git-send-email-julien.grall@linaro.org> 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 Cc: xen-devel@lists.xenproject.org, tim@xen.org, stefano.stabellini@citrix.com List-Id: xen-devel@lists.xenproject.org On Tue, 2015-01-13 at 18:17 +0000, Julien Grall wrote: > When a domain is created on ARM, the grant table code initialized one > grant frame. With a basic load (i.e disk usage), Xen is quickly trying > to expand the number of frames: > > (XEN) grant_table.c:1305:d2v0 Expanding dom (2) grant table from (1) to (2) frames. > (XEN) grant_table.c:1305:d2v0 Expanding dom (2) grant table from (2) to (3) frames. > (XEN) grant_table.c:311:d0v0 Increased maptrack size to 2 frames > (XEN) grant_table.c:311:d0v0 Increased maptrack size to 3 frames > > The x86 code is initialied 4 frames (I didn't find the exact reason). I > think we could use the same default value. > > Signed-off-by: Julien Grall Acked + applied. > --- > xen/include/asm-arm/grant_table.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/include/asm-arm/grant_table.h b/xen/include/asm-arm/grant_table.h > index 88ee3c6..5e076cc 100644 > --- a/xen/include/asm-arm/grant_table.h > +++ b/xen/include/asm-arm/grant_table.h > @@ -3,7 +3,7 @@ > > #include > > -#define INITIAL_NR_GRANT_FRAMES 1 > +#define INITIAL_NR_GRANT_FRAMES 4 > > void gnttab_clear_flag(unsigned long nr, uint16_t *addr); > int create_grant_host_mapping(unsigned long gpaddr,