From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754828AbYISRV3 (ORCPT ); Fri, 19 Sep 2008 13:21:29 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753154AbYISRVE (ORCPT ); Fri, 19 Sep 2008 13:21:04 -0400 Received: from outbound-sin.frontbridge.com ([207.46.51.80]:18160 "EHLO SG2EHSOBE006.bigfish.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752258AbYISRVB (ORCPT ); Fri, 19 Sep 2008 13:21:01 -0400 X-BigFish: VPS-21(z1039oz1418M1432R98dR1805M936fQ873fnzz10d3izz2f39iz32i6bh43j61h) X-Spam-TCS-SCL: 0:0 X-WSS-ID: 0K7GDI8-04-VNA-01 Date: Fri, 19 Sep 2008 19:20:36 +0200 From: Joerg Roedel To: FUJITA Tomonori CC: mingo@elte.hu, linux-kernel@vger.kernel.org Subject: Re: [PATCH] remove fullflush and nofullflush in IOMMU generic option Message-ID: <20080919172036.GT24392@amd.com> References: <20080920012322R.fujita.tomonori@lab.ntt.co.jp> <20080919164541.GS24392@amd.com> <20080920020917O.fujita.tomonori@lab.ntt.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20080920020917O.fujita.tomonori@lab.ntt.co.jp> User-Agent: mutt-ng/devel-r804 (Linux) X-OriginalArrivalTime: 19 Sep 2008 17:20:36.0595 (UTC) FILETIME=[07CD9430:01C91A7C] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Sep 20, 2008 at 02:09:21AM +0900, FUJITA Tomonori wrote: > On Fri, 19 Sep 2008 18:45:41 +0200 > Joerg Roedel wrote: > > > On Sat, Sep 20, 2008 at 01:23:30AM +0900, FUJITA Tomonori wrote: > > > This patch against tip/x86/iommu virtually reverts > > > 2842e5bf3115193f05dc9dac20f940e7abf44c1a. But just reverting the > > > commit breaks AMD IOMMU so this patch also includes some fixes. > > > > NACK. > > > > > The above commit adds new two options to x86 IOMMU generic kernel boot > > > options, fullflush and nofullflush. But such change that affects all > > > the IOMMUs needs more discussion (all IOMMU parties need the chance to > > > discuss it): > > > > It affects only IOMMUs which use the iommu_fullflush variable. This are > > GART, which used it since ages, and AMD IOMMU which was introduced by > > the above commit. It absolutly makes sense to have command line parameters > > which make sense for more than one or most of the IOMMUs in x86 into > > 'iommu='. Ingo agreed with this, see http://lkml.org/lkml/2008/6/30/155 > > I agree with that too. The commit you are trying to revert here was a > > step into this direction. > > The point is we can't remove or rename VT-d option for IOTLB batching > because we already exported it for users. > > I think that once we export a boot option to users, we can't remove or > rename it. It's the user interface. > > You have a different policy for the kernel boot option? You think we > can change or rename it after exporting it? No. But we can have the generic option in parallel. There is no reason to remove the VT-d specific option. > > Yeah, Intel IOMMU need to keep the current option for IOTLB batching > but it also support fullflush. But we don't discuss anything with > Intel developers. That's what I'm against. Thats why my patch does not change VT-d code. So why do you send revert patches and not starting this discussion? If Muli and the Intel people disagree we can still revert it. > > > 'nofullflush' definitely is pointless. 'nofullflush' option doesn't > > > change any kernel behavior and it was added just for GART > > > compatibility. We should not have such generic meaningless option just > > > for GART. > > > > So why do you keep it in this patch then? > > As I wrote, I think that we can't remove the exported user interfacea You keep this option for AMD IOMMU too. If you move it to AMD IOMMU code then you can remove nofullflush there. > > Please keep it for AMD option for now. Please send a patch to make it > generic to other IOMMU people and give them a chance to discuss on > it. > Ok, I will forward the patch to Intel VT-d maintainers and Muli to discuss it. If they disagree we can revert the patch. Joerg -- | AMD Saxony Limited Liability Company & Co. KG Operating | Wilschdorfer Landstr. 101, 01109 Dresden, Germany System | Register Court Dresden: HRA 4896 Research | General Partner authorized to represent: Center | AMD Saxony LLC (Wilmington, Delaware, US) | General Manager of AMD Saxony LLC: Dr. Hans-R. Deppe, Thomas McCoy