Re: [PATCH] drm/xe/uapi: Rename xe perf layer as xe observation layer

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Wed, 03 Jul 2024 09:36:36 -0700, Dixit, Ashutosh wrote:
> 
> On Wed, 03 Jul 2024 09:12:45 -0700, Souza, Jose wrote:
> >
> > On Wed, 2024-07-03 at 08:50 -0700, Dixit, Ashutosh wrote:
> > > On Wed, 03 Jul 2024 08:52:54 -0700, Souza, Jose wrote:
> > > >
> > > > > +static struct ctl_table observation_ctl_table[] = {
> > > > > +	{
> > > > > +	 .procname = "xe_observation_paranoid",
> > > >
> > > > This will lead to /proc/sys/dev/xe/xe_observation_paranoid, is this the intention?
> > > > Before it was /proc/sys/dev/xe/perf_stream_paranoid
> > >
> > > Yes, basically the "perf" name is purged and replaced by "observation" (or
> > > "open a perf stream" is now replaced by "open an observation stream").
> >
> > My main issue is with the "xe/xe_" why duplicate it?
> 
> OK, fair point, I will drop the xe_ so it will be
> /proc/sys/dev/xe/observation_paranoid.

Changed in https://patchwork.freedesktop.org/patch/602182/?series=135667&rev=5

Thanks.
--
Ashutosh