Skip to content

Commit c68feb6

Browse files
committed
KVM: VMX: Add a wrapper around ROL16() to get a vmcs12 from a field encoding
Add a wrapper macro, ENC_TO_VMCS12_IDX(), to get a vmcs12 index given a field encoding in anticipation of adding a macro to get from a vmcs12 index back to the field encoding. And because open coding ROL16(n, 6) everywhere is gross. No functional change intended. Suggested-by: Xiaoyao Li <xiaoyao.li@intel.com> Reviewed-by: Xiaoyao Li <xiaoyao.li@intel.com> Link: https://patch.msgid.link/20260115173427.716021-3-seanjc@google.com Signed-off-by: Sean Christopherson <seanjc@google.com>
1 parent 26304e0 commit c68feb6

5 files changed

Lines changed: 6 additions & 5 deletions

File tree

arch/x86/kvm/vmx/hyperv_evmcs.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
#include "hyperv_evmcs.h"
88

99
#define EVMCS1_OFFSET(x) offsetof(struct hv_enlightened_vmcs, x)
10-
#define EVMCS1_FIELD(number, name, clean_field)[ROL16(number, 6)] = \
10+
#define EVMCS1_FIELD(number, name, clean_field)[ENC_TO_VMCS12_IDX(number)] = \
1111
{EVMCS1_OFFSET(name), clean_field}
1212

1313
const struct evmcs_field vmcs_field_to_evmcs_1[] = {

arch/x86/kvm/vmx/hyperv_evmcs.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ static __always_inline int evmcs_field_offset(unsigned long field,
130130
u16 *clean_field)
131131
{
132132
const struct evmcs_field *evmcs_field;
133-
unsigned int index = ROL16(field, 6);
133+
unsigned int index = ENC_TO_VMCS12_IDX(field);
134134

135135
if (unlikely(index >= nr_evmcs_1_fields))
136136
return -ENOENT;

arch/x86/kvm/vmx/vmcs.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
#include "capabilities.h"
1313

1414
#define ROL16(val, n) ((u16)(((u16)(val) << (n)) | ((u16)(val) >> (16 - (n)))))
15+
#define ENC_TO_VMCS12_IDX(enc) ROL16(enc, 6)
1516

1617
struct vmcs_hdr {
1718
u32 revision_id:31;

arch/x86/kvm/vmx/vmcs12.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@
44
#include "vmcs12.h"
55

66
#define VMCS12_OFFSET(x) offsetof(struct vmcs12, x)
7-
#define FIELD(number, name) [ROL16(number, 6)] = VMCS12_OFFSET(name)
7+
#define FIELD(number, name) [ENC_TO_VMCS12_IDX(number)] = VMCS12_OFFSET(name)
88
#define FIELD64(number, name) \
99
FIELD(number, name), \
10-
[ROL16(number##_HIGH, 6)] = VMCS12_OFFSET(name) + sizeof(u32)
10+
[ENC_TO_VMCS12_IDX(number##_HIGH)] = VMCS12_OFFSET(name) + sizeof(u32)
1111

1212
const unsigned short vmcs12_field_offsets[] = {
1313
FIELD(VIRTUAL_PROCESSOR_ID, virtual_processor_id),

arch/x86/kvm/vmx/vmcs12.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -385,7 +385,7 @@ static inline short get_vmcs12_field_offset(unsigned long field)
385385
if (field >> 15)
386386
return -ENOENT;
387387

388-
index = ROL16(field, 6);
388+
index = ENC_TO_VMCS12_IDX(field);
389389
if (index >= nr_vmcs12_fields)
390390
return -ENOENT;
391391

0 commit comments

Comments
 (0)