From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [Xen-users] XEN/arm XENFB support
Date: Tue, 17 Dec 2013 16:50:43 +0000
Message-ID: <1387299043.1025.13.camel@dagon.hellion.org.uk>
References: <WC20131216081609.85000C@perkbv.com>
	<CAOqnZH5=x0xk5UAyyfW-df-SeSvuo=H99XVSEwdkAgRa6hpdqw@mail.gmail.com>
	<1387187949.20076.37.camel@kazak.uk.xensource.com>
	<WC20131217072023.47000F@perkbv.com>
	<alpine.DEB.2.02.1312171244020.8667@kaball.uk.xensource.com>
	<alpine.DEB.2.02.1312171635470.8667@kaball.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <alpine.DEB.2.02.1312171635470.8667@kaball.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Lars Kurth <lars.kurth.xen@gmail.com>, xen-users@lists.xen.org, peter <peter@perkbv.com>, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

On Tue, 2013-12-17 at 16:42 +0000, Stefano Stabellini wrote:

> diff --git a/xen-all.c b/xen-all.c
> index 4a594bd..d45180d 100644
> --- a/xen-all.c
> +++ b/xen-all.c
> @@ -381,8 +381,6 @@ static int xen_remove_from_physmap(XenIOState *state,
>  
>          rc = xc_domain_add_to_physmap(xen_xc, xen_domid, XENMAPSPACE_gmfn, idx, gpfn);
>          if (rc) {
> -            fprintf(stderr, "add_to_physmap MFN %"PRI_xen_pfn" to PFN %"
> -                    PRI_xen_pfn" failed: %d\n", idx, gpfn, rc);

This message seemed useful to me...

>              return -rc;
>          }
>      }
> 
> diff --git a/xen-mapcache.c b/xen-mapcache.c
> index eda914a..baea376 100644
> --- a/xen-mapcache.c
> +++ b/xen-mapcache.c
> @@ -33,13 +33,8 @@
>  #  define DPRINTF(fmt, ...) do { } while (0)
>  #endif
>  
> -#if defined(__i386__)
>  #  define MCACHE_BUCKET_SHIFT 16
>  #  define MCACHE_MAX_SIZE     (1UL<<31) /* 2GB Cap */
> -#elif defined(__x86_64__)
> -#  define MCACHE_BUCKET_SHIFT 20
> -#  define MCACHE_MAX_SIZE     (1UL<<35) /* 32GB Cap */
> -#endif

This bit seems a unrelated? Or should it really be adding __arm__ and
__aarch64__ bits and this was just a quick hack?

It'd be good if someone would pick up Daniel De Graaf's frontend patches
(see earlier in the thread) to use grants instead of priv mappings and
implement the matching backend stuff in qemu...

Ian.