[PATCH v2 3/3] KVM: arm64: Remove AA64ZFR0_EL1 accessors

2020-11-02 Thread Andrew Jones
The AA64ZFR0_EL1 accessors are just the general accessors with its visibility function open-coded. It also skips the if-else chain in read_id_reg, but there's no reason not to go there. Indeed consolidating ID register accessors and removing lines of code make it worthwhile. No functional change i

Re: [PATCH v2 3/3] KVM: arm64: Remove AA64ZFR0_EL1 accessors

2020-11-03 Thread Dave Martin
On Mon, Nov 02, 2020 at 07:50:37PM +0100, Andrew Jones wrote: > The AA64ZFR0_EL1 accessors are just the general accessors with > its visibility function open-coded. It also skips the if-else > chain in read_id_reg, but there's no reason not to go there. > Indeed consolidating ID register accessors

Re: [PATCH v2 3/3] KVM: arm64: Remove AA64ZFR0_EL1 accessors

2020-11-03 Thread Andrew Jones
On Tue, Nov 03, 2020 at 11:32:08AM +, Dave Martin wrote: > On Mon, Nov 02, 2020 at 07:50:37PM +0100, Andrew Jones wrote: > > The AA64ZFR0_EL1 accessors are just the general accessors with > > its visibility function open-coded. It also skips the if-else > > chain in read_id_reg, but there's no

Re: [PATCH v2 3/3] KVM: arm64: Remove AA64ZFR0_EL1 accessors

2020-11-04 Thread Dave Martin
On Tue, Nov 03, 2020 at 02:46:40PM +0100, Andrew Jones wrote: > On Tue, Nov 03, 2020 at 11:32:08AM +, Dave Martin wrote: > > On Mon, Nov 02, 2020 at 07:50:37PM +0100, Andrew Jones wrote: > > > The AA64ZFR0_EL1 accessors are just the general accessors with > > > its visibility function open-code