[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host
From: |
Ard Biesheuvel |
Subject: |
Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host |
Date: |
Wed, 22 Sep 2021 13:41:13 +0200 |
On Thu, 16 Sept 2021 at 18:17, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Thu, 16 Sept 2021 at 17:05, Ard Biesheuvel <ardb@kernel.org> wrote:
> > I'd argue that compliance with the architecture means that the
> > software should not clear RES1 bits
>
> Architecturally, RES1 means that "software
> * Must not rely on the bit reading as 1.
> * Must use an SBOP policy to write to the bit."
> (SBOP=="should be 1 or preserved", ie you can preserve the existing value,
> as in "read register, change some bits, write back", or you can write a 1.)
>
OVMF preserves the bit, and does not reason or care about its value.
So in this sense, it is compliant.
> > but I don't think we can blame it
> > for not touching bits that were in in invalid state upon entry.
>
> SCTLR_EL1.SPAN == 0 is perfectly valid for a CPU that supports the
> PAN feature. It's just not the value OVMF wants, so OVMF should
> be setting it to what it does want. Also, as the first thing to
> run after reset (ie firmware) OVMF absolutely is responsible for
> dealing with system registers which have UNKNOWN values out of
> reset.
>
Fair enough. But I'd still suggest fixing this at both ends.
> Put another way, if you want to argue that this is an "invalid
> state" you need to point to the specification that defines
> the valid state that OVMF should see when it is handed control.
>
> -- PMM
- [PATCH v11 05/10] arm/hvf: Add a WFI handler, (continued)
- [PATCH v11 05/10] arm/hvf: Add a WFI handler, Alexander Graf, 2021/09/15
- [PATCH v11 06/10] hvf: arm: Implement -cpu host, Alexander Graf, 2021/09/15
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Peter Maydell, 2021/09/16
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Alexander Graf, 2021/09/16
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Peter Maydell, 2021/09/16
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Ard Biesheuvel, 2021/09/16
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Peter Maydell, 2021/09/16
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host,
Ard Biesheuvel <=
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Peter Maydell, 2021/09/22
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Ard Biesheuvel, 2021/09/22
- Re: [PATCH v11 06/10] hvf: arm: Implement -cpu host, Alexander Graf, 2021/09/16
[PATCH v11 03/10] hvf: Introduce hvf_arch_init() callback, Alexander Graf, 2021/09/15
[PATCH v11 08/10] arm: Add Hypervisor.framework build target, Alexander Graf, 2021/09/15
[PATCH v11 09/10] hvf: arm: Add rudimentary PMC support, Alexander Graf, 2021/09/15
[PATCH v11 04/10] hvf: Add Apple Silicon support, Alexander Graf, 2021/09/15