mbox series

[CI,v2,0/4] drm/i915/guc: Update to GuC v49

Message ID 20201015182901.3197788-1-John.C.Harrison@Intel.com (mailing list archive)
Headers show
Series drm/i915/guc: Update to GuC v49 | expand

Message

John Harrison Oct. 15, 2020, 6:28 p.m. UTC
From: John Harrison <John.C.Harrison@Intel.com>

Update to the latest GuC firmware

v2: Rebase to newer tree, updated a commit message (review feedback
from Daniele) and dropped the patch to enable GuC/HuC loading by
default as apparently this is not allowed.

Signed-off-by: John Harrison <John.C.Harrison@Intel.com>


John Harrison (4):
  drm/i915/guc: Update to use firmware v49.0.1
  drm/i915/guc: Improved reporting when GuC fails to load
  drm/i915/guc: Clear pointers on free
  CI: turn on GuC/HuC auto mode by default

 drivers/gpu/drm/i915/gt/intel_engine_cs.c    |   3 +-
 drivers/gpu/drm/i915/gt/uc/intel_guc.c       |  18 ---
 drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c   | 132 +++++++++++++++----
 drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c    |   1 +
 drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c    |  31 +++--
 drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h  |  80 +++++------
 drivers/gpu/drm/i915/gt/uc/intel_guc_reg.h   |   5 +
 drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c     |  29 ++--
 drivers/gpu/drm/i915/gt/uc/intel_uc_fw.h     |   2 +
 drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h |   6 +-
 drivers/gpu/drm/i915/i915_params.h           |   2 +-
 11 files changed, 202 insertions(+), 107 deletions(-)

Comments

Tvrtko Ursulin Oct. 22, 2020, 3:22 p.m. UTC | #1
+ Joonas for maintainer class question.

On 15/10/2020 19:28, John.C.Harrison@Intel.com wrote:
> From: John Harrison <John.C.Harrison@Intel.com>
> 
> Update to the latest GuC firmware
> 
> v2: Rebase to newer tree, updated a commit message (review feedback
> from Daniele) and dropped the patch to enable GuC/HuC loading by
> default as apparently this is not allowed.
> 
> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> 
> 
> John Harrison (4):
>    drm/i915/guc: Update to use firmware v49.0.1
>    drm/i915/guc: Improved reporting when GuC fails to load
>    drm/i915/guc: Clear pointers on free
>    CI: turn on GuC/HuC auto mode by default
> 
>   drivers/gpu/drm/i915/gt/intel_engine_cs.c    |   3 +-
>   drivers/gpu/drm/i915/gt/uc/intel_guc.c       |  18 ---
>   drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c   | 132 +++++++++++++++----
>   drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c    |   1 +
>   drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c    |  31 +++--
>   drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h  |  80 +++++------
>   drivers/gpu/drm/i915/gt/uc/intel_guc_reg.h   |   5 +
>   drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c     |  29 ++--
>   drivers/gpu/drm/i915/gt/uc/intel_uc_fw.h     |   2 +
>   drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h |   6 +-
>   drivers/gpu/drm/i915/i915_params.h           |   2 +-
>   11 files changed, 202 insertions(+), 107 deletions(-)

I tried to merge this for John today but the series applies to 
drm-intel-next-queued and not drm-intel-gt-next, which I thought is 
where GT/GuC should go to. Long story short, where to merge GuC patches 
and how is the question?

Regards,

Tvrtko
Joonas Lahtinen Oct. 23, 2020, 6:24 a.m. UTC | #2
Quoting Tvrtko Ursulin (2020-10-22 18:22:10)
> 
> + Joonas for maintainer class question.
> 
> On 15/10/2020 19:28, John.C.Harrison@Intel.com wrote:
> > From: John Harrison <John.C.Harrison@Intel.com>
> > 
> > Update to the latest GuC firmware
> > 
> > v2: Rebase to newer tree, updated a commit message (review feedback
> > from Daniele) and dropped the patch to enable GuC/HuC loading by
> > default as apparently this is not allowed.
> > 
> > Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> > 
> > 
> > John Harrison (4):
> >    drm/i915/guc: Update to use firmware v49.0.1
> >    drm/i915/guc: Improved reporting when GuC fails to load
> >    drm/i915/guc: Clear pointers on free
> >    CI: turn on GuC/HuC auto mode by default
> > 
> >   drivers/gpu/drm/i915/gt/intel_engine_cs.c    |   3 +-
> >   drivers/gpu/drm/i915/gt/uc/intel_guc.c       |  18 ---
> >   drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c   | 132 +++++++++++++++----
> >   drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c    |   1 +
> >   drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c    |  31 +++--
> >   drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h  |  80 +++++------
> >   drivers/gpu/drm/i915/gt/uc/intel_guc_reg.h   |   5 +
> >   drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c     |  29 ++--
> >   drivers/gpu/drm/i915/gt/uc/intel_uc_fw.h     |   2 +
> >   drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h |   6 +-
> >   drivers/gpu/drm/i915/i915_params.h           |   2 +-
> >   11 files changed, 202 insertions(+), 107 deletions(-)
> 
> I tried to merge this for John today but the series applies to 
> drm-intel-next-queued and not drm-intel-gt-next, which I thought is 
> where GT/GuC should go to. Long story short, where to merge GuC patches 
> and how is the question?

We should definitely merge to drm-intel-gt-next.

If we need to backmerge some changes to gt-next as dependencies, let me know.

Regards, Joonas