> On Jan 18, 2018, at 5:34 PM, Jason Gunthorpe <jgg@xxxxxxxx> 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 <chuck.lever@xxxxxxxxxx> >>> 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? -- Chuck Lever -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
