Return-Path: Received: from mx143.netapp.com ([216.240.21.24]:19472 "EHLO mx143.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752335AbeEGO2U (ORCPT ); Mon, 7 May 2018 10:28:20 -0400 Subject: Re: [PATCH v1 01/19] xprtrdma: Add proper SPDX tags for NetApp-contributed source To: Chuck Lever CC: , Linux NFS Mailing List References: <20180504192748.31688.3678.stgit@manet.1015granger.net> <20180504193432.31688.65935.stgit@manet.1015granger.net> <4da36188-17d2-7b68-2974-80402c96359c@Netapp.com> From: Anna Schumaker Message-ID: <60fdeb9b-f11b-9e29-2609-2f3da2a57d20@Netapp.com> Date: Mon, 7 May 2018 10:28:13 -0400 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Sender: linux-nfs-owner@vger.kernel.org List-ID: On 05/07/2018 10:11 AM, Chuck Lever wrote: > Hi Anna- > > Thanks for the review! > > >> On May 7, 2018, at 9:27 AM, Anna Schumaker wrote: >> >> Hi Chuck, >> >> On 05/04/2018 03:34 PM, Chuck Lever wrote: >>> Signed-off-by: Chuck Lever >>> --- >>> include/linux/sunrpc/rpc_rdma.h | 1 + >>> include/linux/sunrpc/xprtrdma.h | 1 + >>> net/sunrpc/xprtrdma/module.c | 1 + >>> net/sunrpc/xprtrdma/rpc_rdma.c | 1 + >>> net/sunrpc/xprtrdma/transport.c | 1 + >>> net/sunrpc/xprtrdma/verbs.c | 1 + >>> net/sunrpc/xprtrdma/xprt_rdma.h | 1 + >>> 7 files changed, 7 insertions(+) >>> >>> diff --git a/include/linux/sunrpc/rpc_rdma.h b/include/linux/sunrpc/rpc_rdma.h >>> index 8f144db..92d182f 100644 >>> --- a/include/linux/sunrpc/rpc_rdma.h >>> +++ b/include/linux/sunrpc/rpc_rdma.h >>> @@ -1,3 +1,4 @@ >>> +/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */ >>> /* >>> * Copyright (c) 2015-2017 Oracle. All rights reserved. >>> * Copyright (c) 2003-2007 Network Appliance, Inc. All rights reserved. >>> diff --git a/include/linux/sunrpc/xprtrdma.h b/include/linux/sunrpc/xprtrdma.h >>> index 5859563..86fc38f 100644 >>> --- a/include/linux/sunrpc/xprtrdma.h >>> +++ b/include/linux/sunrpc/xprtrdma.h >>> @@ -1,3 +1,4 @@ >>> +/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */ >>> /* >>> * Copyright (c) 2003-2007 Network Appliance, Inc. All rights reserved. >>> * >>> diff --git a/net/sunrpc/xprtrdma/module.c b/net/sunrpc/xprtrdma/module.c >>> index a762d19..f338065 100644 >>> --- a/net/sunrpc/xprtrdma/module.c >>> +++ b/net/sunrpc/xprtrdma/module.c >>> @@ -1,3 +1,4 @@ >>> +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause >> >> I'm not familiar with hte SPDX-License-Identifier tag. Is there a reason it has to exist in a separate comment block at the top of the file instead of getting rolled in with the copyright stuff right below it? > > I believe that each tag is meant to be parsed by a script > to produce a license manifest file for packaging. Thus the > tag is maintained on a separate line using a specific > format. > > More information is at https://spdx.org I'll take a look there, thanks! > > >> Either way, can you use the C-style ("/* ... */") comments here (and in a few other places below) for consistency? > > Here is a mechanical survey of familiar kernel source > files that already have an SPDX tag. > > [cel@klimt linux]$ grep SPDX net/sunrpc/* > grep: net/sunrpc/auth_gss: Is a directory > net/sunrpc/auth_null.c:// SPDX-License-Identifier: GPL-2.0 > net/sunrpc/auth_unix.c:// SPDX-License-Identifier: GPL-2.0 > net/sunrpc/debugfs.c:// SPDX-License-Identifier: GPL-2.0 > net/sunrpc/Makefile:# SPDX-License-Identifier: GPL-2.0 > net/sunrpc/netns.h:/* SPDX-License-Identifier: GPL-2.0 */ > net/sunrpc/xprtmultipath.c:// SPDX-License-Identifier: GPL-2.0 > grep: net/sunrpc/xprtrdma: Is a directory > net/sunrpc/xprtsock.c:// SPDX-License-Identifier: GPL-2.0 > [cel@klimt linux]$ grep SPDX fs/nfs/* > grep: fs/nfs/blocklayout: Is a directory > fs/nfs/cache_lib.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/cache_lib.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/callback.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/callback.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/callback_proc.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/callback_xdr.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/delegation.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/dns_resolve.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/dns_resolve.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/export.c:// SPDX-License-Identifier: GPL-2.0 > grep: fs/nfs/filelayout: Is a directory > grep: fs/nfs/flexfilelayout: Is a directory > fs/nfs/internal.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/io.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/iostat.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/Makefile:# SPDX-License-Identifier: GPL-2.0 > fs/nfs/mount_clnt.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/netns.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfs2xdr.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs3acl.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs3_fs.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfs3proc.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs3xdr.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs42.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfs42proc.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs42xdr.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs4file.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs4_fs.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfs4getroot.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs4namespace.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs4session.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfs4sysctl.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs4trace.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfs4trace.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfs.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/nfsroot.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfstrace.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/nfstrace.h:/* SPDX-License-Identifier: GPL-2.0 */ > fs/nfs/proc.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/symlink.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/sysctl.c:// SPDX-License-Identifier: GPL-2.0 > fs/nfs/unlink.c:// SPDX-License-Identifier: GPL-2.0 > [cel@klimt linux]$ > > The tags I've proposed are consistent with other usage: > > -> .c files use // ... comments > -> .h files use /* ... */ comments > -> Makefiles use # comments > > There were no complaints from checkpatch.pl about the > comment style in my patch. Ah, okay. It's probably best to go with how everybody else uses it (although I still wonder why they use different styles for .c and .h files). I'll take your patch the way it is now. > > >> Thanks, >> Anna >> >>> /* >>> * Copyright (c) 2015, 2017 Oracle. All rights reserved. >>> */ >>> diff --git a/net/sunrpc/xprtrdma/rpc_rdma.c b/net/sunrpc/xprtrdma/rpc_rdma.c >>> index e8adad3..8f89e3f 100644 >>> --- a/net/sunrpc/xprtrdma/rpc_rdma.c >>> +++ b/net/sunrpc/xprtrdma/rpc_rdma.c >>> @@ -1,3 +1,4 @@ >>> +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause >>> /* >>> * Copyright (c) 2014-2017 Oracle. All rights reserved. >>> * Copyright (c) 2003-2007 Network Appliance, Inc. All rights reserved. >>> diff --git a/net/sunrpc/xprtrdma/transport.c b/net/sunrpc/xprtrdma/transport.c >>> index cc1aad3..4717578 100644 >>> --- a/net/sunrpc/xprtrdma/transport.c >>> +++ b/net/sunrpc/xprtrdma/transport.c >>> @@ -1,3 +1,4 @@ >>> +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause >>> /* >>> * Copyright (c) 2014-2017 Oracle. All rights reserved. >>> * Copyright (c) 2003-2007 Network Appliance, Inc. All rights reserved. >>> diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c >>> index c345d36..10f5032 100644 >>> --- a/net/sunrpc/xprtrdma/verbs.c >>> +++ b/net/sunrpc/xprtrdma/verbs.c >>> @@ -1,3 +1,4 @@ >>> +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause >>> /* >>> * Copyright (c) 2014-2017 Oracle. All rights reserved. >>> * Copyright (c) 2003-2007 Network Appliance, Inc. All rights reserved. >>> diff --git a/net/sunrpc/xprtrdma/xprt_rdma.h b/net/sunrpc/xprtrdma/xprt_rdma.h >>> index cb41b12..e83ba758 100644 >>> --- a/net/sunrpc/xprtrdma/xprt_rdma.h >>> +++ b/net/sunrpc/xprtrdma/xprt_rdma.h >>> @@ -1,3 +1,4 @@ >>> +/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */ >>> /* >>> * Copyright (c) 2014-2017 Oracle. All rights reserved. >>> * Copyright (c) 2003-2007 Network Appliance, Inc. All rights reserved. >>> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > Chuck Lever > > >