Skip to content

Commit 54f1c77

Browse files
committed
KVM: nSVM: Merge MSRPM in 64-bit chunks on 64-bit kernels
When merging L0 and L1 MSRPMs as part of nested VMRUN emulation, access the bitmaps using "unsigned long" chunks, i.e. use 8-byte access for 64-bit kernels instead of arbitrarily working on 4-byte chunks. Opportunistically rename local variables in nested_svm_merge_msrpm() to more precisely/accurately reflect their purpose ("offset" in particular is extremely ambiguous). Link: https://lore.kernel.org/r/20250610225737.156318-30-seanjc@google.com Signed-off-by: Sean Christopherson <seanjc@google.com>
1 parent 5904ba5 commit 54f1c77

1 file changed

Lines changed: 11 additions & 10 deletions

File tree

arch/x86/kvm/svm/nested.c

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,7 @@ void recalc_intercepts(struct vcpu_svm *svm)
196196
*/
197197
static int nested_svm_msrpm_merge_offsets[6] __ro_after_init;
198198
static int nested_svm_nr_msrpm_merge_offsets __ro_after_init;
199+
typedef unsigned long nsvm_msrpm_merge_t;
199200

200201
int __init nested_svm_init_msrpm_merge_offsets(void)
201202
{
@@ -230,10 +231,10 @@ int __init nested_svm_init_msrpm_merge_offsets(void)
230231
return -EIO;
231232

232233
/*
233-
* Merging is done in 32-bit chunks to reduce the number of
234-
* accesses to L1's bitmap.
234+
* Merging is done in chunks to reduce the number of accesses
235+
* to L1's bitmap.
235236
*/
236-
offset = bit_nr / BITS_PER_BYTE / sizeof(u32);
237+
offset = bit_nr / BITS_PER_BYTE / sizeof(nsvm_msrpm_merge_t);
237238

238239
for (j = 0; j < nested_svm_nr_msrpm_merge_offsets; j++) {
239240
if (nested_svm_msrpm_merge_offsets[j] == offset)
@@ -261,8 +262,8 @@ int __init nested_svm_init_msrpm_merge_offsets(void)
261262
static bool nested_svm_merge_msrpm(struct kvm_vcpu *vcpu)
262263
{
263264
struct vcpu_svm *svm = to_svm(vcpu);
264-
u32 *msrpm02 = svm->nested.msrpm;
265-
u32 *msrpm01 = svm->msrpm;
265+
nsvm_msrpm_merge_t *msrpm02 = svm->nested.msrpm;
266+
nsvm_msrpm_merge_t *msrpm01 = svm->msrpm;
266267
int i;
267268

268269
/*
@@ -289,15 +290,15 @@ static bool nested_svm_merge_msrpm(struct kvm_vcpu *vcpu)
289290

290291
for (i = 0; i < nested_svm_nr_msrpm_merge_offsets; i++) {
291292
const int p = nested_svm_msrpm_merge_offsets[i];
292-
u32 value;
293-
u64 offset;
293+
nsvm_msrpm_merge_t l1_val;
294+
gpa_t gpa;
294295

295-
offset = svm->nested.ctl.msrpm_base_pa + (p * 4);
296+
gpa = svm->nested.ctl.msrpm_base_pa + (p * sizeof(l1_val));
296297

297-
if (kvm_vcpu_read_guest(vcpu, offset, &value, 4))
298+
if (kvm_vcpu_read_guest(vcpu, gpa, &l1_val, sizeof(l1_val)))
298299
return false;
299300

300-
msrpm02[p] = msrpm01[p] | value;
301+
msrpm02[p] = msrpm01[p] | l1_val;
301302
}
302303

303304
svm->nested.force_msr_bitmap_recalc = false;

0 commit comments

Comments
 (0)