Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753865AbYL0KtM (ORCPT ); Sat, 27 Dec 2008 05:49:12 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751486AbYL0Ks5 (ORCPT ); Sat, 27 Dec 2008 05:48:57 -0500 Received: from mx3.mail.elte.hu ([157.181.1.138]:39570 "EHLO mx3.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751124AbYL0Ks4 (ORCPT ); Sat, 27 Dec 2008 05:48:56 -0500 Date: Sat, 27 Dec 2008 11:48:23 +0100 From: Ingo Molnar To: Jeremy Fitzhardinge , Tony Luck Cc: linux-kernel@vger.kernel.org, Xen-devel , the arch/x86 maintainers , Ian Campbell , Becky Bruce , FUJITA Tomonori Subject: Re: [PATCH 0 of 9] swiotlb: use phys_addr_t for pages Message-ID: <20081227104823.GI14639@elte.hu> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.18 (2008-05-17) X-ELTE-VirusStatus: clean X-ELTE-SpamScore: -1.5 X-ELTE-SpamLevel: X-ELTE-SpamCheck: no X-ELTE-SpamVersion: ELTE 2.0 X-ELTE-SpamCheck-Details: score=-1.5 required=5.9 tests=BAYES_00 autolearn=no SpamAssassin version=3.2.3 -1.5 BAYES_00 BODY: Bayesian spam probability is 0 to 1% [score: 0.0000] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2065 Lines: 53 * Jeremy Fitzhardinge wrote: > Hi all, > > Here's a work in progress series whcih does a partial revert of the > previous swiotlb changes, and does a partial replacement with Becky > Bruce's series. > > The most important difference is Becky's use of phys_addr_t rather than > page+offset to represent arbitrary pages. This turns out to be simpler. > > I didn't replicate the map_single_page changes, since I'm not exactly > sure of ppc's requirements here, and it seemed like something that could > be easily added. > > Quick testing showed no problems, but I haven't had the chance to do > anything extensive. > > I've made some small changes to Becky's patches to make them apply, but > I've separated any functional changes into separate patches with > appropriate authorship. looks good to me in principle. Fujita-san, do you have any principal objections against these generalizations? Also, this patch will interact with ia64 materially: Subject: [PATCH 9 of 9] ia64/x86/swiotlb: use enum dma_data_direciton in dma_ops So we'll need a bit more confidence in the testing status of this queue, and we probably want to wait until Tony merged the pending .29 ia64 changes, and also get an Ack from Tony for these bits: arch/ia64/dig/dig_vtd_iommu.c | 9 +++-- arch/ia64/hp/common/hwsw_iommu.c | 22 ++++++++------ arch/ia64/hp/common/sba_iommu.c | 12 ++++--- arch/ia64/include/asm/dma-mapping.h | 29 +++++++++--------- arch/ia64/include/asm/swiotlb.h | 26 ++++++++++------ arch/ia64/kernel/machvec.c | 6 ++- arch/ia64/sn/kernel/io_common.c | 3 +- arch/ia64/sn/pci/pci_dma.c | 21 ++++++++----- arch/ia64/sn/pci/pcibr/pcibr_dma.c | 3 +- arch/ia64/sn/pci/tioca_provider.c | 3 +- (Tony Cc:-ed) Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/