Re: [Xen-devel] Re: [PATCH] [xen, xencomm] various xencomm fixes (was Re: [XenPPC] Re: [Xen-ia64-devel] [PATCH 1/2] remove xencomm page size limit(xen side))

2007-08-12 Thread Isaku Yamahata
On Fri, Aug 10, 2007 at 01:54:47PM -0500, Hollis Blanchard wrote: I don't see how we'd ever hit this case though, nor why we'd panic the domain. How could paddr_to_maddr() ever return an maddr that doesn't belong to the current domain? If paddr is invalid, an error should be returned from

Re: [PATCH] [xen, xencomm] various xencomm fixes (was Re: [XenPPC] Re: [Xen-ia64-devel] [PATCH 1/2] remove xencomm page size limit(xen side))

2007-08-10 Thread Hollis Blanchard
On Tue, 2007-08-07 at 17:44 +0900, Isaku Yamahata wrote: +/* get_page() to prevent from another vcpu freeing the page */ +static int +xencomm_paddr_to_maddr(unsigned long paddr, unsigned long *maddr, +struct page_info **page) +{ +*maddr = paddr_to_maddr(paddr); +if (*maddr

[PATCH] [xen, xencomm] various xencomm fixes (was Re: [XenPPC] Re: [Xen-ia64-devel] [PATCH 1/2] remove xencomm page size limit(xen side))

2007-08-07 Thread Isaku Yamahata
On Fri, Aug 03, 2007 at 09:43:16AM -0500, Hollis Blanchard wrote: On Fri, 2007-08-03 at 11:12 +0900, Isaku Yamahata wrote: On Thu, Aug 02, 2007 at 10:00:46AM -0500, Hollis Blanchard wrote: On Thu, 2007-08-02 at 11:44 +0900, Isaku Yamahata wrote: But we can issue sequential p2m