[PATCH 1/2] drm/i915/gvt: Refine the snapshort range of I915 MCHBAR to optimize gvt-g boot time

Zhenyu Wang zhenyuw at linux.intel.com
Mon Feb 18 07:54:01 UTC 2019


On 2019.02.14 17:09:16 +0800, Zhao Yakui wrote:
> Currently it will take the snapshot of the MCHBAR registers for gvt-g
> initialization so that it can be used for guest vgpu. And it will cover
> from 0x140000 to 0x17ffff. In fact based on the HW spec most of them are
> meanlingless and some time is wasted to read these register.
> Only the range of 0x144000 to 0x147fff contains the valid definition.
> So the range of capturing I915 MCHBAR register is refined, which helps
> to optimize the gvt-g boot time.
> 
> Signed-off-by: Zhao Yakui <yakui.zhao at intel.com>
> ---
>  drivers/gpu/drm/i915/gvt/handlers.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/gvt/handlers.c b/drivers/gpu/drm/i915/gvt/handlers.c
> index bc64b81..0877678 100644
> --- a/drivers/gpu/drm/i915/gvt/handlers.c
> +++ b/drivers/gpu/drm/i915/gvt/handlers.c
> @@ -47,6 +47,8 @@
>  #define PCH_PP_OFF_DELAYS _MMIO(0xc720c)
>  #define PCH_PP_DIVISOR _MMIO(0xc7210)
>  
> +#define MCHBAR_MEM_BASE		_MMIO(MCHBAR_MIRROR_BASE_SNB + 0x4000)
> +

Pls move this to reg.h which we include all GVT specific define (yeah, I
know those PCH define looks odd, need to clean it up anyway).

>  unsigned long intel_gvt_get_device_type(struct intel_gvt *gvt)
>  {
>  	if (IS_BROADWELL(gvt->dev_priv))
> @@ -3265,7 +3267,7 @@ void intel_gvt_clean_mmio_info(struct intel_gvt *gvt)
>  /* Special MMIO blocks. */
>  static struct gvt_mmio_block mmio_blocks[] = {
>  	{D_SKL_PLUS, _MMIO(CSR_MMIO_START_RANGE), 0x3000, NULL, NULL},
> -	{D_ALL, _MMIO(MCHBAR_MIRROR_BASE_SNB), 0x40000, NULL, NULL},
> +	{D_ALL, MCHBAR_MEM_BASE, 0x4000, NULL, NULL},
>  	{D_ALL, _MMIO(VGT_PVINFO_PAGE), VGT_PVINFO_SIZE,
>  		pvinfo_mmio_read, pvinfo_mmio_write},
>  	{D_ALL, LGC_PALETTE(PIPE_A, 0), 1024, NULL, NULL},
> -- 

I think that's fine, as for recent HW guest drivers shouldn't depend on
anything in head range of MCHBAR.

Acked-by: Zhenyu Wang <zhenyuw at linux.intel.com>

-- 
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/20190218/ecfa57e6/attachment.sig>


More information about the intel-gvt-dev mailing list