Re: [XenPPC] [PATCH] [RFC] Xencomm patch to fix modules

2007-01-18 Thread Hollis Blanchard
On Wed, 2007-01-17 at 21:20 -0600, Jerone Young wrote: I haven't had a chance to fully test this patch. As I have been having problems with my blade today. But here is the code for review. I'll get back to the list once it is fully tested. This looks pretty good; just a couple issues left...

Re: [XenPPC] [PATCH] [RFC] Xencomm patch to fix modules

2007-01-18 Thread Jimi Xenidis
On Jan 18, 2007, at 1:55 PM, Hollis Blanchard wrote: On Thu, 2007-01-18 at 12:17 -0500, Jimi Xenidis wrote: I agree with most of Hollis's comments, but have some of my own. First, I do not think that the implementation of is_phys_contiguous() answers the question in its name and IMNSHO is

Re: [XenPPC] [PATCH] [RFC] Xencomm patch to fix modules

2007-01-18 Thread Hollis Blanchard
On Thu, 2007-01-18 at 16:18 -0500, Jimi Xenidis wrote: On Jan 18, 2007, at 1:55 PM, Hollis Blanchard wrote: On Thu, 2007-01-18 at 12:17 -0500, Jimi Xenidis wrote: I agree with most of Hollis's comments, but have some of my own. First, I do not think that the implementation of

Re: [XenPPC] [PATCH] [RFC] Xencomm patch to fix modules

2007-01-18 Thread Jimi Xenidis
On Jan 18, 2007, at 4:26 PM, Hollis Blanchard wrote: On Thu, 2007-01-18 at 16:18 -0500, Jimi Xenidis wrote: Hey, wouldn't virt_addr_valid() do? I can pass a perfectly valid virtual address that is within a physically discontiguous area of memory, and this function would return 0. hmm,

[XenPPC] [linux-ppc-2.6] [POWERPC][XEN][LINUX] Enable firewall options to avoid breaking init scripts.

2007-01-18 Thread Xen patchbot-linux-ppc-2 . 6
# HG changeset patch # User [EMAIL PROTECTED] # Node ID 5faafe734b0e9e1092400196d3518730d21688ce # Parent bbf2db4ddf5400e908ee6bf92ac798e5cfed82a0 [POWERPC][XEN][LINUX] Enable firewall options to avoid breaking init scripts. Signed-off-by: Jerone Young [EMAIL PROTECTED] Signed-off-by: Hollis