[PATCH] drm/i915/gvt: Add some regs to force-to-nonpriv whitelist

[Zhang, Tina]

Hi,

We need this patch to solve the complains like this:

[70811.201818] gvt: vgpu(1) Invalid FORCE_NONPRIV write 2341 at offset 24d8
[70811.201825] gvt: vgpu(1) Invalid FORCE_NONPRIV write 2351 at offset 24dc
[70811.201831] gvt: vgpu(1) Invalid FORCE_NONPRIV write 10000d82 at offset 24e0
[70811.201837] gvt: vgpu(1) Invalid FORCE_NONPRIV write 10064844 at offset 24e4

BR,
Tina

> -----Original Message-----
> From: Zhang, Tina <tina.zhang at intel.com>
> Sent: Monday, March 2, 2020 4:32 PM
> Cc: Zhang, Tina <tina.zhang at intel.com>; intel-gvt-dev at lists.freedesktop.org;
> zhenyuw at linux.intel.com
> Subject: [PATCH] drm/i915/gvt: Add some regs to force-to-nonpriv whitelist
> 
> Those regs are added in order to slove the following complains:
> 
>  [70811.201818] gvt: vgpu(1) Invalid FORCE_NONPRIV write 2341 at offset
> 24d8  [70811.201825] gvt: vgpu(1) Invalid FORCE_NONPRIV write 2351 at
> offset 24dc  [70811.201831] gvt: vgpu(1) Invalid FORCE_NONPRIV write
> 10000d82 at offset 24e0  [70811.201837] gvt: vgpu(1) Invalid
> FORCE_NONPRIV write 10064844 at offset 24e4
> 
> So solve them by adding the required regs to the whitelist.
> 
> Signed-off-by: Tina Zhang <tina.zhang at intel.com>
> ---
>  drivers/gpu/drm/i915/gvt/handlers.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gvt/handlers.c
> b/drivers/gpu/drm/i915/gvt/handlers.c
> index 1793f6991fa8..0946d5618d29 100644
> --- a/drivers/gpu/drm/i915/gvt/handlers.c
> +++ b/drivers/gpu/drm/i915/gvt/handlers.c
> @@ -460,11 +460,14 @@ static int pipeconf_mmio_write(struct intel_vgpu
> *vgpu, unsigned int offset,
>  	return 0;
>  }
> 
> -/* ascendingly sorted */
> +/* sorted in ascending order */
>  static i915_reg_t force_nonpriv_white_list[] = {
> +	_MMIO(0xd80),
>  	GEN9_CS_DEBUG_MODE1, //_MMIO(0x20ec)
>  	GEN9_CTX_PREEMPT_REG,//_MMIO(0x2248)
> -	PS_INVOCATION_COUNT,//_MMIO(0x2348)
> +	CL_PRIMITIVES_COUNT, //_MMIO(0x2340)
> +	PS_INVOCATION_COUNT, //_MMIO(0x2348)
> +	PS_DEPTH_COUNT, //_MMIO(0x2350)
>  	GEN8_CS_CHICKEN1,//_MMIO(0x2580)
>  	_MMIO(0x2690),
>  	_MMIO(0x2694),
> @@ -489,6 +492,7 @@ static i915_reg_t force_nonpriv_white_list[] = {
>  	_MMIO(0xe18c),
>  	_MMIO(0xe48c),
>  	_MMIO(0xe5f4),
> +	_MMIO(0x64844),
>  };
> 
>  /* a simple bsearch */
> --
> 2.17.1