Skip to content

Commit 99586e3

Browse files
Xiangyang Wulijinxia
authored andcommitted
HV:treewide:rename vm data structure
For data structure types "struct vm", its name is identical with variable name in the same scope. This is a MISRA C violation. Naming convention rule:If the data structure type is used by multi modules, its corresponding logic resource is exposed to external components (such as SOS, UOS), and its name meaning is simplistic (such as vcpu, vm), its name needs prefix "acrn_". The following udpates are made: struct vm *vm-->struct acrn_vm *vm Tracked-On: #861 Signed-off-by: Xiangyang Wu <xiangyang.wu@linux.intel.com>
1 parent ace4f48 commit 99586e3

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

66 files changed

+349
-349
lines changed

hypervisor/arch/x86/assign.c

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
*/
1818
static inline struct ptdev_remapping_info *
1919
ptdev_lookup_entry_by_sid(uint32_t intr_type,
20-
const union source_id *sid,const struct vm *vm)
20+
const union source_id *sid,const struct acrn_vm *vm)
2121
{
2222
struct ptdev_remapping_info *entry;
2323
struct list_head *pos;
@@ -43,7 +43,7 @@ is_entry_active(const struct ptdev_remapping_info *entry)
4343
return atomic_load32(&entry->active) == ACTIVE_FLAG;
4444
}
4545

