Re: [Xen-devel] [PATCH-REWORKED] Added xen_is_contiguous_region

["Jambunathan K" <kjambunathan@xxxxxxxxx>]

On 4/14/07, Keir Fraser <Keir.Fraser@xxxxxxxxxxxx> wrote:

What are you trying to do? There's no reasoning behind this patch, nor any sign off.

 -- Keir

As indicated in the therad ( http://lists.xensource.com/archives/html/xen-devel/2007-04/msg00135.html )  our driver has a 35-bit mask and maintains it's own bounce pools. Our experience with native linux is that this arrangement scales well as against using the shared system pool.

pci_map_single as it stands today sneaks a swiotlb buffer underneath us and we end up implicitly falling back upon the shared pool.

Using the new xen_is_contiguous_region() would enable our driver to fall back upon it's own pool.

This is my first patch to Xen. If you approve of the patch, I will resubmit the changes as required by you.

Jambunathan K.
 

On 13/4/07 20:25, "Jambunathan K" <kjambunathan@xxxxxxxxx> wrote:

Continuation of the following thread
http://lists.xensource.com/archives/html/xen-devel/2007-04/msg00135.html

Find the patch attached.

I haven't compiled the changes for IA64.  Consider changes as indicative and make suitable amends as you deem necessary.

Sorry for the trouble.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel