Commit 8a73b17e authored by Lin Ming's avatar Lin Ming Committed by Len Brown
Browse files

ACPICA: Sleep/Wake interfaces: optionally execute _GTS and _BFS 



Enhanced the sleep/wake interfaces to optionally execute the
_GTS method (Going To Sleep), and the _BFS method (Back From
Sleep).  Windows apparently does not execute these methods, and
therefore these methods are often untested. It has been seen on
some systems where the execution of these methods causes errors
and also prevents the machine from entering S5. It is therefore
suggested that host operating systems do not execute these methods
by default. In the future, perhaps these methods can be optionally
executed based on the age of the system and/or what is the newest
version of Windows that the BIOS asks for via _OSI.

Signed-off-by: default avatarLin Ming <ming.m.lin@intel.com>
Signed-off-by: default avatarBob Moore <robert.moore@intel.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 4acb6884

drivers/acpi/acpica/achware.h

+6 −6
Original line number Original line Diff line number Diff line
@@ -83,22 +83,22 @@ acpi_status acpi_hw_clear_acpi_status(void); 
/*

/*

 * hwsleep - sleep/wake support (Legacy sleep registers)

 * hwsleep - sleep/wake support (Legacy sleep registers)

 */

 */

acpi_status acpi_hw_legacy_sleep(u8 sleep_state);

acpi_status acpi_hw_legacy_sleep(u8 sleep_state, u8 flags);





acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state);

acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state, u8 flags);





acpi_status acpi_hw_legacy_wake(u8 sleep_state);

acpi_status acpi_hw_legacy_wake(u8 sleep_state, u8 flags);





/*

/*

 * hwesleep - sleep/wake support (Extended FADT-V5 sleep registers)

 * hwesleep - sleep/wake support (Extended FADT-V5 sleep registers)

 */

 */

void acpi_hw_execute_sleep_method(char *method_name, u32 integer_argument);

void acpi_hw_execute_sleep_method(char *method_name, u32 integer_argument);





acpi_status acpi_hw_extended_sleep(u8 sleep_state);

acpi_status acpi_hw_extended_sleep(u8 sleep_state, u8 flags);





acpi_status acpi_hw_extended_wake_prep(u8 sleep_state);

acpi_status acpi_hw_extended_wake_prep(u8 sleep_state, u8 flags);





acpi_status acpi_hw_extended_wake(u8 sleep_state);

acpi_status acpi_hw_extended_wake(u8 sleep_state, u8 flags);





/*

/*

 * hwvalid - Port I/O with validation

 * hwvalid - Port I/O with validation

drivers/acpi/acpica/hwesleep.c

+15 −6
Original line number Original line Diff line number Diff line
@@ -103,6 +103,7 @@ void acpi_hw_execute_sleep_method(char *method_pathname, u32 integer_argument) 
 * FUNCTION:    acpi_hw_extended_sleep

 * FUNCTION:    acpi_hw_extended_sleep

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state to enter

 * PARAMETERS:  sleep_state         - Which sleep state to enter

 *              Flags               - ACPI_EXECUTE_GTS to run optional method

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -112,7 +113,7 @@ void acpi_hw_execute_sleep_method(char *method_pathname, u32 integer_argument) 
 *

 *

 ******************************************************************************/

 ******************************************************************************/





acpi_status acpi_hw_extended_sleep(u8 sleep_state)

acpi_status acpi_hw_extended_sleep(u8 sleep_state, u8 flags)

{

{

	acpi_status status;

	acpi_status status;

	u8 sleep_type_value;

	u8 sleep_type_value;
@@ -136,9 +137,11 @@ acpi_status acpi_hw_extended_sleep(u8 sleep_state) 




	acpi_gbl_system_awake_and_running = FALSE;

	acpi_gbl_system_awake_and_running = FALSE;





	/* Execute the _GTS method (Going To Sleep) */

	/* Optionally execute _GTS (Going To Sleep) */





	if (flags & ACPI_EXECUTE_GTS) {

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__GTS, sleep_state);

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__GTS, sleep_state);

	}





	/* Flush caches, as per ACPI specification */

	/* Flush caches, as per ACPI specification */
