Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751259AbaKCXcn (ORCPT ); Mon, 3 Nov 2014 18:32:43 -0500 Received: from www.linutronix.de ([62.245.132.108]:56002 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751168AbaKCXcm (ORCPT ); Mon, 3 Nov 2014 18:32:42 -0500 Date: Tue, 4 Nov 2014 00:32:32 +0100 (CET) From: Thomas Gleixner To: Andy Lutomirski cc: Toshi Kani , "Elliott, Robert (Server Storage)" , "hpa@zytor.com" , "akpm@linux-foundation.org" , "arnd@arndb.de" , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" , "jgross@suse.com" , "stefan.bader@canonical.com" , "hmh@hmh.eng.br" , "yigal@plexistor.com" , "konrad.wilk@oracle.com" , Ingo Molnar , Linus Torvalds , Andrew Morton , Alan Cox , Bdale Garbee Subject: Re: [PATCH v4 4/7] x86, mm, pat: Add pgprot_writethrough() for WT In-Reply-To: Message-ID: References: <1414450545-14028-1-git-send-email-toshi.kani@hp.com> <1414450545-14028-5-git-send-email-toshi.kani@hp.com> <94D0CD8314A33A4D9D801C0FE68B4029593578ED@G9W0745.americas.hpqcorp.net> <1415052905.10958.39.camel@misato.fc.hp.com> User-Agent: Alpine 2.11 (DEB 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001,URIBL_BLOCKED=0.001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 3 Nov 2014, Andy Lutomirski wrote: > On Mon, Nov 3, 2014 at 2:53 PM, Thomas Gleixner wrote: > > On Mon, 3 Nov 2014, Toshi Kani wrote: > >> On Mon, 2014-11-03 at 22:10 +0000, Elliott, Robert (Server Storage) > >> wrote: > >> > > +EXPORT_SYMBOL_GPL(pgprot_writethrough); > >> > ... > >> > > >> > Would you be willing to use EXPORT_SYMBOL for the new > >> > pgprot_writethrough function to provide more flexibility > >> > for modules to utilize the new feature? In x86/mm, 18 of 60 > >> > current exports are GPL and 42 are not GPL. > >> > >> I simply used EXPORT_SYMBOL_GPL() since pgprot_writecombine() used > >> it. :-) This interface is intended to be used along with > >> remap_pfn_range() and ioremap_prot(), which are both exported with > >> EXPORT_SYMBOL(). So, it seems reasonable to export it with > >> EXPORT_SYMBOL() as well. I will make this change. > > > > NAK. > > > > This is new functionality and we really have no reason to give the GPL > > circumventors access to it. > > I have mixed feelings about this. > > On the one hand, I agree with your sentiment. > > On the other hand, I thought that _GPL was supposed to be more about > whether the thing using it is inherently a derived work of the Linux > kernel. Since WT is an Intel concept, not a Linux concept, then I > think that this is a hard argument to make. If your argument stands then almost nothing in Linux which is related to hardware can stand on its own as it is always dependent on the underlying hardware. But that's not true. The software support for that particular hardware feature is always Linux specific. The point about derived work, which Linus made, is that the GPL might not necessarily apply to something which was developed completely independent of Linux in the first place and then adopted via a wrapper layer. This applies pretty much to the odd binary graphics drivers which got retrofitted with a Linux interface by wrapping the existing binary blob. We have always accomodated with this by not changing the replacement interfaces for something with was EXPORT_SYMBOL to EXPORT_SYMBOL_GPL. Though we have forced binary blobs away from abusing stuff by removing such exports; google for the removal of the init_mm export. But that does not mean that we are obliged to expose new Linux infrastucture which supports existing Intel hardware properties to drivers which prefer to be closed for whatever reason. Quite the contrary. We want to expose these new features to the fair players. The HP driver can live with the less performant modes and if it wants to use WT, that's none of our problems at all. Thanks, tglx -- 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/