hollalinux
/
linux-sg2042
mirror of https://github.com/sophgo/linux-riscv.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

x86/boot/e820: Basic cleanup of e820.c 

Over the last decade or so e820.c has become an ureadable mess of
tinkerware. Perform some very basic cleanups before doing more
intricate cleanups, so that my eyes don't start bleeding when I look at it.

Here's some of the excesses:

 - Total disregard of countless aspects of Documentation/CodingStyle.

 - Totally inconsistent hodge-podge of various coding styles and practices.

 - Gems like:

       (unsigned long long) e820_table->entries[i].addr

   ... which is a completely unnecessary type conversion of an u64 value.

 - Incomprehensible comments while there are major functions with absolutely
   no explanation - plus an armada of typos and grammar mistakes.

 - Mindless checkpatch artifacts such as:

         if (append_e820_table(boot_params.e820_table, boot_params.e820_entries)
           < 0) {

           for_each_free_mem_range(u, NUMA_NO_NODE, MEMBLOCK_NONE, &start, &end,
                                   NULL) {

 - Actively misleading comments:

        /* In case someone cares... */
        return who;

   ( The usage site of the return value just a few lines further down makes it
     clear that we very much care about the return value, we use it to print
     out the e820 map... )

 - Colorfully inconsistent capitalization and punctuation throughout.

 - etc.

This patch fixes only the worst excesses - there's more to fix.

No change in functionality.

Cc: Alex Thorlton <athorlton@sgi.com>
Cc: Andy Lutomirski <luto@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Brian Gerst <brgerst@gmail.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: Denys Vlasenko <dvlasenk@redhat.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Huang, Ying <ying.huang@intel.com>
Cc: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: Juergen Gross <jgross@suse.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Paul Jackson <pj@sgi.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Rafael J. Wysocki <rjw@sisk.pl>
Cc: Tejun Heo <tj@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Wei Yang <richard.weiyang@gmail.com>
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
This commit is contained in:
Ingo Molnar 2017-01-28 11:13:08 +01:00
parent 544a0f47e7
commit 640e1b38b0
1 changed files with 207 additions and 240 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

447
arch/x86/kernel/e820.c
View File

@ -1,12 +1,12 @@
/* /*
* Handle the memory map. * Low level x86 E820 memory map handling functions.
* The functions here do the job until bootmem takes over.
* *
* Getting sanitize_e820_table() in sync with i386 version by applying change: * The firmware and bootloader passes us the "E820 table", which is the primary
* - Provisions for empty E820 memory regions (reported by certain BIOSes). * physical memory layout description available about x86 systems.
* Alex Achenbach <xela@slit.de>, December 2002.
* Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
* *
* The kernel takes the E820 memory layout and optionally modifies it with
* quirks and other tweaks, and feeds that into the generic Linux memory
* allocation code routines via a platform independent interface (memblock, etc.).
*/ */
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/types.h> #include <linux/types.h>
@ -27,13 +27,6 @@
#include <asm/cpufeature.h> #include <asm/cpufeature.h>
/* /*
* The firmware and bootloader passes us an E820 table that is the primary
* physical memory layout description available about x86 systems.
*
* The kernel takes the e820 memory layout and optionally modifies it with
* quirks and other tweaks, and feeds that into the generic Linux memory
* allocation code routines via a platform independent interface (memblock, etc.).
*
* We organize the E820 table into two main data structures: * We organize the E820 table into two main data structures:
* *
* - 'e820_table_firmware': the original firmware version passed to us by the * - 'e820_table_firmware': the original firmware version passed to us by the
@ -45,16 +38,16 @@
* - the hibernation code uses it to generate a kernel-independent MD5 * - the hibernation code uses it to generate a kernel-independent MD5
* fingerprint of the physical memory layout of a system. * fingerprint of the physical memory layout of a system.
* *
* - kexec, which is a bootloader in disguise, uses the original e820 * - kexec, which is a bootloader in disguise, uses the original E820
* layout to pass to the kexec-ed kernel. This way the original kernel * layout to pass to the kexec-ed kernel. This way the original kernel
* can have a restricted e820 map while the kexec()-ed kexec-kernel * can have a restricted E820 map while the kexec()-ed kexec-kernel
* can have access to full memory - etc. * can have access to full memory - etc.
* *
* - 'e820_table': this is the main e820 table that is massaged by the * - 'e820_table': this is the main E820 table that is massaged by the
* low level x86 platform code, or modified by boot parameters, before * low level x86 platform code, or modified by boot parameters, before
* passed on to higher level MM layers. * passed on to higher level MM layers.
* *
* Once the e820 map has been converted to the standard Linux memory layout * Once the E820 map has been converted to the standard Linux memory layout
* information its role stops - modifying it has no effect and does not get * information its role stops - modifying it has no effect and does not get
* re-propagated. So itsmain role is a temporary bootstrap storage of firmware * re-propagated. So itsmain role is a temporary bootstrap storage of firmware
* specific memory layout data during early bootup. * specific memory layout data during early bootup.
@ -75,8 +68,7 @@ EXPORT_SYMBOL(pci_mem_start);
* This function checks if any part of the range <start,end> is mapped * This function checks if any part of the range <start,end> is mapped
* with type. * with type.
*/ */
int int e820_any_mapped(u64 start, u64 end, unsigned type)
e820_any_mapped(u64 start, u64 end, unsigned type)
{ {
int i; int i;
@ -94,10 +86,10 @@ e820_any_mapped(u64 start, u64 end, unsigned type)
EXPORT_SYMBOL_GPL(e820_any_mapped); EXPORT_SYMBOL_GPL(e820_any_mapped);
/* /*
* This function checks if the entire range <start,end> is mapped with type. * This function checks if the entire <start,end> range is mapped with 'type'.
* *
* Note: this function only works correct if the e820 table is sorted and * Note: this function only works correctly once the E820 table is sorted and
* not-overlapping, which is the case * not-overlapping (at least for the range specified), which is the case normally.
*/ */
int __init e820_all_mapped(u64 start, u64 end, unsigned type) int __init e820_all_mapped(u64 start, u64 end, unsigned type)
{ {
@ -108,18 +100,21 @@ int __init e820_all_mapped(u64 start, u64 end, unsigned type)
if (type && ei->type != type) if (type && ei->type != type)
continue; continue;
/* is the region (part) in overlap with the current region ?*/
/* Is the region (part) in overlap with the current region? */
if (ei->addr >= end || ei->addr + ei->size <= start) if (ei->addr >= end || ei->addr + ei->size <= start)
continue; continue;
/* if the region is at the beginning of <start,end> we move /*
* start to the end of the region since it's ok until there * If the region is at the beginning of <start,end> we move
* 'start' to the end of the region since it's ok until there
*/ */
if (ei->addr <= start) if (ei->addr <= start)
start = ei->addr + ei->size; start = ei->addr + ei->size;
/* /*
* if start is now at or beyond end, we're done, full * If 'start' is now at or beyond 'end', we're done, full
* coverage * coverage of the desired range exists:
*/ */
if (start >= end) if (start >= end)
return 1; return 1;
@ -128,16 +123,14 @@ int __init e820_all_mapped(u64 start, u64 end, unsigned type)
} }
/* /*
* Add a memory region to the kernel e820 map. * Add a memory region to the kernel E820 map.
*/ */
static void __init __e820_add_region(struct e820_table *table, u64 start, u64 size, int type) static void __init __e820_add_region(struct e820_table *table, u64 start, u64 size, int type)
{ {
int x = table->nr_entries; int x = table->nr_entries;
if (x >= ARRAY_SIZE(table->entries)) { if (x >= ARRAY_SIZE(table->entries)) {
printk(KERN_ERR "e820: too many entries; ignoring [mem %#010llx-%#010llx]\n", printk(KERN_ERR "e820: too many entries; ignoring [mem %#010llx-%#010llx]\n", start, start + size - 1);
(unsigned long long) start,
(unsigned long long) (start + size - 1));
return; return;
} }
@ -155,29 +148,15 @@ void __init e820_add_region(u64 start, u64 size, int type)
static void __init e820_print_type(u32 type) static void __init e820_print_type(u32 type)
{ {
switch (type) { switch (type) {
case E820_RAM: case E820_RAM: /* Fall through: */
case E820_RESERVED_KERN: case E820_RESERVED_KERN: printk(KERN_CONT "usable"); break;
printk(KERN_CONT "usable"); case E820_RESERVED: printk(KERN_CONT "reserved"); break;
break; case E820_ACPI: printk(KERN_CONT "ACPI data"); break;
case E820_RESERVED: case E820_NVS: printk(KERN_CONT "ACPI NVS"); break;
printk(KERN_CONT "reserved"); case E820_UNUSABLE: printk(KERN_CONT "unusable"); break;
break; case E820_PMEM: /* Fall through: */
case E820_ACPI: case E820_PRAM: printk(KERN_CONT "persistent (type %u)", type); break;
printk(KERN_CONT "ACPI data"); default: printk(KERN_CONT "type %u", type); break;
break;
case E820_NVS:
printk(KERN_CONT "ACPI NVS");
break;
case E820_UNUSABLE:
printk(KERN_CONT "unusable");
break;
case E820_PMEM:
case E820_PRAM:
printk(KERN_CONT "persistent (type %u)", type);
break;
default:
printk(KERN_CONT "type %u", type);
break;
} }
} }
@ -187,26 +166,26 @@ void __init e820_print_map(char *who)
for (i = 0; i < e820_table->nr_entries; i++) { for (i = 0; i < e820_table->nr_entries; i++) {
printk(KERN_INFO "%s: [mem %#018Lx-%#018Lx] ", who, printk(KERN_INFO "%s: [mem %#018Lx-%#018Lx] ", who,
(unsigned long long) e820_table->entries[i].addr, e820_table->entries[i].addr,
(unsigned long long) e820_table->entries[i].addr + e820_table->entries[i].size - 1);
(e820_table->entries[i].addr + e820_table->entries[i].size - 1));
e820_print_type(e820_table->entries[i].type); e820_print_type(e820_table->entries[i].type);
printk(KERN_CONT "\n"); printk(KERN_CONT "\n");
} }
} }
/* /*
* Sanitize the BIOS e820 map. * Sanitize the BIOS E820 map.
* *
* Some e820 responses include overlapping entries. The following * Some E820 responses include overlapping entries. The following
* replaces the original e820 map with a new one, removing overlaps, * replaces the original E820 map with a new one, removing overlaps,
* and resolving conflicting memory types in favor of highest * and resolving conflicting memory types in favor of highest
* numbered type. * numbered type.
* *
* The input parameter biosmap points to an array of 'struct * The input parameter biosmap points to an array of 'struct
* e820_entry' which on entry has elements in the range [0, *pnr_map) * e820_entry' which on entry has elements in the range [0, *pnr_map)
* valid, and which has space for up to max_nr_map entries. * valid, and which has space for up to max_nr_map entries.
* On return, the resulting sanitized e820 map entries will be in * On return, the resulting sanitized E820 map entries will be in
* overwritten in the same location, starting at biosmap. * overwritten in the same location, starting at biosmap.
* *
* The integer pointed to by pnr_map must be valid on entry (the * The integer pointed to by pnr_map must be valid on entry (the
@ -257,8 +236,10 @@ void __init e820_print_map(char *who)
* ______________________4_ * ______________________4_
*/ */
struct change_member { struct change_member {
struct e820_entry *pbios; /* pointer to original bios entry */ /* Pointer to the original BIOS entry: */
unsigned long long addr; /* address for this change point */ struct e820_entry *pbios;
/* Address for this change point: */
unsigned long long addr;
}; };
static int __init cpcompare(const void *a, const void *b) static int __init cpcompare(const void *a, const void *b)
@ -268,7 +249,7 @@ static int __init cpcompare(const void *a, const void *b)
/* /*
* Inputs are pointers to two elements of change_point[]. If their * Inputs are pointers to two elements of change_point[]. If their
* addresses are unequal, their difference dominates. If the addresses * addresses are not equal, their difference dominates. If the addresses
* are equal, then consider one that represents the end of its region * are equal, then consider one that represents the end of its region
* to be greater than one that does not. * to be greater than one that does not.
*/ */
@ -278,8 +259,7 @@ static int __init cpcompare(const void *a, const void *b)
return (ap->addr != ap->pbios->addr) - (bp->addr != bp->pbios->addr); return (ap->addr != ap->pbios->addr) - (bp->addr != bp->pbios->addr);
} }
int __init sanitize_e820_table(struct e820_entry *biosmap, int max_nr_map, int __init sanitize_e820_table(struct e820_entry *biosmap, int max_nr_map, u32 *pnr_map)
u32 *pnr_map)
{ {
static struct change_member change_point_list[2*E820_X_MAX] __initdata; static struct change_member change_point_list[2*E820_X_MAX] __initdata;
static struct change_member *change_point[2*E820_X_MAX] __initdata; static struct change_member *change_point[2*E820_X_MAX] __initdata;
@ -293,112 +273,96 @@ int __init sanitize_e820_table(struct e820_entry *biosmap, int max_nr_map,
int old_nr, new_nr, chg_nr; int old_nr, new_nr, chg_nr;
int i; int i;
/* if there's only one memory region, don't bother */ /* If there's only one memory region, don't bother: */
if (*pnr_map < 2) if (*pnr_map < 2)
return -1; return -1;
old_nr = *pnr_map; old_nr = *pnr_map;
BUG_ON(old_nr > max_nr_map); BUG_ON(old_nr > max_nr_map);
/* bail out if we find any unreasonable addresses in bios map */ /* Bail out if we find any unreasonable addresses in the BIOS map: */
for (i = 0; i < old_nr; i++) for (i = 0; i < old_nr; i++) {
if (biosmap[i].addr + biosmap[i].size < biosmap[i].addr) if (biosmap[i].addr + biosmap[i].size < biosmap[i].addr)
return -1; return -1;
}
/* create pointers for initial change-point information (for sorting) */ /* Create pointers for initial change-point information (for sorting): */
for (i = 0; i < 2 * old_nr; i++) for (i = 0; i < 2 * old_nr; i++)
change_point[i] = &change_point_list[i]; change_point[i] = &change_point_list[i];
/* record all known change-points (starting and ending addresses), /*
omitting those that are for empty memory regions */ * Record all known change-points (starting and ending addresses),
* omitting empty memory regions:
*/
chgidx = 0; chgidx = 0;
for (i = 0; i < old_nr; i++) { for (i = 0; i < old_nr; i++) {
if (biosmap[i].size != 0) { if (biosmap[i].size != 0) {
change_point[chgidx]->addr = biosmap[i].addr; change_point[chgidx]->addr = biosmap[i].addr;
change_point[chgidx++]->pbios = &biosmap[i]; change_point[chgidx++]->pbios = &biosmap[i];
change_point[chgidx]->addr = biosmap[i].addr + change_point[chgidx]->addr = biosmap[i].addr + biosmap[i].size;
biosmap[i].size; change_point[chgidx++]->pbios = &biosmap[i];
change_point[chgidx++]->pbios = &biosmap[i];
} }
} }
chg_nr = chgidx; chg_nr = chgidx;
/* sort change-point list by memory addresses (low -> high) */ /* Sort change-point list by memory addresses (low -> high): */
sort(change_point, chg_nr, sizeof *change_point, cpcompare, NULL); sort(change_point, chg_nr, sizeof *change_point, cpcompare, NULL);
/* create a new bios memory map, removing overlaps */ /* Create a new BIOS memory map, removing overlaps: */
overlap_entries = 0; /* number of entries in the overlap table */ overlap_entries = 0; /* Number of entries in the overlap table */
new_bios_entry = 0; /* index for creating new bios map entries */ new_bios_entry = 0; /* Index for creating new bios map entries */
last_type = 0; /* start with undefined memory type */ last_type = 0; /* Start with undefined memory type */
last_addr = 0; /* start with 0 as last starting address */ last_addr = 0; /* Start with 0 as last starting address */
/* loop through change-points, determining affect on the new bios map */ /* Loop through change-points, determining effect on the new BIOS map: */
for (chgidx = 0; chgidx < chg_nr; chgidx++) { for (chgidx = 0; chgidx < chg_nr; chgidx++) {
/* keep track of all overlapping bios entries */ /* Keep track of all overlapping BIOS entries */
if (change_point[chgidx]->addr == if (change_point[chgidx]->addr == change_point[chgidx]->pbios->addr) {
change_point[chgidx]->pbios->addr) { /* Add map entry to overlap list (> 1 entry implies an overlap) */
/* overlap_list[overlap_entries++] = change_point[chgidx]->pbios;
* add map entry to overlap list (> 1 entry
* implies an overlap)
*/
overlap_list[overlap_entries++] =
change_point[chgidx]->pbios;
} else { } else {
/* /* Remove entry from list (order independent, so swap with last): */
* remove entry from list (order independent,
* so swap with last)
*/
for (i = 0; i < overlap_entries; i++) { for (i = 0; i < overlap_entries; i++) {
if (overlap_list[i] == if (overlap_list[i] == change_point[chgidx]->pbios)
change_point[chgidx]->pbios) overlap_list[i] = overlap_list[overlap_entries-1];
overlap_list[i] =
overlap_list[overlap_entries-1];
} }
overlap_entries--; overlap_entries--;
} }
/* /*
* if there are overlapping entries, decide which * If there are overlapping entries, decide which
* "type" to use (larger value takes precedence -- * "type" to use (larger value takes precedence --
* 1=usable, 2,3,4,4+=unusable) * 1=usable, 2,3,4,4+=unusable)
*/ */
current_type = 0; current_type = 0;
for (i = 0; i < overlap_entries; i++) for (i = 0; i < overlap_entries; i++) {
if (overlap_list[i]->type > current_type) if (overlap_list[i]->type > current_type)
current_type = overlap_list[i]->type; current_type = overlap_list[i]->type;
/* }
* continue building up new bios map based on this
* information /* Continue building up new BIOS map based on this information: */
*/
if (current_type != last_type || current_type == E820_PRAM) { if (current_type != last_type || current_type == E820_PRAM) {
if (last_type != 0) { if (last_type != 0) {
new_bios[new_bios_entry].size = new_bios[new_bios_entry].size = change_point[chgidx]->addr - last_addr;
change_point[chgidx]->addr - last_addr; /* Move forward only if the new size was non-zero: */
/*
* move forward only if the new size
* was non-zero
*/
if (new_bios[new_bios_entry].size != 0) if (new_bios[new_bios_entry].size != 0)
/* /* No more space left for new BIOS entries? */
* no more space left for new
* bios entries ?
*/
if (++new_bios_entry >= max_nr_map) if (++new_bios_entry >= max_nr_map)
break; break;
} }
if (current_type != 0) { if (current_type != 0) {
new_bios[new_bios_entry].addr = new_bios[new_bios_entry].addr = change_point[chgidx]->addr;
change_point[chgidx]->addr;
new_bios[new_bios_entry].type = current_type; new_bios[new_bios_entry].type = current_type;
last_addr = change_point[chgidx]->addr; last_addr = change_point[chgidx]->addr;
} }
last_type = current_type; last_type = current_type;
} }
} }
/* retain count for new bios entries */
/* Retain count for new BIOS entries: */
new_nr = new_bios_entry; new_nr = new_bios_entry;
/* copy new bios mapping into original location */ /* Copy new BIOS mapping into the original location: */
memcpy(biosmap, new_bios, new_nr * sizeof(struct e820_entry)); memcpy(biosmap, new_bios, new_nr*sizeof(struct e820_entry));
*pnr_map = new_nr; *pnr_map = new_nr;
return 0; return 0;
@ -412,7 +376,7 @@ static int __init __append_e820_table(struct e820_entry *biosmap, int nr_map)
u64 end = start + size - 1; u64 end = start + size - 1;
u32 type = biosmap->type; u32 type = biosmap->type;
/* Overflow in 64 bits? Ignore the memory map. */ /* Ignore the entry on 64-bit overflow: */
if (start > end && likely(size)) if (start > end && likely(size))
return -1; return -1;
@ -425,7 +389,7 @@ static int __init __append_e820_table(struct e820_entry *biosmap, int nr_map)
} }
/* /*
* Copy the BIOS e820 map into a safe place. * Copy the BIOS E820 map into a safe place.
* *
* Sanity-check it while we're at it.. * Sanity-check it while we're at it..
* *
@ -442,9 +406,8 @@ static int __init append_e820_table(struct e820_entry *biosmap, int nr_map)
return __append_e820_table(biosmap, nr_map); return __append_e820_table(biosmap, nr_map);
} }
static u64 __init __e820_update_range(struct e820_table *table, u64 start, static u64 __init
u64 size, unsigned old_type, __e820_update_range(struct e820_table *table, u64 start, u64 size, unsigned old_type, unsigned new_type)
unsigned new_type)
{ {
u64 end; u64 end;
unsigned int i; unsigned int i;
@ -456,8 +419,7 @@ static u64 __init __e820_update_range(struct e820_table *table, u64 start,
size = ULLONG_MAX - start; size = ULLONG_MAX - start;
end = start + size; end = start + size;
printk(KERN_DEBUG "e820: update [mem %#010Lx-%#010Lx] ", printk(KERN_DEBUG "e820: update [mem %#010Lx-%#010Lx] ", start, end - 1);
(unsigned long long) start, (unsigned long long) (end - 1));
e820_print_type(old_type); e820_print_type(old_type);
printk(KERN_CONT " ==> "); printk(KERN_CONT " ==> ");
e820_print_type(new_type); e820_print_type(new_type);
@ -472,14 +434,15 @@ static u64 __init __e820_update_range(struct e820_table *table, u64 start,
continue; continue;
ei_end = ei->addr + ei->size; ei_end = ei->addr + ei->size;
/* totally covered by new range? */
/* Completely covered by new range? */
if (ei->addr >= start && ei_end <= end) { if (ei->addr >= start && ei_end <= end) {
ei->type = new_type; ei->type = new_type;
real_updated_size += ei->size; real_updated_size += ei->size;
continue; continue;
} }
/* new range is totally covered? */ /* New range is completely covered? */
if (ei->addr < start && ei_end > end) { if (ei->addr < start && ei_end > end) {
__e820_add_region(table, start, size, new_type); __e820_add_region(table, start, size, new_type);
__e820_add_region(table, end, ei_end - end, ei->type); __e820_add_region(table, end, ei_end - end, ei->type);
@ -488,45 +451,41 @@ static u64 __init __e820_update_range(struct e820_table *table, u64 start,
continue; continue;
} }
/* partially covered */ /* Partially covered: */
final_start = max(start, ei->addr); final_start = max(start, ei->addr);
final_end = min(end, ei_end); final_end = min(end, ei_end);
if (final_start >= final_end) if (final_start >= final_end)
continue; continue;
__e820_add_region(table, final_start, final_end - final_start, __e820_add_region(table, final_start, final_end - final_start, new_type);
new_type);
real_updated_size += final_end - final_start; real_updated_size += final_end - final_start;
/* /*
* left range could be head or tail, so need to update * Left range could be head or tail, so need to update
* size at first. * its size first:
*/ */
ei->size -= final_end - final_start; ei->size -= final_end - final_start;
if (ei->addr < final_start) if (ei->addr < final_start)
continue; continue;
ei->addr = final_end; ei->addr = final_end;
} }
return real_updated_size; return real_updated_size;
} }
u64 __init e820_update_range(u64 start, u64 size, unsigned old_type, u64 __init e820_update_range(u64 start, u64 size, unsigned old_type, unsigned new_type)
unsigned new_type)
{ {
return __e820_update_range(e820_table, start, size, old_type, new_type); return __e820_update_range(e820_table, start, size, old_type, new_type);
} }
static u64 __init e820_update_range_firmware(u64 start, u64 size, static u64 __init e820_update_range_firmware(u64 start, u64 size, unsigned old_type, unsigned new_type)
unsigned old_type, unsigned new_type)
{ {
return __e820_update_range(e820_table_firmware, start, size, old_type, return __e820_update_range(e820_table_firmware, start, size, old_type, new_type);
new_type);
} }
/* make e820 not cover the range */ /* Remove a range of memory from the E820 table: */
u64 __init e820_remove_range(u64 start, u64 size, unsigned old_type, u64 __init e820_remove_range(u64 start, u64 size, unsigned old_type, int checktype)
int checktype)
{ {
int i; int i;
u64 end; u64 end;
@ -536,8 +495,7 @@ u64 __init e820_remove_range(u64 start, u64 size, unsigned old_type,
size = ULLONG_MAX - start; size = ULLONG_MAX - start;
end = start + size; end = start + size;
printk(KERN_DEBUG "e820: remove [mem %#010Lx-%#010Lx] ", printk(KERN_DEBUG "e820: remove [mem %#010Lx-%#010Lx] ", start, end - 1);
(unsigned long long) start, (unsigned long long) (end - 1));
if (checktype) if (checktype)
e820_print_type(old_type); e820_print_type(old_type);
printk(KERN_CONT "\n"); printk(KERN_CONT "\n");
@ -551,14 +509,15 @@ u64 __init e820_remove_range(u64 start, u64 size, unsigned old_type,
continue; continue;
ei_end = ei->addr + ei->size; ei_end = ei->addr + ei->size;
/* totally covered? */
/* Completely covered? */
if (ei->addr >= start && ei_end <= end) { if (ei->addr >= start && ei_end <= end) {
real_removed_size += ei->size; real_removed_size += ei->size;
memset(ei, 0, sizeof(struct e820_entry)); memset(ei, 0, sizeof(struct e820_entry));
continue; continue;
} }
/* new range is totally covered? */ /* Is the new range completely covered? */
if (ei->addr < start && ei_end > end) { if (ei->addr < start && ei_end > end) {
e820_add_region(end, ei_end - end, ei->type); e820_add_region(end, ei_end - end, ei->type);
ei->size = start - ei->addr; ei->size = start - ei->addr;
@ -566,20 +525,22 @@ u64 __init e820_remove_range(u64 start, u64 size, unsigned old_type,
continue; continue;
} }
/* partially covered */ /* Partially covered: */
final_start = max(start, ei->addr); final_start = max(start, ei->addr);
final_end = min(end, ei_end); final_end = min(end, ei_end);
if (final_start >= final_end) if (final_start >= final_end)
continue; continue;
real_removed_size += final_end - final_start; real_removed_size += final_end - final_start;
/* /*
* left range could be head or tail, so need to update * Left range could be head or tail, so need to update
* size at first. * the size first:
*/ */
ei->size -= final_end - final_start; ei->size -= final_end - final_start;
if (ei->addr < final_start) if (ei->addr < final_start)
continue; continue;
ei->addr = final_end; ei->addr = final_end;
} }
return real_removed_size; return real_removed_size;
@ -589,19 +550,22 @@ void __init update_e820(void)
{ {
if (sanitize_e820_table(e820_table->entries, ARRAY_SIZE(e820_table->entries), &e820_table->nr_entries)) if (sanitize_e820_table(e820_table->entries, ARRAY_SIZE(e820_table->entries), &e820_table->nr_entries))
return; return;
printk(KERN_INFO "e820: modified physical RAM map:\n"); printk(KERN_INFO "e820: modified physical RAM map:\n");
e820_print_map("modified"); e820_print_map("modified");
} }
static void __init update_e820_table_firmware(void) static void __init update_e820_table_firmware(void)
{ {
sanitize_e820_table(e820_table_firmware->entries, ARRAY_SIZE(e820_table_firmware->entries), &e820_table_firmware->nr_entries); sanitize_e820_table(e820_table_firmware->entries, ARRAY_SIZE(e820_table_firmware->entries), &e820_table_firmware->nr_entries);
} }
#define MAX_GAP_END 0x100000000ull #define MAX_GAP_END 0x100000000ull
/* /*
* Search for a gap in the e820 memory space from 0 to MAX_GAP_END. * Search for a gap in the E820 memory space from 0 to MAX_GAP_END (4GB).
*/ */
static int __init e820_search_gap(unsigned long *gapstart, static int __init e820_search_gap(unsigned long *gapstart, unsigned long *gapsize)
unsigned long *gapsize)
{ {
unsigned long long last = MAX_GAP_END; unsigned long long last = MAX_GAP_END;
int i = e820_table->nr_entries; int i = e820_table->nr_entries;
@ -613,7 +577,7 @@ static int __init e820_search_gap(unsigned long *gapstart,
/* /*
* Since "last" is at most 4GB, we know we'll * Since "last" is at most 4GB, we know we'll
* fit in 32 bits if this condition is true * fit in 32 bits if this condition is true:
*/ */
if (last > end) { if (last > end) {
unsigned long gap = last - end; unsigned long gap = last - end;
@ -631,9 +595,11 @@ static int __init e820_search_gap(unsigned long *gapstart,
} }
/* /*
* Search for the biggest gap in the low 32 bits of the e820 * Search for the biggest gap in the low 32 bits of the E820
* memory space. We pass this space to PCI to assign MMIO resources * memory space. We pass this space to the PCI subsystem, so
* for hotplug or unconfigured devices in. * that it can assign MMIO resources for hotplug or
* unconfigured devices in.
*
* Hopefully the BIOS let enough space left. * Hopefully the BIOS let enough space left.
*/ */
__init void e820_setup_gap(void) __init void e820_setup_gap(void)
@ -648,8 +614,8 @@ __init void e820_setup_gap(void)
#ifdef CONFIG_X86_64 #ifdef CONFIG_X86_64
gapstart = (max_pfn << PAGE_SHIFT) + 1024*1024; gapstart = (max_pfn << PAGE_SHIFT) + 1024*1024;
printk(KERN_ERR printk(KERN_ERR
"e820: cannot find a gap in the 32bit address range\n" "e820: Cannot find an available gap in the 32-bit address range\n"
"e820: PCI devices with unassigned 32bit BARs may break!\n"); "e820: PCI devices with unassigned 32-bit BARs may not work!\n");
#else #else
gapstart = 0x10000000; gapstart = 0x10000000;
#endif #endif
@ -660,43 +626,44 @@ __init void e820_setup_gap(void)
*/ */
pci_mem_start = gapstart; pci_mem_start = gapstart;
printk(KERN_INFO printk(KERN_INFO "e820: [mem %#010lx-%#010lx] available for PCI devices\n", gapstart, gapstart + gapsize - 1);
"e820: [mem %#010lx-%#010lx] available for PCI devices\n",
gapstart, gapstart + gapsize - 1);
} }
/* /*
* Called late during init, in free_initmem(). * Called late during init, in free_initmem().
* *
* Initial e820 and e820_table_firmware are largish __initdata arrays. * Initial e820_table and e820_table_firmware are largish __initdata arrays.
* Copy them to (usually much smaller) dynamically allocated area. *
* This is done after all tweaks we ever do to them: * Copy them to a (usually much smaller) dynamically allocated area that is
* all functions which modify them are __init functions, * sized precisely after the number of e820 entries.
* they won't exist after this point. *
* This is done after we've performed all the fixes and tweaks to the tables.
* All functions which modify them are __init functions, which won't exist
* after free_initmem().
*/ */
__init void e820_reallocate_tables(void) __init void e820_reallocate_tables(void)
{ {
struct e820_table *n; struct e820_table *n;
int size; int size;
size = offsetof(struct e820_table, entries) + sizeof(struct e820_entry) * e820_table->nr_entries; size = offsetof(struct e820_table, entries) + sizeof(struct e820_entry)*e820_table->nr_entries;
n = kmalloc(size, GFP_KERNEL); n = kmalloc(size, GFP_KERNEL);
BUG_ON(!n); BUG_ON(!n);
memcpy(n, e820_table, size); memcpy(n, e820_table, size);
e820_table = n; e820_table = n;
size = offsetof(struct e820_table, entries) + sizeof(struct e820_entry) * e820_table_firmware->nr_entries; size = offsetof(struct e820_table, entries) + sizeof(struct e820_entry)*e820_table_firmware->nr_entries;
n = kmalloc(size, GFP_KERNEL); n = kmalloc(size, GFP_KERNEL);
BUG_ON(!n); BUG_ON(!n);
memcpy(n, e820_table_firmware, size); memcpy(n, e820_table_firmware, size);
e820_table_firmware = n; e820_table_firmware = n;
} }
/** /*
* Because of the size limitation of struct boot_params, only first * Because of the small fixed size of struct boot_params, only the first
* 128 E820 memory entries are passed to kernel via * 128 E820 memory entries are passed to the kernel via boot_params.e820_table,
* boot_params.e820_table, others are passed via SETUP_E820_EXT node of * the remaining (if any) entries are passed via the SETUP_E820_EXT node of
* linked list of struct setup_data, which is parsed here. * struct setup_data, which is parsed here.
*/ */
void __init parse_e820_ext(u64 phys_addr, u32 data_len) void __init parse_e820_ext(u64 phys_addr, u32 data_len)
{ {
@ -707,8 +674,10 @@ void __init parse_e820_ext(u64 phys_addr, u32 data_len)
sdata = early_memremap(phys_addr, data_len); sdata = early_memremap(phys_addr, data_len);
entries = sdata->len / sizeof(struct e820_entry); entries = sdata->len / sizeof(struct e820_entry);
extmap = (struct e820_entry *)(sdata->data); extmap = (struct e820_entry *)(sdata->data);
__append_e820_table(extmap, entries); __append_e820_table(extmap, entries);
sanitize_e820_table(e820_table->entries, ARRAY_SIZE(e820_table->entries), &e820_table->nr_entries); sanitize_e820_table(e820_table->entries, ARRAY_SIZE(e820_table->entries), &e820_table->nr_entries);
early_memunmap(sdata, data_len); early_memunmap(sdata, data_len);
printk(KERN_INFO "e820: extended physical RAM map:\n"); printk(KERN_INFO "e820: extended physical RAM map:\n");
e820_print_map("extended"); e820_print_map("extended");
@ -716,10 +685,10 @@ void __init parse_e820_ext(u64 phys_addr, u32 data_len)
/** /**
* Find the ranges of physical addresses that do not correspond to * Find the ranges of physical addresses that do not correspond to
* e820 RAM areas and mark the corresponding pages as nosave for * E820 RAM areas and mark the corresponding pages as 'nosave' for
* hibernation (32 bit) or software suspend and suspend to RAM (64 bit). * hibernation (32-bit) or software suspend and suspend to RAM (64-bit).
* *
* This function requires the e820 map to be sorted and without any * This function requires the E820 map to be sorted and without any
* overlapping entries. * overlapping entries.
*/ */
void __init e820_mark_nosave_regions(unsigned long limit_pfn) void __init e820_mark_nosave_regions(unsigned long limit_pfn)
@ -744,9 +713,9 @@ void __init e820_mark_nosave_regions(unsigned long limit_pfn)
} }
#ifdef CONFIG_ACPI #ifdef CONFIG_ACPI
/** /*
* Mark ACPI NVS memory region, so that we can save/restore it during * Register ACPI NVS memory regions, so that we can save/restore them during
* hibernation and the subsequent resume. * hibernation and the subsequent resume:
*/ */
static int __init e820_mark_nvs_memory(void) static int __init e820_mark_nvs_memory(void)
{ {
@ -828,6 +797,7 @@ static unsigned long __init e820_end_pfn(unsigned long limit_pfn, unsigned type)
last_pfn, max_arch_pfn); last_pfn, max_arch_pfn);
return last_pfn; return last_pfn;
} }
unsigned long __init e820_end_of_ram_pfn(void) unsigned long __init e820_end_of_ram_pfn(void)
{ {
return e820_end_pfn(MAX_ARCH_PFN, E820_RAM); return e820_end_pfn(MAX_ARCH_PFN, E820_RAM);
@ -846,7 +816,7 @@ static void __init early_panic(char *msg)
static int userdef __initdata; static int userdef __initdata;
/* "mem=nopentium" disables the 4MB page tables. */ /* The "mem=nopentium" boot option disables 4MB page tables on 32-bit kernels: */
static int __init parse_memopt(char *p) static int __init parse_memopt(char *p)
{ {
u64 mem_size; u64 mem_size;
@ -866,9 +836,11 @@ static int __init parse_memopt(char *p)
userdef = 1; userdef = 1;
mem_size = memparse(p, &p); mem_size = memparse(p, &p);
/* don't remove all of memory when handling "mem={invalid}" param */
/* Don't remove all memory when getting "mem={invalid}" parameter: */
if (mem_size == 0) if (mem_size == 0)
return -EINVAL; return -EINVAL;
e820_remove_range(mem_size, ULLONG_MAX - mem_size, E820_RAM, 1); e820_remove_range(mem_size, ULLONG_MAX - mem_size, E820_RAM, 1);
return 0; return 0;
@ -887,7 +859,7 @@ static int __init parse_memmap_one(char *p)
#ifdef CONFIG_CRASH_DUMP #ifdef CONFIG_CRASH_DUMP
/* /*
* If we are doing a crash dump, we still need to know * If we are doing a crash dump, we still need to know
* the real mem size before original memory map is * the real memory size before the original memory map is
* reset. * reset.
*/ */
saved_max_pfn = e820_end_of_ram_pfn(); saved_max_pfn = e820_end_of_ram_pfn();
@ -915,11 +887,13 @@ static int __init parse_memmap_one(char *p)
} else if (*p == '!') { } else if (*p == '!') {
start_at = memparse(p+1, &p); start_at = memparse(p+1, &p);
e820_add_region(start_at, mem_size, E820_PRAM); e820_add_region(start_at, mem_size, E820_PRAM);
} else } else {
e820_remove_range(mem_size, ULLONG_MAX - mem_size, E820_RAM, 1); e820_remove_range(mem_size, ULLONG_MAX - mem_size, E820_RAM, 1);
}
return *p == '\0' ? 0 : -EINVAL; return *p == '\0' ? 0 : -EINVAL;
} }
static int __init parse_memmap_opt(char *str) static int __init parse_memmap_opt(char *str)
{ {
while (str) { while (str) {
@ -939,8 +913,7 @@ early_param("memmap", parse_memmap_opt);
void __init finish_e820_parsing(void) void __init finish_e820_parsing(void)
{ {
if (userdef) { if (userdef) {
if (sanitize_e820_table(e820_table->entries, ARRAY_SIZE(e820_table->entries), if (sanitize_e820_table(e820_table->entries, ARRAY_SIZE(e820_table->entries), &e820_table->nr_entries) < 0)
&e820_table->nr_entries) < 0)
early_panic("Invalid user supplied memory map"); early_panic("Invalid user supplied memory map");
printk(KERN_INFO "e820: user-defined physical RAM map:\n"); printk(KERN_INFO "e820: user-defined physical RAM map:\n");
@ -951,49 +924,42 @@ void __init finish_e820_parsing(void)
static const char *__init e820_type_to_string(int e820_type) static const char *__init e820_type_to_string(int e820_type)
{ {
switch (e820_type) { switch (e820_type) {
case E820_RESERVED_KERN: case E820_RESERVED_KERN: /* Fall-through: */
case E820_RAM: return "System RAM"; case E820_RAM: return "System RAM";
case E820_ACPI: return "ACPI Tables"; case E820_ACPI: return "ACPI Tables";
case E820_NVS: return "ACPI Non-volatile Storage"; case E820_NVS: return "ACPI Non-volatile Storage";
case E820_UNUSABLE: return "Unusable memory"; case E820_UNUSABLE: return "Unusable memory";
case E820_PRAM: return "Persistent Memory (legacy)"; case E820_PRAM: return "Persistent Memory (legacy)";
case E820_PMEM: return "Persistent Memory"; case E820_PMEM: return "Persistent Memory";
default: return "reserved"; default: return "Reserved";
} }
} }
static unsigned long __init e820_type_to_iomem_type(int e820_type) static unsigned long __init e820_type_to_iomem_type(int e820_type)
{ {
switch (e820_type) { switch (e820_type) {
case E820_RESERVED_KERN: case E820_RESERVED_KERN: /* Fall-through: */
case E820_RAM: case E820_RAM: return IORESOURCE_SYSTEM_RAM;
return IORESOURCE_SYSTEM_RAM; case E820_ACPI: /* Fall-through: */
case E820_ACPI: case E820_NVS: /* Fall-through: */
case E820_NVS: case E820_UNUSABLE: /* Fall-through: */
case E820_UNUSABLE: case E820_PRAM: /* Fall-through: */
case E820_PRAM: case E820_PMEM: /* Fall-through: */
case E820_PMEM: default: return IORESOURCE_MEM;
default:
return IORESOURCE_MEM;
} }
} }
static unsigned long __init e820_type_to_iores_desc(int e820_type) static unsigned long __init e820_type_to_iores_desc(int e820_type)
{ {
switch (e820_type) { switch (e820_type) {
case E820_ACPI: case E820_ACPI: return IORES_DESC_ACPI_TABLES;
return IORES_DESC_ACPI_TABLES; case E820_NVS: return IORES_DESC_ACPI_NV_STORAGE;
case E820_NVS: case E820_PMEM: return IORES_DESC_PERSISTENT_MEMORY;
return IORES_DESC_ACPI_NV_STORAGE; case E820_PRAM: return IORES_DESC_PERSISTENT_MEMORY_LEGACY;
case E820_PMEM: case E820_RESERVED_KERN: /* Fall-through: */
return IORES_DESC_PERSISTENT_MEMORY; case E820_RAM: /* Fall-through: */
case E820_PRAM: case E820_UNUSABLE: /* Fall-through: */
return IORES_DESC_PERSISTENT_MEMORY_LEGACY; default: return IORES_DESC_NONE;
case E820_RESERVED_KERN:
case E820_RAM:
case E820_UNUSABLE:
default:
return IORES_DESC_NONE;
} }
} }
@ -1018,9 +984,11 @@ static bool __init do_mark_busy(u32 type, struct resource *res)
} }
/* /*
* Mark e820 reserved areas as busy for the resource manager. * Mark E820 reserved areas as busy for the resource manager:
*/ */
static struct resource __initdata *e820_res; static struct resource __initdata *e820_res;
void __init e820_reserve_resources(void) void __init e820_reserve_resources(void)
{ {
int i; int i;
@ -1056,9 +1024,8 @@ void __init e820_reserve_resources(void)
for (i = 0; i < e820_table_firmware->nr_entries; i++) { for (i = 0; i < e820_table_firmware->nr_entries; i++) {
struct e820_entry *entry = &e820_table_firmware->entries[i]; struct e820_entry *entry = &e820_table_firmware->entries[i];
firmware_map_add_early(entry->addr,
entry->addr + entry->size, firmware_map_add_early(entry->addr, entry->addr + entry->size, e820_type_to_string(entry->type));
e820_type_to_string(entry->type));
} }
} }
@ -1094,7 +1061,7 @@ void __init e820_reserve_resources_late(void)
} }
/* /*
* Try to bump up RAM regions to reasonable boundaries to * Try to bump up RAM regions to reasonable boundaries, to
* avoid stolen RAM: * avoid stolen RAM:
*/ */
for (i = 0; i < e820_table->nr_entries; i++) { for (i = 0; i < e820_table->nr_entries; i++) {
@ -1103,24 +1070,27 @@ void __init e820_reserve_resources_late(void)
if (entry->type != E820_RAM) if (entry->type != E820_RAM)
continue; continue;
start = entry->addr + entry->size; start = entry->addr + entry->size;
end = round_up(start, ram_alignment(start)) - 1; end = round_up(start, ram_alignment(start)) - 1;
if (end > MAX_RESOURCE_SIZE) if (end > MAX_RESOURCE_SIZE)
end = MAX_RESOURCE_SIZE; end = MAX_RESOURCE_SIZE;
if (start >= end) if (start >= end)
continue; continue;
printk(KERN_DEBUG
"e820: reserve RAM buffer [mem %#010llx-%#010llx]\n", printk(KERN_DEBUG "e820: reserve RAM buffer [mem %#010llx-%#010llx]\n", start, end);
start, end); reserve_region_with_split(&iomem_resource, start, end, "RAM buffer");
reserve_region_with_split(&iomem_resource, start, end,
"RAM buffer");
} }
} }
/*
* Pass the firmware (bootloader) E820 map to the kernel and process it:
*/
char *__init e820__memory_setup_default(void) char *__init e820__memory_setup_default(void)
{ {
char *who = "BIOS-e820"; char *who = "BIOS-e820";
u32 new_nr; u32 new_nr;
/* /*
* Try to copy the BIOS-supplied E820-map. * Try to copy the BIOS-supplied E820-map.
* *
@ -1128,17 +1098,14 @@ char *__init e820__memory_setup_default(void)
* the next section from 1mb->appropriate_mem_k * the next section from 1mb->appropriate_mem_k
*/ */
new_nr = boot_params.e820_entries; new_nr = boot_params.e820_entries;
sanitize_e820_table(boot_params.e820_table, sanitize_e820_table(boot_params.e820_table, ARRAY_SIZE(boot_params.e820_table), &new_nr);
ARRAY_SIZE(boot_params.e820_table),
&new_nr);
boot_params.e820_entries = new_nr; boot_params.e820_entries = new_nr;
if (append_e820_table(boot_params.e820_table, boot_params.e820_entries)
< 0) { if (append_e820_table(boot_params.e820_table, boot_params.e820_entries) < 0) {
u64 mem_size; u64 mem_size;
/* compare results from other methods and take the greater */ /* Compare results from other methods and take the one that gives more RAM: */
if (boot_params.alt_mem_k if (boot_params.alt_mem_k < boot_params.screen_info.ext_mem_k) {
< boot_params.screen_info.ext_mem_k) {
mem_size = boot_params.screen_info.ext_mem_k; mem_size = boot_params.screen_info.ext_mem_k;
who = "BIOS-88"; who = "BIOS-88";
} else { } else {
@ -1151,7 +1118,6 @@ char *__init e820__memory_setup_default(void)
e820_add_region(HIGH_MEMORY, mem_size << 10, E820_RAM); e820_add_region(HIGH_MEMORY, mem_size << 10, E820_RAM);
} }
/* In case someone cares... */
return who; return who;
} }
@ -1165,7 +1131,9 @@ void __init e820__memory_setup(void)
char *who; char *who;
who = x86_init.resources.memory_setup(); who = x86_init.resources.memory_setup();
memcpy(e820_table_firmware, e820_table, sizeof(struct e820_table)); memcpy(e820_table_firmware, e820_table, sizeof(struct e820_table));
printk(KERN_INFO "e820: BIOS-provided physical RAM map:\n"); printk(KERN_INFO "e820: BIOS-provided physical RAM map:\n");
e820_print_map(who); e820_print_map(who);
} }
@ -1221,8 +1189,7 @@ void __init memblock_find_dma_reserve(void)
nr_pages += end_pfn - start_pfn; nr_pages += end_pfn - start_pfn;
} }
for_each_free_mem_range(u, NUMA_NO_NODE, MEMBLOCK_NONE, &start, &end, for_each_free_mem_range(u, NUMA_NO_NODE, MEMBLOCK_NONE, &start, &end, NULL) {
NULL) {
start_pfn = min_t(unsigned long, PFN_UP(start), MAX_DMA_PFN); start_pfn = min_t(unsigned long, PFN_UP(start), MAX_DMA_PFN);
end_pfn = min_t(unsigned long, PFN_DOWN(end), MAX_DMA_PFN); end_pfn = min_t(unsigned long, PFN_DOWN(end), MAX_DMA_PFN);
if (start_pfn < end_pfn) if (start_pfn < end_pfn)