@@ -181,6 +184,7 @@ acpi_status acpi_hw_extended_sleep(u8 sleep_state) 
 * FUNCTION:    acpi_hw_extended_wake_prep

 * FUNCTION:    acpi_hw_extended_wake_prep

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 *              Flags               - ACPI_EXECUTE_BFS to run optional method

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -189,7 +193,7 @@ acpi_status acpi_hw_extended_sleep(u8 sleep_state) 
 *

 *

 ******************************************************************************/

 ******************************************************************************/





acpi_status acpi_hw_extended_wake_prep(u8 sleep_state)

acpi_status acpi_hw_extended_wake_prep(u8 sleep_state, u8 flags)

{

{

	acpi_status status;

	acpi_status status;

	u8 sleep_type_value;

	u8 sleep_type_value;
@@ -208,7 +212,11 @@ acpi_status acpi_hw_extended_wake_prep(u8 sleep_state) 
				 &acpi_gbl_FADT.sleep_control);

				 &acpi_gbl_FADT.sleep_control);

	}

	}





	/* Optionally execute _BFS (Back From Sleep) */



	if (flags & ACPI_EXECUTE_BFS) {

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__BFS, sleep_state);

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__BFS, sleep_state);

	}

	return_ACPI_STATUS(AE_OK);

	return_ACPI_STATUS(AE_OK);

}

}
@@ -217,6 +225,7 @@ acpi_status acpi_hw_extended_wake_prep(u8 sleep_state) 
 * FUNCTION:    acpi_hw_extended_wake

 * FUNCTION:    acpi_hw_extended_wake

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 *              Flags               - Reserved, set to zero

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -225,7 +234,7 @@ acpi_status acpi_hw_extended_wake_prep(u8 sleep_state) 
 *

 *

 ******************************************************************************/

 ******************************************************************************/





acpi_status acpi_hw_extended_wake(u8 sleep_state)

acpi_status acpi_hw_extended_wake(u8 sleep_state, u8 flags)

{

{

	ACPI_FUNCTION_TRACE(hw_extended_wake);

	ACPI_FUNCTION_TRACE(hw_extended_wake);

drivers/acpi/acpica/hwsleep.c

+15 −6
Original line number Original line Diff line number Diff line
@@ -56,6 +56,7 @@ ACPI_MODULE_NAME("hwsleep") 
 * FUNCTION:    acpi_hw_legacy_sleep

 * FUNCTION:    acpi_hw_legacy_sleep

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state to enter

 * PARAMETERS:  sleep_state         - Which sleep state to enter

 *              Flags               - ACPI_EXECUTE_GTS to run optional method

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -63,7 +64,7 @@ ACPI_MODULE_NAME("hwsleep") 
 *              THIS FUNCTION MUST BE CALLED WITH INTERRUPTS DISABLED

 *              THIS FUNCTION MUST BE CALLED WITH INTERRUPTS DISABLED

 *

 *

 ******************************************************************************/

 ******************************************************************************/

acpi_status acpi_hw_legacy_sleep(u8 sleep_state)

acpi_status acpi_hw_legacy_sleep(u8 sleep_state, u8 flags)

{

{

	struct acpi_bit_register_info *sleep_type_reg_info;

	struct acpi_bit_register_info *sleep_type_reg_info;

	struct acpi_bit_register_info *sleep_enable_reg_info;

	struct acpi_bit_register_info *sleep_enable_reg_info;
@@ -121,9 +122,11 @@ acpi_status acpi_hw_legacy_sleep(u8 sleep_state) 
		return_ACPI_STATUS(status);

		return_ACPI_STATUS(status);

	}

	}





	/* Execute the _GTS method (Going To Sleep) */

	/* Optionally execute _GTS (Going To Sleep) */





	if (flags & ACPI_EXECUTE_GTS) {

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__GTS, sleep_state);

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__GTS, sleep_state);

	}





	/* Get current value of PM1A control */

	/* Get current value of PM1A control */
@@ -219,6 +222,7 @@ acpi_status acpi_hw_legacy_sleep(u8 sleep_state) 
 * FUNCTION:    acpi_hw_legacy_wake_prep

 * FUNCTION:    acpi_hw_legacy_wake_prep

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 *              Flags               - ACPI_EXECUTE_BFS to run optional method

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -228,7 +232,7 @@ acpi_status acpi_hw_legacy_sleep(u8 sleep_state) 
 *

 *

 ******************************************************************************/

 ******************************************************************************/





acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state)

acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state, u8 flags)

{

{

	acpi_status status;

	acpi_status status;

	struct acpi_bit_register_info *sleep_type_reg_info;

	struct acpi_bit_register_info *sleep_type_reg_info;
@@ -279,7 +283,11 @@ acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state) 
		}

		}

	}

	}





	/* Optionally execute _BFS (Back From Sleep) */



	if (flags & ACPI_EXECUTE_BFS) {

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__BFS, sleep_state);

		acpi_hw_execute_sleep_method(METHOD_PATHNAME__BFS, sleep_state);

	}

	return_ACPI_STATUS(status);

	return_ACPI_STATUS(status);

}

}
@@ -288,6 +296,7 @@ acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state) 
 * FUNCTION:    acpi_hw_legacy_wake

 * FUNCTION:    acpi_hw_legacy_wake

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 * PARAMETERS:  sleep_state         - Which sleep state we just exited

 *              Flags               - Reserved, set to zero

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -296,7 +305,7 @@ acpi_status acpi_hw_legacy_wake_prep(u8 sleep_state) 
 *

 *

 ******************************************************************************/

 ******************************************************************************/





acpi_status acpi_hw_legacy_wake(u8 sleep_state)

acpi_status acpi_hw_legacy_wake(u8 sleep_state, u8 flags)

