WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Enlarge the size of the global mmio_space mmio[]

To: "Cui, Dexuan" <dexuan.cui@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Enlarge the size of the global mmio_space mmio[].
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Sun, 8 Nov 2009 00:07:55 +1100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Sat, 07 Nov 2009 05:08:20 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <EADF0A36011179459010BDF5142A457501CFD04BC0@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <EADF0A36011179459010BDF5142A457501CFD04BC0@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.20 (2009-06-14)
On Fri, Nov 06, 2009 at 05:29:12PM +0800, Cui, Dexuan wrote:
> With the Multi-Function passthrough, we're actually able to assign more than
> 32 functions to guest, so we should enlarge the MAX_MMIO. 1024 should be big
> enough.

Would 265 (= 32 * 8) be big enough?
Or is it harmless to go to 1024 anyway?

> 
> Signed-off-by: Dexuan Cui <dexuan.cui@xxxxxxxxx>
> 
> diff --git a/i386-dm/exec-dm.c b/i386-dm/exec-dm.c
> index d0685af..2603de1 100644
> --- a/i386-dm/exec-dm.c
> +++ b/i386-dm/exec-dm.c
> @@ -275,13 +275,13 @@ void cpu_abort(CPUState *env, const char *fmt, ...)
>  
>  
>  /* XXX: Simple implementation. Fix later */
> -#define MAX_MMIO 32
> +#define MAX_MMIO 1024
>  static struct mmio_space {
>          target_phys_addr_t start;
>          unsigned long size;
>          unsigned long io_index;
>  } mmio[MAX_MMIO];
> -unsigned long mmio_cnt;
> +static unsigned long mmio_cnt;
>  
>  /* register physical memory. 'size' must be a multiple of the target
>     page size. If (phys_offset & ~TARGET_PAGE_MASK) != 0, then it is an


> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel