[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