{

{

	acpi_status status;

	acpi_status status;

drivers/acpi/acpica/hwxfsleep.c

+20 −15
Original line number Original line Diff line number Diff line
@@ -49,15 +49,16 @@ 
ACPI_MODULE_NAME("hwxfsleep")

ACPI_MODULE_NAME("hwxfsleep")





/* Local prototypes */

/* Local prototypes */

static acpi_status acpi_hw_sleep_dispatch(u8 sleep_state, u32 function_id);

static acpi_status

acpi_hw_sleep_dispatch(u8 sleep_state, u8 flags, u32 function_id);





/*

/*

 * Dispatch table used to efficiently branch to the various sleep

 * Dispatch table used to efficiently branch to the various sleep

 * functions.

 * functions.

 */

 */

#define ACPI_SLEEP_FUNCTION         0

#define ACPI_SLEEP_FUNCTION_ID         0

#define ACPI_WAKE_PREP_FUNCTION     1

#define ACPI_WAKE_PREP_FUNCTION_ID     1

#define ACPI_WAKE_FUNCTION          2

#define ACPI_WAKE_FUNCTION_ID          2





/* Legacy functions are optional, based upon ACPI_REDUCED_HARDWARE */

/* Legacy functions are optional, based upon ACPI_REDUCED_HARDWARE */
@@ -233,7 +234,8 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_state_s4bios) 
 *              function.

 *              function.

 *

 *

 ******************************************************************************/

 ******************************************************************************/

static acpi_status acpi_hw_sleep_dispatch(u8 sleep_state, u32 function_id)

static acpi_status

acpi_hw_sleep_dispatch(u8 sleep_state, u8 flags, u32 function_id)

{

{

	acpi_status status;

	acpi_status status;

	struct acpi_sleep_functions *sleep_functions =

	struct acpi_sleep_functions *sleep_functions =
@@ -246,11 +248,11 @@ static acpi_status acpi_hw_sleep_dispatch(u8 sleep_state, u32 function_id) 
	 * use the extended sleep registers

	 * use the extended sleep registers

	 */

	 */

	if (acpi_gbl_reduced_hardware || acpi_gbl_FADT.sleep_control.address) {

	if (acpi_gbl_reduced_hardware || acpi_gbl_FADT.sleep_control.address) {

		status = sleep_functions->extended_function(sleep_state);

		status = sleep_functions->extended_function(sleep_state, flags);

	} else {

	} else {

		/* Legacy sleep */

		/* Legacy sleep */





		status = sleep_functions->legacy_function(sleep_state);

		status = sleep_functions->legacy_function(sleep_state, flags);

	}

	}





	return (status);

	return (status);
@@ -260,7 +262,7 @@ static acpi_status acpi_hw_sleep_dispatch(u8 sleep_state, u32 function_id) 
	 * For the case where reduced-hardware-only code is being generated,

	 * For the case where reduced-hardware-only code is being generated,

	 * we know that only the extended sleep registers are available

	 * we know that only the extended sleep registers are available

	 */

	 */

	status = sleep_functions->extended_function(sleep_state);

	status = sleep_functions->extended_function(sleep_state, flags);

	return (status);

	return (status);





#endif				/* !ACPI_REDUCED_HARDWARE */

#endif				/* !ACPI_REDUCED_HARDWARE */
@@ -290,8 +292,6 @@ acpi_status acpi_enter_sleep_state_prep(u8 sleep_state) 




	ACPI_FUNCTION_TRACE(acpi_enter_sleep_state_prep);

	ACPI_FUNCTION_TRACE(acpi_enter_sleep_state_prep);





	/* _PSW methods could be run here to enable wake-on keyboard, LAN, etc. */



	status = acpi_get_sleep_type_data(sleep_state,

	status = acpi_get_sleep_type_data(sleep_state,

					  &acpi_gbl_sleep_type_a,

					  &acpi_gbl_sleep_type_a,

					  &acpi_gbl_sleep_type_b);

					  &acpi_gbl_sleep_type_b);
@@ -349,6 +349,7 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_state_prep) 
 * FUNCTION:    acpi_enter_sleep_state

 * FUNCTION:    acpi_enter_sleep_state

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state to enter

 * PARAMETERS:  sleep_state         - Which sleep state to enter

 *              Flags               - ACPI_EXECUTE_GTS to run optional method

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -356,7 +357,7 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_state_prep) 
 *              THIS FUNCTION MUST BE CALLED WITH INTERRUPTS DISABLED

 *              THIS FUNCTION MUST BE CALLED WITH INTERRUPTS DISABLED

 *

 *

 ******************************************************************************/

 ******************************************************************************/

acpi_status asmlinkage acpi_enter_sleep_state(u8 sleep_state)

acpi_status asmlinkage acpi_enter_sleep_state(u8 sleep_state, u8 flags)

{

{

	acpi_status status;

	acpi_status status;
@@ -369,7 +370,8 @@ acpi_status asmlinkage acpi_enter_sleep_state(u8 sleep_state) 
		return_ACPI_STATUS(AE_AML_OPERAND_VALUE);

		return_ACPI_STATUS(AE_AML_OPERAND_VALUE);

	}

	}





	status = acpi_hw_sleep_dispatch(sleep_state, ACPI_SLEEP_FUNCTION);

	status =

	    acpi_hw_sleep_dispatch(sleep_state, flags, ACPI_SLEEP_FUNCTION_ID);

	return_ACPI_STATUS(status);

	return_ACPI_STATUS(status);

}

}
@@ -380,6 +382,7 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_state) 
 * FUNCTION:    acpi_leave_sleep_state_prep

 * FUNCTION:    acpi_leave_sleep_state_prep

 *

 *

 * PARAMETERS:  sleep_state         - Which sleep state we are exiting

 * PARAMETERS:  sleep_state         - Which sleep state we are exiting

 *              Flags               - ACPI_EXECUTE_BFS to run optional method

 *

 *

 * RETURN:      Status

 * RETURN:      Status

 *

 *
@@ -388,13 +391,15 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_state) 
 *              Called with interrupts DISABLED.

 *              Called with interrupts DISABLED.

 *

 *

 ******************************************************************************/

 ******************************************************************************/

acpi_status acpi_leave_sleep_state_prep(u8 sleep_state)

acpi_status acpi_leave_sleep_state_prep(u8 sleep_state, u8 flags)

{

{

	acpi_status status;

	acpi_status status;





	ACPI_FUNCTION_TRACE(acpi_leave_sleep_state_prep);

	ACPI_FUNCTION_TRACE(acpi_leave_sleep_state_prep);





	status = acpi_hw_sleep_dispatch(sleep_state, ACPI_WAKE_PREP_FUNCTION);

	status =

	    acpi_hw_sleep_dispatch(sleep_state, flags,

				   ACPI_WAKE_PREP_FUNCTION_ID);

	return_ACPI_STATUS(status);

	return_ACPI_STATUS(status);

}

}
@@ -419,7 +424,7 @@ acpi_status acpi_leave_sleep_state(u8 sleep_state) 
	ACPI_FUNCTION_TRACE(acpi_leave_sleep_state);

	ACPI_FUNCTION_TRACE(acpi_leave_sleep_state);









	status = acpi_hw_sleep_dispatch(sleep_state, ACPI_WAKE_FUNCTION);

	status = acpi_hw_sleep_dispatch(sleep_state, 0, ACPI_WAKE_FUNCTION_ID);

	return_ACPI_STATUS(status);

	return_ACPI_STATUS(status);

}

}

