linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Zhang, Rui" <rui.zhang@intel.com>
To: "tglx@linutronix.de" <tglx@linutronix.de>,
	"linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
	"ldufour@linux.ibm.com" <ldufour@linux.ibm.com>
Cc: "npiggin@gmail.com" <npiggin@gmail.com>,
	"bp@alien8.de" <bp@alien8.de>,
	"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
	"linux-arch@vger.kernel.org" <linux-arch@vger.kernel.org>,
	"christophe.leroy@csgroup.eu" <christophe.leroy@csgroup.eu>,
	"mingo@redhat.com" <mingo@redhat.com>,
	"mpe@ellerman.id.au" <mpe@ellerman.id.au>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 00/10] Introduce SMT level and add PowerPC support
Date: Fri, 28 Jul 2023 14:23:55 +0000	[thread overview]
Message-ID: <beaab9ae25de92bace2f2e30dff5e0d2e7774e56.camel@intel.com> (raw)
In-Reply-To: <87wmykqyam.ffs@tglx>

Hi, Thomas,

On Fri, 2023-07-28 at 09:40 +0200, Thomas Gleixner wrote:
> Rui!
> 
> On Sun, Jul 09 2023 at 15:25, Rui Zhang wrote:
> > I ran into a boot hang regression with latest upstream code, and it
> > took me a while to bisect the offending commit and workaround it.
> 
> Where is the bug report and the analysis? And what's the workaround?

As it is an iwlwifi regression, I didn't paste the link here.

The regression was reported at
https://lore.kernel.org/all/b533071f38804247f06da9e52a04f15cce7a3836.camel@intel.com/

And it was fixed later by below commit in 6.5-rc2.

thanks,
rui

commit 12a89f0177092dbc2a1cb1d05a9790adbcea2309
Author:     Johannes Berg <johannes.berg@intel.com>
AuthorDate: Mon Jul 10 16:50:39 2023 +0200
Commit:     Jakub Kicinski <kuba@kernel.org>
CommitDate: Tue Jul 11 20:26:06 2023 -0700

    wifi: iwlwifi: remove 'use_tfh' config to fix crash
    
    This is equivalent to 'gen2', and it was always confusing to have
    two identical config entries. The split config patch actually had
    been originally developed after removing 'use_tfh" and didn't add
    the use_tfh in the new configs as they'd later been copied to the
    new files. Thus the easiest way to fix the init crash here now is
    to just remove use_tfh (which is erroneously unset in most of the
    configs now) and use 'gen2' in the code instead.
    
    There's possibly still an unwind error in iwl_txq_gen2_init() as
    it crashes if TXQ 0 fails to initialize, but we can deal with it
    later since the original failure is due to the use_tfh confusion.
    
    Tested-by: Xi Ruoyao <xry111@xry111.site>
    Reported-and-tested-by: Niklāvs Koļesņikovs
<pinkflames.linux@gmail.com>
    Reported-and-tested-by: Jeff Chua <jeff.chua.linux@gmail.com>
    Reported-and-tested-by: Zhang Rui <rui.zhang@intel.com>
    Link: https://bugzilla.kernel.org/show_bug.cgi?id=217622
    Link:
https://lore.kernel.org/all/9274d9bd3d080a457649ff5addcc1726f08ef5b2.camel@xry111.site/
    Link:
https://lore.kernel.org/all/CAAJw_Zug6VCS5ZqTWaFSr9sd85k%3DtyPm9DEE%2BmV%3DAKoECZM%2BsQ@mail.gmail.com/
    Fixes: 19898ce9cf8a ("wifi: iwlwifi: split 22000.c into multiple
files")
    Signed-off-by: Johannes Berg <johannes.berg@intel.com>
    Link:
https://lore.kernel.org/r/20230710145038.84186-2-johannes@sipsolutions.net
    Signed-off-by: Jakub Kicinski <kuba@kernel.org>

> 
> Thanks,
> 
>         tglx


  reply	other threads:[~2023-07-28 14:24 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-05 14:51 [PATCH v4 00/10] Introduce SMT level and add PowerPC support Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 01/10] cpu/hotplug: remove dependancy against cpu_primary_thread_mask Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 02/10] cpu/SMT: Move SMT prototypes into cpu_smt.h Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 03/10] cpu/SMT: Move smt/control simple exit cases earlier Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 04/10] cpu/SMT: Store the current/max number of threads Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 05/10] cpu/SMT: Remove topology_smt_supported() Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 06/10] cpu/SMT: Create topology_smt_thread_allowed() Laurent Dufour
2023-07-05 14:51 ` [PATCH v4 07/10] cpu/SMT: Allow enabling partial SMT states via sysfs Laurent Dufour
2024-04-08 14:13   ` Michal Suchánek
2023-07-05 14:51 ` [PATCH v4 08/10] powerpc/pseries: Initialise CPU hotplug callbacks earlier Laurent Dufour
2023-08-14 10:51   ` Srikar Dronamraju
2023-07-05 14:51 ` [PATCH v4 09/10] powerpc: Add HOTPLUG_SMT support Laurent Dufour
2023-08-14 10:52   ` Srikar Dronamraju
2023-07-05 14:51 ` [PATCH v4 10/10] powerpc/pseries: Honour current SMT state when DLPAR onlining CPUs Laurent Dufour
2023-08-14 10:53   ` Srikar Dronamraju
2023-07-09 15:25 ` [PATCH v4 00/10] Introduce SMT level and add PowerPC support Zhang, Rui
2023-07-10  9:08   ` Laurent Dufour
2023-07-28  7:40   ` Thomas Gleixner
2023-07-28 14:23     ` Zhang, Rui [this message]
2023-07-28 14:51       ` Thomas Gleixner
2023-07-28  7:58 ` Thomas Gleixner
2023-07-31 11:55   ` Laurent Dufour
2023-08-10  6:23   ` Michael Ellerman
2023-08-10  8:51     ` Laurent Dufour

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=beaab9ae25de92bace2f2e30dff5e0d2e7774e56.camel@intel.com \
    --to=rui.zhang@intel.com \
    --cc=bp@alien8.de \
    --cc=christophe.leroy@csgroup.eu \
    --cc=dave.hansen@linux.intel.com \
    --cc=ldufour@linux.ibm.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=mingo@redhat.com \
    --cc=mpe@ellerman.id.au \
    --cc=npiggin@gmail.com \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).