Re: [PATCH v1] trace: Relocate event helper files

From: Steven Rostedt
Date: Mon Nov 14 2022 - 10:01:46 EST


On Mon, 14 Nov 2022 09:36:39 -0500
Chuck Lever <chuck.lever@xxxxxxxxxx> wrote:

> drivers/infiniband/core/cm_trace.h | 2
> drivers/infiniband/core/cma_trace.h | 2
> fs/nfs/nfs4trace.h | 6 -
> fs/nfs/nfstrace.h | 6 -
> include/trace/events/fs.h | 122 -----------
> include/trace/events/nfs.h | 375 -----------------------------------
> include/trace/events/rdma.h | 168 ----------------
> include/trace/events/rpcgss.h | 2
> include/trace/events/rpcrdma.h | 4
> include/trace/events/sunrpc.h | 2
> include/trace/events/sunrpc_base.h | 18 --
> include/trace/include/fs.h | 122 +++++++++++
> include/trace/include/nfs.h | 375 +++++++++++++++++++++++++++++++++++
> include/trace/include/rdma.h | 168 ++++++++++++++++
> include/trace/include/sunrpc.h | 18 ++

I'm not so sure I like the "include/trace/include" name. But kernel
developers are known for sucky naming conventions :-p

How about one of:

include/trace/linux/
include/trace/misc/
include/trace/ext/
include/trace/helpers/

?

-- Steve