[Intel-gfx] [CI 0/5] Refactor to expand subslice mask

[Jani Nikula]

On Wed, 29 May 2019, "Saarinen, Jani" <jani.saarinen at intel.com> wrote:
> Hi, 
>
>> -----Original Message-----
>> From: Intel-gfx [mailto:intel-gfx-bounces at lists.freedesktop.org] On Behalf Of
>> Summers, Stuart
>> Sent: tiistai 28. toukokuuta 2019 21.33
>> To: Navare, Manasi D <manasi.d.navare at intel.com>
>> Cc: intel-gfx at lists.freedesktop.org
>> Subject: Re: [Intel-gfx] [CI 0/5] Refactor to expand subslice mask
>> 
>> On Tue, 2019-05-28 at 11:32 -0700, Manasi Navare wrote:
>> > Pushed to dinq, thanks for the patches and the reviews!
>> 
>> Thanks for the push Manasi and the reviews Daniele and others!
> This broke all the ICL systems because CI data was not looked that they did not actually even boot at all.
> All ICL's in BAT and whole ICL shards. 

Commit 1ac159e23c2c ("drm/i915: Expand subslice mask") has now been
reverted and we can carry on.

As discussed, I don't really expect people to dig into the logs after
seeing "Fi.CI.IGT: success". For one I didn't expect patches to be able
to take down machines with just a fairly silent "Participating hosts (10
-> 9)" message. But that discussion is for another forum, another time.

BR,
Jani.


>
>
>> 
>> -Stuart
>> 
>> >
>> > Regards
>> > Manasi
>> >
>> > On Fri, May 24, 2019 at 08:40:17AM -0700, Stuart Summers wrote:
>> > > This patch series contains a few code clean-up patches, followed by
>> > > a patch which changes the storage of the subslice mask to better
>> > > match the userspace access through the I915_QUERY_TOPOLOGY_INFO
>> > > ioctl. The index into the subslice_mask array is then calculated:
>> > >   slice * subslice stride + subslice index / 8
>> > >
>> > > v2: fix i915_pm_sseu test failure
>> > > v3: no changes to patches in the series, just resending to pick up
>> > >     in CI correctly
>> > > v4: rebase
>> > > v5: fix header test
>> > > v6: address review comments from Jari
>> > >     address minor checkpatch warning in existing code
>> > >     use eu_stride for EU div-by-8
>> > > v7: another rebase
>> > > v8: address review comments from Tvrtko and Daniele
>> > > v9: address review comments from Daniele
>> > > v10: add reviewed-by on last patch with minor suggested change,
>> > >      rebase, and repost for CI
>> > >
>> > > Stuart Summers (5):
>> > >   drm/i915: Use local variable for SSEU info in GETPARAM ioctl
>> > >   drm/i915: Add macro for SSEU stride calculation
>> > >   drm/i915: Move calculation of subslices per slice to new function
>> > >   drm/i915: Refactor sseu helper functions
>> > >   drm/i915: Expand subslice mask
>> > >
>> > >  drivers/gpu/drm/i915/gt/intel_engine_cs.c    |  24 ++-
>> > >  drivers/gpu/drm/i915/gt/intel_engine_types.h |  30 ++--
>> > >  drivers/gpu/drm/i915/gt/intel_hangcheck.c    |   3 +-
>> > >  drivers/gpu/drm/i915/gt/intel_sseu.c         |  62 +++++++
>> > >  drivers/gpu/drm/i915/gt/intel_sseu.h         |  35 +++-
>> > >  drivers/gpu/drm/i915/gt/intel_workarounds.c  |   2 +-
>> > >  drivers/gpu/drm/i915/i915_debugfs.c          |  46 ++---
>> > >  drivers/gpu/drm/i915/i915_drv.c              |  15 +-
>> > >  drivers/gpu/drm/i915/i915_gpu_error.c        |   5 +-
>> > >  drivers/gpu/drm/i915/i915_query.c            |  15 +-
>> > >  drivers/gpu/drm/i915/intel_device_info.c     | 176 +++++++++++--
>> > > ------
>> > >  drivers/gpu/drm/i915/intel_device_info.h     |  47 -----
>> > >  12 files changed, 280 insertions(+), 180 deletions(-)
>> > >
>> > > --
>> > > 2.21.0.5.gaeb582a983
>> > >
>> > > _______________________________________________
>> > > Intel-gfx mailing list
>> > > Intel-gfx at lists.freedesktop.org
>> > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Graphics Center