All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] x86/EFI: avoid overrunning mb_modules[]
@ 2017-03-17 10:06 Jan Beulich
  2017-03-17 10:36 ` Andrew Cooper
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2017-03-17 10:06 UTC (permalink / raw)
  To: xen-devel; +Cc: Andrew Cooper

[-- Attachment #1: Type: text/plain, Size: 579 bytes --]

Commit 436fb462ab ("x86/microcode: enable boot time (pre-Dom0)
loading") added a 4th module without providing an array slot for it.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/arch/x86/efi/efi-boot.h
+++ b/xen/arch/x86/efi/efi-boot.h
@@ -13,7 +13,7 @@ static struct file __initdata ucode;
 static multiboot_info_t __initdata mbi = {
     .flags = MBI_MODULES | MBI_LOADERNAME
 };
-static module_t __initdata mb_modules[3];
+static module_t __initdata mb_modules[4];
 
 static void __init edd_put_string(u8 *dst, size_t n, const char *src)
 {




[-- Attachment #2: x86-EFI-mb_modules.patch --]
[-- Type: text/plain, Size: 616 bytes --]

x86/EFI: avoid overrunning mb_modules[]

Commit 436fb462ab ("x86/microcode: enable boot time (pre-Dom0)
loading") added a 4th module without providing an array slot for it.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/arch/x86/efi/efi-boot.h
+++ b/xen/arch/x86/efi/efi-boot.h
@@ -13,7 +13,7 @@ static struct file __initdata ucode;
 static multiboot_info_t __initdata mbi = {
     .flags = MBI_MODULES | MBI_LOADERNAME
 };
-static module_t __initdata mb_modules[3];
+static module_t __initdata mb_modules[4];
 
 static void __init edd_put_string(u8 *dst, size_t n, const char *src)
 {

[-- Attachment #3: Type: text/plain, Size: 127 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] x86/EFI: avoid overrunning mb_modules[]
  2017-03-17 10:06 [PATCH] x86/EFI: avoid overrunning mb_modules[] Jan Beulich
@ 2017-03-17 10:36 ` Andrew Cooper
  0 siblings, 0 replies; 2+ messages in thread
From: Andrew Cooper @ 2017-03-17 10:36 UTC (permalink / raw)
  To: Jan Beulich, xen-devel

On 17/03/17 10:06, Jan Beulich wrote:
> Commit 436fb462ab ("x86/microcode: enable boot time (pre-Dom0)
> loading") added a 4th module without providing an array slot for it.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-03-17 10:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-17 10:06 [PATCH] x86/EFI: avoid overrunning mb_modules[] Jan Beulich
2017-03-17 10:36 ` Andrew Cooper

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.