Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2815941imu; Thu, 29 Nov 2018 10:40:07 -0800 (PST) X-Google-Smtp-Source: AFSGD/WZYVSLJl6ey48hJZtmsvk7JY9Tknv4y94+wYZ4ZJB0J1q/biC9ntRAQuKdAq30mdABToLt X-Received: by 2002:a63:151f:: with SMTP id v31mr2155061pgl.34.1543516807091; Thu, 29 Nov 2018 10:40:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543516807; cv=none; d=google.com; s=arc-20160816; b=GaP/QfXtJBdct+A7Srgno6EBH20rb66LJkTuOVFdvLw4cqoQh9HjTbE/YjPOy0mtsg qFL6ivfBbSBpkI2TWuO+Z7MQjScDVJpK6cW2v8IkqPNwRAzuwlVu4FmTdPplMm7YrtE2 j7O6q7oF5Z6qkY1VgyGJBMRMTOKHUxNo5wsTtxYnvJ7BPwyHHiR7izTW3bO4oNJXIzbR abgrGo4dUwIAB0mRFQ44TU/a3Qv40RDdo6dYANev0SOP/saOF5anOSxIUiQmIzXtXlgt i0c+CRGfSFUHs4zUExVmUtyWwkwfRaaRzEvJNQPIkaXS5BTOyOMUSz3F3Cd9BLZ8cbtB 4Q/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:in-reply-to :mime-version:user-agent:date:message-id:from:cc:references:to :subject; bh=UpQvOTCTbo5G2tRxgHgOU5w5VXLSxLK8kDTOAaRLXa8=; b=xM4BmyCG3Vb7xOZHnudLXFKDWrxX6TD7oVxf1p4F90AgvA0NJrVuAstC5KrfjAtRKG 4CM5AC3yT3giA7UZEjqhuNTJou3TtzxllSmCz2YIo8e+ex9ewrtDEf+7YWTROG2BJUj7 etyoiQdmatpUC/CYlsYugFh9MTnLEBVCZbf625F7Fd8eaeSPOOKQ+ywNY+XzcEG2WNMk GPQsZYxA1QjmJvR3AB6FMOOIa9Etw7ymaxWWJaVAGfEA3jHHIst+JLCYH4k9BD3FuHn1 AwMCNtw7MDkjECD2h1gAG74r7dP7m6wMXTr2MsRPPQQkYedgXwOTqBx+p8mc9Fiawj22 ksNw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y84si2402174pfc.68.2018.11.29.10.39.52; Thu, 29 Nov 2018 10:40:07 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730291AbeK3EhY (ORCPT + 99 others); Thu, 29 Nov 2018 23:37:24 -0500 Received: from szxga07-in.huawei.com ([45.249.212.35]:42898 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728535AbeK3EhY (ORCPT ); Thu, 29 Nov 2018 23:37:24 -0500 Received: from DGGEMS401-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 73E8392566EA; Fri, 30 Nov 2018 01:31:13 +0800 (CST) Received: from [127.0.0.1] (10.202.226.41) by DGGEMS401-HUB.china.huawei.com (10.3.19.201) with Microsoft SMTP Server id 14.3.408.0; Fri, 30 Nov 2018 01:31:07 +0800 Subject: Re: [PATCH] scsi: libsas: Add missing license and update to SPDX license identifier To: James Bottomley , Greg KH References: <1543328132-243266-1-git-send-email-john.garry@huawei.com> <20181127144333.GA11004@kroah.com> <8ae3a984-b4d4-f2eb-4aaa-b460441fa403@huawei.com> <07b65f80-9b73-472f-1fb8-7c0133f1d1ab@huawei.com> <1543506541.2881.16.camel@linux.vnet.ibm.com> CC: , , , From: John Garry Message-ID: <44dc28ee-ad2e-d87d-c2d5-7beb2cf6108e@huawei.com> Date: Thu, 29 Nov 2018 17:31:02 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <1543506541.2881.16.camel@linux.vnet.ibm.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.202.226.41] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 29/11/2018 15:49, James Bottomley wrote: > On Thu, 2018-11-29 at 11:52 +0000, John Garry wrote: > [...] >> Hi Greg, >> >> I also note that currently we have an inconsistency in license of >> sas_init.c: >> >> /* >> * Serial Attached SCSI (SAS) Transport Layer initialization >> * >> * Copyright (C) 2005 Adaptec, Inc. All rights reserved. >> * Copyright (C) 2005 Luben Tuikov >> * >> * This file is licensed under GPLv2. >> * >> * This program is free software; you can redistribute it and/or >> * modify it under the terms of the GNU General Public License as >> * published by the Free Software Foundation; either version 2 of >> the >> * License, or (at your option) any later version. >> * >> * This program is distributed in the hope that it will be useful, >> but >> * WITHOUT ANY WARRANTY; without even the implied warranty of >> * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU >> * General Public License for more details. >> * >> * You should have received a copy of the GNU General Public License >> * along with this program; if not, write to the Free Software >> * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111- >> 1307 >> * USA >> * >> */ >> >> ... >> >> MODULE_AUTHOR("Luben Tuikov "); >> MODULE_DESCRIPTION("SAS Transport Layer"); >> MODULE_LICENSE("GPL v2"); >> >> So the license specifies v2+ but module license states v2. >> >> I could not find a docment for guidance on this. I also note that >> making sas_task.c v2 would mean mixing v2 and v2+ into the module. > > The point here is to get help. There are three pieces of evidence in > the file one for v2+ and two for v2. So you can look in the git tree > to when it was actually contributed: > > commit 2908d778ab3e244900c310974e1fc1c69066e450 > Author: James Bottomley > Date: Tue Aug 29 09:22:51 2006 -0500 > > [SCSI] aic94xx: new driver > > and if you ask the original contributor he can tell you the original > intent was v2 only.If you want to modify all the files in libsas and > aic9xxx to have that SPDX tag. Right, so that would involve modifying the licenses of the currently v2+ libsas files. aic9xx source files are v2 already. > >> At this point I'm reluctant to touch this in case I mess up, but >> there is still the missing license in sas_task.c . > > Again, the tree will tell you. In this case it's > > commit 366ca51f30de1cbb5b356c70b7bb22051c558e41 > Author: James Bottomley > Date: Fri Jan 18 10:47:01 2008 -0600 > > [SCSI] libsas: abstract STP task status into a function > > So that file is a direct extraction from an existing v2 only file in > aic9xxx, so it's licence is also v2 only. ok Thanks, John > > James > > > . >