2008-02-04 23:46:51

by Pavel Machek

[permalink] [raw]
Subject: Small pm documentation cleanups


Small documentation fixes/additions that accumulated in my tree.

Signed-off-by: Pavel Machek <[email protected]>

diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
index cf38689..3be3328 100644
--- a/Documentation/kernel-parameters.txt
+++ b/Documentation/kernel-parameters.txt
@@ -147,8 +147,10 @@ and is between 256 and 4096 characters.
default: 0

acpi_sleep= [HW,ACPI] Sleep options
- Format: { s3_bios, s3_mode }
- See Documentation/power/video.txt
+ Format: { s3_bios, s3_mode, s3_beep }
+ See Documentation/power/video.txt for s3_bios and s3_mode.
+ s3_beep is for debugging; it beeps on PC speaker as soon as
+ kernel's real-mode entry point is called.

acpi_sci= [HW,ACPI] ACPI System Control Interrupt trigger mode
Format: { level | edge | high | low }
diff --git a/Documentation/power/swsusp.txt b/Documentation/power/swsusp.txt
index aea7e92..d3e5e4e 100644
--- a/Documentation/power/swsusp.txt
+++ b/Documentation/power/swsusp.txt
@@ -386,6 +386,11 @@ before suspending; then remount them aft
There is a work-around for this problem. For more information, see
Documentation/usb/persist.txt.

+Q: Can I suspend-to-disk using a swap partition under LVM?
+
+A: No. You can suspend successfully, but you'll not be able to
+resume. uswsusp should be able to work with LVM, see suspend.sf.net.
+
Q: I upgraded the kernel from 2.6.15 to 2.6.16. Both kernels were
compiled with the similar configuration files. Anyway I found that
suspend to disk (and resume) is much slower on 2.6.16 compared to
diff --git a/drivers/acpi/hardware/hwsleep.c b/drivers/acpi/hardware/hwsleep.c
index fd1c4ba..058d0be 100644
--- a/drivers/acpi/hardware/hwsleep.c
+++ b/drivers/acpi/hardware/hwsleep.c
@@ -286,13 +286,13 @@ acpi_status asmlinkage acpi_enter_sleep_
}

/*
+ * 1) Disable/Clear all GPEs
* 2) Enable all wakeup GPEs
*/
status = acpi_hw_disable_all_gpes();
if (ACPI_FAILURE(status)) {
return_ACPI_STATUS(status);
}
-
acpi_gbl_system_awake_and_running = FALSE;

