mbox series

[PULL] gvt-next

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

Pull-request

https://github.com/intel/gvt-linux tags/gvt-next-2020-09-10

Message

Zhenyu Wang Sept. 10, 2020, 5:37 a.m. UTC 
Hi,

As we split pull request for 5.10 this time, here's gvt-next pull
for 5.10. For gvt ww lock fix, Zhi would send another pull based
on gem-next.

This includes current command access flag cleanup for
handlers which would be used for next refined cmd scan. And also
two more recent fixes on workaround cmd access and MIA reset state.

Thanks
--
The following changes since commit ced026e959bec5046afa310d6474e147b6294da2:

  drm/i915: Update DRIVER_DATE to 20200824 (2020-08-24 14:26:38 -0400)

are available in the Git repository at:

  https://github.com/intel/gvt-linux tags/gvt-next-2020-09-10

for you to fetch changes up to df398e33b8fd3ac28b3c7166de555e38d26e7391:

  drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET (2020-09-10 13:49:05 +0800)

----------------------------------------------------------------
gvt-next-2020-09-10

- Cleanup command access flag (Yan)
- New workaround cmd access fix (Colin)
- MIA reset state fix (Colin)

----------------------------------------------------------------
Colin Xu (2):
      drm/i915/gvt: Add F_CMD_ACCESS for some GEN9 SKU WA MMIO access
      drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET

Yan Zhao (4):
      drm/i915/gvt: rename F_IN_CTX flag to F_SR_IN_CTX
      drm/i915/gvt: remove flag F_CMD_ACCESSED
      drm/i915/gvt: add/modify interfaces for flag F_CMD_ACCESS
      drm/i915/gvt: remove F_CMD_ACCESS flag for some registers

 drivers/gpu/drm/i915/gvt/cmd_parser.c   |  6 ++---
 drivers/gpu/drm/i915/gvt/gvt.h          | 44 +++++++++++++++++++--------------
 drivers/gpu/drm/i915/gvt/handlers.c     | 32 +++++++++++++-----------
 drivers/gpu/drm/i915/gvt/mmio.c         |  3 +++
 drivers/gpu/drm/i915/gvt/mmio_context.c |  2 +-
 5 files changed, 49 insertions(+), 38 deletions(-)

Comments

Rodrigo Vivi Sept. 11, 2020, 11:58 p.m. UTC | #1 
On Thu, Sep 10, 2020 at 01:37:20PM +0800, Zhenyu Wang wrote:
> 
> Hi,
> 
> As we split pull request for 5.10 this time, here's gvt-next pull
> for 5.10. For gvt ww lock fix, Zhi would send another pull based
> on gem-next.
> 
> This includes current command access flag cleanup for
> handlers which would be used for next refined cmd scan. And also
> two more recent fixes on workaround cmd access and MIA reset state.
> 
> Thanks
> --
> The following changes since commit ced026e959bec5046afa310d6474e147b6294da2:
> 
>   drm/i915: Update DRIVER_DATE to 20200824 (2020-08-24 14:26:38 -0400)
> 
> are available in the Git repository at:
> 
>   https://github.com/intel/gvt-linux tags/gvt-next-2020-09-10

This is a malformed pull request line which dim doesn't recognize.
Could you please regenerate it?

$ cat /tmp/gvt-next/cur/1599868544.259925_1.rdvivi-losangeles\:2\,S | dim apply-pull drm-intel-next-queued
dim: no pull request found

> 
> for you to fetch changes up to df398e33b8fd3ac28b3c7166de555e38d26e7391:
> 
>   drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET (2020-09-10 13:49:05 +0800)
> 
> ----------------------------------------------------------------
> gvt-next-2020-09-10
> 
> - Cleanup command access flag (Yan)
> - New workaround cmd access fix (Colin)
> - MIA reset state fix (Colin)
> 
> ----------------------------------------------------------------
> Colin Xu (2):
>       drm/i915/gvt: Add F_CMD_ACCESS for some GEN9 SKU WA MMIO access
>       drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET
> 
> Yan Zhao (4):
>       drm/i915/gvt: rename F_IN_CTX flag to F_SR_IN_CTX
>       drm/i915/gvt: remove flag F_CMD_ACCESSED
>       drm/i915/gvt: add/modify interfaces for flag F_CMD_ACCESS
>       drm/i915/gvt: remove F_CMD_ACCESS flag for some registers
> 
>  drivers/gpu/drm/i915/gvt/cmd_parser.c   |  6 ++---
>  drivers/gpu/drm/i915/gvt/gvt.h          | 44 +++++++++++++++++++--------------
>  drivers/gpu/drm/i915/gvt/handlers.c     | 32 +++++++++++++-----------
>  drivers/gpu/drm/i915/gvt/mmio.c         |  3 +++
>  drivers/gpu/drm/i915/gvt/mmio_context.c |  2 +-
>  5 files changed, 49 insertions(+), 38 deletions(-)
> 
> -- 
> 
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
Zhenyu Wang Sept. 14, 2020, 2:34 a.m. UTC | #2 
On 2020.09.11 19:58:25 -0400, Rodrigo Vivi wrote:
> On Thu, Sep 10, 2020 at 01:37:20PM +0800, Zhenyu Wang wrote:
> > 
> > Hi,
> > 
> > As we split pull request for 5.10 this time, here's gvt-next pull
> > for 5.10. For gvt ww lock fix, Zhi would send another pull based
> > on gem-next.
> > 
> > This includes current command access flag cleanup for
> > handlers which would be used for next refined cmd scan. And also
> > two more recent fixes on workaround cmd access and MIA reset state.
> > 
> > Thanks
> > --
> > The following changes since commit ced026e959bec5046afa310d6474e147b6294da2:
> > 
> >   drm/i915: Update DRIVER_DATE to 20200824 (2020-08-24 14:26:38 -0400)
> > 
> > are available in the Git repository at:
> > 
> >   https://github.com/intel/gvt-linux tags/gvt-next-2020-09-10
> 
> This is a malformed pull request line which dim doesn't recognize.
> Could you please regenerate it?
> 
> $ cat /tmp/gvt-next/cur/1599868544.259925_1.rdvivi-losangeles\:2\,S | dim apply-pull drm-intel-next-queued
> dim: no pull request found
>

Hmm, strange, it pulls fine here when I tried this in local. I just copied in mutt and
     cat /tmp/gvt-next.0910 | ./dim apply-pull drm-intel-next-queued

