| Message ID | 20210615133950.693489-7-tabba@google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | KVM: arm64: Fixed features for protected VMs | expand |
On Tue, Jun 15, 2021 at 02:39:43PM +0100, Fuad Tabba wrote: > Add feature register flag definitions to clarify which features > might be supported. > > No functional change intended. > > Signed-off-by: Fuad Tabba <tabba@google.com> > --- > arch/arm64/include/asm/sysreg.h | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm64/include/asm/sysreg.h b/arch/arm64/include/asm/sysreg.h > index 65d15700a168..42bcc5102d10 100644 > --- a/arch/arm64/include/asm/sysreg.h > +++ b/arch/arm64/include/asm/sysreg.h > @@ -789,6 +789,10 @@ > #define ID_AA64PFR0_FP_SUPPORTED 0x0 > #define ID_AA64PFR0_ASIMD_NI 0xf > #define ID_AA64PFR0_ASIMD_SUPPORTED 0x0 > +#define ID_AA64PFR0_EL3_64BIT_ONLY 0x1 > +#define ID_AA64PFR0_EL3_32BIT_64BIT 0x2 > +#define ID_AA64PFR0_EL2_64BIT_ONLY 0x1 > +#define ID_AA64PFR0_EL2_32BIT_64BIT 0x2 > #define ID_AA64PFR0_EL1_64BIT_ONLY 0x1 > #define ID_AA64PFR0_EL1_32BIT_64BIT 0x2 > #define ID_AA64PFR0_EL0_64BIT_ONLY 0x1 Maybe just consolidate all of these into two definitions: #define ID_AA64PFR0_ELx_64BIT_ONLY 0x1 #define ID_AA64PFR0_ELx_32BIT_64BIT 0x2 ? Will
Hi Will, On Thu, Jul 1, 2021 at 2:22 PM Will Deacon <will@kernel.org> wrote: > > On Tue, Jun 15, 2021 at 02:39:43PM +0100, Fuad Tabba wrote: > > Add feature register flag definitions to clarify which features > > might be supported. > > > > No functional change intended. > > > > Signed-off-by: Fuad Tabba <tabba@google.com> > > --- > > arch/arm64/include/asm/sysreg.h | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/arch/arm64/include/asm/sysreg.h b/arch/arm64/include/asm/sysreg.h > > index 65d15700a168..42bcc5102d10 100644 > > --- a/arch/arm64/include/asm/sysreg.h > > +++ b/arch/arm64/include/asm/sysreg.h > > @@ -789,6 +789,10 @@ > > #define ID_AA64PFR0_FP_SUPPORTED 0x0 > > #define ID_AA64PFR0_ASIMD_NI 0xf > > #define ID_AA64PFR0_ASIMD_SUPPORTED 0x0 > > +#define ID_AA64PFR0_EL3_64BIT_ONLY 0x1 > > +#define ID_AA64PFR0_EL3_32BIT_64BIT 0x2 > > +#define ID_AA64PFR0_EL2_64BIT_ONLY 0x1 > > +#define ID_AA64PFR0_EL2_32BIT_64BIT 0x2 > > #define ID_AA64PFR0_EL1_64BIT_ONLY 0x1 > > #define ID_AA64PFR0_EL1_32BIT_64BIT 0x2 > > #define ID_AA64PFR0_EL0_64BIT_ONLY 0x1 > > Maybe just consolidate all of these into two definitions: > > #define ID_AA64PFR0_ELx_64BIT_ONLY 0x1 > #define ID_AA64PFR0_ELx_32BIT_64BIT 0x2 Will do. Cheers, /fuad > ? > > Will
diff --git a/arch/arm64/include/asm/sysreg.h b/arch/arm64/include/asm/sysreg.h index 65d15700a168..42bcc5102d10 100644 --- a/arch/arm64/include/asm/sysreg.h +++ b/arch/arm64/include/asm/sysreg.h @@ -789,6 +789,10 @@ #define ID_AA64PFR0_FP_SUPPORTED 0x0 #define ID_AA64PFR0_ASIMD_NI 0xf #define ID_AA64PFR0_ASIMD_SUPPORTED 0x0 +#define ID_AA64PFR0_EL3_64BIT_ONLY 0x1 +#define ID_AA64PFR0_EL3_32BIT_64BIT 0x2 +#define ID_AA64PFR0_EL2_64BIT_ONLY 0x1 +#define ID_AA64PFR0_EL2_32BIT_64BIT 0x2 #define ID_AA64PFR0_EL1_64BIT_ONLY 0x1 #define ID_AA64PFR0_EL1_32BIT_64BIT 0x2 #define ID_AA64PFR0_EL0_64BIT_ONLY 0x1 @@ -848,12 +852,16 @@ #define ID_AA64MMFR0_ASID_SHIFT 4 #define ID_AA64MMFR0_PARANGE_SHIFT 0 +#define ID_AA64MMFR0_ASID_8 0x0 +#define ID_AA64MMFR0_ASID_16 0x2 + #define ID_AA64MMFR0_TGRAN4_NI 0xf #define ID_AA64MMFR0_TGRAN4_SUPPORTED 0x0 #define ID_AA64MMFR0_TGRAN64_NI 0xf #define ID_AA64MMFR0_TGRAN64_SUPPORTED 0x0 #define ID_AA64MMFR0_TGRAN16_NI 0x0 #define ID_AA64MMFR0_TGRAN16_SUPPORTED 0x1 +#define ID_AA64MMFR0_PARANGE_40 0x2 #define ID_AA64MMFR0_PARANGE_48 0x5 #define ID_AA64MMFR0_PARANGE_52 0x6 @@ -901,6 +909,7 @@ #define ID_AA64MMFR2_CNP_SHIFT 0 /* id_aa64dfr0 */ +#define ID_AA64DFR0_MTPMU_SHIFT 48 #define ID_AA64DFR0_TRBE_SHIFT 44 #define ID_AA64DFR0_TRACE_FILT_SHIFT 40 #define ID_AA64DFR0_DOUBLELOCK_SHIFT 36
Add feature register flag definitions to clarify which features might be supported. No functional change intended. Signed-off-by: Fuad Tabba <tabba@google.com> --- arch/arm64/include/asm/sysreg.h | 9 +++++++++ 1 file changed, 9 insertions(+)