From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
Andrew Cooper <andrew.cooper3@citrix.com>, Wei Liu <wl@xen.org>
Subject: Re: [PATCH] x86/PV32: restore PAE-extended-CR3 logic
Date: Fri, 9 Feb 2024 10:06:57 +0100 [thread overview]
Message-ID: <ZcXrMZ2Nrsna2Kbe@macbook> (raw)
In-Reply-To: <47ab9000-68f6-8925-d814-a3a955b7f6cc@suse.com>
On Wed, Feb 15, 2023 at 03:54:11PM +0100, Jan Beulich wrote:
> While the PAE-extended-CR3 VM assist is a 32-bit only concept, it still
> applies to guests also when run on a 64-bit hypervisor: The "extended
> CR3" format has to be used there as well, to fit the address in the only
> 32-bit wide register there. As a result it was a mistake that the check
> was never enabled for that case, and was then mistakenly deleted in the
> course of removal of 32-bit-Xen code (218adf199e68 ["x86: We can assume
> CONFIG_PAGING_LEVELS==4"]).
>
> Similarly during Dom0 construction kernel awareness needs to be taken
> into account, and respective code was again mistakenly never enabled for
> 32-bit Dom0 when running on 64-bit Xen (and thus wrongly deleted by
> 5d1181a5ea5e ["xen: Remove x86_32 build target"]).
>
> At the same time restrict enabling of the assist for Dom0 to just the
> 32-bit case. Furthermore there's no need for an atomic update there.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Roger Pau Monné <roger.pau@citrix.com>
Thanks, Roger.
prev parent reply other threads:[~2024-02-09 9:07 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-15 14:54 [PATCH] x86/PV32: restore PAE-extended-CR3 logic Jan Beulich
2023-04-04 10:12 ` Roger Pau Monné
2023-04-04 10:31 ` Jan Beulich
2023-04-04 11:41 ` Roger Pau Monné
2023-04-04 14:24 ` Jan Beulich
2023-04-04 15:57 ` Roger Pau Monné
2023-04-04 16:38 ` Roger Pau Monné
2023-04-05 8:11 ` Jan Beulich
2023-04-05 8:59 ` Roger Pau Monné
2023-04-05 9:29 ` Jan Beulich
2023-04-05 9:40 ` Roger Pau Monné
2023-04-04 13:08 ` Andrew Cooper
2023-04-04 14:21 ` Jan Beulich
2023-04-04 20:40 ` Andrew Cooper
2023-04-05 8:44 ` Jan Beulich
2024-02-09 9:06 ` Roger Pau Monné [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=ZcXrMZ2Nrsna2Kbe@macbook \
--to=roger.pau@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=jbeulich@suse.com \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.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.