> > 
> > for you to fetch changes up to df398e33b8fd3ac28b3c7166de555e38d26e7391:
> > 
> >   drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET (2020-09-10 13:49:05 +0800)
> > 
> > ----------------------------------------------------------------
> > gvt-next-2020-09-10
> > 
> > - Cleanup command access flag (Yan)
> > - New workaround cmd access fix (Colin)
> > - MIA reset state fix (Colin)
> > 
> > ----------------------------------------------------------------
> > Colin Xu (2):
> >       drm/i915/gvt: Add F_CMD_ACCESS for some GEN9 SKU WA MMIO access
> >       drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET
> > 
> > Yan Zhao (4):
> >       drm/i915/gvt: rename F_IN_CTX flag to F_SR_IN_CTX
> >       drm/i915/gvt: remove flag F_CMD_ACCESSED
> >       drm/i915/gvt: add/modify interfaces for flag F_CMD_ACCESS
> >       drm/i915/gvt: remove F_CMD_ACCESS flag for some registers
> > 
> >  drivers/gpu/drm/i915/gvt/cmd_parser.c   |  6 ++---
> >  drivers/gpu/drm/i915/gvt/gvt.h          | 44 +++++++++++++++++++--------------
> >  drivers/gpu/drm/i915/gvt/handlers.c     | 32 +++++++++++++-----------
> >  drivers/gpu/drm/i915/gvt/mmio.c         |  3 +++
> >  drivers/gpu/drm/i915/gvt/mmio_context.c |  2 +-
> >  5 files changed, 49 insertions(+), 38 deletions(-)
> > 
> > -- 
> > 
> > $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
> 
> 
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
Rodrigo Vivi Sept. 14, 2020, 6:39 p.m. UTC | #3 
> On Sep 13, 2020, at 7:34 PM, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> 
> On 2020.09.11 19:58:25 -0400, Rodrigo Vivi wrote:
>> On Thu, Sep 10, 2020 at 01:37:20PM +0800, Zhenyu Wang wrote:
>>> 
>>> Hi,
>>> 
>>> As we split pull request for 5.10 this time, here's gvt-next pull
>>> for 5.10. For gvt ww lock fix, Zhi would send another pull based
>>> on gem-next.
>>> 
>>> This includes current command access flag cleanup for
>>> handlers which would be used for next refined cmd scan. And also
>>> two more recent fixes on workaround cmd access and MIA reset state.
>>> 
>>> Thanks
>>> --
>>> The following changes since commit ced026e959bec5046afa310d6474e147b6294da2:
>>> 
>>>  drm/i915: Update DRIVER_DATE to 20200824 (2020-08-24 14:26:38 -0400)
>>> 
>>> are available in the Git repository at:
>>> 
>>>  https://github.com/intel/gvt-linux tags/gvt-next-2020-09-10
>> 
>> This is a malformed pull request line which dim doesn't recognize.
>> Could you please regenerate it?
>> 
>> $ cat /tmp/gvt-next/cur/1599868544.259925_1.rdvivi-losangeles\:2\,S | dim apply-pull drm-intel-next-queued
>> dim: no pull request found
>> 
> 
> Hmm, strange, it pulls fine here when I tried this in local. I just copied in mutt and
>     cat /tmp/gvt-next.0910 | ./dim apply-pull drm-intel-next-queued

sorry... it was dim's fault ;)

dim now accepts python3, but a function used on email parser doesn't work in
the same way...

pulled now, thanks

> 
>>> 
>>> for you to fetch changes up to df398e33b8fd3ac28b3c7166de555e38d26e7391:
>>> 
>>>  drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET (2020-09-10 13:49:05 +0800)
>>> 
>>> ----------------------------------------------------------------
>>> gvt-next-2020-09-10
>>> 
>>> - Cleanup command access flag (Yan)
>>> - New workaround cmd access fix (Colin)
>>> - MIA reset state fix (Colin)
>>> 
>>> ----------------------------------------------------------------
>>> Colin Xu (2):
>>>      drm/i915/gvt: Add F_CMD_ACCESS for some GEN9 SKU WA MMIO access
>>>      drm/i915/gvt: Init vreg GUC_STATUS to GS_MIA_IN_RESET
>>> 
>>> Yan Zhao (4):
>>>      drm/i915/gvt: rename F_IN_CTX flag to F_SR_IN_CTX
>>>      drm/i915/gvt: remove flag F_CMD_ACCESSED
>>>      drm/i915/gvt: add/modify interfaces for flag F_CMD_ACCESS
>>>      drm/i915/gvt: remove F_CMD_ACCESS flag for some registers
>>> 
>>> drivers/gpu/drm/i915/gvt/cmd_parser.c   |  6 ++---
>>> drivers/gpu/drm/i915/gvt/gvt.h          | 44 +++++++++++++++++++--------------
>>> drivers/gpu/drm/i915/gvt/handlers.c     | 32 +++++++++++++-----------
>>> drivers/gpu/drm/i915/gvt/mmio.c         |  3 +++
>>> drivers/gpu/drm/i915/gvt/mmio_context.c |  2 +-
>>> 5 files changed, 49 insertions(+), 38 deletions(-)
>>> 
>>> -- 
>>> 
>>> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
>> 
>> 
>> _______________________________________________
>> intel-gvt-dev mailing list
>> intel-gvt-dev@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
> 
> -- 
> 
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827