[PATCH] drm/i915/gvt: Stop initializing pvinfo through reading mmio

Zhenyu Wang zhenyuw at linux.intel.com
Tue Nov 5 02:54:07 UTC 2019


On 2019.11.04 13:31:48 +0800, Tina Zhang wrote:
> The region of pvinfo is reserved for communication between a VMM and
> the GPU driver executing on a virtual machine. HW doesn't have any
> backing mmio store support for the pvinfo region, thus accessing to
> this range through MMIO read/write from host side is forbidden which
> is regarded as unclaimed register access.
> 
> This patch leaves pvinfo range be initialized with zero.
> 
> Signed-off-by: Tina Zhang <tina.zhang at intel.com>
> ---
>  drivers/gpu/drm/i915/gvt/handlers.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/gvt/handlers.c b/drivers/gpu/drm/i915/gvt/handlers.c
> index afd7f66bdc2d..bd12af349123 100644
> --- a/drivers/gpu/drm/i915/gvt/handlers.c
> +++ b/drivers/gpu/drm/i915/gvt/handlers.c
> @@ -3420,6 +3420,10 @@ int intel_gvt_for_each_tracked_mmio(struct intel_gvt *gvt,
>  	}
>  
>  	for (i = 0; i < gvt->mmio.num_mmio_block; i++, block++) {
> +		/* pvinfo data doesn't come from hw mmio */
> +		if (i915_mmio_reg_offset(block->offset) == VGT_PVINFO_PAGE)
> +			continue;

Equal to remove PVINFO in mmio_block list? Could we just remove it there instead
of this special case?

> +
>  		for (j = 0; j < block->size; j += 4) {
>  			ret = handler(gvt,
>  				      i915_mmio_reg_offset(block->offset) + j,
> -- 
> 2.17.1
> 

-- 
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/20191105/103d21ed/attachment.sig>


More information about the intel-gvt-dev mailing list