Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
SUSE:SLE-12-SP4:Update
xen.612
5502bb30-x86-EFI-allow-reboot-overrides-when-ru...
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 5502bb30-x86-EFI-allow-reboot-overrides-when-running-under-EFI.patch of Package xen.612
# Commit c643fb110a51693e82a36ca9178d54f0b9744024 # Date 2015-03-13 11:25:52 +0100 # Author Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> # Committer Jan Beulich <jbeulich@suse.com> x86/EFI: allow reboot= overrides when running under EFI By default we will always use EFI reboot mechanism when running under EFI platforms. However some EFI platforms are buggy and need to use the ACPI mechanism to reboot (such as Lenovo ThinkCentre M57). As such respect the 'reboot=' override and DMI overrides for EFI platforms. Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> - BOOT_INVALID is just zero - also consider acpi_disabled in BOOT_INVALID resolution - duplicate BOOT_INVALID resolution in machine_restart() - don't fall back from BOOT_ACPI to BOOT_EFI (if it was overridden, it surely was for a reason) - adjust doc change formatting Signed-off-by: Jan Beulich <jbeulich@suse.com> Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> # Commit 8ff330ec11e471919621bce97c069b83b0319d15 # Date 2015-03-23 18:01:51 +0100 # Author Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> # Committer Jan Beulich <jbeulich@suse.com> x86/EFI: fix reboot after c643fb110a acpi_disabled needs to be moved out of .init.data. Reported-by: Ross Lagerwall <ross.lagerwall@citrix.com> From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Signed-off-by: Jan Beulich <jbeulich@suse.com> Tested-by: Ross Lagerwall <ross.lagerwall@citrix.com> --- a/docs/misc/xen-command-line.markdown +++ b/docs/misc/xen-command-line.markdown @@ -818,7 +818,7 @@ This option can be specified more than o > `= <integer>` ### reboot -> `= t[riple] | k[bd] | n[o] [, [w]arm | [c]old]` +> `= t[riple] | k[bd] | a[cpi] | p[ci] | e[fi] | n[o] [, [w]arm | [c]old]` Default: `0` @@ -834,6 +834,11 @@ Specify the host reboot method. `acpi` instructs Xen to reboot the host using RESET_REG in the ACPI FADT. +`pci` instructs Xen to reboot the host using PCI reset register (port CF9). + +'efi' instructs Xen to reboot using the EFI reboot call (in EFI mode by + default it will use that method first). + ### sched > `= credit | credit2 | sedf | arinc653` --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -99,7 +99,7 @@ struct cpuinfo_x86 __read_mostly boot_cp unsigned long __read_mostly mmu_cr4_features = X86_CR4_PSE | X86_CR4_PGE | X86_CR4_PAE; -bool_t __initdata acpi_disabled; +bool_t __read_mostly acpi_disabled; bool_t __initdata acpi_force; static char __initdata acpi_param[10] = ""; static void __init parse_acpi_param(char *s) --- a/xen/arch/x86/shutdown.c +++ b/xen/arch/x86/shutdown.c @@ -28,25 +28,29 @@ #include <asm/apic.h> enum reboot_type { + BOOT_INVALID, BOOT_TRIPLE = 't', BOOT_KBD = 'k', BOOT_ACPI = 'a', BOOT_CF9 = 'p', + BOOT_EFI = 'e', }; static long no_idt[2]; static int reboot_mode; /* - * reboot=b[ios] | t[riple] | k[bd] | n[o] [, [w]arm | [c]old] + * reboot=t[riple] | k[bd] | a[cpi] | p[ci] | n[o] | [e]fi [, [w]arm | [c]old] * warm Don't set the cold reboot flag * cold Set the cold reboot flag + * no Suppress automatic reboot after panics or crashes * triple Force a triple fault (init) * kbd Use the keyboard controller. cold reset (default) * acpi Use the RESET_REG in the FADT * pci Use the so-called "PCI reset register", CF9 + * efi Use the EFI reboot (if running under EFI) */ -static enum reboot_type reboot_type = BOOT_ACPI; +static enum reboot_type reboot_type = BOOT_INVALID; static void __init set_reboot_type(char *str) { for ( ; ; ) @@ -63,6 +67,7 @@ static void __init set_reboot_type(char reboot_mode = 0x0; break; case 'a': + case 'e': case 'k': case 't': case 'p': @@ -101,6 +106,14 @@ void machine_halt(void) __machine_halt(NULL); } +static void default_reboot_type(void) +{ + if ( reboot_type == BOOT_INVALID ) + reboot_type = efi_enabled ? BOOT_EFI + : acpi_disabled ? BOOT_KBD + : BOOT_ACPI; +} + static int __init override_reboot(struct dmi_system_id *d) { enum reboot_type type = (long)d->driver_data; @@ -447,6 +460,7 @@ static struct dmi_system_id __initdata r static int __init reboot_init(void) { + default_reboot_type(); dmi_check_system(reboot_dmi_table); return 0; } @@ -460,7 +474,7 @@ static void __machine_restart(void *pdel void machine_restart(unsigned int delay_millisecs) { unsigned int i, attempt; - enum reboot_type orig_reboot_type = reboot_type; + enum reboot_type orig_reboot_type; watchdog_disable(); console_start_sync(); @@ -495,15 +509,20 @@ void machine_restart(unsigned int delay_ tboot_shutdown(TB_SHUTDOWN_REBOOT); } - efi_reset_system(reboot_mode != 0); + /* Just in case reboot_init() didn't run yet. */ + default_reboot_type(); + orig_reboot_type = reboot_type; /* Rebooting needs to touch the page at absolute address 0. */ - *((unsigned short *)__va(0x472)) = reboot_mode; + if ( reboot_type != BOOT_EFI ) + *((unsigned short *)__va(0x472)) = reboot_mode; for ( attempt = 0; ; attempt++ ) { switch ( reboot_type ) { + case BOOT_INVALID: + ASSERT_UNREACHABLE(); case BOOT_KBD: /* Pulse the keyboard reset line. */ for ( i = 0; i < 100; i++ ) @@ -523,6 +542,11 @@ void machine_restart(unsigned int delay_ reboot_type = (((attempt == 1) && (orig_reboot_type == BOOT_ACPI)) ? BOOT_ACPI : BOOT_TRIPLE); break; + case BOOT_EFI: + reboot_type = acpi_disabled ? BOOT_KBD : BOOT_ACPI; + efi_reset_system(reboot_mode != 0); + *((unsigned short *)__va(0x472)) = reboot_mode; + break; case BOOT_TRIPLE: asm volatile ( "lidt %0 ; int3" : "=m" (no_idt) ); reboot_type = BOOT_KBD; --- a/xen/include/xen/lib.h +++ b/xen/include/xen/lib.h @@ -41,9 +41,11 @@ do { #ifndef NDEBUG #define ASSERT(p) \ do { if ( unlikely(!(p)) ) assert_failed(#p); } while (0) +#define ASSERT_UNREACHABLE() assert_failed("unreachable") #define debug_build() 1 #else #define ASSERT(p) do { if ( 0 && (p) ); } while (0) +#define ASSERT_UNREACHABLE() do { } while (0) #define debug_build() 0 #endif
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor
