[PATCH v1] drm/i915/gvt: Fix guest boot warning

Zhenyu Wang zhenyuw at linux.intel.com
Wed Nov 27 04:53:29 UTC 2019


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) = ...

Looks we can do this better in GUC_STATUS handler that we can clear
IN_RESET value after reading.


>  		engine_mask &= INTEL_INFO(vgpu->gvt->dev_priv)->engine_mask;
>  	}
>  
> -- 
> 2.17.1
> 
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev

-- 
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/20191127/dbb2bf02/attachment.sig>


More information about the intel-gvt-dev mailing list