public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
* [GIT PULL] GVT-g fixes for 4.10-rc5
@ 2017-01-16  7:11 Zhenyu Wang
  2017-01-16  9:34 ` Jani Nikula
  0 siblings, 1 reply; 3+ messages in thread
From: Zhenyu Wang @ 2017-01-16  7:11 UTC (permalink / raw)
  To: Daniel Vetter, Jani Nikula; +Cc: intel-gfx, igvt-g-dev, Lv, Zhiyuan


[-- Attachment #1.1: Type: text/plain, Size: 2303 bytes --]


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.

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(-)


-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 163 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [GIT PULL] GVT-g fixes for 4.10-rc5
  2017-01-16  7:11 [GIT PULL] GVT-g fixes for 4.10-rc5 Zhenyu Wang
@ 2017-01-16  9:34 ` Jani Nikula
  2017-01-26  8:30   ` Daniel Vetter
  0 siblings, 1 reply; 3+ messages in thread
From: Jani Nikula @ 2017-01-16  9:34 UTC (permalink / raw)
  To: Zhenyu Wang, Daniel Vetter; +Cc: intel-gfx, igvt-g-dev, Lv, Zhiyuan

On Mon, 16 Jan 2017, Zhenyu Wang <zhenyuw@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.

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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [GIT PULL] GVT-g fixes for 4.10-rc5
  2017-01-16  9:34 ` Jani Nikula
@ 2017-01-26  8:30   ` Daniel Vetter
  0 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2017-01-26  8:30 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-gfx, Lv, Zhiyuan, igvt-g-dev

On Mon, Jan 16, 2017 at 11:34:46AM +0200, Jani Nikula wrote:
> On Mon, 16 Jan 2017, Zhenyu Wang <zhenyuw@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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-01-26  8:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-16  7:11 [GIT PULL] GVT-g fixes for 4.10-rc5 Zhenyu Wang
2017-01-16  9:34 ` Jani Nikula
2017-01-26  8:30   ` Daniel Vetter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox