[PATCH v2 0/2] drm/i915/gvt: Refactor to per-vGPU accurate vblank emulator

[Zhenyu Wang]

On 2021.02.26 12:45:23 +0800, Colin Xu wrote:
> The patch set includes two patches:
> Patch 1/2: Get accurate vGPU virtual display refresh rate from vreg
> Patch 2/2: Use per-vGPU vblank emulator at dynamic & accurate refresh rate.
> 
> The patch set resovles below issues:
> - 16ms matches the refresh rate at 62.5Hz (instead of 60Hz) which isn't
>   belong to standard timing. This leads to some frame drop or glitch issue
>   during video playback. Since SW expect at least 16.667ms period for 60Hz
>   or may need higher resolution, however vblank emulator only works at 16.
> - Doesn't respect the fact that with current virtual EDID timing set,
>   not all resolutions are enabled at 60Hz. For example, current virtual
>   EDID actually supports refresh rate like 56Hz, 59.97Hz, 60Hz, 75Hz, etc.
> - Current vblank emulator use single hrtimer for all vGPUs. Regardsless
>   how guest changing the resolution, all vsync interrupts are injected at
>   16ms interval.
> 
> With the patch set, the actual refresh rate from guest can now always
> match guest OS resolution settings.
> To run quick test, launch a web browser and goto URL: www.displayhz.com
> 
> V2:
> Rebase to 5.11.
> Correctly calculate DP link rate for BDW and BXT.
> Simplify logic.
> Typo fix.
> 
> Colin Xu (2):
>   drm/i915/gvt: Get accurate vGPU virtual display refresh rate from vreg
>   drm/i915/gvt: Refactor GVT vblank emulator for vGPU virtual display
> 
>  drivers/gpu/drm/i915/gvt/display.c   | 107 ++++++-----
>  drivers/gpu/drm/i915/gvt/display.h   |  14 +-
>  drivers/gpu/drm/i915/gvt/gvt.c       |  25 ++-
>  drivers/gpu/drm/i915/gvt/gvt.h       |  13 +-
>  drivers/gpu/drm/i915/gvt/handlers.c  | 261 ++++++++++++++++++++++++++-
>  drivers/gpu/drm/i915/gvt/interrupt.c |  37 ----
>  drivers/gpu/drm/i915/gvt/interrupt.h |   7 -
>  drivers/gpu/drm/i915/gvt/vgpu.c      |   2 -
>  8 files changed, 353 insertions(+), 113 deletions(-)
> 

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

Thanks!
-------------- 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/20210301/fdfe7605/attachment.sig>