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

[Xen-devel] Re: [ofa-general] Re: mthca use of dma_sync_single is bogus

To: "Michael S. Tsirkin" <mst@xxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [ofa-general] Re: mthca use of dma_sync_single is bogus
From: Roland Dreier <rdreier@xxxxxxxxx>
Date: Tue, 10 Jul 2007 12:25:59 -0700
Authentication-results: sj-dkim-4; header.From=rdreier@xxxxxxxxx; dkim=pass ( sig from cisco.com/sjdkim4002 verified; );
Cc: Lukas Hejtmanek <xhejtman@xxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, Keir Fraser <keir@xxxxxxxxxxxxx>, general@xxxxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 17 Jul 2007 05:03:31 -0700
Dkim-signature: v=0.5; a=rsa-sha256; q=dns/txt; l=766; t=1184095560; x=1184959560; c=relaxed/simple; s=sjdkim4002; h=Content-Type:From:Subject:Content-Transfer-Encoding:MIME-Version; d=cisco.com; i=rdreier@xxxxxxxxx; z=From:=20Roland=20Dreier=20<rdreier@xxxxxxxxx> |Subject:=20Re=3A=20[ofa-general]=20Re=3A=20mthca=20use=20of=20dma_sync_s ingle=20is=20bogus |Sender:=20; bh=zvNyKWwokPW/Nh8lnxYaY2KAjjlvb9JNTDuA8XdisQ8=; b=JK34xbRBLG9cipazFDQv5uZ/J18fTnxrDUjIDS7rhjoVuLDXxsRtFIT4Scm2Y52zbebq7FUF ajIEN8dRAA71EoMlC4t9jX7VdSAtzNGUy+RsE7lzonOsbG3mTpywMuT0;
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20070710183006.GE11320@xxxxxxxxxxxxxx> (Michael S. Tsirkin's message of "Tue, 10 Jul 2007 21:30:06 +0300")
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: <adalkdpxopo.fsf@xxxxxxxxx> <20070709213913.GB20052@xxxxxxxxxxxxxx> <adamyy4vjo9.fsf@xxxxxxxxx> <20070710071547.GA3814@xxxxxxxxxxxxxx> <adabqekuvde.fsf@xxxxxxxxx> <20070710171142.GC11320@xxxxxxxxxxxxxx> <ada3azwb076.fsf@xxxxxxxxx> <20070710183006.GE11320@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Gnus/5.1007 (Gnus v5.10.7) XEmacs/21.4.20 (linux)
 > Hmm. This means there's no way to sync a range within
 > mapping created with map_sg?

It doesn't seem that there is one right now at least.

 > > It actually doesn't look too bad to replace our use of pci_map_sg()
 > > with dma_map_single(), at least at first glance.  I'll try to write a
 > > patch later.
 > 
 > Well, the reason map_sg is there is presumably because on some
 > architectures it's worth it to try and make the region contigious in DMA 
 > space.
 > But I agree this seems the lesser evil at this point ...

Given that we're already trying to allocate big chunks of physically
contiguous memory, I think that any virtual merging we get is likely
to be of very small benefit.

It is kind of a shame to give this up though.

 - R.

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