Re: [XenPPC] RFC: xencomm - linux side

2006-09-08 Thread Hollis Blanchard
On Fri, 2006-08-25 at 09:02 +0200, Tristan Gingold wrote: > Le Jeudi 24 Août 2006 17:43, Hollis Blanchard a écrit : > > On Thu, 2006-08-24 at 09:51 +0200, Tristan Gingold wrote: > > > > My suggestion: have xencomm_create() test IS_KERNEL_ADDR() (in whatever > > > > way is best for portability) and

Re: [XenPPC] RFC: xencomm - linux side

2006-08-24 Thread Tristan Gingold
Le Jeudi 24 Août 2006 17:43, Hollis Blanchard a écrit : > On Thu, 2006-08-24 at 09:51 +0200, Tristan Gingold wrote: > > > My suggestion: have xencomm_create() test IS_KERNEL_ADDR() (in whatever > > > way is best for portability) and if it is, do the "inline" stuff. On > > > the free side, if the de

Re: [XenPPC] RFC: xencomm - linux side

2006-08-24 Thread Hollis Blanchard
On Thu, 2006-08-24 at 09:51 +0200, Tristan Gingold wrote: > > > My suggestion: have xencomm_create() test IS_KERNEL_ADDR() (in whatever > > way is best for portability) and if it is, do the "inline" stuff. On the > > free side, if the descriptor was inline, free can just return. That > > would als

Re: [XenPPC] RFC: xencomm - linux side

2006-08-24 Thread Tristan Gingold
Le Mercredi 23 Août 2006 18:35, Hollis Blanchard a écrit : > On Wed, 2006-08-23 at 09:59 +0200, Tristan Gingold wrote: > > Le Mardi 22 Août 2006 21:03, Hollis Blanchard a écrit : > > > I don't understand the point of all these routines if they just call > > > arch_foo anyways. > > > > Sorry I have

Re: [XenPPC] RFC: xencomm - linux side

2006-08-23 Thread Hollis Blanchard
On Wed, 2006-08-23 at 09:59 +0200, Tristan Gingold wrote: > Le Mardi 22 Août 2006 21:03, Hollis Blanchard a écrit : > > I apologize for my mailer line-wrapping the patch as I quote it below. > > > > On Mon, 2006-08-21 at 17:18 +0200, Tristan Gingold wrote: > > > diff -r b7db009d622c linux-2.6-xen-s

Re: [XenPPC] RFC: xencomm - linux side

2006-08-23 Thread Tristan Gingold
Le Mardi 22 Août 2006 21:03, Hollis Blanchard a écrit : > I apologize for my mailer line-wrapping the patch as I quote it below. > > On Mon, 2006-08-21 at 17:18 +0200, Tristan Gingold wrote: > > diff -r b7db009d622c linux-2.6-xen-sparse/drivers/xen/Kconfig > > --- a/linux-2.6-xen-sparse/drivers/xen

Re: [XenPPC] RFC: xencomm - linux side

2006-08-22 Thread Hollis Blanchard
I apologize for my mailer line-wrapping the patch as I quote it below. On Mon, 2006-08-21 at 17:18 +0200, Tristan Gingold wrote: > diff -r b7db009d622c linux-2.6-xen-sparse/drivers/xen/Kconfig > --- a/linux-2.6-xen-sparse/drivers/xen/Kconfig Mon Aug 21 09:41:24 > 2006 +0200 > +++ b/linux-2.6-xen-

Re: [XenPPC] RFC: xencomm - linux side

2006-08-22 Thread Hollis Blanchard
On Tue, 2006-08-22 at 09:42 +0200, Tristan Gingold wrote: > Le Lundi 21 Août 2006 21:07, Hollis Blanchard a écrit : > > On Mon, 2006-08-21 at 17:18 +0200, Tristan Gingold wrote: > > > I am posting the linux xencomm code for review. I'd plan to submit soon > > > unless comments/remarks. > > > > NAK

Re: [XenPPC] RFC: xencomm - linux side

2006-08-22 Thread Tristan Gingold
Le Lundi 21 Août 2006 21:07, Hollis Blanchard a écrit : > On Mon, 2006-08-21 at 17:18 +0200, Tristan Gingold wrote: > > I am posting the linux xencomm code for review. I'd plan to submit soon > > unless comments/remarks. > > NAK. I'm still waiting to hear back about how you can use > xencomm_inlin

Re: [XenPPC] RFC: xencomm - linux side

2006-08-21 Thread Hollis Blanchard
On Mon, 2006-08-21 at 17:18 +0200, Tristan Gingold wrote: > I am posting the linux xencomm code for review. I'd plan to submit soon > unless comments/remarks. NAK. I'm still waiting to hear back about how you can use xencomm_inline() without worrying about page boundaries. -- Hollis Blanchard

[XenPPC] RFC: xencomm - linux side

2006-08-21 Thread Tristan Gingold
Hi, I am posting the linux xencomm code for review. I'd plan to submit soon unless comments/remarks. This patch has 3 aims: * xencomm descriptor creation (drivers/xen/core/xencomm.c) * linux issued hypercalls translation (drivers/xen/core/xencomm_hcall.c) * privcmd hypercalls translation (drive