Fix the spelling of EFI_PAGE_SIZE
We assume EFI_PAGE_SIZE is the same as PAGE_SIZE, however this may not be the case. Use the former when working with a list of pages from the UEFI firmware so the correct size is used. This will be needed on arm64 where PAGE_SIZE could be 16k or 64k in the future. The other architectures have been updated to be consistent. Reviewed by: imp Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D34510
This commit is contained in:
@@ -816,7 +816,7 @@ add_efi_map_entries(struct efi_map_header *efihdr, vm_paddr_t *physmap,
|
||||
continue;
|
||||
}
|
||||
|
||||
if (!add_physmap_entry(p->md_phys, (p->md_pages * PAGE_SIZE),
|
||||
if (!add_physmap_entry(p->md_phys, p->md_pages * EFI_PAGE_SIZE,
|
||||
physmap, physmap_idx))
|
||||
break;
|
||||
}
|
||||
|
||||
@@ -490,7 +490,7 @@ arm_add_efi_map_entries(struct efi_map_header *efihdr, struct mem_region *mr,
|
||||
break;
|
||||
|
||||
mr[j].mr_start = p->md_phys;
|
||||
mr[j].mr_size = p->md_pages * PAGE_SIZE;
|
||||
mr[j].mr_size = p->md_pages * EFI_PAGE_SIZE;
|
||||
memory_size += mr[j].mr_size;
|
||||
}
|
||||
|
||||
|
||||
@@ -469,7 +469,7 @@ exclude_efi_map_entry(struct efi_md *p)
|
||||
*/
|
||||
break;
|
||||
default:
|
||||
physmem_exclude_region(p->md_phys, p->md_pages * PAGE_SIZE,
|
||||
physmem_exclude_region(p->md_phys, p->md_pages * EFI_PAGE_SIZE,
|
||||
EXFLAG_NOALLOC);
|
||||
}
|
||||
}
|
||||
@@ -501,7 +501,7 @@ add_efi_map_entry(struct efi_md *p)
|
||||
* We're allowed to use any entry with these types.
|
||||
*/
|
||||
physmem_hardware_region(p->md_phys,
|
||||
p->md_pages * PAGE_SIZE);
|
||||
p->md_pages * EFI_PAGE_SIZE);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user