diff options
author | Bernhard Walle <bwalle@suse.de> | 2008-01-30 07:30:32 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-01-30 07:30:32 -0500 |
commit | c9cce83dd1d59f52e2c8f8c7d265ba4854c40785 (patch) | |
tree | 0d6801b5c069e573ee06464c567e838a6764f424 /arch/x86/kernel/setup_64.c | |
parent | 9773db2a301b089bb95907eec5ad1a2ef7fb4099 (diff) |
x86: remove extern declarations for code, data, bss resources
This patch removes the extern struct resource declarations for
data_resource, code_resource and bss_resource on x86 and declares that
three structures as static as done on other architectures like IA64.
On i386, these structures are moved to setup_32.c (from e820_32.c) because
that's code that is not specific to e820 and also required on EFI systems.
That makes the "extern" reference superfluous.
On x86_64, data_resource, code_resource and bss_resource are passed to
e820_reserve_resources() as arguments just as done on i386 and IA64. That
also avoids the "extern" reference and it's possible to make it static.
Signed-off-by: Bernhard Walle <bwalle@suse.de>
Cc: "Luck, Tony" <tony.luck@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/kernel/setup_64.c')
-rw-r--r-- | arch/x86/kernel/setup_64.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/x86/kernel/setup_64.c b/arch/x86/kernel/setup_64.c index bcb5f3aaa097..1acb435a0585 100644 --- a/arch/x86/kernel/setup_64.c +++ b/arch/x86/kernel/setup_64.c | |||
@@ -123,19 +123,19 @@ struct resource standard_io_resources[] = { | |||
123 | 123 | ||
124 | #define IORESOURCE_RAM (IORESOURCE_BUSY | IORESOURCE_MEM) | 124 | #define IORESOURCE_RAM (IORESOURCE_BUSY | IORESOURCE_MEM) |
125 | 125 | ||
126 | struct resource data_resource = { | 126 | static struct resource data_resource = { |
127 | .name = "Kernel data", | 127 | .name = "Kernel data", |
128 | .start = 0, | 128 | .start = 0, |
129 | .end = 0, | 129 | .end = 0, |
130 | .flags = IORESOURCE_RAM, | 130 | .flags = IORESOURCE_RAM, |
131 | }; | 131 | }; |
132 | struct resource code_resource = { | 132 | static struct resource code_resource = { |
133 | .name = "Kernel code", | 133 | .name = "Kernel code", |
134 | .start = 0, | 134 | .start = 0, |
135 | .end = 0, | 135 | .end = 0, |
136 | .flags = IORESOURCE_RAM, | 136 | .flags = IORESOURCE_RAM, |
137 | }; | 137 | }; |
138 | struct resource bss_resource = { | 138 | static struct resource bss_resource = { |
139 | .name = "Kernel bss", | 139 | .name = "Kernel bss", |
140 | .start = 0, | 140 | .start = 0, |
141 | .end = 0, | 141 | .end = 0, |
@@ -438,7 +438,7 @@ void __init setup_arch(char **cmdline_p) | |||
438 | /* | 438 | /* |
439 | * We trust e820 completely. No explicit ROM probing in memory. | 439 | * We trust e820 completely. No explicit ROM probing in memory. |
440 | */ | 440 | */ |
441 | e820_reserve_resources(); | 441 | e820_reserve_resources(&code_resource, &data_resource, &bss_resource); |
442 | e820_mark_nosave_regions(); | 442 | e820_mark_nosave_regions(); |
443 | 443 | ||
444 | { | 444 | { |