Re: [PATCH 05/10] drm/i915: skip DRM_I915_LOW_LEVEL_TRACEPOINTS with NOTRACE

2024-04-10 Thread Steven Rostedt
On Wed, 10 Apr 2024 13:08:57 +0200 Sebastian Andrzej Siewior wrote: > On 2024-04-09 11:55:33 [-0400], Steven Rostedt wrote: > > I believe you need to do it in the .c file: > > > > Can you try something like this? > > > … > > ? > > I tried and nothing changed because the lowlevel config op

Re: [PATCH 05/10] drm/i915: skip DRM_I915_LOW_LEVEL_TRACEPOINTS with NOTRACE

2024-04-10 Thread Sebastian Andrzej Siewior
On 2024-04-09 11:55:33 [-0400], Steven Rostedt wrote: > I believe you need to do it in the .c file: > > Can you try something like this? > … > ? I tried and nothing changed because the lowlevel config option isn't the problem. What about I drop this and replace 4/10 from this series with the pa

Re: [PATCH 05/10] drm/i915: skip DRM_I915_LOW_LEVEL_TRACEPOINTS with NOTRACE

2024-04-09 Thread Steven Rostedt
On Tue, 9 Apr 2024 13:06:01 +0200 Sebastian Andrzej Siewior wrote: > among a few other things I was not aware of. > So yes, this patch is not needed since it makes no difference but I still > have trace points I would rather not have. > If you a clue how to deal with this properly, I am all yours

Re: [PATCH 05/10] drm/i915: skip DRM_I915_LOW_LEVEL_TRACEPOINTS with NOTRACE

2024-04-09 Thread Sebastian Andrzej Siewior
On 2024-04-08 13:06:50 [-0400], Steven Rostedt wrote: > On Fri, 5 Apr 2024 16:18:23 +0200 > Sebastian Andrzej Siewior wrote: > > > The order of the header files is important. If this header file is > > included after tracepoint.h was included then the NOTRACE here becomes a > > nop. Currently th

Re: [PATCH 05/10] drm/i915: skip DRM_I915_LOW_LEVEL_TRACEPOINTS with NOTRACE

2024-04-08 Thread Steven Rostedt
On Fri, 5 Apr 2024 16:18:23 +0200 Sebastian Andrzej Siewior wrote: > The order of the header files is important. If this header file is > included after tracepoint.h was included then the NOTRACE here becomes a > nop. Currently this happens for two .c files which use the tracepoitns > behind DRM

[PATCH 05/10] drm/i915: skip DRM_I915_LOW_LEVEL_TRACEPOINTS with NOTRACE

2024-04-05 Thread Sebastian Andrzej Siewior
The order of the header files is important. If this header file is included after tracepoint.h was included then the NOTRACE here becomes a nop. Currently this happens for two .c files which use the tracepoitns behind DRM_I915_LOW_LEVEL_TRACEPOINTS. Cc: Steven Rostedt Signed-off-by: Sebastian And