[Intel-gfx] [PULL] gvt-fixes

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

This is to fix GVT config workaround introduced during -rc1 via
vfio/mdev change, which exposed dependency issue explicitly that
made current GVT config nasty. So this is to fix dependency issue
and get back original config sanity.

Thanks
--
The following changes since commit e4527420ed087f99c6aa2ac22c6d3458c7dc1a94:

  drm/i915: Use correct downstream caps for check Src-Ctl mode for PCON (2021-05-12 20:53:08 +0300)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2021-05-19

for you to fetch changes up to 145e06b58f8625becc61792a0554726314297a85:

  drm/i915/gvt: Move mdev attribute groups into kvmgt module (2021-05-17 16:37:09 +0800)

----------------------------------------------------------------
gvt-fixes-2021-05-19

- Fix workaround in -rc1 for GVT config (Zhenyu)

----------------------------------------------------------------
Zhenyu Wang (1):
      drm/i915/gvt: Move mdev attribute groups into kvmgt module

 drivers/gpu/drm/i915/Kconfig         |   1 -
 drivers/gpu/drm/i915/gvt/gvt.c       | 124 +----------------------------------
 drivers/gpu/drm/i915/gvt/gvt.h       |   3 -
 drivers/gpu/drm/i915/gvt/hypercall.h |   2 +-
 drivers/gpu/drm/i915/gvt/kvmgt.c     | 122 +++++++++++++++++++++++++++++++---
 drivers/gpu/drm/i915/gvt/mpt.h       |   4 +-
 6 files changed, 118 insertions(+), 138 deletions(-)


[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Wed, 19 May 2021, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> This is to fix GVT config workaround introduced during -rc1 via
> vfio/mdev change, which exposed dependency issue explicitly that
> made current GVT config nasty. So this is to fix dependency issue
> and get back original config sanity.

Pulled to drm-intel-fixes, thanks.

BR,
Jani.

>
> Thanks
> --
> The following changes since commit e4527420ed087f99c6aa2ac22c6d3458c7dc1a94:
>
>   drm/i915: Use correct downstream caps for check Src-Ctl mode for PCON (2021-05-12 20:53:08 +0300)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2021-05-19
>
> for you to fetch changes up to 145e06b58f8625becc61792a0554726314297a85:
>
>   drm/i915/gvt: Move mdev attribute groups into kvmgt module (2021-05-17 16:37:09 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2021-05-19
>
> - Fix workaround in -rc1 for GVT config (Zhenyu)
>
> ----------------------------------------------------------------
> Zhenyu Wang (1):
>       drm/i915/gvt: Move mdev attribute groups into kvmgt module
>
>  drivers/gpu/drm/i915/Kconfig         |   1 -
>  drivers/gpu/drm/i915/gvt/gvt.c       | 124 +----------------------------------
>  drivers/gpu/drm/i915/gvt/gvt.h       |   3 -
>  drivers/gpu/drm/i915/gvt/hypercall.h |   2 +-
>  drivers/gpu/drm/i915/gvt/kvmgt.c     | 122 +++++++++++++++++++++++++++++++---
>  drivers/gpu/drm/i915/gvt/mpt.h       |   4 +-
>  6 files changed, 118 insertions(+), 138 deletions(-)
>

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PULL] gvt-fixes

[Tvrtko Ursulin]

On 02/08/2023 06:54, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here is one gvt fix for bug in AUX CH register message length get.
> Please help to pick.

Pulled, thank you!

Regards,

Tvrtko

> Thanks!
> --
> The following changes since commit e354f67733115b4453268f61e6e072e9b1ea7a2f:
> 
>    drm/i915: Fix an error handling path in igt_write_huge() (2023-07-25 08:38:12 +0100)
> 
> are available in the Git repository at:
> 
>    https://github.com/intel/gvt-linux.git tags/gvt-fixes-2023-08-02
> 
> for you to fetch changes up to 46d14e17095237007b59f56aae2d81ae2dcb0f93:
> 
>    drm/i915/gvt: Fix bug in getting msg length in AUX CH registers handler (2023-08-01 11:21:09 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2023-08-02
> 
> - Fix bug to get AUX CH register message length (Yan)
> 
> ----------------------------------------------------------------
> Yan Zhao (1):
>        drm/i915/gvt: Fix bug in getting msg length in AUX CH registers handler
> 
>   drivers/gpu/drm/i915/gvt/edid.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 945 bytes --]


Hi,

Here is one gvt fix for bug in AUX CH register message length get.
Please help to pick.

Thanks!
--
The following changes since commit e354f67733115b4453268f61e6e072e9b1ea7a2f:

  drm/i915: Fix an error handling path in igt_write_huge() (2023-07-25 08:38:12 +0100)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2023-08-02

for you to fetch changes up to 46d14e17095237007b59f56aae2d81ae2dcb0f93:

  drm/i915/gvt: Fix bug in getting msg length in AUX CH registers handler (2023-08-01 11:21:09 +0800)

----------------------------------------------------------------
gvt-fixes-2023-08-02

- Fix bug to get AUX CH register message length (Yan)

----------------------------------------------------------------
Yan Zhao (1):
      drm/i915/gvt: Fix bug in getting msg length in AUX CH registers handler

 drivers/gpu/drm/i915/gvt/edid.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Thu, Jan 05, 2023 at 08:15:28AM +0800, Zhenyu Wang wrote:
> On 2023.01.04 06:34:28 -0500, Rodrigo Vivi wrote:
> > On Wed, Jan 04, 2023 at 04:05:13PM +0800, Zhenyu Wang wrote:
> > > 
> > > Hi,
> > > 
> > > Here's accumulated current gvt-fixes. Several of them handle
> > > for error or destroy path issues and one from Zhi to fix up
> > > left vgpu status tracking.
> > > 
> > > thanks!
> > > ---
> > > The following changes since commit 6217e9f05a74df48c77ee68993d587cdfdb1feb7:
> > 
> > I'm kind of confused on your baseline here.
> > 
> > It is including a strange merge commit in the middle of the commits:
> > Zhenyu Wang   ??? M?????? Merge tag 'drm-intel-fixes-2022-12-30' into gvt-fixes
> > commit c063c8c07864246ba3831b017cea8d3096e236a8
> > 
> > Please rebase on top of v6.2-rc2 so we have the same base here.
> > (and please remind to sign-off-by when pushing the commits)
> > 
> 
> oh, I tried to sycn up by back merge with vfio iommufd of gvt changes
> to apply Zhi's patch properly, so it should stand on that tag..but anyway

I'd say for the fixes branches probably a rebase is better than back merges.
Keep the rebase in sync with -rc tags is probably the best way. or rebase
on drm-intel-fixes.

> I just refresh with rebase and fixed two invalid r-b tags. Please pull
> below one.

pulled. Thanks

> 
> thanks!
> ---
> The following changes since commit 88603b6dc419445847923fcb7fe5080067a30f98:
> 
>   Linux 6.2-rc2 (2023-01-01 13:53:16 -0800)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2023-01-05
> 
> for you to fetch changes up to 4a61648af68f5ba4884f0e3b494ee1cabc4b6620:
> 
>   drm/i915/gvt: fix double free bug in split_2MB_gtt_entry (2023-01-04 23:21:19 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2023-01-05
> 
> - Fix one missed unpin in error of intel_vgpu_shadow_mm_pin()
> - Fix two debugfs destroy oops issues for vgpu and gvt entries
> - Fix one potential double free issue in gtt shadow pt code
> - Fix to use atomic bit flag for vgpu status
> 
> ----------------------------------------------------------------
> Dan Carpenter (1):
>       drm/i915: unpin on error in intel_vgpu_shadow_mm_pin()
> 
> Zheng Wang (1):
>       drm/i915/gvt: fix double free bug in split_2MB_gtt_entry
> 
> Zhenyu Wang (2):
>       drm/i915/gvt: fix gvt debugfs destroy
>       drm/i915/gvt: fix vgpu debugfs clean in remove
> 
> Zhi Wang (1):
>       drm/i915/gvt: use atomic operations to change the vGPU status
> 
>  drivers/gpu/drm/i915/gvt/debugfs.c   | 36 +++++++++++++++++++++++++++++++-----
>  drivers/gpu/drm/i915/gvt/dmabuf.c    |  3 ++-
>  drivers/gpu/drm/i915/gvt/gtt.c       | 21 +++++++++++++++------
>  drivers/gpu/drm/i915/gvt/gvt.h       | 15 ++++++++++-----
>  drivers/gpu/drm/i915/gvt/interrupt.c |  2 +-
>  drivers/gpu/drm/i915/gvt/kvmgt.c     | 35 +++++++++++++----------------------
>  drivers/gpu/drm/i915/gvt/scheduler.c |  4 +++-
>  drivers/gpu/drm/i915/gvt/vgpu.c      | 12 +++++-------
>  8 files changed, 80 insertions(+), 48 deletions(-)

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 2767 bytes --]

On 2023.01.04 06:34:28 -0500, Rodrigo Vivi wrote:
> On Wed, Jan 04, 2023 at 04:05:13PM +0800, Zhenyu Wang wrote:
> > 
> > Hi,
> > 
> > Here's accumulated current gvt-fixes. Several of them handle
> > for error or destroy path issues and one from Zhi to fix up
> > left vgpu status tracking.
> > 
> > thanks!
> > ---
> > The following changes since commit 6217e9f05a74df48c77ee68993d587cdfdb1feb7:
> 
> I'm kind of confused on your baseline here.
> 
> It is including a strange merge commit in the middle of the commits:
> Zhenyu Wang   ??? M?????? Merge tag 'drm-intel-fixes-2022-12-30' into gvt-fixes
> commit c063c8c07864246ba3831b017cea8d3096e236a8
> 
> Please rebase on top of v6.2-rc2 so we have the same base here.
> (and please remind to sign-off-by when pushing the commits)
> 

oh, I tried to sycn up by back merge with vfio iommufd of gvt changes
to apply Zhi's patch properly, so it should stand on that tag..but anyway
I just refresh with rebase and fixed two invalid r-b tags. Please pull
below one.

thanks!
---
The following changes since commit 88603b6dc419445847923fcb7fe5080067a30f98:

  Linux 6.2-rc2 (2023-01-01 13:53:16 -0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2023-01-05

for you to fetch changes up to 4a61648af68f5ba4884f0e3b494ee1cabc4b6620:

  drm/i915/gvt: fix double free bug in split_2MB_gtt_entry (2023-01-04 23:21:19 +0800)

----------------------------------------------------------------
gvt-fixes-2023-01-05

- Fix one missed unpin in error of intel_vgpu_shadow_mm_pin()
- Fix two debugfs destroy oops issues for vgpu and gvt entries
- Fix one potential double free issue in gtt shadow pt code
- Fix to use atomic bit flag for vgpu status

----------------------------------------------------------------
Dan Carpenter (1):
      drm/i915: unpin on error in intel_vgpu_shadow_mm_pin()

Zheng Wang (1):
      drm/i915/gvt: fix double free bug in split_2MB_gtt_entry

Zhenyu Wang (2):
      drm/i915/gvt: fix gvt debugfs destroy
      drm/i915/gvt: fix vgpu debugfs clean in remove

Zhi Wang (1):
      drm/i915/gvt: use atomic operations to change the vGPU status

 drivers/gpu/drm/i915/gvt/debugfs.c   | 36 +++++++++++++++++++++++++++++++-----
 drivers/gpu/drm/i915/gvt/dmabuf.c    |  3 ++-
 drivers/gpu/drm/i915/gvt/gtt.c       | 21 +++++++++++++++------
 drivers/gpu/drm/i915/gvt/gvt.h       | 15 ++++++++++-----
 drivers/gpu/drm/i915/gvt/interrupt.c |  2 +-
 drivers/gpu/drm/i915/gvt/kvmgt.c     | 35 +++++++++++++----------------------
 drivers/gpu/drm/i915/gvt/scheduler.c |  4 +++-
 drivers/gpu/drm/i915/gvt/vgpu.c      | 12 +++++-------
 8 files changed, 80 insertions(+), 48 deletions(-)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Wed, Jan 04, 2023 at 04:05:13PM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's accumulated current gvt-fixes. Several of them handle
> for error or destroy path issues and one from Zhi to fix up
> left vgpu status tracking.
> 
> thanks!
> ---
> The following changes since commit 6217e9f05a74df48c77ee68993d587cdfdb1feb7:

I'm kind of confused on your baseline here.

It is including a strange merge commit in the middle of the commits:
Zhenyu Wang   │ M─┐ Merge tag 'drm-intel-fixes-2022-12-30' into gvt-fixes
commit c063c8c07864246ba3831b017cea8d3096e236a8

Please rebase on top of v6.2-rc2 so we have the same base here.
(and please remind to sign-off-by when pushing the commits)

> 
>   drm/i915/dsi: fix MIPI_BKLT_EN_1 native GPIO index (2022-12-30 04:28:46 -0500)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2023-01-04
> 
> for you to fetch changes up to 601fd0f6b2a4c776a21ab8300fe0de0726937623:
> 
>   drm/i915/gvt: fix double free bug in split_2MB_gtt_entry (2023-01-04 15:20:09 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2023-01-04
> 
> - Fix one missed unpin in error of intel_vgpu_shadow_mm_pin()
> - Fix two debugfs destroy oops issues for vgpu and gvt entries
> - Fix one potential double free issue in gtt shadow pt code
> - Fix to use atomic bit flag for vgpu status
> 
> ----------------------------------------------------------------
> Dan Carpenter (1):
>       drm/i915: unpin on error in intel_vgpu_shadow_mm_pin()
> 
> Zheng Wang (1):
>       drm/i915/gvt: fix double free bug in split_2MB_gtt_entry
> 
> Zhenyu Wang (3):
>       drm/i915/gvt: fix gvt debugfs destroy
>       drm/i915/gvt: fix vgpu debugfs clean in remove
>       Merge tag 'drm-intel-fixes-2022-12-30' into gvt-fixes
> 
> Zhi Wang (1):
>       drm/i915/gvt: use atomic operations to change the vGPU status
> 
>  drivers/gpu/drm/i915/gvt/debugfs.c   | 36 +++++++++++++++++++++++++++++++-----
>  drivers/gpu/drm/i915/gvt/dmabuf.c    |  3 ++-
>  drivers/gpu/drm/i915/gvt/gtt.c       | 21 +++++++++++++++------
>  drivers/gpu/drm/i915/gvt/gvt.h       | 15 ++++++++++-----
>  drivers/gpu/drm/i915/gvt/interrupt.c |  2 +-
>  drivers/gpu/drm/i915/gvt/kvmgt.c     | 35 +++++++++++++----------------------
>  drivers/gpu/drm/i915/gvt/scheduler.c |  4 +++-
>  drivers/gpu/drm/i915/gvt/vgpu.c      | 12 +++++-------
>  8 files changed, 80 insertions(+), 48 deletions(-)

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 1932 bytes --]


Hi,

Here's accumulated current gvt-fixes. Several of them handle
for error or destroy path issues and one from Zhi to fix up
left vgpu status tracking.

thanks!
---
The following changes since commit 6217e9f05a74df48c77ee68993d587cdfdb1feb7:

  drm/i915/dsi: fix MIPI_BKLT_EN_1 native GPIO index (2022-12-30 04:28:46 -0500)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2023-01-04

for you to fetch changes up to 601fd0f6b2a4c776a21ab8300fe0de0726937623:

  drm/i915/gvt: fix double free bug in split_2MB_gtt_entry (2023-01-04 15:20:09 +0800)

----------------------------------------------------------------
gvt-fixes-2023-01-04

- Fix one missed unpin in error of intel_vgpu_shadow_mm_pin()
- Fix two debugfs destroy oops issues for vgpu and gvt entries
- Fix one potential double free issue in gtt shadow pt code
- Fix to use atomic bit flag for vgpu status

----------------------------------------------------------------
Dan Carpenter (1):
      drm/i915: unpin on error in intel_vgpu_shadow_mm_pin()

Zheng Wang (1):
      drm/i915/gvt: fix double free bug in split_2MB_gtt_entry

Zhenyu Wang (3):
      drm/i915/gvt: fix gvt debugfs destroy
      drm/i915/gvt: fix vgpu debugfs clean in remove
      Merge tag 'drm-intel-fixes-2022-12-30' into gvt-fixes

Zhi Wang (1):
      drm/i915/gvt: use atomic operations to change the vGPU status

 drivers/gpu/drm/i915/gvt/debugfs.c   | 36 +++++++++++++++++++++++++++++++-----
 drivers/gpu/drm/i915/gvt/dmabuf.c    |  3 ++-
 drivers/gpu/drm/i915/gvt/gtt.c       | 21 +++++++++++++++------
 drivers/gpu/drm/i915/gvt/gvt.h       | 15 ++++++++++-----
 drivers/gpu/drm/i915/gvt/interrupt.c |  2 +-
 drivers/gpu/drm/i915/gvt/kvmgt.c     | 35 +++++++++++++----------------------
 drivers/gpu/drm/i915/gvt/scheduler.c |  4 +++-
 drivers/gpu/drm/i915/gvt/vgpu.c      | 12 +++++-------
 8 files changed, 80 insertions(+), 48 deletions(-)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Tvrtko Ursulin]

On 22/11/2022 05:11, Zhenyu Wang wrote:
> 
> ping for this fix pull...

I missed it sorry, pulled now. Will go out this week.

Regards,

Tvrtko

> On 2022.11.11 17:02:08 +0800, Zhenyu Wang wrote:
>> Hi,
>>
>> Here's two fixes from Sean for 6.1 kernel, which is to fix kvm
>> reference in gvt. No regression found in our test.
>>
>> Thanks!
>> ---
>> The following changes since commit f0c4d9fc9cc9462659728d168387191387e903cc:
>>
>>    Linux 6.1-rc4 (2022-11-06 15:07:11 -0800)
>>
>> are available in the Git repository at:
>>
>>    https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-11-11
>>
>> for you to fetch changes up to 3c9fd44b9330adc5006653566f3d386784b2080e:
>>
>>    drm/i915/gvt: Unconditionally put reference to KVM when detaching vGPU (2022-11-11 13:21:52 +0800)
>>
>> ----------------------------------------------------------------
>> gvt-fixes-2022-11-11
>>
>> - kvm reference fix from Sean
>>
>> ----------------------------------------------------------------
>> Sean Christopherson (2):
>>        drm/i915/gvt: Get reference to KVM iff attachment to VM is successful
>>        drm/i915/gvt: Unconditionally put reference to KVM when detaching vGPU
>>
>>   drivers/gpu/drm/i915/gvt/kvmgt.c | 8 +++-----
>>   1 file changed, 3 insertions(+), 5 deletions(-)
>>
> 
> 

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 1180 bytes --]