drivers/acpi/sleep.c

+7 −6
Original line number Original line Diff line number Diff line
@@ -250,7 +250,8 @@ static int acpi_suspend_enter(suspend_state_t pm_state) 
	switch (acpi_state) {

	switch (acpi_state) {

	case ACPI_STATE_S1:

	case ACPI_STATE_S1:

		barrier();

		barrier();

		status = acpi_enter_sleep_state(acpi_state);

		status = acpi_enter_sleep_state(acpi_state,

				ACPI_NO_OPTIONAL_METHODS);

		break;

		break;





	case ACPI_STATE_S3:

	case ACPI_STATE_S3:
@@ -265,7 +266,7 @@ static int acpi_suspend_enter(suspend_state_t pm_state) 
	acpi_write_bit_register(ACPI_BITREG_SCI_ENABLE, 1);

	acpi_write_bit_register(ACPI_BITREG_SCI_ENABLE, 1);





	/* Reprogram control registers and execute _BFS */

	/* Reprogram control registers and execute _BFS */

	acpi_leave_sleep_state_prep(acpi_state);

	acpi_leave_sleep_state_prep(acpi_state, ACPI_NO_OPTIONAL_METHODS);





	/* ACPI 3.0 specs (P62) says that it's the responsibility

	/* ACPI 3.0 specs (P62) says that it's the responsibility

	 * of the OSPM to clear the status bit [ implying that the

	 * of the OSPM to clear the status bit [ implying that the
@@ -534,9 +535,9 @@ static int acpi_hibernation_enter(void) 
	ACPI_FLUSH_CPU_CACHE();

	ACPI_FLUSH_CPU_CACHE();





	/* This shouldn't return.  If it returns, we have a problem */

	/* This shouldn't return.  If it returns, we have a problem */

	status = acpi_enter_sleep_state(ACPI_STATE_S4);

	status = acpi_enter_sleep_state(ACPI_STATE_S4, ACPI_NO_OPTIONAL_METHODS);

	/* Reprogram control registers and execute _BFS */

	/* Reprogram control registers and execute _BFS */

	acpi_leave_sleep_state_prep(ACPI_STATE_S4);

	acpi_leave_sleep_state_prep(ACPI_STATE_S4, ACPI_NO_OPTIONAL_METHODS);





	return ACPI_SUCCESS(status) ? 0 : -EFAULT;

	return ACPI_SUCCESS(status) ? 0 : -EFAULT;

}

}
@@ -549,7 +550,7 @@ static void acpi_hibernation_leave(void) 
	 */

	 */

	acpi_enable();

	acpi_enable();

	/* Reprogram control registers and execute _BFS */

	/* Reprogram control registers and execute _BFS */

	acpi_leave_sleep_state_prep(ACPI_STATE_S4);

	acpi_leave_sleep_state_prep(ACPI_STATE_S4, ACPI_NO_OPTIONAL_METHODS);

	/* Check the hardware signature */

	/* Check the hardware signature */

	if (facs && s4_hardware_signature != facs->hardware_signature) {

	if (facs && s4_hardware_signature != facs->hardware_signature) {

		printk(KERN_EMERG "ACPI: Hardware changed while hibernated, "

		printk(KERN_EMERG "ACPI: Hardware changed while hibernated, "
@@ -773,7 +774,7 @@ static void acpi_power_off(void) 
	/* acpi_sleep_prepare(ACPI_STATE_S5) should have already been called */

	/* acpi_sleep_prepare(ACPI_STATE_S5) should have already been called */

	printk(KERN_DEBUG "%s called\n", __func__);

	printk(KERN_DEBUG "%s called\n", __func__);

	local_irq_disable();

	local_irq_disable();

	acpi_enter_sleep_state(ACPI_STATE_S5);

	acpi_enter_sleep_state(ACPI_STATE_S5, ACPI_NO_OPTIONAL_METHODS);

}

}





/*

/*

CRA Git | Maintained and supported by SUSTech CRA and CCSE