All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marc Zyngier <maz@kernel.org>
To: eric.auger@redhat.com
Cc: Miguel Luis <miguel.luis@oracle.com>,
	"eric.auger.pro@gmail.com" <eric.auger.pro@gmail.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"richard.henderson@linaro.org" <richard.henderson@linaro.org>,
	"gkulkarni@amperecomputing.com" <gkulkarni@amperecomputing.com>,
	"gankulkarni@os.amperecomputing.com"
	<gankulkarni@os.amperecomputing.com>
Subject: Re: [PATCH v5 0/5] ARM Nested Virt Support
Date: Tue, 27 May 2025 15:15:08 +0100	[thread overview]
Message-ID: <86h616dtsj.wl-maz@kernel.org> (raw)
In-Reply-To: <ee464d7c-9b77-4aa7-9afa-627e497c7581@redhat.com>

On Tue, 27 May 2025 14:11:41 +0100,
Eric Auger <eric.auger@redhat.com> wrote:
> 
> Hi Miguel, Marc,
> 
> On 5/27/25 2:54 PM, Miguel Luis wrote:
> >
> >> On 27 May 2025, at 12:01, Marc Zyngier <maz@kernel.org> wrote:
> >>
> >> On Tue, 27 May 2025 12:33:23 +0100,
> >> Miguel Luis <miguel.luis@oracle.com> wrote:
> >>> Hi Eric,
> >>>
> >>>> On 27 May 2025, at 06:24, Eric Auger <eric.auger@redhat.com> wrote:
> >>>>
> >>>> Now that ARM nested virt has landed in kvm/next, let's turn the series
> >>>> into a PATCH series. The linux header update was made against kvm/next.
> >>>>
> >>>> For gaining virt functionality in KVM accelerated L1, The host needs to
> >>>> be booted with "kvm-arm.mode=nested" option and qemu needs to be invoked
> >>>> with: -machine virt,virtualization=on.
> >>> For the current kvmarm/next the guest also needs
> >>> “kvm-arm.mode=nested” I believe.
> >> No, unless you want the guest itself to be NV-capable.
> Effectively you don't need "kvm-arm.mode=nested" except if you want want
> multiple level of nesting.

You need kvm-arm.mode=nested on the command-line of a kernel that acts
as a host exposing KVM_CAP_ARM_EL2. This is obviously a recursive
property, but that's for the user to decide if they want it or not.

> > Correct, I got carried away with some mode combinations. Maybe we should depict
> > here more broadly how NV might be used with different mode combinations. I'll
> > think about this further ahead.
> >
> > As far this series go I couldn't found any issue booting a L1 guest with
> > virtualization=on and a L2 guest with virtualization=off.
> 
> on my end I tested with various untouched L2 guests (debian, fed, rhel)
> in 4kB/4kB/4KB page size mode (host, L1, L2). Those configs were successful.

I'm doing 4/4/4 (x1e) and 16/16/4 (m2). I'm also doing 4/4/4/4/4, but
that's not very reliable yet,

> with 64kB/64kB/64kB configs I am less lucky atm. One one machine I
> cannot boot L1 with virtualization=on. On the other I can boot L1 but
> cannot boot L2.

I think x1e supports 64k pages, but I'd need to try. Not exactly the
highest priority on my list of things to debug.

> Trying my best to debug a little bit further with my setup. Anyway if
> somebody else can try 64kB configs, it would help to confirm whether
> there are pending issues. I don't think they are related to this qemu
> integration series though.

Yeah, definitely a kernel issue IMO.

Thanks,

	M.

-- 
Without deviation from the norm, progress is not possible.

      reply	other threads:[~2025-05-27 14:16 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-27  6:24 [PATCH v5 0/5] ARM Nested Virt Support Eric Auger
2025-05-27  6:24 ` [PATCH v5 1/5] linux-headers: Update against kvm/next Eric Auger
2025-05-27  6:24 ` [PATCH v5 2/5] hw/arm: Allow setting KVM vGIC maintenance IRQ Eric Auger
2025-05-27  6:24 ` [PATCH v5 3/5] target/arm/kvm: Add helper to detect EL2 when using KVM Eric Auger
2025-05-27  6:24 ` [PATCH v5 4/5] target/arm: Enable feature ARM_FEATURE_EL2 if EL2 is supported Eric Auger
2025-05-27  6:24 ` [PATCH v5 5/5] hw/arm/virt: Allow virt extensions with KVM Eric Auger
2025-06-17 14:17   ` Alyssa Ross
2025-06-17 14:52     ` Eric Auger
2025-06-17 15:10     ` Eric Auger
2025-06-17 15:23     ` Miguel Luis
2025-06-17 15:41       ` Eric Auger
2025-06-17 15:50         ` Miguel Luis
2025-06-19  9:40           ` Eric Auger
2025-06-19 13:29           ` Eric Auger
2025-06-19 16:04             ` Alyssa Ross
2025-06-20 16:20             ` Miguel Luis
2025-05-27  7:39 ` [PATCH v5 0/5] ARM Nested Virt Support Marc Zyngier
2025-05-27  9:05   ` Eric Auger
2025-05-27 11:40   ` Miguel Luis
2025-05-27 12:02     ` Marc Zyngier
2025-05-27 13:24       ` Miguel Luis
2025-05-27 13:46         ` Marc Zyngier
2025-05-27 15:55           ` Miguel Luis
2025-05-27 16:52             ` Marc Zyngier
2025-05-27 23:52               ` Miguel Luis
2025-05-28  8:07                 ` Marc Zyngier
2025-06-19  8:19   ` Eric Auger
2025-06-19  8:33     ` Miguel Luis
2025-05-27 11:33 ` Miguel Luis
2025-05-27 12:01   ` Marc Zyngier
2025-05-27 12:54     ` Miguel Luis
2025-05-27 13:11       ` Eric Auger
2025-05-27 14:15         ` Marc Zyngier [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=86h616dtsj.wl-maz@kernel.org \
    --to=maz@kernel.org \
    --cc=eric.auger.pro@gmail.com \
    --cc=eric.auger@redhat.com \
    --cc=gankulkarni@os.amperecomputing.com \
    --cc=gkulkarni@amperecomputing.com \
    --cc=miguel.luis@oracle.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /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.