status = acpi_hw_enable_all_wakeup_gpes();
diff --git a/drivers/acpi/sleep/main.c b/drivers/acpi/sleep/main.c
index 485de13..56e09cf 100644
--- a/drivers/acpi/sleep/main.c
+++ b/drivers/acpi/sleep/main.c
@@ -170,7 +170,7 @@ static int acpi_pm_enter(suspend_state_t
/* Reprogram control registers and execute _BFS */
acpi_leave_sleep_state_prep(acpi_state);

- /* ACPI 3.0 specs (P62) says that it's the responsabilty
+ /* ACPI 3.0 specs (P62) says that it's the responsibilty
* of the OSPM to clear the status bit [ implying that the
* POWER_BUTTON event should not reach userspace ]
*/
diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index ef9b802..b275ffb 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -74,14 +74,14 @@ config PM_TRACE_RTC
RTC across reboots, so that you can debug a machine that just hangs
during suspend (or more commonly, during resume).

- To use this debugging feature you should attempt to suspend the machine,
- then reboot it, then run
+ To use this debugging feature you should attempt to suspend the
+ machine, then reboot it, then run

dmesg -s 1000000 | grep 'hash matches'

CAUTION: this option will cause your machine's real-time clock to be
set to an invalid time after a resume.

config PM_SLEEP_SMP
bool
depends on SMP
@@ -123,7 +129,8 @@ config HIBERNATION
called "hibernation" in user interfaces. STD checkpoints the
system and powers it off; and restores that checkpoint on reboot.

- You can suspend your machine with 'echo disk > /sys/power/state'.
+ You can suspend your machine with 'echo disk > /sys/power/state'
+ after placing resume=/dev/swappartition on kernel command line.
Alternatively, you can use the additional userland tools available
from <http://suspend.sf.net>.

diff --git a/kernel/power/main.c b/kernel/power/main.c
index 6a6d5eb..d3df5af 100644
--- a/kernel/power/main.c
+++ b/kernel/power/main.c
@@ -223,6 +226,7 @@ void __attribute__ ((weak)) arch_suspend
* @state: state to enter
*
* This function should be called after devices have been suspended.
+ * May not sleep.
*/
static int suspend_enter(suspend_state_t state)
{
@@ -250,6 +255,8 @@ static int suspend_enter(suspend_state_t
* suspend_devices_and_enter - suspend devices and enter the desired system
* sleep state.
* @state: state to enter
+ *
+ * Needs to be called from process context and may sleep.
*/
int suspend_devices_and_enter(suspend_state_t state)
{
@@ -341,6 +349,8 @@ static inline int valid_state(suspend_st
* happen when we wake up.
* Then, do the setup for suspend, enter the state, and cleaup (after
* we've woken up).
+ *
+ * Needs to be called from process context, and may sleep.
*/
static int enter_state(suspend_state_t state)
{

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html


2008-02-05 01:26:59

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: Small pm documentation cleanups

Len, please pick this up, thanks.

On Tuesday, 5 of February 2008, Pavel Machek wrote:
>
> Small documentation fixes/additions that accumulated in my tree.
>
> Signed-off-by: Pavel Machek <[email protected]>

Acked-by: Rafael J. Wysocki <[email protected]>

> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> index cf38689..3be3328 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -147,8 +147,10 @@ and is between 256 and 4096 characters.
> default: 0
>
> acpi_sleep= [HW,ACPI] Sleep options
> - Format: { s3_bios, s3_mode }
> - See Documentation/power/video.txt
> + Format: { s3_bios, s3_mode, s3_beep }
> + See Documentation/power/video.txt for s3_bios and s3_mode.
> + s3_beep is for debugging; it beeps on PC speaker as soon as
> + kernel's real-mode entry point is called.
>
> acpi_sci= [HW,ACPI] ACPI System Control Interrupt trigger mode
> Format: { level | edge | high | low }
> diff --git a/Documentation/power/swsusp.txt b/Documentation/power/swsusp.txt
> index aea7e92..d3e5e4e 100644
> --- a/Documentation/power/swsusp.txt
> +++ b/Documentation/power/swsusp.txt
> @@ -386,6 +386,11 @@ before suspending; then remount them aft
> There is a work-around for this problem. For more information, see
> Documentation/usb/persist.txt.
>
> +Q: Can I suspend-to-disk using a swap partition under LVM?
> +
> +A: No. You can suspend successfully, but you'll not be able to
> +resume. uswsusp should be able to work with LVM, see suspend.sf.net.
> +
> Q: I upgraded the kernel from 2.6.15 to 2.6.16. Both kernels were
> compiled with the similar configuration files. Anyway I found that
> suspend to disk (and resume) is much slower on 2.6.16 compared to
> diff --git a/drivers/acpi/hardware/hwsleep.c b/drivers/acpi/hardware/hwsleep.c
> index fd1c4ba..058d0be 100644
> --- a/drivers/acpi/hardware/hwsleep.c
> +++ b/drivers/acpi/hardware/hwsleep.c
> @@ -286,13 +286,13 @@ acpi_status asmlinkage acpi_enter_sleep_
> }
>
> /*
> + * 1) Disable/Clear all GPEs
> * 2) Enable all wakeup GPEs
> */
> status = acpi_hw_disable_all_gpes();
> if (ACPI_FAILURE(status)) {
> return_ACPI_STATUS(status);
> }
> -
> acpi_gbl_system_awake_and_running = FALSE;
>
> status = acpi_hw_enable_all_wakeup_gpes();
> diff --git a/drivers/acpi/sleep/main.c b/drivers/acpi/sleep/main.c
> index 485de13..56e09cf 100644
> --- a/drivers/acpi/sleep/main.c
> +++ b/drivers/acpi/sleep/main.c
> @@ -170,7 +170,7 @@ static int acpi_pm_enter(suspend_state_t
> /* Reprogram control registers and execute _BFS */
> acpi_leave_sleep_state_prep(acpi_state);
>
> - /* ACPI 3.0 specs (P62) says that it's the responsabilty
> + /* ACPI 3.0 specs (P62) says that it's the responsibilty
> * of the OSPM to clear the status bit [ implying that the
> * POWER_BUTTON event should not reach userspace ]
> */
> diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
> index ef9b802..b275ffb 100644
> --- a/kernel/power/Kconfig
> +++ b/kernel/power/Kconfig
> @@ -74,14 +74,14 @@ config PM_TRACE_RTC
> RTC across reboots, so that you can debug a machine that just hangs
> during suspend (or more commonly, during resume).
>
> - To use this debugging feature you should attempt to suspend the machine,
> - then reboot it, then run
> + To use this debugging feature you should attempt to suspend the
> + machine, then reboot it, then run
>
> dmesg -s 1000000 | grep 'hash matches'
>
> CAUTION: this option will cause your machine's real-time clock to be
> set to an invalid time after a resume.
>
> config PM_SLEEP_SMP
> bool
> depends on SMP
> @@ -123,7 +129,8 @@ config HIBERNATION
> called "hibernation" in user interfaces. STD checkpoints the
> system and powers it off; and restores that checkpoint on reboot.
>
> - You can suspend your machine with 'echo disk > /sys/power/state'.
> + You can suspend your machine with 'echo disk > /sys/power/state'
> + after placing resume=/dev/swappartition on kernel command line.
> Alternatively, you can use the additional userland tools available
> from <http://suspend.sf.net>.
>
> diff --git a/kernel/power/main.c b/kernel/power/main.c
> index 6a6d5eb..d3df5af 100644
> --- a/kernel/power/main.c
> +++ b/kernel/power/main.c
> @@ -223,6 +226,7 @@ void __attribute__ ((weak)) arch_suspend
> * @state: state to enter
> *
> * This function should be called after devices have been suspended.
> + * May not sleep.
> */
> static int suspend_enter(suspend_state_t state)
> {
> @@ -250,6 +255,8 @@ static int suspend_enter(suspend_state_t
> * suspend_devices_and_enter - suspend devices and enter the desired system
> * sleep state.
> * @state: state to enter
> + *
> + * Needs to be called from process context and may sleep.
> */
> int suspend_devices_and_enter(suspend_state_t state)
> {
> @@ -341,6 +349,8 @@ static inline int valid_state(suspend_st
> * happen when we wake up.
> * Then, do the setup for suspend, enter the state, and cleaup (after
> * we've woken up).
> + *
> + * Needs to be called from process context, and may sleep.
> */
> static int enter_state(suspend_state_t state)
> {
>



--
"Premature optimization is the root of all evil." - Donald Knuth

2008-02-05 05:35:41

by Nigel Cunningham

[permalink] [raw]
Subject: Re: [linux-pm] Re: Small pm documentation cleanups

Hi.

Rafael J. Wysocki wrote:
> Len, please pick this up, thanks.
>
> On Tuesday, 5 of February 2008, Pavel Machek wrote:
>> Small documentation fixes/additions that accumulated in my tree.
>>
>> Signed-off-by: Pavel Machek <[email protected]>
>
> Acked-by: Rafael J. Wysocki <[email protected]>
>
>> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
>> index cf38689..3be3328 100644
>> --- a/Documentation/kernel-parameters.txt
>> +++ b/Documentation/kernel-parameters.txt
>> @@ -147,8 +147,10 @@ and is between 256 and 4096 characters.
>> default: 0
>>
>> acpi_sleep= [HW,ACPI] Sleep options
>> - Format: { s3_bios, s3_mode }
>> - See Documentation/power/video.txt
>> + Format: { s3_bios, s3_mode, s3_beep }
>> + See Documentation/power/video.txt for s3_bios and s3_mode.
>> + s3_beep is for debugging; it beeps on PC speaker as soon as
>> + kernel's real-mode entry point is called.

s/kernel's/the kernel's/

>>
>> acpi_sci= [HW,ACPI] ACPI System Control Interrupt trigger mode
>> Format: { level | edge | high | low }
>> diff --git a/Documentation/power/swsusp.txt b/Documentation/power/swsusp.txt
>> index aea7e92..d3e5e4e 100644
>> --- a/Documentation/power/swsusp.txt
>> +++ b/Documentation/power/swsusp.txt
>> @@ -386,6 +386,11 @@ before suspending; then remount them aft
>> There is a work-around for this problem. For more information, see
>> Documentation/usb/persist.txt.
>>
>> +Q: Can I suspend-to-disk using a swap partition under LVM?
>> +
>> +A: No. You can suspend successfully, but you'll not be able to
>> +resume. uswsusp should be able to work with LVM, see suspend.sf.net.

s/LVM, see/LVM. See/

>> +
>> Q: I upgraded the kernel from 2.6.15 to 2.6.16. Both kernels were
>> compiled with the similar configuration files. Anyway I found that
>> suspend to disk (and resume) is much slower on 2.6.16 compared to
>> diff --git a/drivers/acpi/hardware/hwsleep.c b/drivers/acpi/hardware/hwsleep.c
>> index fd1c4ba..058d0be 100644
>> --- a/drivers/acpi/hardware/hwsleep.c
>> +++ b/drivers/acpi/hardware/hwsleep.c
>> @@ -286,13 +286,13 @@ acpi_status asmlinkage acpi_enter_sleep_
>> }
>>
>> /*
>> + * 1) Disable/Clear all GPEs
>> * 2) Enable all wakeup GPEs
>> */
>> status = acpi_hw_disable_all_gpes();
>> if (ACPI_FAILURE(status)) {
>> return_ACPI_STATUS(status);
>> }
>> -
>> acpi_gbl_system_awake_and_running = FALSE;
>>
>> status = acpi_hw_enable_all_wakeup_gpes();
>> diff --git a/drivers/acpi/sleep/main.c b/drivers/acpi/sleep/main.c
>> index 485de13..56e09cf 100644
>> --- a/drivers/acpi/sleep/main.c
>> +++ b/drivers/acpi/sleep/main.c
>> @@ -170,7 +170,7 @@ static int acpi_pm_enter(suspend_state_t
>> /* Reprogram control registers and execute _BFS */
>> acpi_leave_sleep_state_prep(acpi_state);
>>
>> - /* ACPI 3.0 specs (P62) says that it's the responsabilty
>> + /* ACPI 3.0 specs (P62) says that it's the responsibilty

s/responsibilty/responsibility/

>> * of the OSPM to clear the status bit [ implying that the
>> * POWER_BUTTON event should not reach userspace ]
>> */
>> diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
>> index ef9b802..b275ffb 100644
>> --- a/kernel/power/Kconfig
>> +++ b/kernel/power/Kconfig
>> @@ -74,14 +74,14 @@ config PM_TRACE_RTC
>> RTC across reboots, so that you can debug a machine that just hangs
>> during suspend (or more commonly, during resume).
>>
>> - To use this debugging feature you should attempt to suspend the machine,
>> - then reboot it, then run
>> + To use this debugging feature you should attempt to suspend the
>> + machine, then reboot it, then run

More readable would be "machine, reboot it and then run"

>>
>> dmesg -s 1000000 | grep 'hash matches'
>>
>> CAUTION: this option will cause your machine's real-time clock to be
>> set to an invalid time after a resume.
>>
>> config PM_SLEEP_SMP
>> bool
>> depends on SMP
>> @@ -123,7 +129,8 @@ config HIBERNATION
>> called "hibernation" in user interfaces. STD checkpoints the
>> system and powers it off; and restores that checkpoint on reboot.
>>
>> - You can suspend your machine with 'echo disk > /sys/power/state'.
>> + You can suspend your machine with 'echo disk > /sys/power/state'
>> + after placing resume=/dev/swappartition on kernel command line.

s/on kernel/on the kernel/

Maybe add "in your bootloader's configuration file"?

>> Alternatively, you can use the additional userland tools available
>> from <http://suspend.sf.net>.
>>
>> diff --git a/kernel/power/main.c b/kernel/power/main.c
>> index 6a6d5eb..d3df5af 100644
>> --- a/kernel/power/main.c
>> +++ b/kernel/power/main.c
>> @@ -223,6 +226,7 @@ void __attribute__ ((weak)) arch_suspend
>> * @state: state to enter
>> *
>> * This function should be called after devices have been suspended.
>> + * May not sleep.
>> */
>> static int suspend_enter(suspend_state_t state)
>> {
>> @@ -250,6 +255,8 @@ static int suspend_enter(suspend_state_t
>> * suspend_devices_and_enter - suspend devices and enter the desired system
>> * sleep state.
>> * @state: state to enter
>> + *
>> + * Needs to be called from process context and may sleep.
>> */
>> int suspend_devices_and_enter(suspend_state_t state)
>> {
>> @@ -341,6 +349,8 @@ static inline int valid_state(suspend_st
>> * happen when we wake up.
>> * Then, do the setup for suspend, enter the state, and cleaup (after
>> * we've woken up).
>> + *
>> + * Needs to be called from process context, and may sleep.
>> */
>> static int enter_state(suspend_state_t state)
>> {
>>
>
>
>

Regards,

Nigel

2008-02-05 18:27:07

by Pavel Machek

[permalink] [raw]
Subject: Re: [linux-pm] Re: Small pm documentation cleanups

Hi!

>> On Tuesday, 5 of February 2008, Pavel Machek wrote:
>>> Small documentation fixes/additions that accumulated in my tree.
>>>
>>> Signed-off-by: Pavel Machek <[email protected]>
>>
>> Acked-by: Rafael J. Wysocki <[email protected]>
>>
...
>>> 0
>>> acpi_sleep= [HW,ACPI] Sleep options
>>> - Format: { s3_bios, s3_mode }
>>> - See Documentation/power/video.txt
>>> + Format: { s3_bios, s3_mode, s3_beep }
>>> + See Documentation/power/video.txt for s3_bios and s3_mode.
>>> + s3_beep is for debugging; it beeps on PC speaker as soon as
>>> + kernel's real-mode entry point is called.
>
> s/kernel's/the kernel's/

Thanks for comments, I applied them.

Small documentation fixes/additions that accumulated in my tree.

Len, please pick this up, thanks.
Pavel


Signed-off-by: Pavel Machek <[email protected]>
Acked-by: Rafael J. Wysocki <[email protected]>

diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
index cf38689..e2fb127 100644
--- a/Documentation/kernel-parameters.txt
+++ b/Documentation/kernel-parameters.txt
@@ -147,8 +147,10 @@ and is between 256 and 4096 characters.
default: 0

acpi_sleep= [HW,ACPI] Sleep options
- Format: { s3_bios, s3_mode }
- See Documentation/power/video.txt
+ Format: { s3_bios, s3_mode, s3_beep }
+ See Documentation/power/video.txt for s3_bios and s3_mode.
+ s3_beep is for debugging; it beeps on PC speaker as soon as
+ the kernel's real-mode entry point is called.

acpi_sci= [HW,ACPI] ACPI System Control Interrupt trigger mode
Format: { level | edge | high | low }
diff --git a/Documentation/power/swsusp.txt b/Documentation/power/swsusp.txt
index aea7e92..9d60ab7 100644
--- a/Documentation/power/swsusp.txt
+++ b/Documentation/power/swsusp.txt
@@ -386,6 +386,11 @@ before suspending; then remount them aft
There is a work-around for this problem. For more information, see
Documentation/usb/persist.txt.

+Q: Can I suspend-to-disk using a swap partition under LVM?
+
+A: No. You can suspend successfully, but you'll not be able to
+resume. uswsusp should be able to work with LVM. See suspend.sf.net.
+
Q: I upgraded the kernel from 2.6.15 to 2.6.16. Both kernels were
compiled with the similar configuration files. Anyway I found that
suspend to disk (and resume) is much slower on 2.6.16 compared to
diff --git a/drivers/acpi/hardware/hwsleep.c b/drivers/acpi/hardware/hwsleep.c
index fd1c4ba..058d0be 100644
--- a/drivers/acpi/hardware/hwsleep.c
+++ b/drivers/acpi/hardware/hwsleep.c
@@ -286,13 +286,13 @@ acpi_status asmlinkage acpi_enter_sleep_
}

/*
+ * 1) Disable/Clear all GPEs
* 2) Enable all wakeup GPEs
*/
status = acpi_hw_disable_all_gpes();
if (ACPI_FAILURE(status)) {
return_ACPI_STATUS(status);
}
-
acpi_gbl_system_awake_and_running = FALSE;

status = acpi_hw_enable_all_wakeup_gpes();
diff --git a/drivers/acpi/sleep/main.c b/drivers/acpi/sleep/main.c
index 485de13..c9a733f 100644
--- a/drivers/acpi/sleep/main.c
+++ b/drivers/acpi/sleep/main.c
@@ -170,7 +170,7 @@ static int acpi_pm_enter(suspend_state_t
/* Reprogram control registers and execute _BFS */
acpi_leave_sleep_state_prep(acpi_state);

- /* ACPI 3.0 specs (P62) says that it's the responsabilty
+ /* ACPI 3.0 specs (P62) says that it's the responsibility
* of the OSPM to clear the status bit [ implying that the
* POWER_BUTTON event should not reach userspace ]
*/
diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index ef9b802..78bb205 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -74,14 +74,14 @@ config PM_TRACE_RTC
RTC across reboots, so that you can debug a machine that just hangs
during suspend (or more commonly, during resume).

- To use this debugging feature you should attempt to suspend the machine,
- then reboot it, then run
+ To use this debugging feature you should attempt to suspend the
+ machine, reboot it and then run

dmesg -s 1000000 | grep 'hash matches'

CAUTION: this option will cause your machine's real-time clock to be
set to an invalid time after a resume.

config PM_SLEEP_SMP
bool
depends on SMP
@@ -123,7 +129,10 @@ config HIBERNATION
called "hibernation" in user interfaces. STD checkpoints the
system and powers it off; and restores that checkpoint on reboot.

- You can suspend your machine with 'echo disk > /sys/power/state'.
+ You can suspend your machine with 'echo disk > /sys/power/state'
+ after placing resume=/dev/swappartition on the kernel command line
+ in your bootloader's configuration file.
+
Alternatively, you can use the additional userland tools available
from <http://suspend.sf.net>.


--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

2008-02-05 21:28:13

by Jesper Juhl

[permalink] [raw]
Subject: Re: [linux-pm] Re: Small pm documentation cleanups

On 05/02/2008, Pavel Machek <[email protected]> wrote:
> Hi!
>
> >> On Tuesday, 5 of February 2008, Pavel Machek wrote:
> >>> Small documentation fixes/additions that accumulated in my tree.
> >>>
> >>> Signed-off-by: Pavel Machek <[email protected]>
> >>
> >> Acked-by: Rafael J. Wysocki <[email protected]>
> >>
> ...
...
>
> Thanks for comments, I applied them.
>
> Small documentation fixes/additions that accumulated in my tree.
>
> Len, please pick this up, thanks.
> Pavel
>
>
> Signed-off-by: Pavel Machek <[email protected]>
> Acked-by: Rafael J. Wysocki <[email protected]>
>

I've put this on my watch list for the trivial tree.
So, if it doesn't get merged elsewhere I'll take care of pushing it
again during the 2.6.26 merge window (and beyond).

--
Jesper Juhl <[email protected]>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html

2008-02-06 21:27:46

by Nigel Cunningham

[permalink] [raw]
Subject: Re: [linux-pm] Re: Small pm documentation cleanups

Hi again.

Pavel Machek wrote:
> Hi!
>
>>> On Tuesday, 5 of February 2008, Pavel Machek wrote:
>>>> Small documentation fixes/additions that accumulated in my tree.
>>>>
>>>> Signed-off-by: Pavel Machek <[email protected]>
>>> Acked-by: Rafael J. Wysocki <[email protected]>
>>>
> ...
>>>> 0
>>>> acpi_sleep= [HW,ACPI] Sleep options
>>>> - Format: { s3_bios, s3_mode }
>>>> - See Documentation/power/video.txt
>>>> + Format: { s3_bios, s3_mode, s3_beep }
>>>> + See Documentation/power/video.txt for s3_bios and s3_mode.
>>>> + s3_beep is for debugging; it beeps on PC speaker as soon as
>>>> + kernel's real-mode entry point is called.
>> s/kernel's/the kernel's/
>
> Thanks for comments, I applied them.

You're welcome. Sorry for replying again, but I just noticed another one
in the line above - it should be something like "it makes the PC's
speaker beep as soon as" ('on PC speaker' isn't right).

Sorry for not getting that last time.

Nigel

2008-02-06 21:31:41

by Pavel Machek

[permalink] [raw]
Subject: Re: [linux-pm] Re: Small pm documentation cleanups

On Thu 2008-02-07 08:27:03, Nigel Cunningham wrote:
> Hi again.
>
> Pavel Machek wrote:
>> Hi!
>>
>>>> On Tuesday, 5 of February 2008, Pavel Machek wrote:
>>>>> Small documentation fixes/additions that accumulated in my tree.
>>>>>
>>>>> Signed-off-by: Pavel Machek <[email protected]>
>>>> Acked-by: Rafael J. Wysocki <[email protected]>
>>>>
>> ...
>>>>> 0
>>>>> acpi_sleep= [HW,ACPI] Sleep options
>>>>> - Format: { s3_bios, s3_mode }
>>>>> - See Documentation/power/video.txt
>>>>> + Format: { s3_bios, s3_mode, s3_beep }
>>>>> + See Documentation/power/video.txt for s3_bios and s3_mode.
>>>>> + s3_beep is for debugging; it beeps on PC speaker as soon as
>>>>> + kernel's real-mode entry point is called.
>>> s/kernel's/the kernel's/
>>
>> Thanks for comments, I applied them.
>
> You're welcome. Sorry for replying again, but I just noticed another one in
> the line above - it should be something like "it makes the PC's speaker
> beep as soon as" ('on PC speaker' isn't right).
>
> Sorry for not getting that last time.

Ok, lets fix that in followup patch...
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

2008-02-07 06:29:29

by Len Brown

[permalink] [raw]
Subject: Re: [linux-pm] Re: Small pm documentation cleanups

applied
(with nigel's last comment included)

thanks,
-len

On Wednesday 06 February 2008 16:27, Nigel Cunningham wrote:
> Hi again.
>
> Pavel Machek wrote:
> > Hi!
> >
> >>> On Tuesday, 5 of February 2008, Pavel Machek wrote:
> >>>> Small documentation fixes/additions that accumulated in my tree.
> >>>>
> >>>> Signed-off-by: Pavel Machek <[email protected]>
> >>> Acked-by: Rafael J. Wysocki <[email protected]>
> >>>
> > ...
> >>>> 0
> >>>> acpi_sleep= [HW,ACPI] Sleep options
> >>>> - Format: { s3_bios, s3_mode }
> >>>> - See Documentation/power/video.txt
> >>>> + Format: { s3_bios, s3_mode, s3_beep }
> >>>> + See Documentation/power/video.txt for s3_bios and s3_mode.
> >>>> + s3_beep is for debugging; it beeps on PC speaker as soon as
> >>>> + kernel's real-mode entry point is called.
> >> s/kernel's/the kernel's/
> >
> > Thanks for comments, I applied them.
>
> You're welcome. Sorry for replying again, but I just noticed another one
> in the line above - it should be something like "it makes the PC's
> speaker beep as soon as" ('on PC speaker' isn't right).
>
> Sorry for not getting that last time.
>
> Nigel
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>