Skip to content

Commit 349c330

Browse files
author
Marc Zyngier
committed
KVM: arm64: Don't use empty structures as CPU reset state
Keeping empty structure as the vcpu state initializer is slightly wasteful: we only want to set pstate, and zero everything else. Just do that. Signed-off-by: Marc Zyngier <[email protected]>
1 parent bb44a8d commit 349c330

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

arch/arm64/kvm/reset.c

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,15 +36,11 @@ static u32 kvm_ipa_limit;
3636
/*
3737
* ARMv8 Reset Values
3838
*/
39-
static const struct kvm_regs default_regs_reset = {
40-
.regs.pstate = (PSR_MODE_EL1h | PSR_A_BIT | PSR_I_BIT |
41-
PSR_F_BIT | PSR_D_BIT),
42-
};
39+
#define VCPU_RESET_PSTATE_EL1 (PSR_MODE_EL1h | PSR_A_BIT | PSR_I_BIT | \
40+
PSR_F_BIT | PSR_D_BIT)
4341

44-
static const struct kvm_regs default_regs_reset32 = {
45-
.regs.pstate = (PSR_AA32_MODE_SVC | PSR_AA32_A_BIT |
46-
PSR_AA32_I_BIT | PSR_AA32_F_BIT),
47-
};
42+
#define VCPU_RESET_PSTATE_SVC (PSR_AA32_MODE_SVC | PSR_AA32_A_BIT | \
43+
PSR_AA32_I_BIT | PSR_AA32_F_BIT)
4844

4945
static bool cpu_has_32bit_el1(void)
5046
{
@@ -257,9 +253,9 @@ static int kvm_vcpu_enable_ptrauth(struct kvm_vcpu *vcpu)
257253
*/
258254
int kvm_reset_vcpu(struct kvm_vcpu *vcpu)
259255
{
260-
const struct kvm_regs *cpu_reset;
261256
int ret = -EINVAL;
262257
bool loaded;
258+
u32 pstate;
263259

264260
/* Reset PMU outside of the non-preemptible section */
265261
kvm_pmu_vcpu_reset(vcpu);
@@ -290,16 +286,17 @@ int kvm_reset_vcpu(struct kvm_vcpu *vcpu)
290286
if (test_bit(KVM_ARM_VCPU_EL1_32BIT, vcpu->arch.features)) {
291287
if (!cpu_has_32bit_el1())
292288
goto out;
293-
cpu_reset = &default_regs_reset32;
289+
pstate = VCPU_RESET_PSTATE_SVC;
294290
} else {
295-
cpu_reset = &default_regs_reset;
291+
pstate = VCPU_RESET_PSTATE_EL1;
296292
}
297293

298294
break;
299295
}
300296

301297
/* Reset core registers */
302-
memcpy(vcpu_gp_regs(vcpu), cpu_reset, sizeof(*cpu_reset));
298+
memset(vcpu_gp_regs(vcpu), 0, sizeof(*vcpu_gp_regs(vcpu)));
299+
vcpu_gp_regs(vcpu)->regs.pstate = pstate;
303300

304301
/* Reset system registers */
305302
kvm_reset_sys_regs(vcpu);

0 commit comments

Comments
 (0)