Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Date:   Wed, 29 Nov 2023 10:25:45 -0500
From:   Steven Rostedt <rostedt@goodmis.org>
To:     "Masami Hiramatsu (Google)" <mhiramat@kernel.org>
Cc:     LKML <linux-kernel@vger.kernel.org>,
        Linux Trace Kernel <linux-trace-kernel@vger.kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
        dmaluka@google.com, Sean Paul <seanpaul@chromium.org>,
        Arun Easi <aeasi@marvell.com>, Daniel Wagner <dwagner@suse.de>
Subject: Re: [PATCH] tracing: Allow creating instances with specified system
 events
Message-ID: <20231129102545.56901df5@gandalf.local.home>
In-Reply-To: <20231129235821.d99da161644525a2fa988938@kernel.org>
References: <20231128122117.2276f4a7@gandalf.local.home>
        <20231129235821.d99da161644525a2fa988938@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Wed, 29 Nov 2023 23:58:21 +0900
Masami Hiramatsu (Google) <mhiramat@kernel.org> wrote:

> > - Dynamic events had to be specified directly to still allow them to be
> >   created.  
> 
> I have a question about this point. Does this mean the dynamic event files
> will be created in the instance which limits the "system"?
> For this point, I would like to allow limiting dynamic events on instance too.
> If user needs to use specific one, e.g. synthetic, then it is possible to
> add it to the filter.

I'm going back and forth on this. Here's my thoughts about it.

Arguments for allowing all dynamic events:

1. There's not many. This code is basically a way to keep the overhead of
   an instance down. By removing all static events, it can substantially lower
   the memory footprint.

   As synthetic events are user created, there shouldn't be too many that
   would cause a memory foot print issue, and the user also has a bit of
   control of what events are created.

2. We have no idea what a user may want to do with those events. What if
   the user wants to see an event in that buffer and creates a synthetic
   event for it? Should the kernel create a policy for that?

Arguments against defaulting dynamic events in:

1. The list is created in the kernel. The instance is not created by the
   user and thus the kernel could have more control over it.

2. If the user really wants to have dynamic events, they can create another
   instance with the needed events or use the top level.

Hmm, writing this out, I am now leaning more toward not defaulting dynamic
events in. And if we can create a way to dynamically add and remove events
systems from an instance (via mkdir and rmdir), this should not be an issue.

Anyone else have any thoughts about this?

-- Steve