46-
static bool ptdev_hv_owned_intx(const struct vm *vm, const union source_id *virt_sid)
46+
static bool ptdev_hv_owned_intx(const struct acrn_vm *vm, const union source_id *virt_sid)
4747
{
4848
/* vm0 vuart pin is owned by hypervisor under debug version */
4949
if (is_vm0(vm) && (virt_sid->intx_id.pin == COM1_IRQ)) {
@@ -68,7 +68,7 @@ static uint64_t calculate_logical_dest_mask(uint64_t pdmask)
6868
return dest_mask;
6969
}
7070

71-
static void ptdev_build_physical_msi(struct vm *vm, struct ptdev_msi_info *info,
71+
static void ptdev_build_physical_msi(struct acrn_vm *vm, struct ptdev_msi_info *info,
7272
uint32_t vector)
7373
{
7474
uint64_t vdmask, pdmask, dest_mask;
@@ -105,7 +105,7 @@ static void ptdev_build_physical_msi(struct vm *vm, struct ptdev_msi_info *info,
105105
}
106106

107107
static union ioapic_rte
108-
ptdev_build_physical_rte(struct vm *vm,
108+
ptdev_build_physical_rte(struct acrn_vm *vm,
109109
struct ptdev_remapping_info *entry)
110110
{
111111
union ioapic_rte rte;
@@ -191,7 +191,7 @@ ptdev_build_physical_rte(struct vm *vm,
191191
* - if the entry already be added by other vm, return NULL
192192
*/
193193
static struct ptdev_remapping_info *
194-
add_msix_remapping(struct vm *vm, uint16_t virt_bdf, uint16_t phys_bdf,
194+
add_msix_remapping(struct acrn_vm *vm, uint16_t virt_bdf, uint16_t phys_bdf,
195195
uint32_t entry_nr)
196196
{
197197
struct ptdev_remapping_info *entry;
@@ -246,7 +246,7 @@ add_msix_remapping(struct vm *vm, uint16_t virt_bdf, uint16_t phys_bdf,
246246

247247
/* deactive & remove mapping entry of vbdf:entry_nr for vm */
248248
static void
249-
remove_msix_remapping(const struct vm *vm, uint16_t virt_bdf, uint32_t entry_nr)
249+
remove_msix_remapping(const struct acrn_vm *vm, uint16_t virt_bdf, uint32_t entry_nr)
250250
{
251251
struct ptdev_remapping_info *entry;
252252
DEFINE_MSI_SID(virt_sid, virt_bdf, entry_nr);
@@ -280,7 +280,7 @@ remove_msix_remapping(const struct vm *vm, uint16_t virt_bdf, uint32_t entry_nr)
280280
* - if the entry already be added by other vm, return NULL
281281
*/
282282
static struct ptdev_remapping_info *
283-
add_intx_remapping(struct vm *vm, uint8_t virt_pin,
283+
add_intx_remapping(struct acrn_vm *vm, uint8_t virt_pin,
284284
uint8_t phys_pin, bool pic_pin)
285285
{
286286
struct ptdev_remapping_info *entry;
@@ -340,7 +340,7 @@ add_intx_remapping(struct vm *vm, uint8_t virt_pin,
340340
}
341341

342342
/* deactive & remove mapping entry of vpin for vm */
343-
static void remove_intx_remapping(const struct vm *vm, uint8_t virt_pin, bool pic_pin)
343+
static void remove_intx_remapping(const struct acrn_vm *vm, uint8_t virt_pin, bool pic_pin)
344344
{
345345
uint32_t phys_irq;
346346
struct ptdev_remapping_info *entry;
@@ -375,7 +375,7 @@ static void remove_intx_remapping(const struct vm *vm, uint8_t virt_pin, bool pi
375375
spinlock_release(&ptdev_lock);
376376
}
377377

378-
static void ptdev_intr_handle_irq(struct vm *vm,
378+
static void ptdev_intr_handle_irq(struct acrn_vm *vm,
379379
const struct ptdev_remapping_info *entry)
380380
{
381381
const union source_id *virt_sid = &entry->virt_sid;
@@ -443,7 +443,7 @@ static void ptdev_intr_handle_irq(struct vm *vm,
443443
void ptdev_softirq(uint16_t pcpu_id)
444444
{
445445
struct acrn_vcpu *vcpu = (struct acrn_vcpu *)per_cpu(vcpu, pcpu_id);
446-
struct vm *vm = vcpu->vm;
446+
struct acrn_vm *vm = vcpu->vm;
447447

448448
while (1) {
449449
struct ptdev_remapping_info *entry = ptdev_dequeue_softirq(vm);
@@ -482,7 +482,7 @@ void ptdev_softirq(uint16_t pcpu_id)
482482
}
483483
}
484484

485-
void ptdev_intx_ack(struct vm *vm, uint8_t virt_pin,
485+
void ptdev_intx_ack(struct acrn_vm *vm, uint8_t virt_pin,
486486
enum ptdev_vpin_source vpin_src)
487487
{
488488
uint32_t phys_irq;
@@ -533,7 +533,7 @@ void ptdev_intx_ack(struct vm *vm, uint8_t virt_pin,
533533
*
534534
* This function is called by SOS pci MSI config routine through hcall
535535
*/
536-
int ptdev_msix_remap(struct vm *vm, uint16_t virt_bdf,
536+
int ptdev_msix_remap(struct acrn_vm *vm, uint16_t virt_bdf,
537537
uint16_t entry_nr, struct ptdev_msi_info *info)
538538
{
539539
struct ptdev_remapping_info *entry;
@@ -589,7 +589,7 @@ int ptdev_msix_remap(struct vm *vm, uint16_t virt_bdf,
589589
return 0;
590590
}
591591

592-
static void activate_physical_ioapic(struct vm *vm,
592+
static void activate_physical_ioapic(struct acrn_vm *vm,
593593
struct ptdev_remapping_info *entry)
594594
{
595595
union ioapic_rte rte;
@@ -622,7 +622,7 @@ static void activate_physical_ioapic(struct vm *vm,
622622
/* Main entry for PCI/Legacy device assignment with INTx, calling from vIOAPIC
623623
* or vPIC
624624
*/
625-
int ptdev_intx_pin_remap(struct vm *vm, uint8_t virt_pin,
625+
int ptdev_intx_pin_remap(struct acrn_vm *vm, uint8_t virt_pin,
626626
enum ptdev_vpin_source vpin_src)
627627
{
628628
struct ptdev_remapping_info *entry;
@@ -729,7 +729,7 @@ int ptdev_intx_pin_remap(struct vm *vm, uint8_t virt_pin,
729729
* - currently, one phys_pin can only be held by one pin source (vPIC or
730730
* vIOAPIC)
731731
*/
732-
int ptdev_add_intx_remapping(struct vm *vm, uint8_t virt_pin, uint8_t phys_pin,
732+
int ptdev_add_intx_remapping(struct acrn_vm *vm, uint8_t virt_pin, uint8_t phys_pin,
733733
bool pic_pin)
734734
{
735735
struct ptdev_remapping_info *entry;
@@ -748,7 +748,7 @@ int ptdev_add_intx_remapping(struct vm *vm, uint8_t virt_pin, uint8_t phys_pin,
748748
/*
749749
* @pre vm != NULL
750750
*/
751-
void ptdev_remove_intx_remapping(const struct vm *vm, uint8_t virt_pin, bool pic_pin)
751+
void ptdev_remove_intx_remapping(const struct acrn_vm *vm, uint8_t virt_pin, bool pic_pin)
752752
{
753753
remove_intx_remapping(vm, virt_pin, pic_pin);
754754
}
@@ -758,7 +758,7 @@ void ptdev_remove_intx_remapping(const struct vm *vm, uint8_t virt_pin, bool pic
758758
* - the entry is identified by phys_bdf:msi_idx:
759759
* one entry vs. one phys_bdf:msi_idx
760760
*/
761-
int ptdev_add_msix_remapping(struct vm *vm, uint16_t virt_bdf,
761+
int ptdev_add_msix_remapping(struct acrn_vm *vm, uint16_t virt_bdf,
762762
uint16_t phys_bdf, uint32_t vector_count)
763763
{
764764
struct ptdev_remapping_info *entry;
@@ -777,7 +777,7 @@ int ptdev_add_msix_remapping(struct vm *vm, uint16_t virt_bdf,
777777
/*
778778
* @pre vm != NULL
779779
*/
780-
void ptdev_remove_msix_remapping(const struct vm *vm, uint16_t virt_bdf,
780+
void ptdev_remove_msix_remapping(const struct acrn_vm *vm, uint16_t virt_bdf,
781781
uint32_t vector_count)
782782
{
783783
uint32_t i;

hypervisor/arch/x86/cpuid.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ static inline struct vcpuid_entry *find_vcpuid_entry(const struct acrn_vcpu *vcp
1111
{
1212
uint32_t i = 0U, nr, half;
1313
struct vcpuid_entry *entry = NULL;
14-
struct vm *vm = vcpu->vm;
14+
struct acrn_vm *vm = vcpu->vm;
1515
uint32_t leaf = leaf_arg;
1616

1717
nr = vm->vcpuid_entry_nr;
@@ -63,7 +63,7 @@ static inline struct vcpuid_entry *find_vcpuid_entry(const struct acrn_vcpu *vcp
6363
return entry;
6464
}
6565

66-
static inline int set_vcpuid_entry(struct vm *vm,
66+
static inline int set_vcpuid_entry(struct acrn_vm *vm,
6767
const struct vcpuid_entry *entry)
6868
{
6969
struct vcpuid_entry *tmp;
@@ -170,7 +170,7 @@ static void init_vcpuid_entry(uint32_t leaf, uint32_t subleaf,
170170
}
171171
}
172172

173-
int set_vcpuid_entries(struct vm *vm)
173+
int set_vcpuid_entries(struct acrn_vm *vm)
174174
{
175175
int result;
176176
struct vcpuid_entry entry;

hypervisor/arch/x86/ept.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
#define ACRN_DBG_EPT 6U
1212

13-
void destroy_ept(struct vm *vm)
13+
void destroy_ept(struct acrn_vm *vm)
1414
{
1515
/* Destroy secure world */
1616
if (vm->sworld_control.flag.active != 0UL) {
@@ -23,7 +23,7 @@ void destroy_ept(struct vm *vm)
2323
}
2424

2525
/* using return value INVALID_HPA as error code */
26-
uint64_t local_gpa2hpa(struct vm *vm, uint64_t gpa, uint32_t *size)
26+
uint64_t local_gpa2hpa(struct acrn_vm *vm, uint64_t gpa, uint32_t *size)
2727
{
2828
uint64_t hpa = INVALID_HPA;
2929
uint64_t *pgentry, pg_size = 0UL;
@@ -58,7 +58,7 @@ uint64_t local_gpa2hpa(struct vm *vm, uint64_t gpa, uint32_t *size)
5858
}
5959

6060
/* using return value INVALID_HPA as error code */
61-
uint64_t gpa2hpa(struct vm *vm, uint64_t gpa)
61+
uint64_t gpa2hpa(struct acrn_vm *vm, uint64_t gpa)
6262
{
6363
return local_gpa2hpa(vm, gpa, NULL);
6464
}
@@ -178,7 +178,7 @@ int ept_misconfig_vmexit_handler(__unused struct acrn_vcpu *vcpu)
178178
return status;
179179
}
180180

181-
void ept_mr_add(struct vm *vm, uint64_t *pml4_page,
181+
void ept_mr_add(struct acrn_vm *vm, uint64_t *pml4_page,
182182
uint64_t hpa, uint64_t gpa, uint64_t size, uint64_t prot_orig)
183183
{
184184
uint16_t i;
@@ -203,7 +203,7 @@ void ept_mr_add(struct vm *vm, uint64_t *pml4_page,
203203
}
204204
}
205205

206-
void ept_mr_modify(struct vm *vm, uint64_t *pml4_page,
206+
void ept_mr_modify(struct acrn_vm *vm, uint64_t *pml4_page,
207207
uint64_t gpa, uint64_t size,
208208
uint64_t prot_set, uint64_t prot_clr)
209209
{
@@ -221,7 +221,7 @@ void ept_mr_modify(struct vm *vm, uint64_t *pml4_page,
221221
/**
222222
* @pre [gpa,gpa+size) has been mapped into host physical memory region
223223
*/
224-
void ept_mr_del(struct vm *vm, uint64_t *pml4_page, uint64_t gpa, uint64_t size)
224+
void ept_mr_del(struct acrn_vm *vm, uint64_t *pml4_page, uint64_t gpa, uint64_t size)
225225
{
226226
struct acrn_vcpu *vcpu;
227227
uint16_t i;

hypervisor/arch/x86/guest/guest.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ struct page_walk_info {
3232
bool is_smep_on;
3333
};
3434

35-
uint64_t vcpumask2pcpumask(struct vm *vm, uint64_t vdmask)
35+
uint64_t vcpumask2pcpumask(struct acrn_vm *vm, uint64_t vdmask)
3636
{
3737
uint16_t vcpu_id;
3838
uint64_t dmask = 0UL;
@@ -327,7 +327,7 @@ int gva2gpa(struct acrn_vcpu *vcpu, uint64_t gva, uint64_t *gpa,
327327
return ret;
328328
}
329329

330-
static inline uint32_t local_copy_gpa(struct vm *vm, void *h_ptr, uint64_t gpa,
330+
static inline uint32_t local_copy_gpa(struct acrn_vm *vm, void *h_ptr, uint64_t gpa,
331331
uint32_t size, uint32_t fix_pg_size, bool cp_from_vm)
332332
{
333333
uint64_t hpa;
@@ -360,7 +360,7 @@ static inline uint32_t local_copy_gpa(struct vm *vm, void *h_ptr, uint64_t gpa,
360360
return len;
361361
}
362362

363-
static inline int copy_gpa(struct vm *vm, void *h_ptr_arg, uint64_t gpa_arg,
363+
static inline int copy_gpa(struct acrn_vm *vm, void *h_ptr_arg, uint64_t gpa_arg,
364364
uint32_t size_arg, bool cp_from_vm)
365365
{
366366
void *h_ptr = h_ptr_arg;
@@ -427,7 +427,7 @@ static inline int copy_gva(struct acrn_vcpu *vcpu, void *h_ptr_arg, uint64_t gva
427427
* continuous
428428
* @pre Pointer vm is non-NULL
429429
*/
430-
int copy_from_gpa(struct vm *vm, void *h_ptr, uint64_t gpa, uint32_t size)
430+
int copy_from_gpa(struct acrn_vm *vm, void *h_ptr, uint64_t gpa, uint32_t size)
431431
{
432432
return copy_gpa(vm, h_ptr, gpa, size, 1);
433433
}
@@ -438,7 +438,7 @@ int copy_from_gpa(struct vm *vm, void *h_ptr, uint64_t gpa, uint32_t size)
438438
* continuous
439439
* @pre Pointer vm is non-NULL
440440
*/
441-
int copy_to_gpa(struct vm *vm, void *h_ptr, uint64_t gpa, uint32_t size)
441+
int copy_to_gpa(struct acrn_vm *vm, void *h_ptr, uint64_t gpa, uint32_t size)
442442
{
443443
return copy_gpa(vm, h_ptr, gpa, size, 0);
444444
}
@@ -606,7 +606,7 @@ static void rebuild_vm0_e820(void)
606606
* @pre vm != NULL
607607
* @pre is_vm0(vm) == true
608608
*/
609-
int prepare_vm0_memmap_and_e820(struct vm *vm)
609+
int prepare_vm0_memmap_and_e820(struct acrn_vm *vm)
610610
{
611611
uint32_t i;
612612
uint64_t attr_uc = (EPT_RWX | EPT_UNCACHED);

hypervisor/arch/x86/guest/mptable.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -288,7 +288,7 @@ static uint8_t mpt_compute_checksum(void *base, size_t len)
288288
return (256U - sum);
289289
}
290290

291-
int mptable_build(struct vm *vm)
291+
int mptable_build(struct acrn_vm *vm)
292292
{
293293
char *startaddr;
294294
char *curraddr;

hypervisor/arch/x86/guest/pm.c

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
#include <hypervisor.h>
88

9-
int validate_pstate(const struct vm *vm, uint64_t perf_ctl)
9+
int validate_pstate(const struct acrn_vm *vm, uint64_t perf_ctl)
1010
{
1111
const struct cpu_px_data *px_data;
1212
int i, px_cnt;
@@ -31,7 +31,7 @@ int validate_pstate(const struct vm *vm, uint64_t perf_ctl)
3131
return -1;
3232
}
3333

34-
static void vm_setup_cpu_px(struct vm *vm)
34+
static void vm_setup_cpu_px(struct acrn_vm *vm)
3535
{
3636
uint32_t px_data_size;
3737

@@ -56,7 +56,7 @@ static void vm_setup_cpu_px(struct vm *vm)
5656

5757
}
5858

59-
static void vm_setup_cpu_cx(struct vm *vm)
59+
static void vm_setup_cpu_cx(struct acrn_vm *vm)
6060
{
6161
uint32_t cx_data_size;
6262

@@ -84,7 +84,7 @@ static void vm_setup_cpu_cx(struct vm *vm)
8484

8585
}
8686

87-
static inline void init_cx_port(struct vm *vm)
87+
static inline void init_cx_port(struct acrn_vm *vm)
8888
{
8989
uint8_t cx_idx;
9090

@@ -99,7 +99,7 @@ static inline void init_cx_port(struct vm *vm)
9999
}
100100
}
101101

102-
void vm_setup_cpu_state(struct vm *vm)
102+
void vm_setup_cpu_state(struct acrn_vm *vm)
103103
{
104104
vm_setup_cpu_px(vm);
105105
vm_setup_cpu_cx(vm);
@@ -109,7 +109,7 @@ void vm_setup_cpu_state(struct vm *vm)
109109
/* This function is for power management Sx state implementation,
110110
* VM need to load the Sx state data to implement S3/S5.
111111
*/
112-
int vm_load_pm_s_state(struct vm *vm)
112+
int vm_load_pm_s_state(struct acrn_vm *vm)
113113
{
114114
#ifdef ACPI_INFO_VALIDATED
115115
vm->pm.sx_state_data = (struct pm_s_state_data *)&host_pm_s_state;
@@ -132,7 +132,7 @@ static inline uint8_t get_slp_typx(uint32_t pm1_cnt)
132132
return (uint8_t)((pm1_cnt & 0x1fffU) >> BIT_SLP_TYPx);
133133
}
134134

135-
static uint32_t pm1ab_io_read(__unused struct vm *vm, uint16_t addr,
135+
static uint32_t pm1ab_io_read(__unused struct acrn_vm *vm, uint16_t addr,
136136
size_t width)
137137
{
138138
uint32_t val = pio_read(addr, width);
@@ -148,7 +148,7 @@ static uint32_t pm1ab_io_read(__unused struct vm *vm, uint16_t addr,
148148
return val;
149149
}
150150

151-
static void pm1ab_io_write(__unused struct vm *vm, uint16_t addr, size_t width,
151+
static void pm1ab_io_write(__unused struct acrn_vm *vm, uint16_t addr, size_t width,
152152
uint32_t v)
153153
{
154154
static uint32_t pm1a_cnt_ready = 0U;
@@ -187,7 +187,7 @@ static void pm1ab_io_write(__unused struct vm *vm, uint16_t addr, size_t width,
187187
}
188188

189189
static void
190-
register_gas_io_handler(struct vm *vm, const struct acpi_generic_address *gas)
190+
register_gas_io_handler(struct acrn_vm *vm, const struct acpi_generic_address *gas)
191191
{
192192
uint8_t io_len[5] = {0, 1, 2, 4, 8};
193193
struct vm_io_range gas_io;
@@ -210,7 +210,7 @@ register_gas_io_handler(struct vm *vm, const struct acpi_generic_address *gas)
210210
vm->vm_id, gas_io.base, gas_io.len);
211211
}
212212

213-
void register_pm1ab_handler(struct vm *vm)
213+
void register_pm1ab_handler(struct acrn_vm *vm)
214214
{
215215
struct pm_s_state_data *sx_data = vm->pm.sx_state_data;
216216

0 commit comments

Comments
 (0)