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>]

On Tue, 2023-03-28 at 09:14 +0800, Zhang Rui wrote:
> 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.

Is this topology fix patch the only patch applied?
or together with some other patches?

I can hardly imagine that the fix patch can trigger such issues, so I
suspect they are intermittent issues. say
is the regression 100% reproducible?
does the warning/failure ever show without the patch?

BTW, I just happened to see this thread
https://lore.kernel.org/all/DM8PR11MB565580BCF44661B6A392F0CEE08B9@DM8PR11MB5655.namprd11.prod.outlook.com/
If the problem on hand has been verified to be not related with the
topology fix, can we update in this thread as well?
https://lore.kernel.org/all/20230323015640.27906-1-rui.zhang@intel.com/
This is another issue that the patch fixes. And it's better to have a
Buglink/Tested-by tag in the commit.

thanks,
rui

> 
> 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