mbox series

[PULL] gvt-fixes for 5.1-rc1

Message ID 20190307060810.GT12380@zhen-hp.sh.intel.com (mailing list archive)
State New, archived
Headers show
Series [PULL] gvt-fixes for 5.1-rc1 | expand

Pull-request

https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-03-07

Message

Zhenyu Wang March 7, 2019, 6:08 a.m. UTC
Hi,

Here's gvt-fixes for 5.1-rc1. This contains fixes for newer
version of Windows driver, e.g fix parser for MI_FLUSH_DW and
font render error. And with other stable fix in error path,
fix unexpected workload submission when vGPU idle, etc. Details
are below.

Thanks
--
The following changes since commit 0f75551216091223efe1f18295f655aff6415385:

  drm/i915/gvt: release shadow batch buffer and wa_ctx before destroy one workload (2019-01-23 13:58:46 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-03-07

for you to fetch changes up to 72aabfb862e40ee83c136c4f87877c207e6859b7:

  drm/i915/gvt: Add mutual lock for ppgtt mm LRU list (2019-03-04 14:45:28 +0800)

----------------------------------------------------------------
gvt-fixes-2019-03-07

- Fix MI_FLUSH_DW cmd parser on index check (Zhenyu)
- Fix Windows guest font render error (Colin)
- Fix unexpected workload submission for inactive vGPU (Weinan)
- Fix incorrect workload submission in error path (Zhenyu)
- Fix warning for shadow ppgtt mm reclaim list walk with locking (Zhenyu)

----------------------------------------------------------------
Colin Xu (1):
      drm/i915/gvt: Add in context mmio 0x20D8 to gen9 mmio list

Weinan Li (1):
      drm/i915/gvt: stop scheduling workload when vgpu is inactive

Zhenyu Wang (4):
      drm/i915/gvt: Fix MI_FLUSH_DW parsing with correct index check
      drm/i915/gvt: Don't submit request for error workload dispatch
      drm/i915/gvt: Only assign ppgtt root at dispatch time
      drm/i915/gvt: Add mutual lock for ppgtt mm LRU list

 drivers/gpu/drm/i915/gvt/cmd_parser.c   |  2 +-
 drivers/gpu/drm/i915/gvt/gtt.c          | 14 +++++++++++++-
 drivers/gpu/drm/i915/gvt/gtt.h          |  1 +
 drivers/gpu/drm/i915/gvt/mmio_context.c |  1 +
 drivers/gpu/drm/i915/gvt/scheduler.c    | 28 ++++++++++++++++++++--------
 5 files changed, 36 insertions(+), 10 deletions(-)

Comments

Rodrigo Vivi March 7, 2019, 11:44 p.m. UTC | #1
On Thu, Mar 07, 2019 at 02:08:10PM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's gvt-fixes for 5.1-rc1.

I'm kind of confused here. this should be -next-fixes right?

$ dim apply-pull drm-intel-next-fixes

Pulling https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-03-07 ...
From https://github.com/intel/gvt-linux
 * tag                         gvt-fixes-2019-03-07 -> FETCH_HEAD
dim: Pull request contains commits from drm/drm-fixes
dim: Please backmerge first
dim: ERROR: Issues in pull request detected, aborting

or am I missing something?

Thanks,
Rodrigo.

> This contains fixes for newer
> version of Windows driver, e.g fix parser for MI_FLUSH_DW and
> font render error. And with other stable fix in error path,
> fix unexpected workload submission when vGPU idle, etc. Details
> are below.
> 
> Thanks
> --
> The following changes since commit 0f75551216091223efe1f18295f655aff6415385:
> 
>   drm/i915/gvt: release shadow batch buffer and wa_ctx before destroy one workload (2019-01-23 13:58:46 +0800)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-03-07
> 
> for you to fetch changes up to 72aabfb862e40ee83c136c4f87877c207e6859b7:
> 
>   drm/i915/gvt: Add mutual lock for ppgtt mm LRU list (2019-03-04 14:45:28 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2019-03-07
> 
> - Fix MI_FLUSH_DW cmd parser on index check (Zhenyu)
> - Fix Windows guest font render error (Colin)
> - Fix unexpected workload submission for inactive vGPU (Weinan)
> - Fix incorrect workload submission in error path (Zhenyu)
> - Fix warning for shadow ppgtt mm reclaim list walk with locking (Zhenyu)
> 
> ----------------------------------------------------------------
> Colin Xu (1):
>       drm/i915/gvt: Add in context mmio 0x20D8 to gen9 mmio list
> 
> Weinan Li (1):
>       drm/i915/gvt: stop scheduling workload when vgpu is inactive
> 
> Zhenyu Wang (4):
>       drm/i915/gvt: Fix MI_FLUSH_DW parsing with correct index check
>       drm/i915/gvt: Don't submit request for error workload dispatch
>       drm/i915/gvt: Only assign ppgtt root at dispatch time
>       drm/i915/gvt: Add mutual lock for ppgtt mm LRU list
> 
>  drivers/gpu/drm/i915/gvt/cmd_parser.c   |  2 +-
>  drivers/gpu/drm/i915/gvt/gtt.c          | 14 +++++++++++++-
>  drivers/gpu/drm/i915/gvt/gtt.h          |  1 +
>  drivers/gpu/drm/i915/gvt/mmio_context.c |  1 +
>  drivers/gpu/drm/i915/gvt/scheduler.c    | 28 ++++++++++++++++++++--------
>  5 files changed, 36 insertions(+), 10 deletions(-)
> 
> 
> -- 
> Open Source Technology Center, Intel ltd.
> 
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
Zhenyu Wang March 8, 2019, 2:04 a.m. UTC | #2
On 2019.03.07 15:44:35 -0800, Rodrigo Vivi wrote:
> On Thu, Mar 07, 2019 at 02:08:10PM +0800, Zhenyu Wang wrote:
> > 
> > Hi,
> > 
> > Here's gvt-fixes for 5.1-rc1.
> 
> I'm kind of confused here. this should be -next-fixes right?
>

oops, you're right, it should be -next-fixes

> $ dim apply-pull drm-intel-next-fixes
> 
> Pulling https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-03-07 ...
> From https://github.com/intel/gvt-linux
>  * tag                         gvt-fixes-2019-03-07 -> FETCH_HEAD
> dim: Pull request contains commits from drm/drm-fixes
> dim: Please backmerge first
> dim: ERROR: Issues in pull request detected, aborting
> 
> or am I missing something?

I generated against drm-intel-fixes, will resend for -next-fixes.

Thanks
Zhenyu Wang March 8, 2019, 2:47 a.m. UTC | #3
On 2019.03.07 15:44:35 -0800, Rodrigo Vivi wrote:
> On Thu, Mar 07, 2019 at 02:08:10PM +0800, Zhenyu Wang wrote:
> > 
> > Hi,
> > 
> > Here's gvt-fixes for 5.1-rc1.
> 
> I'm kind of confused here. this should be -next-fixes right?
> 
> $ dim apply-pull drm-intel-next-fixes
> 
> Pulling https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-03-07 ...
> From https://github.com/intel/gvt-linux
>  * tag                         gvt-fixes-2019-03-07 -> FETCH_HEAD
> dim: Pull request contains commits from drm/drm-fixes
> dim: Please backmerge first
> dim: ERROR: Issues in pull request detected, aborting
> 
> or am I missing something?
> 

Looks drm-intel-next-queued has backmerge of 5.0-rc7 from drm-next,
but drm-intel-next-fixes doesn't, some fixes depend on upstreamn
commit, e.g for f0e994372518 ("drm/i915/gvt: Fix workload request
allocation before request add"). For current window, I might hold till
drm-intel-fixes updated.