Skip to content

Commit bff592d

Browse files
mgcaowenlingz
authored andcommitted
HV: rename e820_entries to e820_entries_count
to make its meaning more matched. Tracked-On: #1842 Signed-off-by: Minggui Cao <minggui.cao@intel.com> Acked-by: Anthony Xu <anthony.xu@intel.com>
1 parent 9b58b9d commit bff592d

File tree

1 file changed

+19
-19
lines changed

1 file changed

+19
-19
lines changed

hypervisor/arch/x86/e820.c

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
* and hide HV memory from VM0...
1515
*/
1616

17-
static uint32_t e820_entries;
17+
static uint32_t e820_entries_count;
1818
static struct e820_entry e820[E820_MAX_ENTRIES];
1919
static struct e820_mem_params e820_mem;
2020

@@ -31,7 +31,7 @@ static void obtain_e820_mem_info(void)
3131
e820_mem.max_ram_blk_base = 0UL;
3232
e820_mem.max_ram_blk_size = 0UL;
3333

34-
for (i = 0U; i < e820_entries; i++) {
34+
for (i = 0U; i < e820_entries_count; i++) {
3535
entry = &e820[i];
3636
if (e820_mem.mem_bottom > entry->baseaddr) {
3737
e820_mem.mem_bottom = entry->baseaddr;
@@ -64,7 +64,7 @@ void rebuild_vm0_e820(void)
6464
/* hypervisor mem need be filter out from e820 table
6565
* it's hv itself + other hv reserved mem like vgt etc
6666
*/
67-
for (i = 0U; i < e820_entries; i++) {
67+
for (i = 0U; i < e820_entries_count; i++) {
6868
entry = &e820[i];
6969
entry_start = entry->baseaddr;
7070
entry_end = entry->baseaddr + entry->length;
@@ -105,9 +105,9 @@ void rebuild_vm0_e820(void)
105105
}
106106

107107
if (new_entry.length > 0UL) {
108-
e820_entries++;
109-
ASSERT(e820_entries <= E820_MAX_ENTRIES, "e820 entry overflow");
110-
entry = &e820[e820_entries - 1];
108+
e820_entries_count++;
109+
ASSERT(e820_entries_count <= E820_MAX_ENTRIES, "e820 entry overflow");
110+
entry = &e820[e820_entries_count - 1];
111111
entry->baseaddr = new_entry.baseaddr;
112112
entry->length = new_entry.length;
113113
entry->type = new_entry.type;
@@ -126,7 +126,7 @@ uint64_t e820_alloc_low_memory(uint32_t size_arg)
126126
/* We want memory in page boundary and integral multiple of pages */
127127
size = (((size + PAGE_SIZE) - 1U) >> PAGE_SHIFT) << PAGE_SHIFT;
128128

129-
for (i = 0U; i < e820_entries; i++) {
129+
for (i = 0U; i < e820_entries_count; i++) {
130130
entry = &e820[i];
131131
uint64_t start, end, length;
132132

@@ -151,15 +151,15 @@ uint64_t e820_alloc_low_memory(uint32_t size_arg)
151151
* found entry with available memory larger than requested
152152
* allocate memory from the end of this entry at page boundary
153153
*/
154-
new_entry = &e820[e820_entries];
154+
new_entry = &e820[e820_entries_count];
155155
new_entry->type = E820_TYPE_RESERVED;
156156
new_entry->baseaddr = end - size;
157157
new_entry->length = (entry->baseaddr + entry->length) - new_entry->baseaddr;
158158

159159
/* Shrink the existing entry and total available memory */
160160
entry->length -= new_entry->length;
161161
e820_mem.total_mem_size -= new_entry->length;
162-
e820_entries++;
162+
e820_entries_count++;
163163

164164
return new_entry->baseaddr;
165165
}
@@ -180,16 +180,16 @@ void init_e820(void)
180180
if ((mbi->mi_flags & MULTIBOOT_INFO_HAS_MMAP) != 0U) {
181181
struct multiboot_mmap *mmap = (struct multiboot_mmap *)hpa2hva((uint64_t)mbi->mi_mmap_addr);
182182

183-
e820_entries = mbi->mi_mmap_length / sizeof(struct multiboot_mmap);
184-
if (e820_entries > E820_MAX_ENTRIES) {
185-
pr_err("Too many E820 entries %d\n", e820_entries);
186-
e820_entries = E820_MAX_ENTRIES;
183+
e820_entries_count = mbi->mi_mmap_length / sizeof(struct multiboot_mmap);
184+
if (e820_entries_count > E820_MAX_ENTRIES) {
185+
pr_err("Too many E820 entries %d\n", e820_entries_count);
186+
e820_entries_count = E820_MAX_ENTRIES;
187187
}
188188

189189
dev_dbg(ACRN_DBG_E820, "mmap length 0x%x addr 0x%x entries %d\n",
190-
mbi->mi_mmap_length, mbi->mi_mmap_addr, e820_entries);
190+
mbi->mi_mmap_length, mbi->mi_mmap_addr, e820_entries_count);
191191

192-
for (i = 0U; i < e820_entries; i++) {
192+
for (i = 0U; i < e820_entries_count; i++) {
193193
e820[i].baseaddr = mmap[i].baseaddr;
194194
e820[i].length = mmap[i].length;
195195
e820[i].type = mmap[i].type;
@@ -208,7 +208,7 @@ void init_e820(void)
208208

209209
uint32_t get_e820_entries_count(void)
210210
{
211-
return e820_entries;
211+
return e820_entries_count;
212212
}
213213

214214
const struct e820_entry *get_e820_entry(void)
@@ -239,14 +239,14 @@ uint32_t create_e820_table(struct e820_entry *param_e820)
239239
{
240240
uint32_t i;
241241

242-
ASSERT(e820_entries > 0U, "e820 should be inited");
242+
ASSERT(e820_entries_count > 0U, "e820 should be inited");
243243

244-
for (i = 0U; i < e820_entries; i++) {
244+
for (i = 0U; i < e820_entries_count; i++) {
245245
param_e820[i].baseaddr = e820[i].baseaddr;
246246
param_e820[i].length = e820[i].length;
247247
param_e820[i].type = e820[i].type;
248248
}
249249

250-
return e820_entries;
250+
return e820_entries_count;
251251
}
252252
#endif

0 commit comments

Comments
 (0)