Re: [PATCH v2] xen/arm: don't read aarch32 regs when aarch32 isn't available

2021-01-13 Thread Julien Grall
Hi Stefano, I think you forgot to update the title to say the state is EL0. On 12/01/2021 23:44, Stefano Stabellini wrote: Don't read aarch32 system registers at boot time when the aarch32 state is not available at EL0. They are UNKNOWN, so it is not useful to read them. Moreover, on Cavium Th

Re: [PATCH v2] xen/arm: don't read aarch32 regs when aarch32 isn't available

2021-01-13 Thread Bertrand Marquis
Hi Stefano, > On 12 Jan 2021, at 23:44, Stefano Stabellini wrote: > > Don't read aarch32 system registers at boot time when the aarch32 state > is not available at EL0. They are UNKNOWN, so it is not useful to read > them. Moreover, on Cavium ThunderX reading ID_PFR2_EL1 causes a Xen > crash.

[PATCH v2] xen/arm: don't read aarch32 regs when aarch32 isn't available

2021-01-12 Thread Stefano Stabellini
Don't read aarch32 system registers at boot time when the aarch32 state is not available at EL0. They are UNKNOWN, so it is not useful to read them. Moreover, on Cavium ThunderX reading ID_PFR2_EL1 causes a Xen crash. Instead, only read them when aarch32 is available. Leave the corresponding fie