[PATCH v1] drm/i915/gvt: Fix guest boot warning
Zhenyu Wang
zhenyuw at linux.intel.com
Thu Nov 28 02:39:43 UTC 2019
On 2019.11.27 08:51:49 +0000, Gao, Fred wrote:
> Thanks, comments in line.
>
> -----Original Message-----
> From: Zhenyu Wang <zhenyuw at linux.intel.com>
> Sent: 2019年11月27日 12:53
> To: Gao, Fred <fred.gao at intel.com>
> Cc: intel-gvt-dev at lists.freedesktop.org
> Subject: Re: [PATCH v1] drm/i915/gvt: Fix guest boot warning
>
> On 2019.11.27 00:19:04 +0800, Gao, Fred wrote:
> > Simulate MIA core in reset status once GUC engine is reset.
> >
> > Signed-off-by: Gao Fred <fred.gao at intel.com>
> > ---
> > drivers/gpu/drm/i915/gvt/handlers.c | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/gvt/handlers.c
> > b/drivers/gpu/drm/i915/gvt/handlers.c
> > index 3de664deb2f8..78f4d41b9005 100644
> > --- a/drivers/gpu/drm/i915/gvt/handlers.c
> > +++ b/drivers/gpu/drm/i915/gvt/handlers.c
> > @@ -341,6 +341,11 @@ static int gdrst_mmio_write(struct intel_vgpu *vgpu, unsigned int offset,
> > gvt_dbg_mmio("vgpu%d: request VCS2 Reset\n", vgpu->id);
> > engine_mask |= BIT(VCS1);
> > }
> > + if (data & GEN9_GRDOM_GUC) {
> > + gvt_dbg_mmio("vgpu%d: request GUC Reset\n", vgpu->id);
> > + vgpu_vreg(vgpu, i915_mmio_reg_offset(GUC_STATUS))
> > + |= GS_MIA_IN_RESET;
> > + }
>
> Can write as vgpu_vreg_t(vgpu, GUC_STATUS) = ...
> [fred] ok
> Looks we can do this better in GUC_STATUS handler that we can clear IN_RESET value after reading.
> [Fred] since the value is always 1 after guc_reset in native i915, so I do not do the change as well.
> (0x0000c000): 0x00000001
>
As we don't really handle GuC reset now, we just emulate reset status,
if always leaving in_reset bit, guest might think reset is not done yet,
so by clear it after once reading should tell guest reset is not in progress
any more.
--
Open Source Technology Center, Intel ltd.
$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20191128/27d70415/attachment.sig>
More information about the intel-gvt-dev
mailing list