ping for this fix pull...

On 2022.11.11 17:02:08 +0800, Zhenyu Wang wrote:
> Hi,
> 
> Here's two fixes from Sean for 6.1 kernel, which is to fix kvm
> reference in gvt. No regression found in our test.
> 
> Thanks!
> ---
> The following changes since commit f0c4d9fc9cc9462659728d168387191387e903cc:
> 
>   Linux 6.1-rc4 (2022-11-06 15:07:11 -0800)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-11-11
> 
> for you to fetch changes up to 3c9fd44b9330adc5006653566f3d386784b2080e:
> 
>   drm/i915/gvt: Unconditionally put reference to KVM when detaching vGPU (2022-11-11 13:21:52 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2022-11-11
> 
> - kvm reference fix from Sean
> 
> ----------------------------------------------------------------
> Sean Christopherson (2):
>       drm/i915/gvt: Get reference to KVM iff attachment to VM is successful
>       drm/i915/gvt: Unconditionally put reference to KVM when detaching vGPU
> 
>  drivers/gpu/drm/i915/gvt/kvmgt.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 



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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 999 bytes --]

Hi,

Here's two fixes from Sean for 6.1 kernel, which is to fix kvm
reference in gvt. No regression found in our test.

Thanks!
---
The following changes since commit f0c4d9fc9cc9462659728d168387191387e903cc:

  Linux 6.1-rc4 (2022-11-06 15:07:11 -0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-11-11

for you to fetch changes up to 3c9fd44b9330adc5006653566f3d386784b2080e:

  drm/i915/gvt: Unconditionally put reference to KVM when detaching vGPU (2022-11-11 13:21:52 +0800)

----------------------------------------------------------------
gvt-fixes-2022-11-11

- kvm reference fix from Sean

----------------------------------------------------------------
Sean Christopherson (2):
      drm/i915/gvt: Get reference to KVM iff attachment to VM is successful
      drm/i915/gvt: Unconditionally put reference to KVM when detaching vGPU

 drivers/gpu/drm/i915/gvt/kvmgt.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)


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

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 3284 bytes --]

