From: Oliver Upton <oupton@kernel.org>
To: Marc Zyngier <maz@kernel.org>
Cc: Leonardo Bras <leo.bras@arm.com>,
kvmarm@lists.linux.dev, Joey Gouly <joey.gouly@arm.com>,
Suzuki K Poulose <suzuki.poulose@arm.com>,
Zenghui Yu <yuzenghui@huawei.com>,
Wei-Lin Chang <weilin.chang@arm.com>,
Steffen Eiden <seiden@linux.ibm.com>
Subject: Re: [PATCH 00/22] KVM: arm64: nv: Implement FEAT_HAFDBS, FEAT_HAFT
Date: Fri, 26 Jun 2026 10:45:03 -0700 [thread overview]
Message-ID: <aj66nzo3gTHYInp8@kernel.org> (raw)
In-Reply-To: <8633y9ql8q.wl-maz@kernel.org>
On Fri, Jun 26, 2026 at 06:12:21PM +0100, Marc Zyngier wrote:
> On Fri, 26 Jun 2026 16:31:50 +0100,
> Leonardo Bras <leo.bras@arm.com> wrote:
> >
> > On Tue, Jun 23, 2026 at 11:41:39AM -0700, Oliver Upton wrote:
> > > KVM's support for hardware descriptor updates has been lacking, owing in
> > > large part to the fact that the most widely available NV2 hardware
> > > actually lacks HAFDBS :)
> > >
> > > This series brings support for hardware dirty state and table Access
> > > flags to the nested MMU. While KVM uses neither of these features, the
> > > kernel definitely does in the stage-1 page tables. Since there is only
> > > one hypervisor to rule them all, implementing this stuff at stage-2 is
> > > mostly motivated by the desire to enable guest stage-1 :)
> > >
> > > Implementing dirty state at stage-2 is a bit more challenging than
> > > anticipated, I'm in the middle of seeking a relaxation from Arm that
> > > would make this all architectural. See the last patch for the details.
> >
> > Hi Oliver,
> >
> > Haven't read your patchset yet, but in regards to stage-2 dirty-state
> > tracking, there have already been some efforts in the area, also
> > implementing HDBSS and HACDBS in a way of improving performance / reducing
> > impact of live migration on a system.
>
> I'm worried you are reading this series the wrong way. This is mostly
> orthogonal to the whole HDBSS/HACDBS/SLAUA (Super Long And
> Unpronounceable Acronym). Oh wait, you haven't read it yet. Great :-(.
>
> This series is about *implementing* HAFDBS and HAFT in NV. We can't
> rely on the HW setting the AF and DB in the guest's own S2 PTs,
> because these page tables are never live. So we have to do it
> ourselves.
>
> So this has nothing to do with S2 dirty tracking for the purpose of
> migration. The guest could implement its own migration based on that,
> but that has nothing to do with what this series is doing.
Sorry, I haven't looked at the HDBSS/HACDBS stuff yet. Just want to be
clear, neither of these features make sense for the host to use in the
context of NV.
KVM's dirty tracking is done on the L1 IPA space, however nested MMUs
translate the L2 IPA space. The descriptors emitted by HDBSS would be of
the L2 IPA space and would require reverse-translation for each entry...
Software dirty tracking is the way to go for nested MMUs. Maybe in the
distant future we can emulate HDBSS/HACDBS for the L1 hypervisor. Maybe.
Thanks,
Oliver
prev parent reply other threads:[~2026-06-26 17:45 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-23 18:41 [PATCH 00/22] KVM: arm64: nv: Implement FEAT_HAFDBS, FEAT_HAFT Oliver Upton
2026-06-23 18:41 ` [PATCH 01/22] KVM: arm64: nv: Introduce struct for stage-2 walk step Oliver Upton
2026-06-23 18:41 ` [PATCH 02/22] KVM: arm64: nv: Consolidate computation of stage-2 permissions Oliver Upton
2026-06-23 18:57 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 03/22] KVM: arm64: nv: Get rid of kvm_s2_trans*() accessors Oliver Upton
2026-06-23 18:41 ` [PATCH 04/22] KVM: arm64: nv: Only shadow writable-dirty guest descs as writable Oliver Upton
2026-06-23 18:58 ` sashiko-bot
2026-06-23 20:05 ` Oliver Upton
2026-06-23 18:41 ` [PATCH 05/22] KVM: arm64: nv: Pass an access descriptor for stage-2 walks Oliver Upton
2026-06-23 19:06 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 06/22] KVM: arm64: nv: Use a helper for stage-2 descriptor updates Oliver Upton
2026-06-23 18:41 ` [PATCH 07/22] KVM: arm64: nv: Set dirty state at stage-2 Oliver Upton
2026-06-23 19:03 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 08/22] KVM: arm64: nv: Treat DBM as writable " Oliver Upton
2026-06-23 18:55 ` sashiko-bot
2026-06-23 20:08 ` Oliver Upton
2026-06-23 18:41 ` [PATCH 09/22] KVM: arm64: Compute S1 permissions as part of s1_walk() Oliver Upton
2026-06-23 18:41 ` [PATCH 10/22] KVM: arm64: Plumb through access descriptor for stage-1 Oliver Upton
2026-06-23 18:41 ` [PATCH 11/22] KVM: arm64: Use a struct for stage-1 walk context Oliver Upton
2026-06-23 18:41 ` [PATCH 12/22] KVM: arm64: Create helper for stage-1 descriptor updates Oliver Upton
2026-06-23 18:55 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 13/22] KVM: arm64: Set dirty state at stage-1 Oliver Upton
2026-06-23 18:54 ` sashiko-bot
2026-06-26 15:49 ` Leonardo Bras
2026-06-26 16:03 ` Marc Zyngier
2026-06-26 17:35 ` Oliver Upton
2026-06-23 18:41 ` [PATCH 14/22] KVM: arm64: Grant write permission when DBM is set at S1 Oliver Upton
2026-06-23 18:57 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 15/22] KVM: arm64: Don't update descriptors for "non-arch" access Oliver Upton
2026-06-23 18:41 ` [PATCH 16/22] KVM: arm64: nv: Expose FEAT_HAFDBS Oliver Upton
2026-06-23 19:01 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 17/22] KVM: arm64: Set Access flag on table descriptors at stage-1 Oliver Upton
2026-06-23 20:56 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 18/22] KVM: arm64: nv: Set access flag on table descriptors at stage-2 Oliver Upton
2026-06-23 19:05 ` sashiko-bot
2026-06-23 20:14 ` Oliver Upton
2026-06-23 18:41 ` [PATCH 19/22] KVM: arm64: nv: Expose FEAT_HAFT Oliver Upton
2026-06-23 19:05 ` sashiko-bot
2026-06-23 18:41 ` [PATCH 20/22] KVM: arm64: selftests: Only test AF behavior for emulated AT insns Oliver Upton
2026-06-23 18:42 ` [PATCH 21/22] KVM: arm64: selftests: Test AT emulation for FEAT_HAFT Oliver Upton
2026-06-23 19:05 ` sashiko-bot
2026-06-23 20:17 ` Oliver Upton
2026-06-23 18:42 ` [PATCH 22/22] HACK: KVM: arm64: nv: Set the dirty state for CMOs that fetch for write Oliver Upton
2026-06-26 15:31 ` [PATCH 00/22] KVM: arm64: nv: Implement FEAT_HAFDBS, FEAT_HAFT Leonardo Bras
2026-06-26 17:12 ` Marc Zyngier
2026-06-26 17:45 ` Oliver Upton [this message]
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=aj66nzo3gTHYInp8@kernel.org \
--to=oupton@kernel.org \
--cc=joey.gouly@arm.com \
--cc=kvmarm@lists.linux.dev \
--cc=leo.bras@arm.com \
--cc=maz@kernel.org \
--cc=seiden@linux.ibm.com \
--cc=suzuki.poulose@arm.com \
--cc=weilin.chang@arm.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.