From: Marc Zyngier <maz@kernel.org>
To: Thomas Gleixner <tglx@kernel.org>
Cc: Fuad Tabba <fuad.tabba@linux.dev>,
Oliver Upton <oupton@kernel.org>, Eric Auger <eauger@redhat.com>,
Joey Gouly <joey.gouly@arm.com>,
Steffen Eiden <seiden@linux.ibm.com>,
Suzuki K Poulose <suzuki.poulose@arm.com>,
Zenghui Yu <yuzenghui@huawei.com>, Will Deacon <will@kernel.org>,
Sascha Bischoff <Sascha.Bischoff@arm.com>,
Sebastian Ene <sebastianene@google.com>,
Yuan Yao <yaoyuan@linux.alibaba.com>,
Fuad Tabba <tabba@google.com>,
kvmarm@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/4] irqchip/gic-v4, KVM: arm64: Fix the vgic init error paths
Date: Thu, 15 Oct 2026 11:18:56 +0100 [thread overview]
Message-ID: <86o6cv1d2n.wl-maz@kernel.org> (raw)
In-Reply-To: <87wlrje2cx.ffs@fw13>
On Thu, 15 Oct 2026 10:31:58 +0100,
Thomas Gleixner <tglx@kernel.org> wrote:
>
> On Mon, Sep 21 2026 at 08:29, Fuad Tabba wrote:
> > Hi folks,
> >
> > Changes since v2 [1]:
> > - Patch 3: a vgic_v4_init() failure now frees only the SPI array,
> > since that function unwinds its own state; the teardown label is
> > for the routing failure. (Yuan Yao)
> > - Rebased on kvmarm/next, where the routing failure already tears
> > down the GICv5 state. Patch 3 moves that into the shared label, so
> > the SPI array is freed on that path too.
> > - Collected Reviewed-by from Yuan Yao on patches 1 and 2. (thanks!)
> >
> > Four fixes on the paths that run when vgic init fails partway. Each one
> > needs an allocation failure to reach.
>
> Marc?
I was about to say that I had already queued that, except that it
appears it only happened in my head, and nowhere else... :-(
If that's OK with you, I'll queue (for real!) the whole thing via the
kvmarm tree, targeting 7.4-rc1.
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
next prev parent reply other threads:[~2026-10-15 10:18 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-21 7:29 [PATCH v3 0/4] irqchip/gic-v4, KVM: arm64: Fix the vgic init error paths Fuad Tabba
2026-09-21 7:29 ` [PATCH v3 1/4] irqchip/gic-v4: Clear the domain and fwnode pointers after freeing them Fuad Tabba
2026-09-21 7:29 ` [PATCH v3 2/4] irqchip/gic-v4: Unwind what its_alloc_vcpu_irqs() allocated on failure Fuad Tabba
2026-09-21 7:29 ` [PATCH v3 3/4] KVM: arm64: vgic: Tear down what vgic_init() created when it fails Fuad Tabba
2026-09-21 7:29 ` [PATCH v3 4/4] KVM: arm64: vgic-v4: Restore nr_vpes before freeing the vPE resources Fuad Tabba
2026-10-15 9:31 ` [PATCH v3 0/4] irqchip/gic-v4, KVM: arm64: Fix the vgic init error paths Thomas Gleixner
2026-10-15 10:18 ` Marc Zyngier [this message]
2026-10-15 12:32 ` Thomas Gleixner
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=86o6cv1d2n.wl-maz@kernel.org \
--to=maz@kernel.org \
--cc=Sascha.Bischoff@arm.com \
--cc=eauger@redhat.com \
--cc=fuad.tabba@linux.dev \
--cc=joey.gouly@arm.com \
--cc=kvmarm@lists.linux.dev \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oupton@kernel.org \
--cc=sebastianene@google.com \
--cc=seiden@linux.ibm.com \
--cc=suzuki.poulose@arm.com \
--cc=tabba@google.com \
--cc=tglx@kernel.org \
--cc=will@kernel.org \
--cc=yaoyuan@linux.alibaba.com \
--cc=yuzenghui@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.