> -----Original Message----- > From: intel-gvt-dev [mailto:intel-gvt-dev-bounces@xxxxxxxxxxxxxxxxxxxxx] On > Behalf Of Chris Wilson > Sent: Thursday, March 9, 2017 8:45 PM > To: Dong, Chuanxiao <chuanxiao.dong@xxxxxxxxx> > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH] drm/i915/gvt: add enable_execlists check > before enable gvt > > On Thu, Mar 09, 2017 at 12:32:18PM +0000, Dong, Chuanxiao wrote: > > Hi, > > > > Any comments to this patch? > > > > Thanks > > Chuanxiao > > > > > -----Original Message----- > > > From: intel-gvt-dev > > > [mailto:intel-gvt-dev-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of > > > Chuanxiao Dong > > > Sent: Monday, March 6, 2017 1:16 PM > > > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; > > > intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx > > > Subject: [PATCH] drm/i915/gvt: add enable_execlists check before > > > enable gvt > > > > > > The GVT-g needs execlists to be enabled otherwise gvt should be disabled. > > > Add a check for enable_execlists before enabling gvt. > > > > > > Signed-off-by: Chuanxiao Dong <chuanxiao.dong@xxxxxxxxx> > > > --- > > > drivers/gpu/drm/i915/intel_gvt.c | 5 +++++ > > > 1 file changed, 5 insertions(+) > > > > > > diff --git a/drivers/gpu/drm/i915/intel_gvt.c > > > b/drivers/gpu/drm/i915/intel_gvt.c > > > index d23c0fc..3799cb3 100644 > > > --- a/drivers/gpu/drm/i915/intel_gvt.c > > > +++ b/drivers/gpu/drm/i915/intel_gvt.c > > > @@ -77,6 +77,11 @@ int intel_gvt_init(struct drm_i915_private > *dev_priv) > > > goto bail; > > > } > > > > > > + if (!i915.enable_execlists) { > > > + DRM_DEBUG_DRIVER("Execlists unsupported, GVT-g is > > > disabled\n"); > > Should be DRM_INFO() in response to the user action, and should be treated > as a user visible string. > > DRM_INFO("GPU guest virtualisation [GVT-g] disabled due to disabled > execlist submission [i915.enable_execlists module parameter]\n"); -Chris Thanks Chris for the comments. Use "DRM_DEBUG_DRIVER" is just following the same print level with the other fail case in intel_gvt_init(). Anyway, I will take your suggestion to use DRM_INFO instead. The message printed is longer than 80 characters, and checkpatch tool also complain with " quoted string split across lines" by splitting to multiple lines. So should I use the message print like below? DRM_INFO("GPU guest virtualisation [GVT-g] disabled due to "); DRM_INFO("disabled execlist submission "); DRM_INFO("[i915.enable_execlists module parameter]\n"); Or any better idea? Thanks Chuanxiao > > -- > Chris Wilson, Intel Open Source Technology Centre > _______________________________________________ > intel-gvt-dev mailing list > intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