On 2022.08.18 15:43:32 +0000, Vivi, Rodrigo wrote:
> On Thu, 2022-08-18 at 17:27 +0300, Jani Nikula wrote:
> 
>     On Thu, 18 Aug 2022, Jani Nikula <jani.nikula@intel.com> wrote:
> 
>         On Wed, 17 Aug 2022, "Colin King (gmail)" <colin.i.king@gmail.com>
>         wrote:
> 
>             On 17/08/2022 21:07, Vivi, Rodrigo wrote:
> 
>                 On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote:
> 
>                     On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
> 
>                         On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
> 
>                             On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu
>                             Wang wrote:
> 
>                     oh, surprise! I just found Colin's email is actually
>                     defined in
>                     .mailmap,
>                     so all his commits in kernel are changed for @intel.com
>                     address as in
>                     mailmap...
> 
> 
>                 Colin, would you mind to get the Sign-off-by in the patches the
>                 same
>                 as your authorship so the tools don't get confused?
>                 (starting with modifying in tree this already merged patch)
> 
> 
>             Since my patches are generally trivial janitorial fixed done in my
>             spare
>             time I'm going to get the .mailmap changed to use my gmail email
>             address
>             rather than my Intel one (since I don't do kernel work in my
>             current role).
> 
>             This should clean up the confusion. Apologies.
> 
> 
>         Everyone hold your horses.
> 
>         I think our tooling should handle the mailmap stuff. The commit *is*
>         fine, it's just that the when we check it, we let mailmap alter it. We
>         should check the commit without mailmap modifications.
> 
> 
>     In this case, it's actually not about Colin's Signed-off-by or mailmap
>     at all! Like the error message from dim says, "committer Signed-off-by
>     missing". Committer, not author!
> 
>     $ git show -s tags/gvt-fixes-2022-08-15^ --pretty=fuller
>     commit d6632370536d0b80be3bfc90dd67e1f693335a75
>     Author:     Colin Ian King <colin.king@intel.com>
>     AuthorDate: Tue Mar 15 20:24:49 2022 +0000
>     Commit:     Zhenyu Wang <zhenyuw@linux.intel.com>
>     CommitDate: Mon Aug 15 10:51:15 2022 +0800
> 
>         drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"
>        
>         There is a spelling mistake in a gvt_vgpu_err error message. Fix it.
>        
>         Fixes: 695fbc08d80f ("drm/i915/gvt: replace the gvt_err with
>     gvt_vgpu_err")
>         Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
>         Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
>         Link: http://patchwork.freedesktop.org/patch/msgid/
>     20220315202449.2952845-1-colin.i.king@gmail.com
>         Reviewed-by: Zhi Wang <zhi.a.wang@intel.com>
> 
>     Committed by Zhenyu, Signed-off-by Zhi. Maybe caused by rebase by Zhenyu
>     after being committed by Zhi?
> 
> 
> Probably easier if you could rebase it again signing it then?
>

Oops, sorry about that, will re-send.

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Vivi, Rodrigo]

