Re: [GIT PATCH] ACPI patches for 2.6.26-rc5

Previous thread: [PATCH] kernel/signal.c: change vars pid and tgid types to pid_t by Gustavo Fernando Padovan on Wednesday, June 11, 2008 - 7:11 pm. (1 message)

Next thread: Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets by David Miller on Wednesday, June 11, 2008 - 7:52 pm. (27 messages)
To: Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-acpi@...>, Linux Power Management List <linux-pm@...>
Date: Wednesday, June 11, 2008 - 7:44 pm

Hi Linus,

please pull from:

git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release

This will update the files shown below.

thanks!

-Len

ps. individual patches are available on linux-acpi@vger.kernel.org
and a consolidated plain patch is available here:
ftp://ftp.kernel.org/pub/linux/kernel/people/lenb/acpi/patches/release/2...

Documentation/laptops/thinkpad-acpi.txt | 2 +-
MAINTAINERS | 10 +-
arch/ia64/kernel/acpi.c | 9 ++-
arch/x86/mm/srat_64.c | 27 ------
drivers/acpi/bay.c | 16 ++--
drivers/acpi/dispatcher/dsfield.c | 5 +-
drivers/acpi/dock.c | 2 +-
drivers/acpi/ec.c | 2 +-
drivers/acpi/executer/exconfig.c | 10 ++-
drivers/acpi/glue.c | 6 ++
drivers/acpi/hardware/hwsleep.c | 8 +-
drivers/acpi/numa.c | 31 ++++++-
drivers/acpi/parser/psargs.c | 4 +-
drivers/acpi/processor_core.c | 1 -
drivers/acpi/processor_idle.c | 7 +-
drivers/acpi/sleep/proc.c | 6 ++
drivers/acpi/system.c | 15 +---
drivers/acpi/tables/tbinstal.c | 25 ++----
drivers/acpi/tables/tbxface.c | 2 +-
drivers/acpi/thermal.c | 11 ++-
drivers/acpi/utilities/utmisc.c | 2 +-
drivers/cpuidle/cpuidle.c | 40 ++++++++-
drivers/misc/thinkpad_acpi.c | 140 ++++++++++++++++++-------------
drivers/pnp/pnpacpi/rsparser.c | 46 +++++++----
include/acpi/processor.h | 1 +
include/linux/cpuidle.h | 1 +
include/linux/ioport.h | 6 +-
27 files changed, 255 insertions(+), 180 deletions(-)

through these commits:

Adrian Bunk (2):
MAINTAINERS: update ACPI homepage
proper prototy...

To: Len Brown <lenb@...>
Cc: Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-acpi@...>, Linux Power Management List <linux-pm@...>, Lin Ming <ming.m.lin@...>, Rafael J. Wysocki <rjw@...>
Date: Thursday, June 12, 2008 - 1:31 am

Hi,

What about the "ACPICA: Fixes for Unload and DDBHandles" patch?

http://bugzilla.kernel.org/show_bug.cgi?id=10669

The patch was reverted in mainline, but is now fixed, shouldn't it be
brought back at some point? It's (still) listed as a regression,
anyway.

Vegard

--
"The animistic metaphor of the bug that maliciously sneaked in while
the programmer was not looking is intellectually dishonest as it
disguises that the error is the programmer's own creation."
-- E. W. Dijkstra, EWD1036
--

To: Len Brown <lenb@...>
Cc: Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-acpi@...>, Linux Power Management List <linux-pm@...>, Lin Ming <ming.m.lin@...>, Rafael J. Wysocki <rjw@...>
Date: Thursday, June 12, 2008 - 1:37 am

Hm, sorry, I only looked for the patch title in your series. It seems
to be this entry:

commit 8410565f540db87ca938f56f92780d251e4f157d
Author: Bob Moore <robert.moore@intel.com>
Date: Tue Jun 10 14:29:26 2008 +0800

ACPICA: Fix for access to deleted object <regression>

Fixes problem introduced in 20080123, with fix for Unload operator.
Parse tree object can be already deleted; must use the opcode
within the WalkState.

ACPI: kmemcheck: Caught 16-bit read from freed memory
http://bugzilla.kernel.org/show_bug.cgi?id=10669

Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>

but now it has a completely new title and a new author? This looks
like just the fix, I couldn't find the original patch (which had a
nice title and description on its own). Hm.

Thanks, and sorry for the noise.

Vegard

--
"The animistic metaphor of the bug that maliciously sneaked in while
the programmer was not looking is intellectually dishonest as it
disguises that the error is the programmer's own creation."
-- E. W. Dijkstra, EWD1036
--

To: Len Brown <lenb@...>
Cc: Andrew Morton <akpm@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-acpi@...>, Linux Power Management List <linux-pm@...>
Date: Wednesday, June 11, 2008 - 8:24 pm

Hmm. I get very different statistics:

27 files changed, 434 insertions(+), 359 deletions(-)

You seem to use "git diff -w" to ignore whitespace changes, which explains
it, but please don't - it makes it harder to compare with expected
results, and it hides some changes entirely.

I do agree that using '-w' is often useful, but especially for the purpose
of letting people know what changed it kind of avoids the point by hiding
many things.

Linus
--

To: Linus Torvalds <torvalds@...>
Cc: Andrew Morton <akpm@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-acpi@...>, Linux Power Management List <linux-pm@...>
Date: Wednesday, June 11, 2008 - 11:08 pm

Yes, I've been using "-b" recently:

git diff -b -M --stat linus..$RELEASE

and if I remove the "-b", we match:

27 files changed, 434 insertions(+), 359 deletions(-)

I have a vague memory of having a good reason to cut over to "-b"
a while back and noting that in my pull request, but if you prefer
no -b, no problem.

thanks,
-Len
--

Previous thread: [PATCH] kernel/signal.c: change vars pid and tgid types to pid_t by Gustavo Fernando Padovan on Wednesday, June 11, 2008 - 7:11 pm. (1 message)

Next thread: Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets by David Miller on Wednesday, June 11, 2008 - 7:52 pm. (27 messages)