From: Marc Zyngier <maz@kernel.org>
To: Luiz Capitulino <luizcap@amazon.com>
Cc: <stable@vger.kernel.org>, <tglx@linutronix.de>, <lcapitulino@gmail.com>
Subject: Re: [PATH stable 5.15,5.10 0/4] Fix EBS volume attach on AWS ARM instances
Date: Mon, 28 Nov 2022 17:53:55 +0000 [thread overview]
Message-ID: <86h6yjm0cs.wl-maz@kernel.org> (raw)
In-Reply-To: <cover.1669655291.git.luizcap@amazon.com>
On Mon, 28 Nov 2022 17:08:31 +0000,
Luiz Capitulino <luizcap@amazon.com> wrote:
>
> Hi,
>
> [ Marc, can you help reviewing? Esp. the first patch? ]
>
> This series of backports from upstream to stable 5.15 and 5.10 fixes an issue
> we're seeing on AWS ARM instances where attaching an EBS volume (which is a
> nvme device) to the instance after offlining CPUs causes the device to take
> several minutes to show up and eventually nvme kworkers and other threads start
> getting stuck.
>
> This series fixes the issue for 5.15.79 and 5.10.155. I can't reproduce it
> on 5.4. Also, I couldn't reproduce this on x86 even w/ affected kernels.
That's because x86 has a very different allocation policy compared to
what the ITS does. The x86 vector space is tiny, so vectors are only
allocated when required. In your case, that's when the CPUs are
onlined.
With the ITS, all the vectors are allocated upfront, as this is
essentially free. But in the case of managed interrupts, these vectors
are now pointing to offline CPUs. The ITS tries to fix that, but
doesn't nearly have enough information. And the correct course of
action is to keep these interrupts in the shutdown state, which is
what the series is doing.
>
> An easy reproducer is:
>
> 1. Start an ARM instance with 32 CPUs
To satisfy my own curiosity, is that in a guest or bare metal? It
shouldn't make any difference, but hey...
Anyway, patch #1 looks OK to me, but I haven't tried to dig further
into something that is "oh so last year" ;-). Specially as we're
rewriting the whole of the MSI stack! FWIW:
Acked-by: Marc Zyngier <maz@kernel.org>
M.
--
Without deviation from the norm, progress is not possible.
next prev parent reply other threads:[~2022-11-28 18:11 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-28 17:08 [PATH stable 5.15,5.10 0/4] Fix EBS volume attach on AWS ARM instances Luiz Capitulino
2022-11-28 17:08 ` [PATH stable 5.15,5.10 1/4] genirq/msi: Shutdown managed interrupts with unsatifiable affinities Luiz Capitulino
2022-11-28 17:08 ` [PATH stable 5.15,5.10 2/4] genirq: Always limit the affinity to online CPUs Luiz Capitulino
2022-11-28 17:08 ` [PATH stable 5.15,5.10 3/4] irqchip/gic-v3: Always trust the managed affinity provided by the core code Luiz Capitulino
2022-11-28 17:08 ` [PATH stable 5.15,5.10 4/4] genirq: Take the proposed affinity at face value if force==true Luiz Capitulino
2022-11-28 17:53 ` Marc Zyngier [this message]
2022-11-28 18:27 ` [PATH stable 5.15, 5.10 0/4] Fix EBS volume attach on AWS ARM instances Luiz Capitulino
2022-11-30 3:12 ` Luiz Capitulino
2022-11-30 17:11 ` [PATH stable 5.15,5.10 " Greg KH
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=86h6yjm0cs.wl-maz@kernel.org \
--to=maz@kernel.org \
--cc=lcapitulino@gmail.com \
--cc=luizcap@amazon.com \
--cc=stable@vger.kernel.org \
--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).