* [PATCH, resend] x86/EFI: properly pre-initialize table pointers
@ 2011-07-05 11:22 Jan Beulich
2011-07-05 12:57 ` [tip:x86/efi] x86, efi: Properly " tip-bot for Jan Beulich
0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2011-07-05 11:22 UTC (permalink / raw)
To: mingo, tglx, hpa; +Cc: linux-kernel
Consumers of the table pointers in struct efi check for
EFI_INVALID_TABLE_ADDR to determine validity, hence these pointers
should all be pre-initialized to this value (rather than zero).
Noticed by the discrepancy between efivars' systab sysfs entry showing
all tables (and their pointers) despite the code intending to only
display the valid ones. No other bad effects known, but having the
various table parsing routines bogusly access physical address zero is
certainly not very desirable (even though they're unlikely to find
anything useful there).
Signed-off-by: Jan Beulich <jbeulich@novell.com>
---
arch/x86/platform/efi/efi.c | 12 +++++++++++-
1 file changed, 11 insertions(+), 1 deletion(-)
--- 3.0-rc6/arch/x86/platform/efi/efi.c
+++ 3.0-rc6-x86-EFI/arch/x86/platform/efi/efi.c
@@ -51,7 +51,17 @@
int efi_enabled;
EXPORT_SYMBOL(efi_enabled);
-struct efi efi;
+struct efi __read_mostly efi = {
+ .mps = EFI_INVALID_TABLE_ADDR,
+ .acpi = EFI_INVALID_TABLE_ADDR,
+ .acpi20 = EFI_INVALID_TABLE_ADDR,
+ .smbios = EFI_INVALID_TABLE_ADDR,
+ .sal_systab = EFI_INVALID_TABLE_ADDR,
+ .boot_info = EFI_INVALID_TABLE_ADDR,
+ .hcdp = EFI_INVALID_TABLE_ADDR,
+ .uga = EFI_INVALID_TABLE_ADDR,
+ .uv_systab = EFI_INVALID_TABLE_ADDR,
+};
EXPORT_SYMBOL(efi);
struct efi_memory_map memmap;
^ permalink raw reply [flat|nested] 2+ messages in thread
* [tip:x86/efi] x86, efi: Properly pre-initialize table pointers
2011-07-05 11:22 [PATCH, resend] x86/EFI: properly pre-initialize table pointers Jan Beulich
@ 2011-07-05 12:57 ` tip-bot for Jan Beulich
0 siblings, 0 replies; 2+ messages in thread
From: tip-bot for Jan Beulich @ 2011-07-05 12:57 UTC (permalink / raw)
To: linux-tip-commits
Cc: linux-kernel, hpa, mingo, jbeulich, JBeulich, tglx, mingo
Commit-ID: d80603c9d876efafd8b07469c891076de470e323
Gitweb: http://git.kernel.org/tip/d80603c9d876efafd8b07469c891076de470e323
Author: Jan Beulich <JBeulich@novell.com>
AuthorDate: Tue, 5 Jul 2011 12:22:18 +0100
Committer: Ingo Molnar <mingo@elte.hu>
CommitDate: Tue, 5 Jul 2011 13:40:34 +0200
x86, efi: Properly pre-initialize table pointers
Consumers of the table pointers in struct efi check for
EFI_INVALID_TABLE_ADDR to determine validity, hence these
pointers should all be pre-initialized to this value (rather
than zero).
Noticed by the discrepancy between efivars' systab sysfs entry
showing all tables (and their pointers) despite the code
intending to only display the valid ones. No other bad effects
known, but having the various table parsing routines bogusly
access physical address zero is certainly not very desirable
(even though they're unlikely to find anything useful there).
Signed-off-by: Jan Beulich <jbeulich@novell.com>
Link: http://lkml.kernel.org/r/4E13100A020000780004C256@nat28.tlf.novell.com
Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
arch/x86/platform/efi/efi.c | 12 +++++++++++-
1 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c
index a0e4244..11e766d 100644
--- a/arch/x86/platform/efi/efi.c
+++ b/arch/x86/platform/efi/efi.c
@@ -51,7 +51,17 @@
int efi_enabled;
EXPORT_SYMBOL(efi_enabled);
-struct efi efi;
+struct efi __read_mostly efi = {
+ .mps = EFI_INVALID_TABLE_ADDR,
+ .acpi = EFI_INVALID_TABLE_ADDR,
+ .acpi20 = EFI_INVALID_TABLE_ADDR,
+ .smbios = EFI_INVALID_TABLE_ADDR,
+ .sal_systab = EFI_INVALID_TABLE_ADDR,
+ .boot_info = EFI_INVALID_TABLE_ADDR,
+ .hcdp = EFI_INVALID_TABLE_ADDR,
+ .uga = EFI_INVALID_TABLE_ADDR,
+ .uv_systab = EFI_INVALID_TABLE_ADDR,
+};
EXPORT_SYMBOL(efi);
struct efi_memory_map memmap;
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-07-05 12:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-05 11:22 [PATCH, resend] x86/EFI: properly pre-initialize table pointers Jan Beulich
2011-07-05 12:57 ` [tip:x86/efi] x86, efi: Properly " tip-bot for Jan Beulich
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.