WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH-REWORKED] Added xen_is_contiguous_region
From: "Jambunathan K" <kjambunathan@xxxxxxxxx>
Date: Sat, 14 Apr 2007 01:26:16 +0530
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 13 Apr 2007 12:54:55 -0700
Dkim-signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:references; b=OgQv4P1jmMDdfvRL15GVgGE3mXAesPbGdxD45CZJwobPMq95KKhom/oliKaPPpSg3EJX8/wGuK7IveeCGNAtcvdSMvGHQC7zA5oIaKqBR4OQ3Wn2FirLVzTDl+t4ZmVBlVmVXPYi2vhKHezwdAQyB9FlX06CmqatpC5TBNy7JEo=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:references; b=S+8PLnxrAuJIrw1mR+dhd/EHHfhiDjuQKG63TwOzTJ9+9wA/U9NF0wZGt0angYtQoPShRixE4GGeVAPjOUoSClgCu5/ux3jQHIPHFBB/iJquDcB0MLQJzvy6FfVYTHr6b0fOi0x0JcFbjfjq39LbzWfPhsRq/qBlzWEV4cBysNA=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C2459958.5D6C%Keir.Fraser@xxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <b8e946230704131225s5bdb78depb401085f221c56c1@xxxxxxxxxxxxxx> <C2459958.5D6C%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx


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