[-- Attachment #1: Type: text/plain, Size: 2795 bytes --]

On Thu, 2022-08-18 at 17:27 +0300, Jani Nikula wrote:
On Thu, 18 Aug 2022, Jani Nikula <jani.nikula@intel.com<mailto:jani.nikula@intel.com>> wrote:
On Wed, 17 Aug 2022, "Colin King (gmail)" <colin.i.king@gmail.com<mailto:colin.i.king@gmail.com>> wrote:
On 17/08/2022 21:07, Vivi, Rodrigo wrote:
On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote:
On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
oh, surprise! I just found Colin's email is actually defined in
.mailmap,
so all his commits in kernel are changed for @intel.com address as in
mailmap...

Colin, would you mind to get the Sign-off-by in the patches the same
as your authorship so the tools don't get confused?
(starting with modifying in tree this already merged patch)

Since my patches are generally trivial janitorial fixed done in my spare
time I'm going to get the .mailmap changed to use my gmail email address
rather than my Intel one (since I don't do kernel work in my current role).

This should clean up the confusion. Apologies.

Everyone hold your horses.

I think our tooling should handle the mailmap stuff. The commit *is*
fine, it's just that the when we check it, we let mailmap alter it. We
should check the commit without mailmap modifications.

In this case, it's actually not about Colin's Signed-off-by or mailmap
at all! Like the error message from dim says, "committer Signed-off-by
missing". Committer, not author!

$ git show -s tags/gvt-fixes-2022-08-15^ --pretty=fuller
commit d6632370536d0b80be3bfc90dd67e1f693335a75
Author:     Colin Ian King <colin.king@intel.com<mailto:colin.king@intel.com>>
AuthorDate: Tue Mar 15 20:24:49 2022 +0000
Commit:     Zhenyu Wang <zhenyuw@linux.intel.com<mailto:zhenyuw@linux.intel.com>>
CommitDate: Mon Aug 15 10:51:15 2022 +0800

    drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"

    There is a spelling mistake in a gvt_vgpu_err error message. Fix it.

    Fixes: 695fbc08d80f ("drm/i915/gvt: replace the gvt_err with gvt_vgpu_err")
    Signed-off-by: Colin Ian King <colin.i.king@gmail.com<mailto:colin.i.king@gmail.com>>
    Signed-off-by: Zhi Wang <zhi.a.wang@intel.com<mailto:zhi.a.wang@intel.com>>
    Link: http://patchwork.freedesktop.org/patch/msgid/20220315202449.2952845-1-colin.i.king@gmail.com
    Reviewed-by: Zhi Wang <zhi.a.wang@intel.com<mailto:zhi.a.wang@intel.com>>

Committed by Zhenyu, Signed-off-by Zhi. Maybe caused by rebase by Zhenyu
after being committed by Zhi?

Probably easier if you could rebase it again signing it then?

I could bypass dim here, but this will likely affect someone else later in the upstream chain as well..


BR,
Jani.




[-- Attachment #2: Type: text/html, Size: 5394 bytes --]

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Thu, 18 Aug 2022, Jani Nikula <jani.nikula@intel.com> wrote:
> On Wed, 17 Aug 2022, "Colin King (gmail)" <colin.i.king@gmail.com> wrote:
>> On 17/08/2022 21:07, Vivi, Rodrigo wrote:
>>> On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote:
>>>> On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
>>>>> On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
>>>>>> On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
>>>> oh, surprise! I just found Colin's email is actually defined in
>>>> .mailmap,
>>>> so all his commits in kernel are changed for @intel.com address as in
>>>> mailmap...
>>> 
>>> Colin, would you mind to get the Sign-off-by in the patches the same
>>> as your authorship so the tools don't get confused?
>>> (starting with modifying in tree this already merged patch)
>>
>> Since my patches are generally trivial janitorial fixed done in my spare 
>> time I'm going to get the .mailmap changed to use my gmail email address 
>> rather than my Intel one (since I don't do kernel work in my current role).
>>
>> This should clean up the confusion. Apologies.
>
> Everyone hold your horses.
>
> I think our tooling should handle the mailmap stuff. The commit *is*
> fine, it's just that the when we check it, we let mailmap alter it. We
> should check the commit without mailmap modifications.

In this case, it's actually not about Colin's Signed-off-by or mailmap
at all! Like the error message from dim says, "committer Signed-off-by
missing". Committer, not author!

$ git show -s tags/gvt-fixes-2022-08-15^ --pretty=fuller
commit d6632370536d0b80be3bfc90dd67e1f693335a75
Author:     Colin Ian King <colin.king@intel.com>
AuthorDate: Tue Mar 15 20:24:49 2022 +0000
Commit:     Zhenyu Wang <zhenyuw@linux.intel.com>
CommitDate: Mon Aug 15 10:51:15 2022 +0800

    drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"
    
    There is a spelling mistake in a gvt_vgpu_err error message. Fix it.
    
    Fixes: 695fbc08d80f ("drm/i915/gvt: replace the gvt_err with gvt_vgpu_err")
    Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
    Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
    Link: http://patchwork.freedesktop.org/patch/msgid/20220315202449.2952845-1-colin.i.king@gmail.com
    Reviewed-by: Zhi Wang <zhi.a.wang@intel.com>

Committed by Zhenyu, Signed-off-by Zhi. Maybe caused by rebase by Zhenyu
after being committed by Zhi?

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Graphics Center

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Wed, 17 Aug 2022, "Colin King (gmail)" <colin.i.king@gmail.com> wrote:
> On 17/08/2022 21:07, Vivi, Rodrigo wrote:
>> On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote:
>>> On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
>>>> On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
>>>>> On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
>>> oh, surprise! I just found Colin's email is actually defined in
>>> .mailmap,
>>> so all his commits in kernel are changed for @intel.com address as in
>>> mailmap...
>> 
>> Colin, would you mind to get the Sign-off-by in the patches the same
>> as your authorship so the tools don't get confused?
>> (starting with modifying in tree this already merged patch)
>
> Since my patches are generally trivial janitorial fixed done in my spare 
> time I'm going to get the .mailmap changed to use my gmail email address 
> rather than my Intel one (since I don't do kernel work in my current role).
>
> This should clean up the confusion. Apologies.

Everyone hold your horses.

I think our tooling should handle the mailmap stuff. The commit *is*
fine, it's just that the when we check it, we let mailmap alter it. We
should check the commit without mailmap modifications.

BR,
Jani.



-- 
Jani Nikula, Intel Open Source Graphics Center

Re: [Intel-gfx] [PULL] gvt-fixes

[Colin King (gmail)]

On 17/08/2022 21:07, Vivi, Rodrigo wrote:
> On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote:
>> On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
>>> On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
>>>> On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
>>>>>
>>>>> Hi,
>>>>>
>>>>> Here's one gvt-fixes pull for 6.0-rc. Major one is Cometlake
>>>>> regression
>>>>> fix for mmio table rework, and others are left kernel doc fixes
>>>>> not pushed yet.
>>>>>
>>>>> Thanks
>>>>> --
>>>>> The following changes since commit
>>>>> a7a47a5dfa9a9692a41764ee9ab4054f12924a42:
>>>>>
>>>>>    drm/i915/reset: Add additional steps for Wa_22011802037 for
>>>>> execlist backend (2022-07-25 15:57:54 +0100)
>>>>>
>>>>> are available in the Git repository at:
>>>>>
>>>>>    https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-
>>>>> 08-15
>>>>>
>>>>> for you to fetch changes up to
>>>>> 394f0560a76298842defd1d95bd64b203a5fdcc4:
>>>>>
>>>>>    drm/i915/gvt: Fix Comet Lake (2022-08-15 10:54:03 +0800)
>>>>>
>>>>> ---------------------------------------------------------------
>>>>> -
>>>>> gvt-fixes-2022-08-15
>>>>>
>>>>> - CometLake regression fix in mmio table rework (Alex)
>>>>> - misc kernel doc and typo fixes
>>>>>
>>>>> ---------------------------------------------------------------
>>>>> -
>>>>> Alex Williamson (1):
>>>>>        drm/i915/gvt: Fix Comet Lake
>>>>>
>>>>> Colin Ian King (1):
>>>>>        drm/i915/reg: Fix spelling mistake "Unsupport" ->
>>>>> "Unsupported"
>>>>
>>>> dim: d6632370536d ("drm/i915/reg: Fix spelling mistake
>>>> "Unsupport" -> "Unsupported""): committer Signed-off-by missing.
>>>>
>>>> is it possible to fix this in your tree?
>>>
>>> Sorry about that. Let me re-generate.
>>
>> oh, surprise! I just found Colin's email is actually defined in
>> .mailmap,
>> so all his commits in kernel are changed for @intel.com address as in
>> mailmap...
> 
> Colin, would you mind to get the Sign-off-by in the patches the same
> as your authorship so the tools don't get confused?
> (starting with modifying in tree this already merged patch)

Since my patches are generally trivial janitorial fixed done in my spare 
time I'm going to get the .mailmap changed to use my gmail email address 
rather than my Intel one (since I don't do kernel work in my current role).

This should clean up the confusion. Apologies.

Colin

> 
> Thanks,
> Rodrigo.
> 
>>
>> So maybe I can't change that?
>>
>>
> 

Re: [Intel-gfx] [PULL] gvt-fixes

[Vivi, Rodrigo]

On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote:
> On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
> > On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
> > > On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
> > > > 
> > > > Hi,
> > > > 
> > > > Here's one gvt-fixes pull for 6.0-rc. Major one is Cometlake
> > > > regression
> > > > fix for mmio table rework, and others are left kernel doc fixes
> > > > not pushed yet.
> > > > 
> > > > Thanks
> > > > --
> > > > The following changes since commit
> > > > a7a47a5dfa9a9692a41764ee9ab4054f12924a42:
> > > > 
> > > >   drm/i915/reset: Add additional steps for Wa_22011802037 for
> > > > execlist backend (2022-07-25 15:57:54 +0100)
> > > > 
> > > > are available in the Git repository at:
> > > > 
> > > >   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-
> > > > 08-15
> > > > 
> > > > for you to fetch changes up to
> > > > 394f0560a76298842defd1d95bd64b203a5fdcc4:
> > > > 
> > > >   drm/i915/gvt: Fix Comet Lake (2022-08-15 10:54:03 +0800)
> > > > 
> > > > ---------------------------------------------------------------
> > > > -
> > > > gvt-fixes-2022-08-15
> > > > 
> > > > - CometLake regression fix in mmio table rework (Alex)
> > > > - misc kernel doc and typo fixes
> > > > 
> > > > ---------------------------------------------------------------
> > > > -
> > > > Alex Williamson (1):
> > > >       drm/i915/gvt: Fix Comet Lake
> > > > 
> > > > Colin Ian King (1):
> > > >       drm/i915/reg: Fix spelling mistake "Unsupport" ->
> > > > "Unsupported"
> > > 
> > > dim: d6632370536d ("drm/i915/reg: Fix spelling mistake
> > > "Unsupport" -> "Unsupported""): committer Signed-off-by missing.
> > > 
> > > is it possible to fix this in your tree?
> > 
> > Sorry about that. Let me re-generate.
> 
> oh, surprise! I just found Colin's email is actually defined in
> .mailmap,
> so all his commits in kernel are changed for @intel.com address as in
> mailmap...

Colin, would you mind to get the Sign-off-by in the patches the same
as your authorship so the tools don't get confused?
(starting with modifying in tree this already merged patch)

Thanks,
Rodrigo.

> 
> So maybe I can't change that?
> 
> 

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 1803 bytes --]

On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote:
> On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
> > On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
> > > 
> > > Hi,
> > > 
> > > Here's one gvt-fixes pull for 6.0-rc. Major one is Cometlake regression
> > > fix for mmio table rework, and others are left kernel doc fixes not pushed yet.
> > > 
> > > Thanks
> > > --
> > > The following changes since commit a7a47a5dfa9a9692a41764ee9ab4054f12924a42:
> > > 
> > >   drm/i915/reset: Add additional steps for Wa_22011802037 for execlist backend (2022-07-25 15:57:54 +0100)
> > > 
> > > are available in the Git repository at:
> > > 
> > >   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-08-15
> > > 
> > > for you to fetch changes up to 394f0560a76298842defd1d95bd64b203a5fdcc4:
> > > 
> > >   drm/i915/gvt: Fix Comet Lake (2022-08-15 10:54:03 +0800)
> > > 
> > > ----------------------------------------------------------------
> > > gvt-fixes-2022-08-15
> > > 
> > > - CometLake regression fix in mmio table rework (Alex)
> > > - misc kernel doc and typo fixes
> > > 
> > > ----------------------------------------------------------------
> > > Alex Williamson (1):
> > >       drm/i915/gvt: Fix Comet Lake
> > > 
> > > Colin Ian King (1):
> > >       drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"
> > 
> > dim: d6632370536d ("drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported""): committer Signed-off-by missing.
> > 
> > is it possible to fix this in your tree?
> 
> Sorry about that. Let me re-generate.

oh, surprise! I just found Colin's email is actually defined in .mailmap,
so all his commits in kernel are changed for @intel.com address as in mailmap...

So maybe I can't change that?



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

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 1477 bytes --]

On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote:
> On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
> > 
> > Hi,
> > 
> > Here's one gvt-fixes pull for 6.0-rc. Major one is Cometlake regression
> > fix for mmio table rework, and others are left kernel doc fixes not pushed yet.
> > 
> > Thanks
> > --
> > The following changes since commit a7a47a5dfa9a9692a41764ee9ab4054f12924a42:
> > 
> >   drm/i915/reset: Add additional steps for Wa_22011802037 for execlist backend (2022-07-25 15:57:54 +0100)
> > 
> > are available in the Git repository at:
> > 
> >   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-08-15
> > 
> > for you to fetch changes up to 394f0560a76298842defd1d95bd64b203a5fdcc4:
> > 
> >   drm/i915/gvt: Fix Comet Lake (2022-08-15 10:54:03 +0800)
> > 
> > ----------------------------------------------------------------
> > gvt-fixes-2022-08-15
> > 
> > - CometLake regression fix in mmio table rework (Alex)
> > - misc kernel doc and typo fixes
> > 
> > ----------------------------------------------------------------
> > Alex Williamson (1):
> >       drm/i915/gvt: Fix Comet Lake
> > 
> > Colin Ian King (1):
> >       drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"
> 
> dim: d6632370536d ("drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported""): committer Signed-off-by missing.
> 
> is it possible to fix this in your tree?

Sorry about that. Let me re-generate.

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's one gvt-fixes pull for 6.0-rc. Major one is Cometlake regression
> fix for mmio table rework, and others are left kernel doc fixes not pushed yet.
> 
> Thanks
> --
> The following changes since commit a7a47a5dfa9a9692a41764ee9ab4054f12924a42:
> 
>   drm/i915/reset: Add additional steps for Wa_22011802037 for execlist backend (2022-07-25 15:57:54 +0100)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-08-15
> 
> for you to fetch changes up to 394f0560a76298842defd1d95bd64b203a5fdcc4:
> 
>   drm/i915/gvt: Fix Comet Lake (2022-08-15 10:54:03 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2022-08-15
> 
> - CometLake regression fix in mmio table rework (Alex)
> - misc kernel doc and typo fixes
> 
> ----------------------------------------------------------------
> Alex Williamson (1):
>       drm/i915/gvt: Fix Comet Lake
> 
> Colin Ian King (1):
>       drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"

dim: d6632370536d ("drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported""): committer Signed-off-by missing.

is it possible to fix this in your tree?

Thanks,
Rodrigo.

> 
> Jiapeng Chong (3):
>       drm/i915/gvt: Fix kernel-doc
>       drm/i915/gvt: Fix kernel-doc
>       drm/i915/gvt: Fix kernel-doc
> 
> Julia Lawall (1):
>       drm/i915/gvt: fix typo in comment
> 
>  drivers/gpu/drm/i915/gvt/aperture_gm.c      | 4 ++--
>  drivers/gpu/drm/i915/gvt/gtt.c              | 2 +-
>  drivers/gpu/drm/i915/gvt/handlers.c         | 4 ++--
>  drivers/gpu/drm/i915/gvt/mmio_context.c     | 2 +-
>  drivers/gpu/drm/i915/intel_gvt_mmio_table.c | 3 ++-
>  5 files changed, 8 insertions(+), 7 deletions(-)

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 1486 bytes --]


Hi,

Here's one gvt-fixes pull for 6.0-rc. Major one is Cometlake regression
fix for mmio table rework, and others are left kernel doc fixes not pushed yet.

Thanks
--
The following changes since commit a7a47a5dfa9a9692a41764ee9ab4054f12924a42:

  drm/i915/reset: Add additional steps for Wa_22011802037 for execlist backend (2022-07-25 15:57:54 +0100)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-08-15

for you to fetch changes up to 394f0560a76298842defd1d95bd64b203a5fdcc4:

  drm/i915/gvt: Fix Comet Lake (2022-08-15 10:54:03 +0800)

----------------------------------------------------------------
gvt-fixes-2022-08-15

- CometLake regression fix in mmio table rework (Alex)
- misc kernel doc and typo fixes

----------------------------------------------------------------
Alex Williamson (1):
      drm/i915/gvt: Fix Comet Lake

Colin Ian King (1):
      drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported"

Jiapeng Chong (3):
      drm/i915/gvt: Fix kernel-doc
      drm/i915/gvt: Fix kernel-doc
      drm/i915/gvt: Fix kernel-doc

Julia Lawall (1):
      drm/i915/gvt: fix typo in comment

 drivers/gpu/drm/i915/gvt/aperture_gm.c      | 4 ++--
 drivers/gpu/drm/i915/gvt/gtt.c              | 2 +-
 drivers/gpu/drm/i915/gvt/handlers.c         | 4 ++--
 drivers/gpu/drm/i915/gvt/mmio_context.c     | 2 +-
 drivers/gpu/drm/i915/intel_gvt_mmio_table.c | 3 ++-
 5 files changed, 8 insertions(+), 7 deletions(-)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Mon, Jul 11, 2022 at 01:20:21PM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's one gvt fix for 5.19, from Dan for shmem_pin_map() return check bug.
> 
> Thanks!
> ---
> 
> The following changes since commit d72d69abfdb6e0375981cfdda8eb45143f12c77d:
> 
>   drm/i915/gvt: Make DRM_I915_GVT depend on X86 (2022-01-13 18:13:12 +0000)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-07-11
> 
> for you to fetch changes up to e87197fbd137c888fd6c871c72fe7e89445dd015:
> 
>   drm/i915/gvt: IS_ERR() vs NULL bug in intel_gvt_update_reg_whitelist() (2022-07-11 13:05:05 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2022-07-11
> 
> - Fix return value for shmem_pin_map()

pulled to drm-intel-fixes.

Thanks,
Rodrigo.

> 
> ----------------------------------------------------------------
> Dan Carpenter (1):
>       drm/i915/gvt: IS_ERR() vs NULL bug in intel_gvt_update_reg_whitelist()
> 
>  drivers/gpu/drm/i915/gvt/cmd_parser.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 924 bytes --]


Hi,

Here's one gvt fix for 5.19, from Dan for shmem_pin_map() return check bug.

Thanks!
---

The following changes since commit d72d69abfdb6e0375981cfdda8eb45143f12c77d:

  drm/i915/gvt: Make DRM_I915_GVT depend on X86 (2022-01-13 18:13:12 +0000)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2022-07-11

for you to fetch changes up to e87197fbd137c888fd6c871c72fe7e89445dd015:

  drm/i915/gvt: IS_ERR() vs NULL bug in intel_gvt_update_reg_whitelist() (2022-07-11 13:05:05 +0800)

----------------------------------------------------------------
gvt-fixes-2022-07-11

- Fix return value for shmem_pin_map()

----------------------------------------------------------------
Dan Carpenter (1):
      drm/i915/gvt: IS_ERR() vs NULL bug in intel_gvt_update_reg_whitelist()

 drivers/gpu/drm/i915/gvt/cmd_parser.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Sat, 18 Sep 2021, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> Here's one ww lock fini fix from Zhi which resolved recent regression
> with i915 change.

Thanks, pulled and pushed to drm-intel-fixes.

BR,
Jani.

>
> Thanks
> --
> The following changes since commit 71de496cc489b6bae2f51f89da7f28849bf2836e:
>
>   drm/i915/dp: Drop redundant debug print (2021-08-26 07:31:52 -0400)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2021-09-18
>
> for you to fetch changes up to d168cd797982db9db617113644c87b8f5f3cf27e:
>
>   drm/i915/gvt: fix the usage of ww lock in gvt scheduler. (2021-09-13 21:59:31 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2021-09-18
>
> - ww locking fix from Zhi
>
> ----------------------------------------------------------------
> Zhi A Wang (1):
>       drm/i915/gvt: fix the usage of ww lock in gvt scheduler.
>
>  drivers/gpu/drm/i915/gvt/scheduler.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 876 bytes --]


Hi,

Here's one ww lock fini fix from Zhi which resolved recent regression
with i915 change.

Thanks
--
The following changes since commit 71de496cc489b6bae2f51f89da7f28849bf2836e:

  drm/i915/dp: Drop redundant debug print (2021-08-26 07:31:52 -0400)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2021-09-18

for you to fetch changes up to d168cd797982db9db617113644c87b8f5f3cf27e:

  drm/i915/gvt: fix the usage of ww lock in gvt scheduler. (2021-09-13 21:59:31 +0800)

----------------------------------------------------------------
gvt-fixes-2021-09-18

- ww locking fix from Zhi

----------------------------------------------------------------
Zhi A Wang (1):
      drm/i915/gvt: fix the usage of ww lock in gvt scheduler.

 drivers/gpu/drm/i915/gvt/scheduler.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Tue, Aug 10, 2021 at 01:01:33PM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's one regression fix for windows VM hang issue on recent drivers.
> 
> Thanks
> --
> The following changes since commit c90b4503ccf42d9d367e843c223df44aa550e82a:
> 
>   drm/i915/gvt: Clear d3_entered on elsp cmd submission. (2021-07-08 16:42:34 +0800)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2021-08-10
> 
> for you to fetch changes up to 699aa57b35672c3b2f230e2b7e5d0ab8c2bde80a:
> 
>   drm/i915/gvt: Fix cached atomics setting for Windows VM (2021-08-09 14:42:09 +0800)

pulled, thanks

> 
> ----------------------------------------------------------------
> gvt-fixes-2021-08-10
> 
> - Fix windows VM hang issue for atomics workaround (Zhenyu)
> 
> ----------------------------------------------------------------
> Zhenyu Wang (1):
>       drm/i915/gvt: Fix cached atomics setting for Windows VM
> 
>  drivers/gpu/drm/i915/gvt/handlers.c     | 1 +
>  drivers/gpu/drm/i915/gvt/mmio_context.c | 2 ++
>  2 files changed, 3 insertions(+)

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

[-- Attachment #1: Type: text/plain, Size: 940 bytes --]


Hi,

Here's one regression fix for windows VM hang issue on recent drivers.

Thanks
--
The following changes since commit c90b4503ccf42d9d367e843c223df44aa550e82a:

  drm/i915/gvt: Clear d3_entered on elsp cmd submission. (2021-07-08 16:42:34 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2021-08-10

for you to fetch changes up to 699aa57b35672c3b2f230e2b7e5d0ab8c2bde80a:

  drm/i915/gvt: Fix cached atomics setting for Windows VM (2021-08-09 14:42:09 +0800)

----------------------------------------------------------------
gvt-fixes-2021-08-10

- Fix windows VM hang issue for atomics workaround (Zhenyu)

----------------------------------------------------------------
Zhenyu Wang (1):
      drm/i915/gvt: Fix cached atomics setting for Windows VM

 drivers/gpu/drm/i915/gvt/handlers.c     | 1 +
 drivers/gpu/drm/i915/gvt/mmio_context.c | 2 ++
 2 files changed, 3 insertions(+)

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Thu, Jul 15, 2021 at 10:22:37AM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's one fix of shadow ppgtt invalidation with proper vGPU D3 state tracking.
> 
> Thanks

I only noticed this after sending this week's pull request.
I'm sorry about that.

It is now pulled and queued for next week.

Sorry and Thanks,
Rodrigo.

> --
> The following changes since commit 62fb9874f5da54fdb243003b386128037319b219:
> 
>   Linux 5.13 (2021-06-27 15:21:11 -0700)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2021-07-15
> 
> for you to fetch changes up to c90b4503ccf42d9d367e843c223df44aa550e82a:
> 
>   drm/i915/gvt: Clear d3_entered on elsp cmd submission. (2021-07-08 16:42:34 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2021-07-15
> 
> - Fix shadow ppgtt invalidation with proper D3 state tracking (Colin)
> 
> ----------------------------------------------------------------
> Colin Xu (1):
>       drm/i915/gvt: Clear d3_entered on elsp cmd submission.
> 
>  drivers/gpu/drm/i915/gvt/handlers.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)



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

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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's one fix of shadow ppgtt invalidation with proper vGPU D3 state tracking.

Thanks
--
The following changes since commit 62fb9874f5da54fdb243003b386128037319b219:

  Linux 5.13 (2021-06-27 15:21:11 -0700)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2021-07-15

for you to fetch changes up to c90b4503ccf42d9d367e843c223df44aa550e82a:

  drm/i915/gvt: Clear d3_entered on elsp cmd submission. (2021-07-08 16:42:34 +0800)

----------------------------------------------------------------
gvt-fixes-2021-07-15

- Fix shadow ppgtt invalidation with proper D3 state tracking (Colin)

----------------------------------------------------------------
Colin Xu (1):
      drm/i915/gvt: Clear d3_entered on elsp cmd submission.

 drivers/gpu/drm/i915/gvt/handlers.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Tue, Apr 20, 2021 at 10:33:12AM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's one BDW regression fix for cmd parser rework in 5.12 on recent
> Windows guest driver update. Sorry to be a bit late, hopefully can
> still catch up.

pulled, thanks!

> 
> Thanks!
> ---
> The following changes since commit b6a37a93c9ac3900987c79b726d0bb3699d8db4e:
> 
>   drm/i915: Fix invalid access to ACPI _DSM objects (2021-04-07 19:07:44 -0400)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2021-04-20
> 
> for you to fetch changes up to 6b5b2a5bcfe9a250da19bac4ef7cabdc81d154ec:
> 
>   drm/i915/gvt: Fix BDW command parser regression (2021-04-16 13:41:41 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2021-04-20
> 
> - Fix cmd parser regression on BDW (Zhenyu)
> 
> ----------------------------------------------------------------
> Zhenyu Wang (1):
>       drm/i915/gvt: Fix BDW command parser regression
> 
>  drivers/gpu/drm/i915/gvt/cmd_parser.c | 19 +++++++++++++------
>  1 file changed, 13 insertions(+), 6 deletions(-)
> 


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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's one BDW regression fix for cmd parser rework in 5.12 on recent
Windows guest driver update. Sorry to be a bit late, hopefully can
still catch up.

Thanks!
---
The following changes since commit b6a37a93c9ac3900987c79b726d0bb3699d8db4e:

  drm/i915: Fix invalid access to ACPI _DSM objects (2021-04-07 19:07:44 -0400)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2021-04-20

for you to fetch changes up to 6b5b2a5bcfe9a250da19bac4ef7cabdc81d154ec:

  drm/i915/gvt: Fix BDW command parser regression (2021-04-16 13:41:41 +0800)

----------------------------------------------------------------
gvt-fixes-2021-04-20

- Fix cmd parser regression on BDW (Zhenyu)

----------------------------------------------------------------
Zhenyu Wang (1):
      drm/i915/gvt: Fix BDW command parser regression

 drivers/gpu/drm/i915/gvt/cmd_parser.c | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)


[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Fri, 08 Jan 2021, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> Here's one gvt fixes for VFIO edid on APL/BXT with virtual display
> hotplug fixed that feature is enabled again.

Thanks, merged to drm-intel-fixes.

BR,
Jani.

>
> Thanks.
> --
> The following changes since commit e71ba9452f0b5b2e8dc8aa5445198cd9214a6a62:
>
>   Linux 5.11-rc2 (2021-01-03 15:55:30 -0800)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-01-08
>
> for you to fetch changes up to 4ceb06e7c336f4a8d3f3b6ac9a4fea2e9c97dc07:
>
>   drm/i915/gvt: Fix vfio_edid issue for BXT/APL (2021-01-06 11:19:15 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2020-01-08
>
> - Fix VFIO EDID on APL/BXT (Colin)
>
> ----------------------------------------------------------------
> Colin Xu (1):
>       drm/i915/gvt: Fix vfio_edid issue for BXT/APL
>
>  drivers/gpu/drm/i915/gvt/display.c | 81 +++++++++++++++++++++++++++-----------
>  drivers/gpu/drm/i915/gvt/vgpu.c    |  5 +--
>  2 files changed, 59 insertions(+), 27 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's one gvt fixes for VFIO edid on APL/BXT with virtual display
hotplug fixed that feature is enabled again.

Thanks.
--
The following changes since commit e71ba9452f0b5b2e8dc8aa5445198cd9214a6a62:

  Linux 5.11-rc2 (2021-01-03 15:55:30 -0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-01-08

for you to fetch changes up to 4ceb06e7c336f4a8d3f3b6ac9a4fea2e9c97dc07:

  drm/i915/gvt: Fix vfio_edid issue for BXT/APL (2021-01-06 11:19:15 +0800)

----------------------------------------------------------------
gvt-fixes-2020-01-08

- Fix VFIO EDID on APL/BXT (Colin)

----------------------------------------------------------------
Colin Xu (1):
      drm/i915/gvt: Fix vfio_edid issue for BXT/APL

 drivers/gpu/drm/i915/gvt/display.c | 81 +++++++++++++++++++++++++++-----------
 drivers/gpu/drm/i915/gvt/vgpu.c    |  5 +--
 2 files changed, 59 insertions(+), 27 deletions(-)

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Tue, Nov 17, 2020 at 10:39:18AM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's more gvt fixes for 5.10. It temporarily disables VFIO edid
> feature on BXT/APL until its virtual display is really fixed to make
> it work properly. And fixes for DPCD 1.2 and error return in taking
> module reference.
> 
> Thanks
> --
> The following changes since commit 92010a97098c4c9fd777408cc98064d26b32695b:
> 
>   drm/i915/gvt: Fix mmio handler break on BXT/APL. (2020-10-30 11:50:06 +0800)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-11-17
> 
> for you to fetch changes up to 4ec2b69da5e1544dbadb30cddb49c8df60209b0c:
> 
>   drm/i915/gvt: return error when failing to take the module reference (2020-11-13 12:16:52 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2020-11-17
> 
> - Temporarily disable VFIO edid on BXT/APL (Colin)
> - Fix emulated DPCD for version 1.2 (Tina)
> - Fix error return when failing to take module reference (Xiongfeng)

pulled, thanks!

> 
> ----------------------------------------------------------------
> Colin Xu (1):
>       drm/i915/gvt: Temporarily disable vfio_edid for BXT/APL
> 
> Tina Zhang (1):
>       drm/i915/gvt: Set ENHANCED_FRAME_CAP bit
> 
> Xiongfeng Wang (1):
>       drm/i915/gvt: return error when failing to take the module reference
> 
>  drivers/gpu/drm/i915/gvt/display.c | 2 +-
>  drivers/gpu/drm/i915/gvt/kvmgt.c   | 4 +++-
>  drivers/gpu/drm/i915/gvt/vgpu.c    | 3 ++-
>  3 files changed, 6 insertions(+), 3 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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's more gvt fixes for 5.10. It temporarily disables VFIO edid
feature on BXT/APL until its virtual display is really fixed to make
it work properly. And fixes for DPCD 1.2 and error return in taking
module reference.

Thanks
--
The following changes since commit 92010a97098c4c9fd777408cc98064d26b32695b:

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

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-11-17

for you to fetch changes up to 4ec2b69da5e1544dbadb30cddb49c8df60209b0c:

  drm/i915/gvt: return error when failing to take the module reference (2020-11-13 12:16:52 +0800)

----------------------------------------------------------------
gvt-fixes-2020-11-17

- Temporarily disable VFIO edid on BXT/APL (Colin)
- Fix emulated DPCD for version 1.2 (Tina)
- Fix error return when failing to take module reference (Xiongfeng)

----------------------------------------------------------------
Colin Xu (1):
      drm/i915/gvt: Temporarily disable vfio_edid for BXT/APL

Tina Zhang (1):
      drm/i915/gvt: Set ENHANCED_FRAME_CAP bit

Xiongfeng Wang (1):
      drm/i915/gvt: return error when failing to take the module reference

 drivers/gpu/drm/i915/gvt/display.c | 2 +-
 drivers/gpu/drm/i915/gvt/kvmgt.c   | 4 +++-
 drivers/gpu/drm/i915/gvt/vgpu.c    | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

-- 

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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Vivi, Rodrigo]

> 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

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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

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


-- 

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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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

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

-- 

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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Vivi, Rodrigo]

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



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


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


[-- Attachment #1.2: Type: text/html, Size: 11201 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

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


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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


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


-- 

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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Thu, 17 Sep 2020, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> Here's one left GVT fix against 5.9 is for kernel oops with VFIO edid
> on BDW.

Pulled, thanks.

BR,
Jani.

>
> Thanks
> --
> The following changes since commit a291e4fba259a56a6a274c1989997acb6f0bb03a:
>
>   drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context (2020-06-17 12:36:19 +0800)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-09-17
>
> for you to fetch changes up to 28284943ac94014767ecc2f7b3c5747c4a5617a0:
>
>   drm/i915/gvt: Fix port number for BDW on EDID region setup (2020-09-14 16:44:39 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2020-09-17
>
> - Fix kernel oops for VFIO edid on BDW (Zhenyu)
>
> ----------------------------------------------------------------
> Zhenyu Wang (1):
>       drm/i915/gvt: Fix port number for BDW on EDID region setup
>
>  drivers/gpu/drm/i915/gvt/vgpu.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's one left GVT fix against 5.9 is for kernel oops with VFIO edid
on BDW.

Thanks
--
The following changes since commit a291e4fba259a56a6a274c1989997acb6f0bb03a:

  drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context (2020-06-17 12:36:19 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-09-17

for you to fetch changes up to 28284943ac94014767ecc2f7b3c5747c4a5617a0:

  drm/i915/gvt: Fix port number for BDW on EDID region setup (2020-09-14 16:44:39 +0800)

----------------------------------------------------------------
gvt-fixes-2020-09-17

- Fix kernel oops for VFIO edid on BDW (Zhenyu)

----------------------------------------------------------------
Zhenyu Wang (1):
      drm/i915/gvt: Fix port number for BDW on EDID region setup

 drivers/gpu/drm/i915/gvt/vgpu.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)


-- 

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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Tue, 23 Jun 2020, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> On 2020.06.18 15:48:01 +0300, Joonas Lahtinen wrote:
>> Quoting Zhenyu Wang (2020-06-17 07:34:18)
>> > 
>> > Hi,
>> > 
>> > This contains misc fixes for gvt. Two MMIO handler fixes on SKL/CFL,
>> > one mask register bit checking fix exposed in suspend/resume path and
>> > one lockdep error fix for debugfs entry access.
>> 
>> Could not pull this one due to the extra hassle with CI this week.
>> 
>> Jani, can you please pull this next week.
>> 
>
> Got it. Please help to pull then.

Pulled, thanks.

BR,
Jani.


>
> One thing I forgot to mention that change in "drm/i915/gvt: Fix incorrect check of enabled bits in mask registers"
> would cause a minor conflict if backmerging from linux master to dinq, which
> is because of new IS_COMETLAKE. Change like below could resolve that.
>
> diff --git a/drivers/gpu/drm/i915/gvt/handlers.c b/drivers/gpu/drm/i915/gvt/handlers.c
> index 9f48db0bf9d5..78ba2857144e 100644
> --- a/drivers/gpu/drm/i915/gvt/handlers.c
> +++ b/drivers/gpu/drm/i915/gvt/handlers.c
> @@ -1734,14 +1734,9 @@ static int ring_mode_mmio_write(struct intel_vgpu *vgpu, unsigned int offset,
>  		return 0;
>  	}
>  
> -<<<<<<< HEAD
>  	if ((IS_COFFEELAKE(vgpu->gvt->gt->i915) ||
>  	     IS_COMETLAKE(vgpu->gvt->gt->i915)) &&
> -	    data & _MASKED_BIT_ENABLE(2)) {
> -=======
> -	if (IS_COFFEELAKE(vgpu->gvt->gt->i915) &&
>  	    IS_MASKED_BITS_ENABLED(data, 2)) {
> ->>>>>>> origin/gvt-next-fixes
>  		enter_failsafe_mode(vgpu, GVT_FAILSAFE_UNSUPPORTED_GUEST);
>  		return 0;
>  	}
>
>> 
>> > Thanks.
>> > --
>> > The following changes since commit 8e68c6340d5833077b3753eabedab40755571383:
>> > 
>> >   drm/i915/display: Fix the encoder type check (2020-06-16 11:34:24 +0300)
>> > 
>> > are available in the Git repository at:
>> > 
>> >   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-06-17
>> > 
>> > for you to fetch changes up to a291e4fba259a56a6a274c1989997acb6f0bb03a:
>> > 
>> >   drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context (2020-06-17 12:36:19 +0800)
>> > 
>> > ----------------------------------------------------------------
>> > gvt-fixes-2020-06-17
>> > 
>> > - Two missed MMIO handler fixes for SKL/CFL (Colin)
>> > - Fix mask register bits check (Colin)
>> > - Fix one lockdep error for debugfs entry access (Colin)
>> > 
>> > ----------------------------------------------------------------
>> > Colin Xu (4):
>> >       drm/i915/gvt: Add one missing MMIO handler for D_SKL_PLUS
>> >       drm/i915/gvt: Fix two CFL MMIO handling caused by regression.
>> >       drm/i915/gvt: Fix incorrect check of enabled bits in mask registers
>> >       drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context
>> > 
>> >  drivers/gpu/drm/i915/gvt/debugfs.c      |  2 +-
>> >  drivers/gpu/drm/i915/gvt/handlers.c     | 24 +++++++++++++-----------
>> >  drivers/gpu/drm/i915/gvt/mmio_context.h |  6 +++---
>> >  drivers/gpu/drm/i915/gvt/reg.h          |  5 +++++
>> >  4 files changed, 22 insertions(+), 15 deletions(-)
>> _______________________________________________
>> intel-gvt-dev mailing list
>> intel-gvt-dev@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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

On 2020.06.18 15:48:01 +0300, Joonas Lahtinen wrote:
> Quoting Zhenyu Wang (2020-06-17 07:34:18)
> > 
> > Hi,
> > 
> > This contains misc fixes for gvt. Two MMIO handler fixes on SKL/CFL,
> > one mask register bit checking fix exposed in suspend/resume path and
> > one lockdep error fix for debugfs entry access.
> 
> Could not pull this one due to the extra hassle with CI this week.
> 
> Jani, can you please pull this next week.
> 

Got it. Please help to pull then.

One thing I forgot to mention that change in "drm/i915/gvt: Fix incorrect check of enabled bits in mask registers"
would cause a minor conflict if backmerging from linux master to dinq, which
is because of new IS_COMETLAKE. Change like below could resolve that.

diff --git a/drivers/gpu/drm/i915/gvt/handlers.c b/drivers/gpu/drm/i915/gvt/handlers.c
index 9f48db0bf9d5..78ba2857144e 100644
--- a/drivers/gpu/drm/i915/gvt/handlers.c
+++ b/drivers/gpu/drm/i915/gvt/handlers.c
@@ -1734,14 +1734,9 @@ static int ring_mode_mmio_write(struct intel_vgpu *vgpu, unsigned int offset,
 		return 0;
 	}
 
-<<<<<<< HEAD
 	if ((IS_COFFEELAKE(vgpu->gvt->gt->i915) ||
 	     IS_COMETLAKE(vgpu->gvt->gt->i915)) &&
-	    data & _MASKED_BIT_ENABLE(2)) {
-=======
-	if (IS_COFFEELAKE(vgpu->gvt->gt->i915) &&
 	    IS_MASKED_BITS_ENABLED(data, 2)) {
->>>>>>> origin/gvt-next-fixes
 		enter_failsafe_mode(vgpu, GVT_FAILSAFE_UNSUPPORTED_GUEST);
 		return 0;
 	}

> 
> > Thanks.
> > --
> > The following changes since commit 8e68c6340d5833077b3753eabedab40755571383:
> > 
> >   drm/i915/display: Fix the encoder type check (2020-06-16 11:34:24 +0300)
> > 
> > are available in the Git repository at:
> > 
> >   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-06-17
> > 
> > for you to fetch changes up to a291e4fba259a56a6a274c1989997acb6f0bb03a:
> > 
> >   drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context (2020-06-17 12:36:19 +0800)
> > 
> > ----------------------------------------------------------------
> > gvt-fixes-2020-06-17
> > 
> > - Two missed MMIO handler fixes for SKL/CFL (Colin)
> > - Fix mask register bits check (Colin)
> > - Fix one lockdep error for debugfs entry access (Colin)
> > 
> > ----------------------------------------------------------------
> > Colin Xu (4):
> >       drm/i915/gvt: Add one missing MMIO handler for D_SKL_PLUS
> >       drm/i915/gvt: Fix two CFL MMIO handling caused by regression.
> >       drm/i915/gvt: Fix incorrect check of enabled bits in mask registers
> >       drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context
> > 
> >  drivers/gpu/drm/i915/gvt/debugfs.c      |  2 +-
> >  drivers/gpu/drm/i915/gvt/handlers.c     | 24 +++++++++++++-----------
> >  drivers/gpu/drm/i915/gvt/mmio_context.h |  6 +++---
> >  drivers/gpu/drm/i915/gvt/reg.h          |  5 +++++
> >  4 files changed, 22 insertions(+), 15 deletions(-)
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev

-- 
Open Source Technology Center, Intel ltd.

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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Joonas Lahtinen]

Quoting Zhenyu Wang (2020-06-17 07:34:18)
> 
> Hi,
> 
> This contains misc fixes for gvt. Two MMIO handler fixes on SKL/CFL,
> one mask register bit checking fix exposed in suspend/resume path and
> one lockdep error fix for debugfs entry access.

Could not pull this one due to the extra hassle with CI this week.

Jani, can you please pull this next week.

Regards, Joonas

> Thanks.
> --
> The following changes since commit 8e68c6340d5833077b3753eabedab40755571383:
> 
>   drm/i915/display: Fix the encoder type check (2020-06-16 11:34:24 +0300)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-06-17
> 
> for you to fetch changes up to a291e4fba259a56a6a274c1989997acb6f0bb03a:
> 
>   drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context (2020-06-17 12:36:19 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2020-06-17
> 
> - Two missed MMIO handler fixes for SKL/CFL (Colin)
> - Fix mask register bits check (Colin)
> - Fix one lockdep error for debugfs entry access (Colin)
> 
> ----------------------------------------------------------------
> Colin Xu (4):
>       drm/i915/gvt: Add one missing MMIO handler for D_SKL_PLUS
>       drm/i915/gvt: Fix two CFL MMIO handling caused by regression.
>       drm/i915/gvt: Fix incorrect check of enabled bits in mask registers
>       drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context
> 
>  drivers/gpu/drm/i915/gvt/debugfs.c      |  2 +-
>  drivers/gpu/drm/i915/gvt/handlers.c     | 24 +++++++++++++-----------
>  drivers/gpu/drm/i915/gvt/mmio_context.h |  6 +++---
>  drivers/gpu/drm/i915/gvt/reg.h          |  5 +++++
>  4 files changed, 22 insertions(+), 15 deletions(-)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

This contains misc fixes for gvt. Two MMIO handler fixes on SKL/CFL,
one mask register bit checking fix exposed in suspend/resume path and
one lockdep error fix for debugfs entry access.

Thanks.
--
The following changes since commit 8e68c6340d5833077b3753eabedab40755571383:

  drm/i915/display: Fix the encoder type check (2020-06-16 11:34:24 +0300)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-06-17

for you to fetch changes up to a291e4fba259a56a6a274c1989997acb6f0bb03a:

  drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context (2020-06-17 12:36:19 +0800)

----------------------------------------------------------------
gvt-fixes-2020-06-17

- Two missed MMIO handler fixes for SKL/CFL (Colin)
- Fix mask register bits check (Colin)
- Fix one lockdep error for debugfs entry access (Colin)

----------------------------------------------------------------
Colin Xu (4):
      drm/i915/gvt: Add one missing MMIO handler for D_SKL_PLUS
      drm/i915/gvt: Fix two CFL MMIO handling caused by regression.
      drm/i915/gvt: Fix incorrect check of enabled bits in mask registers
      drm/i915/gvt: Use GFP_ATOMIC instead of GFP_KERNEL in atomic context

 drivers/gpu/drm/i915/gvt/debugfs.c      |  2 +-
 drivers/gpu/drm/i915/gvt/handlers.c     | 24 +++++++++++++-----------
 drivers/gpu/drm/i915/gvt/mmio_context.h |  6 +++---
 drivers/gpu/drm/i915/gvt/reg.h          |  5 +++++
 4 files changed, 22 insertions(+), 15 deletions(-)

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Tue, May 12, 2020 at 10:48:03AM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> Here's two more fixes for 5.7. One is for recent guest display probe failure,
> which is fixed by setting correct transcoder and DPLL clock against virtual
> display. Another is regression to fix kernel oops for older aliasing ppgtt guest.

Puled to drm-intel-fixes.
Thanks,
Rodrigo.

> 
> Thanks
> --
> 
> The following changes since commit ec7301d5146c9abe8aaf6e16e420ea3951018503:
> 
>   drm/i915/gvt: switch to user vfio_group_pin/upin_pages (2020-04-14 16:30:17 +0800)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-05-12
> 
> for you to fetch changes up to 72a7a9925e2beea09b109dffb3384c9bf920d9da:
> 
>   drm/i915/gvt: Fix kernel oops for 3-level ppgtt guest (2020-05-11 17:07:25 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2020-05-12
> 
> - Correct transcoder and DPLL initial clock to fix recent guest
>   display probe failure. (Colin)
> - Fix kernel oops on older guest using aliasing ppgtt. (Zhenyu)
> 
> ----------------------------------------------------------------
> Colin Xu (1):
>       drm/i915/gvt: Init DPLL/DDI vreg for virtual display instead of inheritance.
> 
> Zhenyu Wang (1):
>       drm/i915/gvt: Fix kernel oops for 3-level ppgtt guest
> 
>  drivers/gpu/drm/i915/gvt/display.c   | 49 ++++++++++++++++++++++++++++++++----
>  drivers/gpu/drm/i915/gvt/scheduler.c |  6 ++++-
>  2 files changed, 49 insertions(+), 6 deletions(-)
> 
> 
> -- 
> Open Source Technology Center, Intel ltd.
> 
> $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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's two more fixes for 5.7. One is for recent guest display probe failure,
which is fixed by setting correct transcoder and DPLL clock against virtual
display. Another is regression to fix kernel oops for older aliasing ppgtt guest.

Thanks
--

The following changes since commit ec7301d5146c9abe8aaf6e16e420ea3951018503:

  drm/i915/gvt: switch to user vfio_group_pin/upin_pages (2020-04-14 16:30:17 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-05-12

for you to fetch changes up to 72a7a9925e2beea09b109dffb3384c9bf920d9da:

  drm/i915/gvt: Fix kernel oops for 3-level ppgtt guest (2020-05-11 17:07:25 +0800)

----------------------------------------------------------------
gvt-fixes-2020-05-12

- Correct transcoder and DPLL initial clock to fix recent guest
  display probe failure. (Colin)
- Fix kernel oops on older guest using aliasing ppgtt. (Zhenyu)

----------------------------------------------------------------
Colin Xu (1):
      drm/i915/gvt: Init DPLL/DDI vreg for virtual display instead of inheritance.

Zhenyu Wang (1):
      drm/i915/gvt: Fix kernel oops for 3-level ppgtt guest

 drivers/gpu/drm/i915/gvt/display.c   | 49 ++++++++++++++++++++++++++++++++----
 drivers/gpu/drm/i915/gvt/scheduler.c |  6 ++++-
 2 files changed, 49 insertions(+), 6 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: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Rodrigo Vivi]

On Tue, Apr 14, 2020 at 04:36:26PM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> As I stated in last pull
> (https://lists.freedesktop.org/archives/intel-gvt-dev/2020-April/006542.html),
> this one includes gvt fixes for guest page access by using VFIO
> interface instead, so remove the concern from KVM and do the right thing.

Looking the patches now with more calm and the vfio part merged there
it makes sense to get this for -rc1 imho.

So, pulled to drm-intel-fixes.

thanks for the patches and sorry for the previous confusion,
Rodrigo.

> 
> Thanks
> --
> The following changes since commit bcad588dea538a4fc173d16a90a005536ec8dbf2:
> 
>   drm/i915/perf: Do not clear pollin for small user read buffers (2020-04-13 14:09:48 -0700)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-04-14
> 
> for you to fetch changes up to ec7301d5146c9abe8aaf6e16e420ea3951018503:
> 
>   drm/i915/gvt: switch to user vfio_group_pin/upin_pages (2020-04-14 16:30:17 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2020-04-14
> 
> - Fix guest page access by using VFIO dma r/w interface (Yan)
> 
> ----------------------------------------------------------------
> Yan Zhao (3):
>       drm/i915/gvt: hold reference of VFIO group during opening of vgpu
>       drm/i915/gvt: subsitute kvm_read/write_guest with vfio_dma_rw
>       drm/i915/gvt: switch to user vfio_group_pin/upin_pages
> 
>  drivers/gpu/drm/i915/gvt/kvmgt.c | 46 +++++++++++++++++++---------------------
>  1 file changed, 22 insertions(+), 24 deletions(-)
> 
> -- 
> Open Source Technology Center, Intel ltd.
> 
> $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

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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

As I stated in last pull
(https://lists.freedesktop.org/archives/intel-gvt-dev/2020-April/006542.html),
this one includes gvt fixes for guest page access by using VFIO
interface instead, so remove the concern from KVM and do the right thing.

Thanks
--
The following changes since commit bcad588dea538a4fc173d16a90a005536ec8dbf2:

  drm/i915/perf: Do not clear pollin for small user read buffers (2020-04-13 14:09:48 -0700)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-04-14

for you to fetch changes up to ec7301d5146c9abe8aaf6e16e420ea3951018503:

  drm/i915/gvt: switch to user vfio_group_pin/upin_pages (2020-04-14 16:30:17 +0800)

----------------------------------------------------------------
gvt-fixes-2020-04-14

- Fix guest page access by using VFIO dma r/w interface (Yan)

----------------------------------------------------------------
Yan Zhao (3):
      drm/i915/gvt: hold reference of VFIO group during opening of vgpu
      drm/i915/gvt: subsitute kvm_read/write_guest with vfio_dma_rw
      drm/i915/gvt: switch to user vfio_group_pin/upin_pages

 drivers/gpu/drm/i915/gvt/kvmgt.c | 46 +++++++++++++++++++---------------------
 1 file changed, 22 insertions(+), 24 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: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Tue, 10 Mar 2020, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> Here's more gvt fixes for 5.6. Fix timer issue caused by idr destroy
> change and VBT size error.

Pulled and pushed to drm-intel-fixes, thanks.

BR,
Jani.


>
> Thanks
> --
>
> The following changes since commit b549c252b1292aea959cd9b83537fcb9384a6112:
>
>   drm/i915/gvt: Fix orphan vgpu dmabuf_objs' lifetime (2020-02-25 16:14:20 +0800)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-03-10
>
> for you to fetch changes up to 2fa7e15c5f466fdd0c0b196b1dc4a65d191efd96:
>
>   drm/i915/gvt: Fix emulated vbt size issue (2020-03-06 09:35:30 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2020-03-10
>
> - Fix vgpu idr destroy causing timer destroy failure (Zhenyu)
> - Fix VBT size (Tina)
>
> ----------------------------------------------------------------
> Tina Zhang (1):
>       drm/i915/gvt: Fix emulated vbt size issue
>
> Zhenyu Wang (1):
>       drm/i915/gvt: Fix unnecessary schedule timer when no vGPU exits
>
>  drivers/gpu/drm/i915/gvt/opregion.c |  5 ++---
>  drivers/gpu/drm/i915/gvt/vgpu.c     | 12 +++++++++---
>  2 files changed, 11 insertions(+), 6 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's more gvt fixes for 5.6. Fix timer issue caused by idr destroy
change and VBT size error.

Thanks
--

The following changes since commit b549c252b1292aea959cd9b83537fcb9384a6112:

  drm/i915/gvt: Fix orphan vgpu dmabuf_objs' lifetime (2020-02-25 16:14:20 +0800)

are available in the Git repository at:

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

for you to fetch changes up to 2fa7e15c5f466fdd0c0b196b1dc4a65d191efd96:

  drm/i915/gvt: Fix emulated vbt size issue (2020-03-06 09:35:30 +0800)

----------------------------------------------------------------
gvt-fixes-2020-03-10

- Fix vgpu idr destroy causing timer destroy failure (Zhenyu)
- Fix VBT size (Tina)

----------------------------------------------------------------
Tina Zhang (1):
      drm/i915/gvt: Fix emulated vbt size issue

Zhenyu Wang (1):
      drm/i915/gvt: Fix unnecessary schedule timer when no vGPU exits

 drivers/gpu/drm/i915/gvt/opregion.c |  5 ++---
 drivers/gpu/drm/i915/gvt/vgpu.c     | 12 +++++++++---
 2 files changed, 11 insertions(+), 6 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: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Wed, 26 Feb 2020, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> Here's gvt-fixes for 5.6-rc with two fixes. One to resolve virtual
> display reset and another one for use-after-free in dmabuf destroy
> function.

Pulled, thanks.

BR,
Jani.

>
> Thanks
> --
> The following changes since commit 0e9d7bb293f3f9c3ee376b126141407efb265f31:
>
>   drm/i915/gvt: more locking for ppgtt mm LRU list (2020-02-10 10:04:34 +0800)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux tags/gvt-fixes-2020-02-26
>
> for you to fetch changes up to b549c252b1292aea959cd9b83537fcb9384a6112:
>
>   drm/i915/gvt: Fix orphan vgpu dmabuf_objs' lifetime (2020-02-25 16:14:20 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2020-02-26
>
> - Fix virtual display reset (Tina)
> - Fix one use-after-free for dmabuf (Tina)
>
> ----------------------------------------------------------------
> Tina Zhang (2):
>       drm/i915/gvt: Separate display reset from ALL_ENGINES reset
>       drm/i915/gvt: Fix orphan vgpu dmabuf_objs' lifetime
>
>  drivers/gpu/drm/i915/gvt/dmabuf.c | 2 +-
>  drivers/gpu/drm/i915/gvt/vgpu.c   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's gvt-fixes for 5.6-rc with two fixes. One to resolve virtual
display reset and another one for use-after-free in dmabuf destroy
function.

Thanks
--
The following changes since commit 0e9d7bb293f3f9c3ee376b126141407efb265f31:

  drm/i915/gvt: more locking for ppgtt mm LRU list (2020-02-10 10:04:34 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2020-02-26

for you to fetch changes up to b549c252b1292aea959cd9b83537fcb9384a6112:

  drm/i915/gvt: Fix orphan vgpu dmabuf_objs' lifetime (2020-02-25 16:14:20 +0800)

----------------------------------------------------------------
gvt-fixes-2020-02-26

- Fix virtual display reset (Tina)
- Fix one use-after-free for dmabuf (Tina)

----------------------------------------------------------------
Tina Zhang (2):
      drm/i915/gvt: Separate display reset from ALL_ENGINES reset
      drm/i915/gvt: Fix orphan vgpu dmabuf_objs' lifetime

 drivers/gpu/drm/i915/gvt/dmabuf.c | 2 +-
 drivers/gpu/drm/i915/gvt/vgpu.c   | 2 +-
 2 files changed, 2 insertions(+), 2 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: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Jani Nikula]

On Wed, 12 Feb 2020, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> Hi,
>
> Here's two fixes for gvt. One for missed locking and another for
> firmware allocation change for late load.

Pulled, thanks.

BR,
Jani.

>
> Thanks
> --
> The following changes since commit 5e822e44cecec1ea48925630aa31dfac827fd202:
>
>   drm/i915/gvt: Fix guest boot warning (2019-12-17 11:19:58 +0800)
>
> are available in the Git repository at:
>
>   https://github.com/intel/gvt-linux.git tags/gvt-fixes-2020-02-12
>
> for you to fetch changes up to 0e9d7bb293f3f9c3ee376b126141407efb265f31:
>
>   drm/i915/gvt: more locking for ppgtt mm LRU list (2020-02-10 10:04:34 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2020-02-12
>
> - fix possible high-order allocation fail for late load (Igor)
> - fix one missed lock for ppgtt mm LRU list (Igor)
>
> ----------------------------------------------------------------
> Igor Druzhinin (2):
>       drm/i915/gvt: fix high-order allocation failure on late load
>       drm/i915/gvt: more locking for ppgtt mm LRU list
>
>  drivers/gpu/drm/i915/gvt/firmware.c | 4 ++--
>  drivers/gpu/drm/i915/gvt/gtt.c      | 4 ++++
>  2 files changed, 6 insertions(+), 2 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's two fixes for gvt. One for missed locking and another for
firmware allocation change for late load.

Thanks
--
The following changes since commit 5e822e44cecec1ea48925630aa31dfac827fd202:

  drm/i915/gvt: Fix guest boot warning (2019-12-17 11:19:58 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2020-02-12

for you to fetch changes up to 0e9d7bb293f3f9c3ee376b126141407efb265f31:

  drm/i915/gvt: more locking for ppgtt mm LRU list (2020-02-10 10:04:34 +0800)

----------------------------------------------------------------
gvt-fixes-2020-02-12

- fix possible high-order allocation fail for late load (Igor)
- fix one missed lock for ppgtt mm LRU list (Igor)

----------------------------------------------------------------
Igor Druzhinin (2):
      drm/i915/gvt: fix high-order allocation failure on late load
      drm/i915/gvt: more locking for ppgtt mm LRU list

 drivers/gpu/drm/i915/gvt/firmware.c | 4 ++--
 drivers/gpu/drm/i915/gvt/gtt.c      | 4 ++++
 2 files changed, 6 insertions(+), 2 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: 195 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

Re: [Intel-gfx] [PULL] gvt-fixes

[Joonas Lahtinen]

Quoting Zhenyu Wang (2019-12-18 07:16:57)
> 
> Hi,
> 
> Here's current gvt fixes which has fix for vGPU display dmabuf,
> one guest reset warning and one locking issue. Details below.

Pulled, thanks for the PR. I'll send out this tomorrow along with
other fixes once CI results arrive.

I'll be out of office for next two weeks then, please ping Jani/
Rodrigo if there is something that needs reaction before that.

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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's current gvt fixes which has fix for vGPU display dmabuf,
one guest reset warning and one locking issue. Details below.

Thanks
--
The following changes since commit f26a9e959a7b1588c59f7a919b41b67175b211d8:

  drm/i915/gt: Detect if we miss WaIdleLiteRestore (2019-12-10 12:37:53 +0200)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2019-12-18

for you to fetch changes up to 5e822e44cecec1ea48925630aa31dfac827fd202:

  drm/i915/gvt: Fix guest boot warning (2019-12-17 11:19:58 +0800)

----------------------------------------------------------------
gvt-fixes-2019-12-18

- vGPU state setting locking fix (Zhenyu)
- Fix vGPU display dmabuf as read-only (Zhenyu)
- Properly handle vGPU display dmabuf page pin when rendering (Tina)
- Fix one guest boot warning to handle guc reset state (Fred)

----------------------------------------------------------------
Gao Fred (1):
      drm/i915/gvt: Fix guest boot warning

Tina Zhang (1):
      drm/i915/gvt: Pin vgpu dma address before using

Zhenyu Wang (2):
      drm/i915/gvt: use vgpu lock for active state setting
      drm/i915/gvt: set guest display buffer as readonly

 drivers/gpu/drm/i915/gvt/dmabuf.c    | 64 +++++++++++++++++++++++++++++++++---
 drivers/gpu/drm/i915/gvt/handlers.c  | 16 +++++++++
 drivers/gpu/drm/i915/gvt/hypercall.h |  2 ++
 drivers/gpu/drm/i915/gvt/kvmgt.c     | 23 +++++++++++++
 drivers/gpu/drm/i915/gvt/mpt.h       | 15 +++++++++
 drivers/gpu/drm/i915/gvt/vgpu.c      |  4 +--
 6 files changed, 118 insertions(+), 6 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: 195 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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Yesterday Alex reported one windows guest failure with older windows
driver version that was missed in our validation, as newer windows
driver version doesn't use MI_ATOMIC to trigger the issue. This one
fixes the failure.

I'm not sure if this can still catch up 5.4, but it's cc stable anyway.

Thanks.
--
The following changes since commit 41d931459b53e32c67a1f8838d1e6826a69ee745:

  drm/i915/gvt: fix dropping obj reference twice (2019-11-08 11:04:07 +0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-fixes-2019-11-22

for you to fetch changes up to a40c273d80f6aff28e619cdcd4b34e9080034c70:

  drm/i915/gvt: Fix cmd length check for MI_ATOMIC (2019-11-22 10:54:19 +0800)

----------------------------------------------------------------
gvt-fixes-2019-11-22

- Fix MI_ATOMIC cmd parser for length (Zhenyu)

----------------------------------------------------------------
Zhenyu Wang (1):
      drm/i915/gvt: Fix cmd length check for MI_ATOMIC

 drivers/gpu/drm/i915/gvt/cmd_parser.c | 6 +++---
 1 file changed, 3 insertions(+), 3 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: 195 bytes --]

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

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

Re: [Intel-gfx] [PULL] gvt-fixes

[Vivi, Rodrigo]

pulled, thanks

> On Nov 11, 2019, at 10:18 PM, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> 
> 
> Hi,
> 
> Here's one GVT dmabuf reference drop fix from Pan Bian.
> 
> Thanks
> --
> The following changes since commit ee2c5ef8a9d640ee1617ec97b84fe2f634284e51:
> 
>  drm/i915/dp: Do not switch aux to TBT mode for non-TC ports (2019-11-04 13:24:14 -0800)
> 
> are available in the Git repository at:
> 
>  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-11-12
> 
> for you to fetch changes up to 41d931459b53e32c67a1f8838d1e6826a69ee745:
> 
>  drm/i915/gvt: fix dropping obj reference twice (2019-11-08 11:04:07 +0800)
> 
> ----------------------------------------------------------------
> gvt-fixes-2019-11-12
> 
> - Fix dmabuf reference drop (Pan)
> 
> ----------------------------------------------------------------
> Pan Bian (1):
>      drm/i915/gvt: fix dropping obj reference twice
> 
> drivers/gpu/drm/i915/gvt/dmabuf.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> -- 
> Open Source Technology Center, Intel ltd.
> 
> $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

[Intel-gfx] [PULL] gvt-fixes

[Zhenyu Wang]

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


Hi,

Here's one GVT dmabuf reference drop fix from Pan Bian.

Thanks
--
The following changes since commit ee2c5ef8a9d640ee1617ec97b84fe2f634284e51:

  drm/i915/dp: Do not switch aux to TBT mode for non-TC ports (2019-11-04 13:24:14 -0800)

are available in the Git repository at:

  https://github.com/intel/gvt-linux.git tags/gvt-fixes-2019-11-12

for you to fetch changes up to 41d931459b53e32c67a1f8838d1e6826a69ee745:

  drm/i915/gvt: fix dropping obj reference twice (2019-11-08 11:04:07 +0800)

----------------------------------------------------------------
gvt-fixes-2019-11-12

- Fix dmabuf reference drop (Pan)

----------------------------------------------------------------
Pan Bian (1):
      drm/i915/gvt: fix dropping obj reference twice

 drivers/gpu/drm/i915/gvt/dmabuf.c | 4 ++--
 1 file changed, 2 insertions(+), 2 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: 195 bytes --]

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

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