From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhenyu Wang Subject: Re: [PATCH] drm/i915/gvt: remove duplicate entry of trace Date: Wed, 12 Jun 2019 11:22:36 +0800 Message-ID: <20190612032236.GH9684@zhen-hp.sh.intel.com> References: <20190526075633.GA9245@hari-Inspiron-1545> Reply-To: Zhenyu Wang Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="JSVXQxoTSdH0Ya++" Return-path: Content-Disposition: inline In-Reply-To: <20190526075633.GA9245@hari-Inspiron-1545> Sender: linux-kernel-owner@vger.kernel.org To: Hariprasad Kelam Cc: Zhi Wang , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , David Airlie , Daniel Vetter , intel-gvt-dev@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org List-Id: intel-gfx@lists.freedesktop.org --JSVXQxoTSdH0Ya++ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2019.05.26 13:26:33 +0530, Hariprasad Kelam wrote: > Remove duplicate include of trace.h >=20 > Issue identified by includecheck >=20 > Signed-off-by: Hariprasad Kelam > --- > drivers/gpu/drm/i915/gvt/trace_points.c | 1 - > 1 file changed, 1 deletion(-) >=20 > diff --git a/drivers/gpu/drm/i915/gvt/trace_points.c b/drivers/gpu/drm/i9= 15/gvt/trace_points.c > index a3deed69..569f5e3 100644 > --- a/drivers/gpu/drm/i915/gvt/trace_points.c > +++ b/drivers/gpu/drm/i915/gvt/trace_points.c > @@ -32,5 +32,4 @@ > =20 > #ifndef __CHECKER__ > #define CREATE_TRACE_POINTS > -#include "trace.h" > #endif > --=20 This actually caused build issue like ERROR: "__tracepoint_gma_index" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_render_mmio" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_gvt_command" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_spt_guest_change" [drivers/gpu/drm/i915/i915.ko] undef= ined! ERROR: "__tracepoint_gma_translate" [drivers/gpu/drm/i915/i915.ko] undefine= d! ERROR: "__tracepoint_spt_alloc" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_spt_change" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_oos_sync" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_write_ir" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_propagate_event" [drivers/gpu/drm/i915/i915.ko] undefi= ned! ERROR: "__tracepoint_inject_msi" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_spt_refcount" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_spt_free" [drivers/gpu/drm/i915/i915.ko] undefined! ERROR: "__tracepoint_oos_change" [drivers/gpu/drm/i915/i915.ko] undefined! scripts/Makefile.modpost:91: recipe for target '__modpost' failed Looks we need fix like below. Subject: [PATCH] drm/i915/gvt: remove duplicate include of trace.h This removes duplicate include of trace.h. Found by Hariprasad Kelam with includecheck. Reported-by: Hariprasad Kelam Signed-off-by: Zhenyu Wang --- drivers/gpu/drm/i915/gvt/trace_points.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i915/gvt/trace_points.c b/drivers/gpu/drm/i915= /gvt/trace_points.c index a3deed692b9c..fe552e877e09 100644 --- a/drivers/gpu/drm/i915/gvt/trace_points.c +++ b/drivers/gpu/drm/i915/gvt/trace_points.c @@ -28,8 +28,6 @@ * */ =20 -#include "trace.h" - #ifndef __CHECKER__ #define CREATE_TRACE_POINTS #include "trace.h" --=20 2.20.1 --=20 Open Source Technology Center, Intel ltd. $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827 --JSVXQxoTSdH0Ya++ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EARECAB0WIQTXuabgHDW6LPt9CICxBBozTXgYJwUCXQBv/AAKCRCxBBozTXgY J6qvAJ9q4SpHBMazUVGyrExwIyT3tac9zQCbB8m/t9a9wlOqkm0W0bss9NDOds0= =lNiG -----END PGP SIGNATURE----- --JSVXQxoTSdH0Ya++--