From: Marc Zyngier <maz@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Marek Szyprowski <m.szyprowski@samsung.com>,
linux-kernel <linux-kernel@vger.kernel.org>,
'Linux Samsung SOC' <linux-samsung-soc@vger.kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
John Garry <john.garry@huawei.com>,
Xiongfeng Wang <wangxiongfeng2@huawei.com>,
David Decotigny <ddecotig@google.com>
Subject: Re: [PATCH v3 2/3] genirq: Always limit the affinity to online CPUs
Date: Wed, 20 Apr 2022 10:40:43 +0100 [thread overview]
Message-ID: <87mtgg9jok.wl-maz@kernel.org> (raw)
In-Reply-To: <70123e98-046f-f10e-1032-8d112edd3ecf@linaro.org>
Hi Krzysztof,
On Wed, 20 Apr 2022 10:13:52 +0100,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
>
> On 14/04/2022 13:08, Marek Szyprowski wrote:
> >> Thanks for all of the debug, super helpful. The issue is that we don't
> >> handle the 'force' case, which a handful of drivers are using when
> >> bringing up CPUs (and doing so before the CPUs are marked online).
> >>
> >> Can you please give the below hack a go?
> >
> > This patch fixed the issue. Thanks! Feel free to add my:
> >
> > Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
> >
> > Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
>
> Hi Marc,
>
> Linux-next still fails to boot on Exynos5422 boards, so I wonder if you
> applied the fix?
It was picked up by Thomas and pushed out into tip, which is pulled by
-next:
maz@hot-poop:~/arm-platforms$ git describe --contains c48c8b829d2b966a6649827426bcdba082ccf922
next-20220420~51^2~3^2
So it definitely is in today's -next.
> Instead of silent fail there is now "Unable to handle kernel paging
> request at virtual address f0836644", so it is slightly different.
>
> See the dmesg:
> https://krzk.eu/#/builders/21/builds/3542/steps/15/logs/serial0
This looks completely unrelated:
[ 10.382010] Unable to handle kernel paging request at virtual address f0836644
[ 10.388597] [f0836644] *pgd=41c83811, *pte=00000000, *ppte=00000000
[ 10.394482] Internal error: Oops: 807 [#1] PREEMPT SMP ARM
[ 10.399567] Modules linked in:
[ 10.402583] CPU: 2 PID: 1 Comm: swapper/0 Not tainted 5.18.0-rc3-next-20220420 #2
[ 10.410060] Hardware name: Samsung Exynos (Flattened Device Tree)
[ 10.416106] PC is at cpu_ca15_set_pte_ext+0x4c/0x58
[ 10.420952] LR is at handle_pte_fault+0x218/0x260
[ 10.425631] pc : [<c011d588>] lr : [<c02ab188>] psr: 40000113
[ 10.431874] sp : f0835df0 ip : f0835e5c fp : 00000081
[ 10.437069] r10: c0f2eafc r9 : c1d31000 r8 : 00000000
[ 10.442268] r7 : c1d58000 r6 : 00000081 r5 : befffff6 r4 : f0835e24
[ 10.448773] r3 : 00000000 r2 : 00000000 r1 : 00000040 r0 : f0835e44
[ 10.455273] Flags: nZcv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none
[ 10.462381] Control: 10c5387d Table: 4000406a DAC: 00000051
This is a crash in cpu_ca15_set_pte_ext() when populating the
userspace page tables, which seems unrelated to interrupt affinity.
I suggest you bisect this to find the actual problem.
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
next prev parent reply other threads:[~2022-04-20 9:40 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-05 18:50 [PATCH v3 0/3] genirq: Managed affinity fixes Marc Zyngier
2022-04-05 18:50 ` [PATCH v3 1/3] genirq/msi: Shutdown managed interrupts with unsatifiable affinities Marc Zyngier
2022-04-10 19:13 ` [tip: irq/core] " tip-bot2 for Marc Zyngier
2022-04-05 18:50 ` [PATCH v3 2/3] genirq: Always limit the affinity to online CPUs Marc Zyngier
2022-04-10 19:12 ` [tip: irq/core] " tip-bot2 for Marc Zyngier
2022-04-13 14:59 ` [PATCH v3 2/3] " Marek Szyprowski
2022-04-13 17:26 ` Marc Zyngier
2022-04-14 9:09 ` Marek Szyprowski
2022-04-14 10:35 ` Marc Zyngier
2022-04-14 11:08 ` Marek Szyprowski
2022-04-20 9:13 ` Krzysztof Kozlowski
2022-04-20 9:40 ` Marc Zyngier [this message]
2022-04-20 9:42 ` Krzysztof Kozlowski
2022-04-20 9:47 ` Marek Szyprowski
2022-04-20 9:50 ` Krzysztof Kozlowski
2022-04-14 10:49 ` Thomas Gleixner
2022-04-14 14:14 ` [tip: irq/core] genirq: Take the proposed affinity at face value if force==true tip-bot2 for Marc Zyngier
2022-04-05 18:50 ` [PATCH v3 3/3] irqchip/gic-v3: Always trust the managed affinity provided by the core code Marc Zyngier
2022-04-10 19:12 ` [tip: irq/core] " tip-bot2 for Marc Zyngier
2022-04-07 17:29 ` [PATCH v3 0/3] genirq: Managed affinity fixes John Garry
2022-04-08 1:02 ` Xiongfeng Wang
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=87mtgg9jok.wl-maz@kernel.org \
--to=maz@kernel.org \
--cc=ddecotig@google.com \
--cc=john.garry@huawei.com \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=tglx@linutronix.de \
--cc=wangxiongfeng2@huawei.com \
/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