mbox series

[PULL] gvt-fixes

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

Pull-request

https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-27

Message

Zhenyu Wang Oct. 27, 2020, 3:17 a.m. UTC 
Hi,

Here's first gvt fixes for 5.10 which includes more vGPU
suspend/resume fix in HWSP reset handling, and also fix for host i915
suspend regression when vGPU is created (not need to be active), and
one workaround for APL guest hang issue.

Thanks
--
The following changes since commit 16cce04cdb200ba905d1241b425ac48da5a9ace5:

  drm/i915/selftests: Push the fake iommu device from the stack to data (2020-09-23 10:15:46 +0300)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-27

for you to fetch changes up to 401ccfa87856656b874c737522ea92721394a348:

  drm/i915/gvt: Only pin/unpin intel_context along with workload (2020-10-19 16:54:28 +0800)

----------------------------------------------------------------
gvt-fixes-2020-10-27

- Fix HWSP reset handling during vGPU suspend/resume (Colin)
- Apply flush workaround on APL now for possible guest hang (Colin)
- Fix vGPU context pin/unpin also for host suspend regression with
  vGPU created (Colin)

----------------------------------------------------------------
Colin Xu (3):
      drm/i915/gvt: Allow zero out HWSP addr on hws_pga_write
      drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU BB hang
      drm/i915/gvt: Only pin/unpin intel_context along with workload

 drivers/gpu/drm/i915/gvt/handlers.c  | 35 +++++++++++++++++++++++++++++++++--
 drivers/gpu/drm/i915/gvt/scheduler.c | 15 ++++++++-------
 2 files changed, 41 insertions(+), 9 deletions(-)

Comments

Rodrigo Vivi Oct. 27, 2020, 8:46 p.m. UTC | #1 
On Tue, Oct 27, 2020 at 11:17:40AM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's first gvt fixes for 5.10 which includes more vGPU
> suspend/resume fix in HWSP reset handling, and also fix for host i915
> suspend regression when vGPU is created (not need to be active), and
> one workaround for APL guest hang issue.

pulled to drm-intel-fixes
thanks

