> -----Original Message-----
> From: intel-gvt-dev [mailto:[email protected]] On
> Behalf Of Chris Wilson
> Sent: Thursday, March 9, 2017 8:45 PM
> To: Dong, Chuanxiao <[email protected]>
> Cc: [email protected]; [email protected]
> Subject: Re: [Intel-gfx] [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:[email protected]] On Behalf Of
> > > Chuanxiao Dong
> > > Sent: Monday, March 6, 2017 1:16 PM
> > > To: [email protected];
> > > [email protected]
> > > 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 <[email protected]>
> > > ---
> > > 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
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
