Re: [PATCH] asus-laptop: add suspend/resume handlers

2008-02-18 Thread Corentin CHARY
Hi, This patch seems ok for me. Thanks for your work Fabien ;) - To unsubscribe from this list: send the line unsubscribe linux-acpi in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: 2.6.25-rc2-mm1 (x64 thermal build failure)

2008-02-18 Thread Thomas Petazzoni
Hi, Le Sat, 16 Feb 2008 21:44:10 -0800, Andrew Morton [EMAIL PROTECTED] a écrit : Bustage in x86-configurable-dmi-scanning-code.patch. Previously, DMI=y was just hardwired. Now, it becomes selectable and stuff breaks. I guess the DMI=n version of dmi_check_system() could become a macro

Re: 2.6.25-rc2-mm1 (x64 thermal build failure)

2008-02-18 Thread Andrew Morton
On Mon, 18 Feb 2008 11:15:36 +0100 Thomas Petazzoni [EMAIL PROTECTED] wrote: Hi, Le Sat, 16 Feb 2008 21:44:10 -0800, Andrew Morton [EMAIL PROTECTED] a __crit : Bustage in x86-configurable-dmi-scanning-code.patch. Previously, DMI=y was just hardwired. Now, it becomes selectable and

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Andrew Morton
On Sun, 17 Feb 2008 00:54:08 + (GMT) Chris Rankin [EMAIL PROTECTED] wrote: [Try this again, except this time I'll force the attachment as inline text!] Hi, I have managed to boot 2.6.24.1 on this machine, with the NMI watchdog enabled, by using the acpi=noirq option. (There does

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Andrew Morton
On Sun, 17 Feb 2008 00:54:08 + (GMT) Chris Rankin [EMAIL PROTECTED] wrote: [Try this again, except this time I'll force the attachment as inline text!] Hi, I have managed to boot 2.6.24.1 on this machine, with the NMI watchdog enabled, by using the acpi=noirq option. (There does

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Chris Rankin
--- Andrew Morton [EMAIL PROTECTED] wrote: and here it hangs, I assume? Oops, I think you have misunderstood. The hang happens if I *don't* specify acpi=noirq, whereas in this case I did. I have already reported the original hang under threads:

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Chris Rankin
--- Andrew Morton [EMAIL PROTECTED] wrote: sysfs: duplicate filename 'bridge' can not be created WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() Pid: 1, comm: swapper Not tainted 2.6.24.1 #1 [c0105020] show_trace_log_lvl+0x1a/0x2f [c0105990] show_trace+0x12/0x14 [c010613d]

Re: Problem with the Fn+F3 key. acpi problem ?

2008-02-18 Thread Thomas Renninger
On Sat, 2008-01-12 at 16:02 +0100, giggz wrote: Matthew Garrett a écrit : On Sat, Jan 12, 2008 at 01:33:38PM +0100, giggz wrote: What is the acpi video module ? If you have /proc/acpi/video, it's loaded. I don't have any /var/log/acpid.log it seems to be normal that I don't

Re: [PATCH 5/5] ACPI: add DMI to enable OSI(Linux) on ThinkPad T61

2008-02-18 Thread Thomas Renninger
Hi, On Sun, 2008-01-20 at 17:49 -0200, Henrique de Moraes Holschuh wrote: On Sat, 19 Jan 2008, Theodore Tso wrote: Sure, and that means they have to *tell* *us* what they are doing so we can be compatible with Windows. The concern is that they may be doing stuff that isn't in the standard

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Stephen Hemminger
On Mon, 18 Feb 2008 05:00:49 -0800 Andrew Morton [EMAIL PROTECTED] wrote: On Sun, 17 Feb 2008 00:54:08 + (GMT) Chris Rankin [EMAIL PROTECTED] wrote: [Try this again, except this time I'll force the attachment as inline text!] Hi, I have managed to boot 2.6.24.1 on this

Re: [PATCH 5/5] ACPI: add DMI to enable OSI(Linux) on ThinkPad T61

2008-02-18 Thread Henrique de Moraes Holschuh
On Mon, 18 Feb 2008, Thomas Renninger wrote: The problem is that OSI is used by Windows to pass the exact Windows (not OS) version they are running, this function should be called WOSI. We of course want to run on the latest fix-ups here and should pass Windows 2006 (or whatever latest string

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Chris Rankin
--- Stephen Hemminger [EMAIL PROTECTED] wrote: sysfs: duplicate filename 'bridge' can not be created WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() Pid: 1, comm: swapper Not tainted 2.6.24.1 #1 [c0105020] show_trace_log_lvl+0x1a/0x2f [c0105990] show_trace+0x12/0x14 [c010613d]

Re: Linux 2.6.24.1 - kernel does not boot; IRQ trouble?

2008-02-18 Thread Stephen Hemminger
On Mon, 18 Feb 2008 19:42:25 + (GMT) Chris Rankin [EMAIL PROTECTED] wrote: --- Stephen Hemminger [EMAIL PROTECTED] wrote: sysfs: duplicate filename 'bridge' can not be created WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() Pid: 1, comm: swapper Not tainted 2.6.24.1 #1

[PATCH 3/4] provide error handling for unclean objects

2008-02-18 Thread Glauber Costa
Previously, if acpi_processor_start() failed to create sysfs nodes, it would leave stuff to be cleaned, that won't perish in acpi_processor_remove(), since we'll have pr-cdev == NULL. This patch cleans those objects. Signed-off-by: Glauber Costa [EMAIL PROTECTED] ---

[PATCH 0/4] Avoid panic in case of buggy bios

2008-02-18 Thread Glauber Costa
Hi, I've came across a panic today after failing acpi_processor_start() in this condition: if (processor_device_array[pr-id] != NULL processor_device_array[pr-id] != device) { printk(KERN_WARNING BIOS reported wrong ACPI id for the

[PATCH 2/4] use pr-cdev as a condition for cleanup

2008-02-18 Thread Glauber Costa
The acpi_processor_start() function can fail before creating the sysfs nodes for thermal cooling. In this case, pr-cdev will be NULL, and the removal code will break. This patch uses pr-cdev as a criteria for termination of the mentioned code, and avoids it, if it was never initialized.

[PATCH 1/4] make cdev attribuition the last step

2008-02-18 Thread Glauber Costa
This patch uses a temporary variable cdev instead of using directly pr-cdev. Through it, we can tell later whether or not this code was completed properly: by checking for pr-cdev != NULL Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- drivers/acpi/processor_core.c | 14 -- 1

Re: [PATCH 5/5] ACPI: add DMI to enable OSI(Linux) on ThinkPad T61

2008-02-18 Thread Thomas Renninger
On Mon, 2008-02-18 at 16:17 -0300, Henrique de Moraes Holschuh wrote: On Mon, 18 Feb 2008, Thomas Renninger wrote: The problem is that OSI is used by Windows to pass the exact Windows (not OS) version they are running, this function should be called WOSI. We of course want to run on the

Re: [PATCH 5/5] ACPI: add DMI to enable OSI(Linux) on ThinkPad T61

2008-02-18 Thread Theodore Tso
On Tue, Feb 19, 2008 at 01:00:59AM +0100, Thomas Renninger wrote: Most stuff that gets fixed by these workarounds would make no sense to backport, because backports are much too intrusive, e.g.: Many of the workarounds really aren't that hard to backport, and the reality is after the distro

Re: [PATCH 4/4] remove goto statement

2008-02-18 Thread Li Zefan
Glauber Costa 写道: This patch removes goto statements in favour of plain returns in places that had nothing left behind that would justify such construction --- drivers/acpi/processor_core.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 4/4] remove goto statement

2008-02-18 Thread Glauber Costa
Li Zefan wrote: Glauber Costa 写道: This patch removes goto statements in favour of plain returns in places that had nothing left behind that would justify such construction --- drivers/acpi/processor_core.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 4/4] [PATCH] remove goto statement

2008-02-18 Thread Glauber Costa
This patch removes goto statements in favour of plain returns in places that had nothing left behind that would justify such construction --- drivers/acpi/processor_core.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/acpi/processor_core.c

Re: [PATCH 4/4] remove goto statement

2008-02-18 Thread Li Zefan
Glauber Costa wrote: Li Zefan wrote: Glauber Costa 写道: This patch removes goto statements in favour of plain returns in places that had nothing left behind that would justify such construction --- drivers/acpi/processor_core.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)

Re: [PATCH 5/5] ACPI: add DMI to enable OSI(Linux) on ThinkPad T61

2008-02-18 Thread Thomas Renninger
On Mon, 2008-02-18 at 19:26 -0500, Theodore Tso wrote: On Tue, Feb 19, 2008 at 01:00:59AM +0100, Thomas Renninger wrote: Most stuff that gets fixed by these workarounds would make no sense to backport, because backports are much too intrusive, e.g.: Many of the workarounds really