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

[Simon Horman <horms@xxxxxxxxxxxx>]

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