[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Xen-devel] Re: [ofa-general] Re: mthca use of dma_sync_single is bogus
- To: "Michael S. Tsirkin" <mst@xxxxxxxxxxxxxxxxxx>
- 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;
- List-id: Xen developer discussion <xen-devel.lists.xensource.com>
> 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