On Thu, Jan 18, 2018 at 03:34:46PM -0700, 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 <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? It can be easily start to be API, for example "this battle" is not finished yet: https://lwn.net/Articles/442113/ > > > > +++ 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. > > Jason > -- > 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
Attachment:
signature.asc
Description: PGP signature
