Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for x86/topology: fix erroneous smp_num_siblings on Intel Hybrid platform

["Zhang, Rui" <rui.zhang@intel.com>]

Hi, Imre,

Thanks for raising this.

On Tue, 2023-03-28 at 00:26 +0300, Imre Deak wrote:
> Hi Rui,
> 
> after applying your
> "x86/topology: fix erroneous smp_num_siblings on Intel Hybrid
> platform"
> 
> fix on the drm-tip tree (see the patchwork URL below) the CI tests
> show
> some regression on a HSW and a KBL machine (see [2] and [4] below) in
> the i915 driver. However I think they can't be related to your
> changes,
> since on these machines all cores will report the same number of CPU
> siblings.

Right.

>  Could you confirm this and that in general the reported
> siblings can only vary on platforms with both E and P cores (ADL-P
> being
> the first such platform)?

Right.

I don't think the patch could bring any change related.
It only affects hybrid platforms.

Thanks,
rui
> 
> Thanks,
> Imre
> 
> On Mon, Mar 27, 2023 at 07:02:25PM +0000, Patchwork wrote:
> > == Series Details ==
> > 
> > Series: x86/topology: fix erroneous smp_num_siblings on Intel
> > Hybrid platform
> > URL   : https://patchwork.freedesktop.org/series/115661/
> > State : failure
> > 
> > == Summary ==
> > 
> > CI Bug Log - changes from CI_DRM_12921 -> Patchwork_115661v1
> > ====================================================
> > 
> > Summary
> > -------
> > 
> >   **FAILURE**
> > 
> >   Serious unknown changes coming with Patchwork_115661v1 absolutely
> > need to be
> >   verified manually.
> >   
> >   If you think the reported changes have nothing to do with the
> > changes
> >   introduced in Patchwork_115661v1, please notify your bug team to
> > allow them
> >   to document this new failure mode, which will reduce false
> > positives in CI.
> > 
> >   External URL: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/index.html
> > 
> > Participating hosts (37 -> 37)
> > ------------------------------
> > 
> >   No changes in participating hosts
> > 
> > Possible new issues
> > -------------------
> > 
> >   Here are the unknown changes that may have been introduced in
> > Patchwork_115661v1:
> > 
> > ### IGT changes ###
> > 
> > #### Possible regressions ####
> > 
> >   * igt@i915_selftest@live@hangcheck:
> >     - fi-hsw-4770:        [PASS][1] -> [DMESG-WARN][2]
> >    [1]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
> >    [2]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
> > 
> >   
> > #### Suppressed ####
> > 
> >   The following results come from untrusted machines, tests, or
> > statuses.
> >   They do not affect the overall result.
> > 
> >   * igt@fbdev@info:
> >     - {bat-kbl-2}:        [SKIP][3] ([fdo#109271]) -> [ABORT][4]
> >    [3]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/bat-kbl-2/igt@fbdev@info.html
> >    [4]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-kbl-2/igt@fbdev@info.html
> > 
> >   
> > Known issues
> > ------------
> > 
> >   Here are the changes found in Patchwork_115661v1 that come from
> > known issues:
> > 
> > ### IGT changes ###
> > 
> > #### Issues hit ####
> > 
> >   * igt@gem_exec_suspend@basic-s3@lmem0:
> >     - bat-dg2-11:         [PASS][5] -> [INCOMPLETE][6]
> > ([i915#6311])
> >    [5]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/bat-dg2-11/igt@gem_exec_suspend@basic-s3@lmem0.html
> >    [6]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-dg2-11/igt@gem_exec_suspend@basic-s3@lmem0.html
> > 
> >   * igt@i915_selftest@live@reset:
> >     - bat-rpls-1:         [PASS][7] -> [ABORT][8] ([i915#4983])
> >    [7]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/bat-rpls-1/igt@i915_selftest@live@reset.html
> >    [8]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-rpls-1/igt@i915_selftest@live@reset.html
> > 
> >   * igt@i915_selftest@live@slpc:
> >     - bat-rpls-2:         NOTRUN -> [DMESG-FAIL][9] ([i915#6367] /
> > [i915#7913] / [i915#7996])
> >    [9]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-rpls-2/igt@i915_selftest@live@slpc.html
> > 
> >   * igt@i915_suspend@basic-s3-without-i915:
> >     - bat-rpls-2:         NOTRUN -> [ABORT][10] ([i915#6687] /
> > [i915#7978])
> >    [10]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-rpls-2/igt@i915_suspend@basic-s3-without-i915.html
> > 
> >   * igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-d-
> > dp-1:
> >     - bat-dg2-8:          [PASS][11] -> [FAIL][12] ([i915#7932])
> >    [11]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-d-dp-1.html
> >    [12]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-d-dp-1.html
> > 
> >   
> > #### Possible fixes ####
> > 
> >   * igt@i915_selftest@live@gt_heartbeat:
> >     - fi-kbl-soraka:      [DMESG-FAIL][13] ([i915#5334] /
> > [i915#7872]) -> [PASS][14]
> >    [13]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/fi-kbl-soraka/igt@i915_selftest@live@gt_heartbeat.html
> >    [14]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/fi-kbl-soraka/igt@i915_selftest@live@gt_heartbeat.html
> > 
> >   * igt@i915_selftest@live@reset:
> >     - bat-rpls-2:         [ABORT][15] ([i915#4983] / [i915#7913])
> > -> [PASS][16]
> >    [15]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12921/bat-rpls-2/igt@i915_selftest@live@reset.html
> >    [16]: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/bat-rpls-2/igt@i915_selftest@live@reset.html
> > 
> >   
> >   {name}: This element is suppressed. This means it is ignored when
> > computing
> >           the status of the difference (SUCCESS, WARNING, or
> > FAILURE).
> > 
> >   [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
> >   [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
> >   [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
> >   [i915#6311]: https://gitlab.freedesktop.org/drm/intel/issues/6311
> >   [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
> >   [i915#6687]: https://gitlab.freedesktop.org/drm/intel/issues/6687
> >   [i915#7872]: https://gitlab.freedesktop.org/drm/intel/issues/7872
> >   [i915#7913]: https://gitlab.freedesktop.org/drm/intel/issues/7913
> >   [i915#7932]: https://gitlab.freedesktop.org/drm/intel/issues/7932
> >   [i915#7978]: https://gitlab.freedesktop.org/drm/intel/issues/7978
> >   [i915#7996]: https://gitlab.freedesktop.org/drm/intel/issues/7996
> > 
> > 
> > Build changes
> > -------------
> > 
> >   * Linux: CI_DRM_12921 -> Patchwork_115661v1
> > 
> >   CI-20190529: 20190529
> >   CI_DRM_12921: 3de6040ce9900a94ec626662d5c6a227b37eeb1c @
> > git://anongit.freedesktop.org/gfx-ci/linux
> >   IGT_7221: 4b77c6d85024d22ca521d510f8eee574128fe04f @ 
> > https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> >   Patchwork_115661v1: 3de6040ce9900a94ec626662d5c6a227b37eeb1c @
> > git://anongit.freedesktop.org/gfx-ci/linux
> > 
> > 
> > ### Linux commits
> > 
> > 83d9e76610d5 x86/topology: fix erroneous smp_num_siblings on Intel
> > Hybrid platform
> > 
> > == Logs ==
> > 
> > For more details see: 
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_115661v1/index.html