From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH v1 01/12] rdma/ib: Add trace point macros to display human-readable values Date: Fri, 19 Jan 2018 12:58:20 -0700 Message-ID: <20180119195820.GC3437@ziepe.ca> References: <20171220210236.29321.59307.stgit@manet.1015granger.net> <20171220213023.29321.97527.stgit@manet.1015granger.net> <1245d090-99ee-c517-5f8f-afdb7ffc0456@Netapp.com> <20180118223446.GB17294@ziepe.ca> <73857AF2-BE14-451B-AC9D-1F2B2395CCB7@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <73857AF2-BE14-451B-AC9D-1F2B2395CCB7-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Chuck Lever Cc: Anna Schumaker , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linux NFS Mailing List List-Id: linux-rdma@vger.kernel.org On Fri, Jan 19, 2018 at 11:19:20AM -0500, Chuck Lever wrote: > > > > On Jan 18, 2018, at 5:34 PM, Jason Gunthorpe wrote: > > > > On Thu, Jan 18, 2018 at 04:49:20PM -0500, Anna Schumaker wrote: > >> Hi Folks, > >> > >> On 12/20/2017 04:30 PM, Chuck Lever wrote: > >>> These can be shared with all kernel ULPs, and more can easily be > >>> added as needed. > >>> > >>> Note: checkpatch.pl has some heartburn with the TRACE_DEFINE_ENUM > >>> macros and the LIST macros. These follow the same style as other > >>> header files under include/tracing/events , thus should be > >>> considered acceptable exceptions. > >>> > >>> Signed-off-by: Chuck Lever > >>> include/trace/events/rdma.h | 128 +++++++++++++++++++++++++++++++++++++++++++ > >>> 1 file changed, 128 insertions(+) > >>> create mode 100644 include/trace/events/rdma.h > >> > >> Just checking that it's okay to send this through the NFS tree, > >> since it goes along with the rest of Chuck's patches? > > > > New file, won't create any conflicts, so should be fine. > > > > Is this API or anything that needs a really special look? > > > >>> +++ b/include/trace/events/rdma.h > >>> @@ -0,0 +1,128 @@ > >>> +/* > >>> + * Copyright (c) 2017 Oracle. All rights reserved. > >>> + */ > > > > This should have a SPDX header though, from what I understand. > > If Anna adds SPDX headers to the files introduced in this series, > can we get an Acked-by for this patch from one of the RDMA > maintainers? Or is that not necessary? Probably a MAINTAINERS file entry too. The patch looks OK to me, it is just boilerplate to make the enums work right. Yes, you should have Ack's from maintainers when going through another tree. Send a v2 of this patch and I can ack it for you. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f68.google.com ([74.125.82.68]:35139 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755940AbeAST6Z (ORCPT ); Fri, 19 Jan 2018 14:58:25 -0500 Received: by mail-wm0-f68.google.com with SMTP id r78so5790704wme.0 for ; Fri, 19 Jan 2018 11:58:25 -0800 (PST) Date: Fri, 19 Jan 2018 12:58:20 -0700 From: Jason Gunthorpe To: Chuck Lever Cc: Anna Schumaker , linux-rdma@vger.kernel.org, Linux NFS Mailing List Subject: Re: [PATCH v1 01/12] rdma/ib: Add trace point macros to display human-readable values Message-ID: <20180119195820.GC3437@ziepe.ca> References: <20171220210236.29321.59307.stgit@manet.1015granger.net> <20171220213023.29321.97527.stgit@manet.1015granger.net> <1245d090-99ee-c517-5f8f-afdb7ffc0456@Netapp.com> <20180118223446.GB17294@ziepe.ca> <73857AF2-BE14-451B-AC9D-1F2B2395CCB7@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <73857AF2-BE14-451B-AC9D-1F2B2395CCB7@oracle.com> Sender: linux-nfs-owner@vger.kernel.org List-ID: On Fri, Jan 19, 2018 at 11:19:20AM -0500, Chuck Lever wrote: > > > > On Jan 18, 2018, at 5:34 PM, Jason Gunthorpe wrote: > > > > On Thu, Jan 18, 2018 at 04:49:20PM -0500, Anna Schumaker wrote: > >> Hi Folks, > >> > >> On 12/20/2017 04:30 PM, Chuck Lever wrote: > >>> These can be shared with all kernel ULPs, and more can easily be > >>> added as needed. > >>> > >>> Note: checkpatch.pl has some heartburn with the TRACE_DEFINE_ENUM > >>> macros and the LIST macros. These follow the same style as other > >>> header files under include/tracing/events , thus should be > >>> considered acceptable exceptions. > >>> > >>> Signed-off-by: Chuck Lever > >>> include/trace/events/rdma.h | 128 +++++++++++++++++++++++++++++++++++++++++++ > >>> 1 file changed, 128 insertions(+) > >>> create mode 100644 include/trace/events/rdma.h > >> > >> Just checking that it's okay to send this through the NFS tree, > >> since it goes along with the rest of Chuck's patches? > > > > New file, won't create any conflicts, so should be fine. > > > > Is this API or anything that needs a really special look? > > > >>> +++ b/include/trace/events/rdma.h > >>> @@ -0,0 +1,128 @@ > >>> +/* > >>> + * Copyright (c) 2017 Oracle. All rights reserved. > >>> + */ > > > > This should have a SPDX header though, from what I understand. > > If Anna adds SPDX headers to the files introduced in this series, > can we get an Acked-by for this patch from one of the RDMA > maintainers? Or is that not necessary? Probably a MAINTAINERS file entry too. The patch looks OK to me, it is just boilerplate to make the enums work right. Yes, you should have Ack's from maintainers when going through another tree. Send a v2 of this patch and I can ack it for you. Jason