Sorry I sent the wrong patch file. Correct one is attached.

On Tue, 2007-01-09 at 15:12 -0600, Jerone Young wrote:
> This patch removes an invalid optimization that works great if you are a
> kernel address (which is contiguous), but if you are module then you
> have a kernel address but you are not contiguous. So this check is
> invalid.
> _______________________________________________
> Xen-ppc-devel mailing list
> Xen-ppc-devel@lists.xensource.com
> http://lists.xensource.com/xen-ppc-devel
# HG changeset patch
# User [EMAIL PROTECTED]
# Date 1168376635 21600
# Node ID 86833d9eb368e1a8a41fd97e155da58be6b37d54
# Parent  bbf2db4ddf5400e908ee6bf92ac798e5cfed82a0
Remove invalid optimization since this does not correctly check if the address maybe from a kernel module, in which makes this optimization invalid.

Signed-off-by: Jerone Young <[EMAIL PROTECTED]>

diff -r bbf2db4ddf54 -r 86833d9eb368 arch/powerpc/platforms/xen/xencomm.c
--- a/arch/powerpc/platforms/xen/xencomm.c	Tue Dec 19 09:22:37 2006 -0500
+++ b/arch/powerpc/platforms/xen/xencomm.c	Tue Jan 09 15:03:55 2007 -0600
@@ -36,9 +36,6 @@ unsigned long xencomm_vtop(unsigned long
 	if (vaddr == 0)
 		return 0;
 
-	if (is_kernel_addr(vaddr))
-		return __pa(vaddr);
-
 	/* XXX double-check (lack of) locking */
 	vma = find_extend_vma(current->mm, vaddr);
 	BUG_ON(!vma);
_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@lists.xensource.com
http://lists.xensource.com/xen-ppc-devel

Reply via email to