[Intel-gfx] [GIT PULL] GVT-g fixes for 4.10-rc5

Daniel Vetter daniel at ffwll.ch
Thu Jan 26 08:30:27 UTC 2017


On Mon, Jan 16, 2017 at 11:34:46AM +0200, Jani Nikula wrote:
> On Mon, 16 Jan 2017, Zhenyu Wang <zhenyuw at linux.intel.com> wrote:
> > Hi,
> >
> > This pull contains vGPU/mdev reset fixes from Changbin to fix vGPU hang
> > for VM reboot and vGPU/mdev instance reuse issues that might impact usage. 
> > Several preparation patches are to align reset code for each functional part
> > and actual fix is just to reset device model state for all components.
> 
> Pulled to drm-intel-fixes, thanks.
> 
> Please note that commits for -rc kernels should mostly be just
> fixes. Especially towards the bigger -rc, the rules are about the same
> as for stable kernels, i.e. keep it rather conservative. Since GVT-g is
> new and isolated, I'm giving you more leeway, but please keep this in
> mind going forward.

Yeah for 4.10 this is still ok-ish since gvt is all new and it's better to
fix it than to ship it broken. But for 4.11+ this looks like way too much,
and smells a bit like refactoring/feature work instead of minimal
bugfixes.
-Daniel
> 
> BR,
> Jani.
> 
> 
> 
> 
> >
> > Thanks.
> >
> > ---
> > The following changes since commit 9631739f8196ec80b5d9bf955f79b711490c0205:
> >
> >   drm/i915/gvt: cleanup GFP flags (2017-01-09 17:31:05 +0800)
> >
> > are available in the git repository at:
> >
> >   https://github.com/01org/gvt-linux.git tags/gvt-fixes-2017-01-16
> >
> > for you to fetch changes up to c34eaa8d0f9d9ae26a4a6af7bc3aca57310cf483:
> >
> >   drm/i915/gvt: rewrite gt reset handler using new function intel_gvt_reset_vgpu_locked (2017-01-13 15:05:38 +0800)
> >
> > ----------------------------------------------------------------
> > gvt-fixes-2017-01-16
> >
> > vGPU reset fixes from Changbin.
> >
> > ----------------------------------------------------------------
> > Changbin Du (8):
> >       drm/i915/gvt: introudce intel_vgpu_reset_resource() to reset vgpu resource state
> >       drm/i915/gvt: introuduce intel_vgpu_reset_gtt() to reset gtt
> >       drm/i915/gvt: move cfg space inititation function to cfg_space.c
> >       drm/i915/gvt: introduce intel_vgpu_reset_cfg_space to reset configuration space
> >       drm/i915/gvt: move mmio init/clean function to mmio.c
> >       drm/i915/gvt: introduce intel_vgpu_reset_mmio() to reset mmio space
> >       drm/i915/gvt: fix vGPU instance reuse issues by vGPU reset function
> >       drm/i915/gvt: rewrite gt reset handler using new function intel_gvt_reset_vgpu_locked
> >
> >  drivers/gpu/drm/i915/gvt/aperture_gm.c |  29 ++++++-
> >  drivers/gpu/drm/i915/gvt/cfg_space.c   |  74 ++++++++++++++++
> >  drivers/gpu/drm/i915/gvt/gtt.c         |  27 ++++++
> >  drivers/gpu/drm/i915/gvt/gtt.h         |   1 +
> >  drivers/gpu/drm/i915/gvt/gvt.h         |   8 +-
> >  drivers/gpu/drm/i915/gvt/handlers.c    |  90 +++++++------------
> >  drivers/gpu/drm/i915/gvt/mmio.c        |  53 ++++++++++++
> >  drivers/gpu/drm/i915/gvt/mmio.h        |   4 +
> >  drivers/gpu/drm/i915/gvt/vgpu.c        | 154 ++++++++++++++++-----------------
> >  9 files changed, 298 insertions(+), 142 deletions(-)
> 
> -- 
> Jani Nikula, Intel Open Source Technology Center

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the Intel-gfx mailing list