On Mon, 11 Apr 2022, Christoph Hellwig <hch@xxxxxx> wrote: > drivers/gpu/drm/i915/gvt/Makefile is included > from drivers/gpu/drm/i915/Makefile and thus inherits the normal include > path relative to drivers/gpu/drm/i915/. Fix up the gvt-specific trace > header and just do away with the include path manipulation. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > --- > drivers/gpu/drm/i915/gvt/Makefile | 1 - > drivers/gpu/drm/i915/gvt/trace.h | 2 +- > 2 files changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gvt/Makefile b/drivers/gpu/drm/i915/gvt/Makefile > index ea8324abc784a..4d70f4689479c 100644 > --- a/drivers/gpu/drm/i915/gvt/Makefile > +++ b/drivers/gpu/drm/i915/gvt/Makefile > @@ -5,5 +5,4 @@ GVT_SOURCE := gvt.o aperture_gm.o handlers.o vgpu.o trace_points.o firmware.o \ > execlist.o scheduler.o sched_policy.o mmio_context.o cmd_parser.o debugfs.o \ > fb_decoder.o dmabuf.o page_track.o > > -ccflags-y += -I $(srctree)/$(src) -I $(srctree)/$(src)/$(GVT_DIR)/ > i915-y += $(addprefix $(GVT_DIR)/, $(GVT_SOURCE)) > diff --git a/drivers/gpu/drm/i915/gvt/trace.h b/drivers/gpu/drm/i915/gvt/trace.h > index 6d787750d279f..348f57f8301db 100644 > --- a/drivers/gpu/drm/i915/gvt/trace.h > +++ b/drivers/gpu/drm/i915/gvt/trace.h > @@ -379,5 +379,5 @@ TRACE_EVENT(render_mmio, > #undef TRACE_INCLUDE_PATH > #define TRACE_INCLUDE_PATH . > #undef TRACE_INCLUDE_FILE > -#define TRACE_INCLUDE_FILE trace > +#define TRACE_INCLUDE_FILE gvt/trace This is not actually correct. Please see my gvt Makefile cleanup [1][2]. BR, Jani. [1] https://lore.kernel.org/all/20220331082127.432171-1-jani.nikula@xxxxxxxxx/ [2] https://lore.kernel.org/all/20220331082127.432171-2-jani.nikula@xxxxxxxxx/ > #include <trace/define_trace.h> -- Jani Nikula, Intel Open Source Graphics Center