Re: [Xen-devel] RFC: xencomm in common

[Tristan Gingold <Tristan.Gingold@xxxxxxxx>]

Le Lundi 21 Août 2006 10:11, Keir Fraser a écrit :
> On 21/8/06 9:03 am, "Tristan Gingold" <Tristan.Gingold@xxxxxxxx> wrote:
> > xencomm is the ppc infrastructure to do hypercalls with guest physical
> > addresses instead of virtual address.
> >
> > Because ia64 should also use physicall address, I think it's better to
> > re-use the ppc code and to put into common code.
> >
> > I'd propose to submit this patch is every part is OK (ie no NAK).
> >
> > This patch creates include/xen/xencomm_access.h which is to be included
> > by asm/guest_access.h.  It also creates common/xencomm.c which is
> > compiled only if HAS_XENCOMM is y in Rules.mk.
>
> Fine in principle. Specific comments:
>  * powerpc should be cleaned up at the same time to use the common
> infrastructure. I don't want duplicated code hanging around in arch/powerpc
I have attached a blindly-made patch again powerpc files.
If ppc people could check this, I'd be happy.

>  * The code you add to common/ should obey the coding style of other files
> in that directory
>  * Arguably we should have an asm-generic for the xencomm guest-access
> macros. That's a Linux-ism which I think fits well in this particular case.
Taken into account in this updated patch.

Tristan.

Attachment: 1xencommon.patch
Description: Text Data

Attachment: 3xenppc.patch
Description: Text Data

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