Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932075AbWCNX0c (ORCPT ); Tue, 14 Mar 2006 18:26:32 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932082AbWCNX0c (ORCPT ); Tue, 14 Mar 2006 18:26:32 -0500 Received: from gprs189-60.eurotel.cz ([160.218.189.60]:2450 "EHLO amd.ucw.cz") by vger.kernel.org with ESMTP id S932075AbWCNX0c (ORCPT ); Tue, 14 Mar 2006 18:26:32 -0500 Date: Wed, 15 Mar 2006 00:26:12 +0100 From: Pavel Machek To: Jon Mason Cc: Muli Ben-Yehuda , Andi Kleen , Muli Ben-Yehuda , Linux-Kernel , discuss@x86-64.org, Andrew Morton Subject: Re: [RFC PATCH 3/3] x86-64: Calgary IOMMU - hook it in Message-ID: <20060314232612.GC1785@elf.ucw.cz> References: <20060314082432.GE23631@granada.merseine.nu> <20060314082552.GF23631@granada.merseine.nu> <20060314082634.GG23631@granada.merseine.nu> <20060314230348.GC1579@elf.ucw.cz> <20060314232247.GB7699@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20060314232247.GB7699@us.ibm.com> X-Warning: Reading this can be dangerous to your mental health. User-Agent: Mutt/1.5.9i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2308 Lines: 55 On ?t 14-03-06 17:22:48, Jon Mason wrote: > On Wed, Mar 15, 2006 at 12:03:48AM +0100, Pavel Machek wrote: > > On ?t 14-03-06 10:26:34, Muli Ben-Yehuda wrote: > > > This patch hooks Calgary into the build and the x86-64 IOMMU > > > initialization paths. > > > > > > Signed-Off-By: Muli Ben-Yehuda > > > Signed-Off-By: Jon Mason > > > > > > diff -Naurp --exclude-from /home/muli/w/dontdiff iommu_detected/arch/x86_64/Kconfig linux/arch/x86_64/Kconfig > > > --- iommu_detected/arch/x86_64/Kconfig 2006-03-14 08:58:23.000000000 +0200 > > > +++ linux/arch/x86_64/Kconfig 2006-03-12 10:49:04.000000000 +0200 > > > @@ -372,6 +372,16 @@ config GART_IOMMU > > > and a software emulation used on other systems. > > > If unsure, say Y. > > > > > > +config CALGARY_IOMMU > > > + bool "IBM x366 server IOMMU" > > > + default y > > > + depends on PCI && MPSC && EXPERIMENTAL > > > + help > > > + Support for hardware IOMMUs in IBM's x366 server > > > + systems. The IOMMU can be turned off at runtime with the > > > + iommu=off parameter. Normally the kernel will make the right > > > > Runtime? I think you meant boottime. > > Yes, thanks for pointing it out. > > > > > > + choice by itself. If unsure, say Y. > > > > Eh? How common are those machines? > > While this code is specific to IBM's xSeries systems, it will not hurt > to have it enabled on other systems. The code is intelligent enough to > detect the existence of Calgary chips and, if not there, will go down > the standard path of no-iommu (providing that swiotlb has not been > specified at boottime). If this isn't clear enough in the description, > I can remedy that. No, it was pretty clear. But unless these machines are pretty common, I'd suggest users to say N. ... its like most drivers, it takes space but no other harm. Still we don't want to say "say Y" on all drivers. ...it is not required for boot on IBM x366 machines, is it? Pavel -- 56: MD5.TransformBlock( adNAME, 0, strName.IndexOf( '\0' ), adNAME, 0 ); - 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/