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-ppc-devel

Re: [XenPPC] moving HYPERVISOR_grant_table_op may not be the best idea?

To: jyoung5@xxxxxxxxxx
Subject: Re: [XenPPC] moving HYPERVISOR_grant_table_op may not be the best idea?
From: Jimi Xenidis <jimix@xxxxxxxxxxxxxx>
Date: Tue, 23 Jan 2007 21:24:19 -0500
Cc: xen-ppc-devel <xen-ppc-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 23 Jan 2007 18:23:48 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1169569394.12609.2.camel@thinkpad>
List-help: <mailto:xen-ppc-devel-request@lists.xensource.com?subject=help>
List-id: Xen PPC development <xen-ppc-devel.lists.xensource.com>
List-post: <mailto:xen-ppc-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ppc-devel>, <mailto:xen-ppc-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ppc-devel>, <mailto:xen-ppc-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1169569394.12609.2.camel@thinkpad>
Sender: xen-ppc-devel-bounces@xxxxxxxxxxxxxxxxxxx
since those are arch/powerpc/platforms/xen specific stick the protos in setup.h in that directory. That file will need some tending to later, but for now I'd just put it there.
-JX
On Jan 23, 2007, at 11:23 AM, Jerone Young wrote:

So I started to move HYPERVISOR_grant_table_op from
arch/powerpc/platforms/xen/gttab.c to hcall.c. Well this looks like it
may not be the best idea.

The reason is that it calls a static function,
gnttab_pre_unmap_grant_ref, in gttab.c that would need to be added to
include/xen/gnttab.h, and this would be the only needed for power. Where folks can be a bit annoyed by. Don't know if it is the best idea to move
this now? What do think guys think.

Having HYPERVISOR_grant_table_op in hcall.c would have all the
hypercalls in one place, but would require a nice ifdef in a header that
some have been know to take issue with.


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


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

<Prev in Thread] Current Thread [Next in Thread>