> 
> Thanks
> --
> The following changes since commit 16cce04cdb200ba905d1241b425ac48da5a9ace5:
> 
>   drm/i915/selftests: Push the fake iommu device from the stack to data (2020-09-23 10:15:46 +0300)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-27
> 
> for you to fetch changes up to 401ccfa87856656b874c737522ea92721394a348:
> 
>   drm/i915/gvt: Only pin/unpin intel_context along with workload (2020-10-19 16:54:28 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2020-10-27
> 
> - Fix HWSP reset handling during vGPU suspend/resume (Colin)
> - Apply flush workaround on APL now for possible guest hang (Colin)
> - Fix vGPU context pin/unpin also for host suspend regression with
>   vGPU created (Colin)
> 
> ----------------------------------------------------------------
> Colin Xu (3):
>       drm/i915/gvt: Allow zero out HWSP addr on hws_pga_write
>       drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU BB hang
>       drm/i915/gvt: Only pin/unpin intel_context along with workload
> 
>  drivers/gpu/drm/i915/gvt/handlers.c  | 35 +++++++++++++++++++++++++++++++++--
>  drivers/gpu/drm/i915/gvt/scheduler.c | 15 ++++++++-------
>  2 files changed, 41 insertions(+), 9 deletions(-)
> 
> 
> -- 
> 
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
Rodrigo Vivi Oct. 28, 2020, 11:18 a.m. UTC | #2 
On Oct 27, 2020, at 1:46 PM, Rodrigo Vivi <rodrigo.vivi@intel.com<mailto:rodrigo.vivi@intel.com>> wrote:

On Tue, Oct 27, 2020 at 11:17:40AM +0800, Zhenyu Wang wrote:

Hi,

Here's first gvt fixes for 5.10 which includes more vGPU
suspend/resume fix in HWSP reset handling, and also fix for host i915
suspend regression when vGPU is created (not need to be active), and
one workaround for APL guest hang issue.

pulled to drm-intel-fixes

I'm actually pulling it off. I had bypassed dim, considering this was an old issue with our email decoder,
but it happens that

$ git show 401ccfa87856 | grep Fixes
Fixes: e6ba76480299 (drm/i915: Remove i915->kernel_context)

And this is what it should have:

$ dim fixes e6ba76480299 | grep Fixes
Fixes: e6ba76480299 ("drm/i915: Remove i915->kernel_context")

Sorry for the trouble.
Let's fix this in place so we don't propagate bad tag that might break other scripts on the way

Sorry,
Rodrigo.

thanks


Thanks
--
The following changes since commit 16cce04cdb200ba905d1241b425ac48da5a9ace5:

 drm/i915/selftests: Push the fake iommu device from the stack to data (2020-09-23 10:15:46 +0300)

are available in the Git repository at:

 https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-27

for you to fetch changes up to 401ccfa87856656b874c737522ea92721394a348:

 drm/i915/gvt: Only pin/unpin intel_context along with workload (2020-10-19 16:54:28 +0800)

----------------------------------------------------------------
gvt-fixes-2020-10-27

- Fix HWSP reset handling during vGPU suspend/resume (Colin)
- Apply flush workaround on APL now for possible guest hang (Colin)
- Fix vGPU context pin/unpin also for host suspend regression with
 vGPU created (Colin)

----------------------------------------------------------------
Colin Xu (3):
     drm/i915/gvt: Allow zero out HWSP addr on hws_pga_write
     drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU BB hang
     drm/i915/gvt: Only pin/unpin intel_context along with workload

drivers/gpu/drm/i915/gvt/handlers.c  | 35 +++++++++++++++++++++++++++++++++--
drivers/gpu/drm/i915/gvt/scheduler.c | 15 ++++++++-------
2 files changed, 41 insertions(+), 9 deletions(-)


--

$gpg --keyserver wwwkeys.pgp.net<http://wwwkeys.pgp.net> --recv-keys 4D781827
Zhenyu Wang Oct. 30, 2020, 3:09 a.m. UTC | #3 
On 2020.10.28 11:18:45 +0000, Vivi, Rodrigo wrote:
> 
> 
>     On Oct 27, 2020, at 1:46 PM, Rodrigo Vivi <rodrigo.vivi@intel.com> wrote:
> 
>     On Tue, Oct 27, 2020 at 11:17:40AM +0800, Zhenyu Wang wrote:
> 
> 
>         Hi,
> 
>         Here's first gvt fixes for 5.10 which includes more vGPU
>         suspend/resume fix in HWSP reset handling, and also fix for host i915
>         suspend regression when vGPU is created (not need to be active), and
>         one workaround for APL guest hang issue.
> 
> 
>     pulled to drm-intel-fixes
> 
> 
> I'm actually pulling it off. I had bypassed dim, considering this was an old
> issue with our email decoder,
> but it happens that 
> 
> $ git show 401ccfa87856 | grep Fixes
> Fixes: e6ba76480299 (drm/i915: Remove i915->kernel_context)
> 
> And this is what it should have:
> 
> $ dim fixes e6ba76480299 | grep Fixes
> Fixes: e6ba76480299 ("drm/i915: Remove i915->kernel_context")
> 
> Sorry for the trouble.
> Let's fix this in place so we don't propagate bad tag that might break other
> scripts on the way
>

Oops, sorry I missed this one. I'll refresh.

Thanks

> 
> 
> 
>         Thanks
>         --
>         The following changes since commit
>         16cce04cdb200ba905d1241b425ac48da5a9ace5:
> 
>          drm/i915/selftests: Push the fake iommu device from the stack to data
>         (2020-09-23 10:15:46 +0300)
> 
>         are available in the Git repository at:
> 
>          https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-27
> 
>         for you to fetch changes up to
>         401ccfa87856656b874c737522ea92721394a348:
> 
>          drm/i915/gvt: Only pin/unpin intel_context along with workload
>         (2020-10-19 16:54:28 +0800)
> 
>         ----------------------------------------------------------------
>         gvt-fixes-2020-10-27
> 
>         - Fix HWSP reset handling during vGPU suspend/resume (Colin)
>         - Apply flush workaround on APL now for possible guest hang (Colin)
>         - Fix vGPU context pin/unpin also for host suspend regression with
>          vGPU created (Colin)
> 
>         ----------------------------------------------------------------
>         Colin Xu (3):
>              drm/i915/gvt: Allow zero out HWSP addr on hws_pga_write
>              drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU BB
>         hang
>              drm/i915/gvt: Only pin/unpin intel_context along with workload
> 
>         drivers/gpu/drm/i915/gvt/handlers.c  | 35
>         +++++++++++++++++++++++++++++++++--
>         drivers/gpu/drm/i915/gvt/scheduler.c | 15 ++++++++-------
>         2 files changed, 41 insertions(+), 9 deletions(-)
> 
> 
>         -- 
> 
>         $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
> 
> 
> 
>     _______________________________________________
>     Intel-gfx mailing list
>     Intel-gfx@lists.freedesktop.org
>     https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
>
Zhenyu Wang Oct. 30, 2020, 5:21 a.m. UTC | #4 
On 2020.10.28 11:18:45 +0000, Vivi, Rodrigo wrote:

> 
> I'm actually pulling it off. I had bypassed dim, considering this was an old
> issue with our email decoder,
> but it happens that 
> 
> $ git show 401ccfa87856 | grep Fixes
> Fixes: e6ba76480299 (drm/i915: Remove i915->kernel_context)
> 
> And this is what it should have:
> 
> $ dim fixes e6ba76480299 | grep Fixes
> Fixes: e6ba76480299 ("drm/i915: Remove i915->kernel_context")
> 
> Sorry for the trouble.
> Let's fix this in place so we don't propagate bad tag that might break other
> scripts on the way
> 

Hi, Rodrigo, here's re-generated pull. I also picked another fixes for
MMIO cmd access flag on BXT/APL, which required a backmerge.

Thanks!
--
The following changes since commit 61334ed227a5852100115180f5535b1396ed5227:

  drm/i915: Reject 90/270 degree rotated initial fbs (2020-10-29 14:20:24 -0400)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-30

for you to fetch changes up to 92010a97098c4c9fd777408cc98064d26b32695b:

  drm/i915/gvt: Fix mmio handler break on BXT/APL. (2020-10-30 11:50:06 +0800)

----------------------------------------------------------------
gvt-fixes-2020-10-30

- Fix HWSP reset handling during vGPU suspend/resume (Colin)
- Apply flush workaround on APL now for possible guest hang (Colin)
- Fix vGPU context pin/unpin also for host suspend regression with
  vGPU created (Colin)
- more BXT/APL mmio cmd access fixes (Colin)

----------------------------------------------------------------
Colin Xu (4):
      drm/i915/gvt: Allow zero out HWSP addr on hws_pga_write
      drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU BB hang
      drm/i915/gvt: Only pin/unpin intel_context along with workload
      drm/i915/gvt: Fix mmio handler break on BXT/APL.

Zhenyu Wang (1):
      Merge tag 'drm-intel-fixes-2020-10-29' into gvt-fixes

 drivers/gpu/drm/i915/gvt/handlers.c  | 47 +++++++++++++++++++++++++++++++++---
 drivers/gpu/drm/i915/gvt/scheduler.c | 15 ++++++------
 2 files changed, 52 insertions(+), 10 deletions(-)
Rodrigo Vivi Oct. 30, 2020, 10:57 p.m. UTC | #5 
> On Oct 29, 2020, at 10:21 PM, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> 
> On 2020.10.28 11:18:45 +0000, Vivi, Rodrigo wrote:
> 
>> 
>> I'm actually pulling it off. I had bypassed dim, considering this was an old
>> issue with our email decoder,
>> but it happens that 
>> 
>> $ git show 401ccfa87856 | grep Fixes
>> Fixes: e6ba76480299 (drm/i915: Remove i915->kernel_context)
>> 
>> And this is what it should have:
>> 
>> $ dim fixes e6ba76480299 | grep Fixes
>> Fixes: e6ba76480299 ("drm/i915: Remove i915->kernel_context")
>> 
>> Sorry for the trouble.
>> Let's fix this in place so we don't propagate bad tag that might break other
>> scripts on the way
>> 
> 
> Hi, Rodrigo, here's re-generated pull. I also picked another fixes for
> MMIO cmd access flag on BXT/APL, which required a backmerge.
> 
> Thanks!
> --
> The following changes since commit 61334ed227a5852100115180f5535b1396ed5227:
> 
>  drm/i915: Reject 90/270 degree rotated initial fbs (2020-10-29 14:20:24 -0400)
> 
> are available in the Git repository at:
> 
>  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-10-30
> 
> for you to fetch changes up to 92010a97098c4c9fd777408cc98064d26b32695b:
> 
>  drm/i915/gvt: Fix mmio handler break on BXT/APL. (2020-10-30 11:50:06 +0800)


pulled, thanks!


> 
> ----------------------------------------------------------------
> gvt-fixes-2020-10-30
> 
> - Fix HWSP reset handling during vGPU suspend/resume (Colin)
> - Apply flush workaround on APL now for possible guest hang (Colin)
> - Fix vGPU context pin/unpin also for host suspend regression with
>  vGPU created (Colin)
> - more BXT/APL mmio cmd access fixes (Colin)
> 
> ----------------------------------------------------------------
> Colin Xu (4):
>      drm/i915/gvt: Allow zero out HWSP addr on hws_pga_write
>      drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU BB hang
>      drm/i915/gvt: Only pin/unpin intel_context along with workload
>      drm/i915/gvt: Fix mmio handler break on BXT/APL.
> 
> Zhenyu Wang (1):
>      Merge tag 'drm-intel-fixes-2020-10-29' into gvt-fixes
> 
> drivers/gpu/drm/i915/gvt/handlers.c  | 47 +++++++++++++++++++++++++++++++++---
> drivers/gpu/drm/i915/gvt/scheduler.c | 15 ++++++------
> 2 files changed, 52 insertions(+), 10 deletions(-)
> 
> 
> -- 
> 
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827