Message ID | 20210610043649.144416-1-matthew.brost@intel.com (mailing list archive) |
---|---|
Headers | show |
Series | Update firmware to v62.0.0 | expand |
On Wed, Jun 09, 2021 at 09:36:36PM -0700, Matthew Brost wrote: > As part of enabling GuC submission [1] we need to update to the latest > and greatest firmware. This series does that. This is a destructive > change. e.g. Without all the patches in this series it will break the > i915 driver. As such, after we review most of these patches they will > squashed into a single patch for merging. > > v2: Address comments, looking for remaning RBs so patches can be > squashed and sent for CI > Ugh, forgot to include some RBs in this rev. Just looking for RBs 1-2, and 6-8 in this rev. Matt > Signed-off-by: Matthew Brost <matthew.brost@intel.com> > > [1] https://patchwork.freedesktop.org/series/89844/i > > John Harrison (3): > drm/i915/guc: Support per context scheduling policies > drm/i915/guc: Unified GuC log > drm/i915/guc: Update firmware to v62.0.0 > > Michal Wajdeczko (10): > drm/i915/guc: Introduce unified HXG messages > drm/i915/guc: Update MMIO based communication > drm/i915/guc: Update CTB response status definition > drm/i915/guc: Add flag for mark broken CTB > drm/i915/guc: New definition of the CTB descriptor > drm/i915/guc: New definition of the CTB registration action > drm/i915/guc: New CTB based communication > drm/i915/doc: Include GuC ABI documentation > drm/i915/guc: Kill guc_clients.ct_pool > drm/i915/guc: Kill ads.client_info > > Documentation/gpu/i915.rst | 8 + > .../gpu/drm/i915/gt/uc/abi/guc_actions_abi.h | 107 ++++++ > .../gt/uc/abi/guc_communication_ctb_abi.h | 128 +++++-- > .../gt/uc/abi/guc_communication_mmio_abi.h | 65 ++-- > .../gpu/drm/i915/gt/uc/abi/guc_messages_abi.h | 213 +++++++++++ > drivers/gpu/drm/i915/gt/uc/intel_guc.c | 107 ++++-- > drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 45 +-- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 356 +++++++++--------- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.h | 6 +- > drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 75 +--- > drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 29 +- > drivers/gpu/drm/i915/gt/uc/intel_guc_log.h | 6 +- > drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 26 +- > 13 files changed, 750 insertions(+), 421 deletions(-) > > -- > 2.28.0 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On 10.06.2021 06:36, Matthew Brost wrote: > As part of enabling GuC submission [1] we need to update to the latest > and greatest firmware. This series does that. This is a destructive > change. e.g. Without all the patches in this series it will break the not really 'all' > i915 driver. As such, after we review most of these patches they will > squashed into a single patch for merging. I assume final series will looks as follows: 1 = hxg 2+3+5+6+7+8+4+10+11+13 = ctb/ads/fw 12 = log 9 = rst > > v2: Address comments, looking for remaning RBs so patches can be typo and likely series should be generated with -v 2 option too > squashed and sent for CI > > Signed-off-by: Matthew Brost <matthew.brost@intel.com> > > [1] https://patchwork.freedesktop.org/series/89844/i > > John Harrison (3): > drm/i915/guc: Support per context scheduling policies > drm/i915/guc: Unified GuC log > drm/i915/guc: Update firmware to v62.0.0 > > Michal Wajdeczko (10): > drm/i915/guc: Introduce unified HXG messages > drm/i915/guc: Update MMIO based communication > drm/i915/guc: Update CTB response status definition > drm/i915/guc: Add flag for mark broken CTB > drm/i915/guc: New definition of the CTB descriptor > drm/i915/guc: New definition of the CTB registration action > drm/i915/guc: New CTB based communication > drm/i915/doc: Include GuC ABI documentation > drm/i915/guc: Kill guc_clients.ct_pool > drm/i915/guc: Kill ads.client_info > > Documentation/gpu/i915.rst | 8 + > .../gpu/drm/i915/gt/uc/abi/guc_actions_abi.h | 107 ++++++ > .../gt/uc/abi/guc_communication_ctb_abi.h | 128 +++++-- > .../gt/uc/abi/guc_communication_mmio_abi.h | 65 ++-- > .../gpu/drm/i915/gt/uc/abi/guc_messages_abi.h | 213 +++++++++++ > drivers/gpu/drm/i915/gt/uc/intel_guc.c | 107 ++++-- > drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 45 +-- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 356 +++++++++--------- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.h | 6 +- > drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 75 +--- > drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 29 +- > drivers/gpu/drm/i915/gt/uc/intel_guc_log.h | 6 +- > drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 26 +- > 13 files changed, 750 insertions(+), 421 deletions(-) >
On Thu, Jun 10, 2021 at 03:35:57PM +0200, Michal Wajdeczko wrote: > > > On 10.06.2021 06:36, Matthew Brost wrote: > > As part of enabling GuC submission [1] we need to update to the latest > > and greatest firmware. This series does that. This is a destructive > > change. e.g. Without all the patches in this series it will break the > > not really 'all' > Yea, typo. I updated the comment below to say 'most' but missed this one. > > i915 driver. As such, after we review most of these patches they will > > squashed into a single patch for merging. > > I assume final series will looks as follows: > > 1 = hxg > 2+3+5+6+7+8+4+10+11+13 = ctb/ads/fw > 12 = log > 9 = rst > Seems right to me. > > > > v2: Address comments, looking for remaning RBs so patches can be > > typo > Not seeing the typo. > and likely series should be generated with -v 2 option too > Didn't know about -v2 option. Will use going forward. Matt > > > squashed and sent for CI > > > > Signed-off-by: Matthew Brost <matthew.brost@intel.com> > > > > [1] https://patchwork.freedesktop.org/series/89844/i > > > > John Harrison (3): > > drm/i915/guc: Support per context scheduling policies > > drm/i915/guc: Unified GuC log > > drm/i915/guc: Update firmware to v62.0.0 > > > > Michal Wajdeczko (10): > > drm/i915/guc: Introduce unified HXG messages > > drm/i915/guc: Update MMIO based communication > > drm/i915/guc: Update CTB response status definition > > drm/i915/guc: Add flag for mark broken CTB > > drm/i915/guc: New definition of the CTB descriptor > > drm/i915/guc: New definition of the CTB registration action > > drm/i915/guc: New CTB based communication > > drm/i915/doc: Include GuC ABI documentation > > drm/i915/guc: Kill guc_clients.ct_pool > > drm/i915/guc: Kill ads.client_info > > > > Documentation/gpu/i915.rst | 8 + > > .../gpu/drm/i915/gt/uc/abi/guc_actions_abi.h | 107 ++++++ > > .../gt/uc/abi/guc_communication_ctb_abi.h | 128 +++++-- > > .../gt/uc/abi/guc_communication_mmio_abi.h | 65 ++-- > > .../gpu/drm/i915/gt/uc/abi/guc_messages_abi.h | 213 +++++++++++ > > drivers/gpu/drm/i915/gt/uc/intel_guc.c | 107 ++++-- > > drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 45 +-- > > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 356 +++++++++--------- > > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.h | 6 +- > > drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 75 +--- > > drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 29 +- > > drivers/gpu/drm/i915/gt/uc/intel_guc_log.h | 6 +- > > drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 26 +- > > 13 files changed, 750 insertions(+), 421 deletions(-) > >
As part of enabling GuC submission [1] we need to update to the latest and greatest firmware. This series does that. This is a destructive change. e.g. Without all the patches in this series it will break the i915 driver. As such, after we review most of these patches they will squashed into a single patch for merging. v2: Address comments, looking for remaning RBs so patches can be squashed and sent for CI Signed-off-by: Matthew Brost <matthew.brost@intel.com> [1] https://patchwork.freedesktop.org/series/89844/i John Harrison (3): drm/i915/guc: Support per context scheduling policies drm/i915/guc: Unified GuC log drm/i915/guc: Update firmware to v62.0.0 Michal Wajdeczko (10): drm/i915/guc: Introduce unified HXG messages drm/i915/guc: Update MMIO based communication drm/i915/guc: Update CTB response status definition drm/i915/guc: Add flag for mark broken CTB drm/i915/guc: New definition of the CTB descriptor drm/i915/guc: New definition of the CTB registration action drm/i915/guc: New CTB based communication drm/i915/doc: Include GuC ABI documentation drm/i915/guc: Kill guc_clients.ct_pool drm/i915/guc: Kill ads.client_info Documentation/gpu/i915.rst | 8 + .../gpu/drm/i915/gt/uc/abi/guc_actions_abi.h | 107 ++++++ .../gt/uc/abi/guc_communication_ctb_abi.h | 128 +++++-- .../gt/uc/abi/guc_communication_mmio_abi.h | 65 ++-- .../gpu/drm/i915/gt/uc/abi/guc_messages_abi.h | 213 +++++++++++ drivers/gpu/drm/i915/gt/uc/intel_guc.c | 107 ++++-- drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 45 +-- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 356 +++++++++--------- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.h | 6 +- drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 75 +--- drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 29 +- drivers/gpu/drm/i915/gt/uc/intel_guc_log.h | 6 +- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 26 +- 13 files changed, 750 insertions(+), 421 deletions(-)