RE: Silent data corruption caused by XPC V2.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+	/* ensure that the physically mapped memory is contiguous */
+	DBUG_ON(REGION_NUMBER(vdst) != RGN_KERNEL &&
+		REGION_NUMBER(vdst) != RGN_UNCACHED &&
+		PAGE_SIZE - (vdst & ~PAGE_MASK) < len);
 
I thought that part of the point of this patch was that
by using kmalloc() for the buffer, you actually guaranteed
that the whole thing is contiguous (since kmalloc will give
you an order(N) page for requests greater than a page.

This test is a bit convoluted, but looks it will ping you
whenever a transfer crosses a page boundary, regardless of
whether the pages are contiguous or not.  Certainly the
comment seems tenuously connected to the code.

-Tony
-
: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Sparc Linux]     [DCCP]     [Linux ARM]     [Yosemite News]     [Linux SCSI]     [Linux x86_64]     [Linux for Ham Radio]

  Powered by Linux