Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2630906imu; Thu, 29 Nov 2018 07:55:47 -0800 (PST) X-Google-Smtp-Source: AFSGD/WOLhnNBfT9APpsrICwXIaOzRIGOAgbu4bjfb43TXFhesAaTQsQJEESIbNwiHB17pqD5LxZ X-Received: by 2002:a63:5153:: with SMTP id r19mr1655571pgl.281.1543506947666; Thu, 29 Nov 2018 07:55:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543506947; cv=none; d=google.com; s=arc-20160816; b=0l+Si00uaEKr0Flw0cB9cPXk7nkLmtq5+8TotRTJGrVPjNnolbtBskDnwH7gw5UkMc 7IPtsHGM/o0GfJ0TERTB8v1+2iv5aM1qDdpBiaJnKvxC3g+ZVmxEtS8Yom5gLhlmn/lZ SxT9w0Igq5WJ/jCZK38q4Pb3ch9DRR4guKyz/MenBMR0nODFecKrlWyz/coNxrzEHPtR ++vAdnUJ/3d+yvAjfuMyASEJp1UOZlRQVRf0e7Yvxdz5bXC+xNFdzX+RNhTmI+3sA70o /dMmjiT85kdsBCD1jQZIUoMkPYtk6Mn6zVw87xwIyWHOnEuUx8DH7VIO6m1eBJLRT7bW 6AVg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:content-transfer-encoding :mime-version:references:in-reply-to:date:cc:to:from:subject; bh=tB1Vtbf0gJ4x0Bk2b4E1g0OHYIwYCKr6tGgCExxt8ME=; b=Wzo9tGHEIYgPmd6H7nI8PwWOg/f63ChnzCcEQVG34MNcXVCWVAbiLqnOXb+R7+JdhW oCccggQVfqboXSAT0PtRH2czdrvaH3jH1eD+XVSCbWn85iWhwY3Kb4qtGN+dCXBd0wlr UhUFzBD8GG3aeLJcuOP0vEe9DZyEhyvsZQmh/3uM7YGOe8OISyIErG4qlnN2gZoehjac W6bj2cq/Xgq0myl/uDuay/aMYB5hJ9tVOr8p5kuviIqoppXyTkqHkYksNbvQbLL7AWPE mS7TP9BgiwBKd6FNDofA5tPe2+5YtKNkfOir7YMu32ZeLcKsNyRsxegELbPGzLo4cpHt 8EQw== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=ibm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id w12si2530328pfn.212.2018.11.29.07.55.30; Thu, 29 Nov 2018 07:55:47 -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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=ibm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729364AbeK3C7G (ORCPT + 99 others); Thu, 29 Nov 2018 21:59:06 -0500 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:52894 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728363AbeK3C7G (ORCPT ); Thu, 29 Nov 2018 21:59:06 -0500 Received: from pps.filterd (m0098393.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id wATFdoXf060506 for ; Thu, 29 Nov 2018 10:53:17 -0500 Received: from e34.co.us.ibm.com (e34.co.us.ibm.com [32.97.110.152]) by mx0a-001b2d01.pphosted.com with ESMTP id 2p2g4x8xby-1 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=NOT) for ; Thu, 29 Nov 2018 10:50:06 -0500 Received: from localhost by e34.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 29 Nov 2018 15:49:07 -0000 Received: from b03cxnp07029.gho.boulder.ibm.com (9.17.130.16) by e34.co.us.ibm.com (192.168.1.134) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; (version=TLSv1/SSLv3 cipher=AES256-GCM-SHA384 bits=256/256) Thu, 29 Nov 2018 15:49:04 -0000 Received: from b03ledav004.gho.boulder.ibm.com (b03ledav004.gho.boulder.ibm.com [9.17.130.235]) by b03cxnp07029.gho.boulder.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id wATFn4dc15925274 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 29 Nov 2018 15:49:04 GMT Received: from b03ledav004.gho.boulder.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id EE2067805E; Thu, 29 Nov 2018 15:49:03 +0000 (GMT) Received: from b03ledav004.gho.boulder.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id BD4CE78066; Thu, 29 Nov 2018 15:49:02 +0000 (GMT) Received: from [153.66.254.194] (unknown [9.85.139.109]) by b03ledav004.gho.boulder.ibm.com (Postfix) with ESMTP; Thu, 29 Nov 2018 15:49:02 +0000 (GMT) Subject: Re: [PATCH] scsi: libsas: Add missing license and update to SPDX license identifier From: James Bottomley To: John Garry , Greg KH Cc: martin.petersen@oracle.com, linuxarm@huawei.com, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org Date: Thu, 29 Nov 2018 07:49:01 -0800 In-Reply-To: <07b65f80-9b73-472f-1fb8-7c0133f1d1ab@huawei.com> 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> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.26.6 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 x-cbid: 18112915-0016-0000-0000-0000095CE2DA X-IBM-SpamModules-Scores: X-IBM-SpamModules-Versions: BY=3.00010143; HX=3.00000242; KW=3.00000007; PH=3.00000004; SC=3.00000270; SDB=6.01124513; UDB=6.00583849; IPR=6.00904640; MB=3.00024384; MTD=3.00000008; XFM=3.00000015; UTC=2018-11-29 15:49:07 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 18112915-0017-0000-0000-0000413B7536 Message-Id: <1543506541.2881.16.camel@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:,, definitions=2018-11-29_09:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1810050000 definitions=main-1811290131 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. > 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. James