Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752838AbdCHMRj (ORCPT ); Wed, 8 Mar 2017 07:17:39 -0500 Received: from hqemgate15.nvidia.com ([216.228.121.64]:14143 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750728AbdCHMRf (ORCPT ); Wed, 8 Mar 2017 07:17:35 -0500 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Wed, 08 Mar 2017 02:53:13 -0800 Subject: Re: [PATCH 0/1] refcount: restore kref_get and kref_put to non-GPL status To: Greg Kroah-Hartman References: <20170308092549.28982-1-jhubbard@nvidia.com> <20170308094810.GB30552@kroah.com> <20170308101215.GA15198@kroah.com> CC: , Peter Zijlstra , Thomas Gleixner , Linus Torvalds , Ingo Molnar , From: John Hubbard X-Nvconfidentiality: public Message-ID: <6b0551c9-dc34-9e87-c8c9-7bdfea85e1f6@nvidia.com> Date: Wed, 8 Mar 2017 02:52:42 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <20170308101215.GA15198@kroah.com> X-Originating-IP: [10.2.166.22] X-ClientProxiedBy: DRHQMAIL104.nvidia.com (10.27.9.13) To HQMAIL107.nvidia.com (172.20.187.13) Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1790 Lines: 52 On 03/08/2017 02:12 AM, Greg Kroah-Hartman wrote: > On Wed, Mar 08, 2017 at 01:59:33AM -0800, John Hubbard wrote: >> >> >> On 03/08/2017 01:48 AM, Greg Kroah-Hartman wrote: >>> On Wed, Mar 08, 2017 at 01:25:48AM -0800, john.hubbard@gmail.com wrote: >>>> From: John Hubbard >>>> >>>> Hi, >>>> >>>> Say, I'm 99% sure that this was just an oversight, so >>>> I'm sticking my neck out here and floating a patch to >>>> Put Things Back. I'm hoping that there is not some >>>> firm reason to GPL-protect the basic kref_get and >>>> kref_put routines, because when designing some >>>> recent new (open-source, yay!) device drivers, we relied >>>> on this being available, even for MIT-licensed code. >>> >>> MIT-licensed code should be just fine with GPL symbols, just use the >>> correct MODULE_LICENSE() setting and all is good. >> >> Actually, we're still using this license string: >> >> MODULE_LICENSE("MIT"); >> >> which I understand does *not* grant access to GPL symbols. So I guess we'd >> have to switch over to "MIT/GPL", if I understand correctly, in order to be >> all correct here. > > You need to write this as: > MODULE_LICENSE("Dual MIT/GPL"); > for the linker to handle this properly as that is the string it is > looking for. Yes, understood. I thought from your previous response that maybe "MIT" alone was *intended* to provide access, but this makes sense and matches what I'd thought earlier. And of course, your other point (about the header itself being licensed) overrides this, but it's nice to get clarification on this detail, as long as I'm stuck--for now--on "MIT". Appreciate your time and quick responses on this, I know it's probably tiresome to answer these sorts of questions. thanks again, john h > > thanks, > > greg k-h >