[0/2] KVM: arm64: Minor improvements to RAZ register handling

Message

Alexandru Elisei Sept. 27, 2021, 12:49 p.m. UTC

What sparked these two small patches is the series that fixed the PMU reset
values and their visibility from userspace, more specifically the
discussion around the patch that removed the PMSWINC_EL0 shadow register
[1].

The patches are straightforward cleanups without any changes in
functionality.

Tested on a rockpro64, by running kvm-unit-tests under qemu.

[1] https://www.spinics.net/lists/kvm-arm/msg47976.html

Alexandru Elisei (2):
  KVM: arm64: Return early from read_id_reg() if register is RAZ
  KVM: arm64: Use get_raz_reg() for userspace reads of PMSWINC_EL0

 arch/arm64/kvm/sys_regs.c | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

Comments

Marc Zyngier Oct. 11, 2021, 9:47 a.m. UTC | #1

Hi Alexandru,

On Mon, 27 Sep 2021 13:49:09 +0100,
Alexandru Elisei <[e-mail]> wrote:
> 
> What sparked these two small patches is the series that fixed the PMU reset
> values and their visibility from userspace, more specifically the
> discussion around the patch that removed the PMSWINC_EL0 shadow register
> [1].
> 
> The patches are straightforward cleanups without any changes in
> functionality.
> 
> Tested on a rockpro64, by running kvm-unit-tests under qemu.
> 
> [1] https://www.spinics.net/lists/kvm-arm/msg47976.html
> 
> Alexandru Elisei (2):
>   KVM: arm64: Return early from read_id_reg() if register is RAZ
>   KVM: arm64: Use get_raz_reg() for userspace reads of PMSWINC_EL0
> 
>  arch/arm64/kvm/sys_regs.c | 18 ++++++++++++++++--
>  1 file changed, 16 insertions(+), 2 deletions(-)
> 

If you are going to respin this one, now would be the time! :-)

Thanks,

	M.