* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
@ 2004-03-08 5:21 ` Yu, Luming
2004-03-08 19:10 ` David Mosberger
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Yu, Luming @ 2004-03-08 5:21 UTC (permalink / raw)
To: linux-ia64
Regarding unalignment access issue, Bob have the following patch
integrated into 20040220.
Would it help this cases?
diff -Bru 2.6-acpi/include/acpi/acglobal.h
patched/include/acpi/acglobal.h
--- 2.6-acpi/include/acpi/acglobal.h 2003-12-18 13:19:56.000000000
+0800
+++ patched/include/acpi/acglobal.h 2003-12-22 15:31:29.000000000
+0800
@@ -58,6 +58,11 @@
#endif
+/* Keep local copies of these FADT-based registers */
+
+ACPI_EXTERN struct acpi_generic_address acpi_gbl_xpm1a_enable;
+ACPI_EXTERN struct acpi_generic_address acpi_gbl_xpm1b_enable;
+
/***********************************************************************
******
*
* Debug support
@@ -107,10 +112,6 @@
ACPI_EXTERN u8
acpi_gbl_integer_byte_width;
ACPI_EXTERN u8
acpi_gbl_integer_nybble_width;
-/* Keep local copies of these FADT-based registers */
-
-ACPI_EXTERN struct acpi_generic_address acpi_gbl_xpm1a_enable;
-ACPI_EXTERN struct acpi_generic_address acpi_gbl_xpm1b_enable;
/*
* Since there may be multiple SSDTs and PSDTS, a single pointer is not
> -----Original Message-----
> From: linux-ia64-owner@vger.kernel.org
> [mailto:linux-ia64-owner@vger.kernel.org] On Behalf Of David Mosberger
> Sent: Friday, March 05, 2004 8:16 AM
> To: Brown, Len
> Cc: linux-ia64@vger.kernel.org
> Subject: cat /proc/acpi/events bad for your system's health!
>
>
> Hi Len,
>
> While tracking down another ACPI problem, I thought I'd try this:
>
> # cat /proc/acpi/events
>
> To my surprise pushing the power-button then caused "cat" to crash.
> The exact failure more seems to vary a bit but variously, you'll get a
> segfault in "cat", possible along with some kind of machine check
> error, or the machine dies. I confirmed this behavior both on
> zx1-based platforms and on a Tiger. This used to work fine (well,
> last time I tried it was probably a 2.4 kernel, but still...).
>
> I attached the console output that I got when doing this on the tiger.
> It looks to me like a more or less random address is being accessed.
>
> The kernel was 2.6.4-rc1.
>
> If you don't have physical access to a machine, I think the bug
> can also be triggered by simply hitting Ctrl-C when "cat" is
> running.
>
> It's a good thing access to /proc/acpi/events is privileged...
>
> --david
>
>
> kernel unaligned access to 0xffffffffffffffff, ip=0xa0000001000f7f30
> cat[628]: error during unaligned kernel access
> -1 [1]
> CPU 1: SAL log contains CPE error record
>
> Pid: 628, CPU 2, comm: cat
> psr : 0000101008022018 ifs : 8000000000000308 ip :
> [<a0000001000f7f30>] Not tainted
> ip is at kfree+0xb0/0x1c0
> unat: 0000000000000000 pfs : 0000000000000288 rsc : 0000000000000003
> rnat: 0000000000000000 bsps: 0000000000000000 pr : 000000000009aa59
> ldrs: 0000000000000000 ccv : 0000000000000000 fpsr: 0009804c0270033f
> csd : 0000000000000000 ssd : 0000000000000000
> b0 : a00000010039ea50 b6 : a0000001000f2f40 b7 : a00000010000c8c0
> f6 : 000000000000000000000 f7 : 1003e0fc0fc0fc0fc0fc1
> f8 : 1003e0000000000002490 f9 : 1003e000000000ea008e2
> f10 : 1003e00000000367b9beb f11 : 1003e44b831eee7285baf
> r1 : a000000100a94e30 r2 : 0000000000000003 r3 : e0000007ffe880f8
> r8 : 000000009fffffff r9 : e000000103ccdb50 r10 : e000000103ccdb40
> r11 : 00000000003bb5b4 r12 : e0000002fb88fd80 r13 : e0000002fb888008
> r14 : 0000000000004000 r15 : 0000000000004000 r16 : e000000100118000
> r17 : e0000002fb888eac r18 : 000000000000000f r19 : a0000001008a9b80
> r20 : a0000001008a9b80 r21 : 0000000000000018 r22 : a0000001008461d0
> r23 : 4652575000000000 r24 : 0000008000000000 r25 : 0000000000000001
> r26 : 0000000000004000 r27 : 0000000000004000 r28 : 0000000000004000
> r29 : 0000000000000001 r30 : 0000000000000018 r31 : 0000000000000288
>
> Call Trace:
> [<a000000100014a20>] show_stack+0x80/0xa0
> [<a00000010003de20>] die+0x1a0/0x2a0
> [<a000000100043470>] ia64_handle_unaligned+0x1410/0x2600
> [<a00000010000d610>] ia64_prepare_handle_unaligned+0x30/0x60
> [<a00000010000d040>] ia64_leave_kernel+0x0/0x260
> [<a0000001000f7f30>] kfree+0xb0/0x1c0
> [<a00000010039ea50>] acpi_bus_receive_event+0x2d0/0x300
> [<a0000001003ac1a0>] acpi_system_read_event+0xc0/0x2a0
> [<a000000100133040>] vfs_read+0x1c0/0x2e0
> [<a000000100133620>] sys_read+0x60/0xe0
> [<a00000010000cec0>] ia64_ret_from_syscall+0x0/0x20
> -
> To unsubscribe from this list: send the line "unsubscribe
> linux-ia64" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 8+ messages in thread* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
2004-03-08 5:21 ` Yu, Luming
@ 2004-03-08 19:10 ` David Mosberger
2004-03-10 11:13 ` Yu, Luming
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: David Mosberger @ 2004-03-08 19:10 UTC (permalink / raw)
To: linux-ia64
>>>>> On Mon, 8 Mar 2004 13:21:34 +0800, "Yu, Luming" <luming.yu@intel.com> said:
Luming> Regarding unalignment access issue, Bob have the following
Luming> patch integrated into 20040220. Would it help this cases?
The primary problem is not the unaligned access, it's an access to
a bogus address:
>> kernel unaligned access to 0xffffffffffffffff,
Are you saying that /proc/acpi/events works for you as expected with
the updated ACPI?
--david
^ permalink raw reply [flat|nested] 8+ messages in thread* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
2004-03-08 5:21 ` Yu, Luming
2004-03-08 19:10 ` David Mosberger
@ 2004-03-10 11:13 ` Yu, Luming
2004-03-10 21:05 ` David Mosberger
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Yu, Luming @ 2004-03-10 11:13 UTC (permalink / raw)
To: linux-ia64
I confirm this issue here on tiger with 2.6.4-rc3 kernel , but 2.4 is
ok.
Not sure this is a regression.
PS . please forget my previous reply against this thread. :(
Thanks,
Luming
> -----Original Message-----
> From: David Mosberger [mailto:davidm@napali.hpl.hp.com]
> Sent: Tuesday, March 09, 2004 3:10 AM
> To: Yu, Luming
> Cc: davidm@hpl.hp.com; Brown, Len; linux-ia64@vger.kernel.org
> Subject: RE: cat /proc/acpi/events bad for your system's health!
>
>
> >>>>> On Mon, 8 Mar 2004 13:21:34 +0800, "Yu, Luming"
> <luming.yu@intel.com> said:
>
> Luming> Regarding unalignment access issue, Bob have the following
> Luming> patch integrated into 20040220. Would it help this cases?
>
> The primary problem is not the unaligned access, it's an access to
> a bogus address:
>
> >> kernel unaligned access to 0xffffffffffffffff,
>
> Are you saying that /proc/acpi/events works for you as expected with
> the updated ACPI?
>
> --david
>
^ permalink raw reply [flat|nested] 8+ messages in thread* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
` (2 preceding siblings ...)
2004-03-10 11:13 ` Yu, Luming
@ 2004-03-10 21:05 ` David Mosberger
2004-03-12 15:15 ` Yu, Luming
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: David Mosberger @ 2004-03-10 21:05 UTC (permalink / raw)
To: linux-ia64
>>>>> On Wed, 10 Mar 2004 19:13:00 +0800, "Yu, Luming" <luming.yu@intel.com> said:
Luming> I confirm this issue here on tiger with 2.6.4-rc3 kernel ,
Luming> but 2.4 is ok. Not sure this is a regression.
OK. I assume you (or Len) will take care of it?
Thanks,
--david
^ permalink raw reply [flat|nested] 8+ messages in thread* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
` (3 preceding siblings ...)
2004-03-10 21:05 ` David Mosberger
@ 2004-03-12 15:15 ` Yu, Luming
2004-03-12 18:02 ` David Mosberger
2004-03-13 2:20 ` Yu, Luming
6 siblings, 0 replies; 8+ messages in thread
From: Yu, Luming @ 2004-03-12 15:15 UTC (permalink / raw)
To: linux-ia64
>
> Luming> I confirm this issue here on tiger with 2.6.4-rc3 kernel ,
> Luming> but 2.4 is ok. Not sure this is a regression.
>
> OK. I assume you (or Len) will take care of it?
>
The following patch make crash on my tiger gone. However I didn't dig
out the true culprit. Maybe this is due to excessive optimization.
BTW, my gcc version is 3.2.3 .
Thanks,
Luming
diff -Bru 2.6-acpi/drivers/acpi/bus.c patched/drivers/acpi/bus.c
--- 2.6-acpi/drivers/acpi/bus.c 2004-02-27 09:17:52.000000000 +0800
+++ patched/drivers/acpi/bus.c 2004-03-12 23:03:05.000000000 +0800
@@ -335,6 +335,7 @@
remove_wait_queue(&acpi_bus_event_queue, &wait);
set_current_state(TASK_RUNNING);
+ acpi_os_stall(100);
if (signal_pending(current))
return_VALUE(-ERESTARTSYS);
}
^ permalink raw reply [flat|nested] 8+ messages in thread* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
` (4 preceding siblings ...)
2004-03-12 15:15 ` Yu, Luming
@ 2004-03-12 18:02 ` David Mosberger
2004-03-13 2:20 ` Yu, Luming
6 siblings, 0 replies; 8+ messages in thread
From: David Mosberger @ 2004-03-12 18:02 UTC (permalink / raw)
To: linux-ia64
>>>>> On Fri, 12 Mar 2004 23:15:01 +0800, "Yu, Luming" <luming.yu@intel.com> said:
Luming> I confirm this issue here on tiger with 2.6.4-rc3 kernel ,
Luming> but 2.4 is ok. Not sure this is a regression.
>> OK. I assume you (or Len) will take care of it?
Luming> The following patch make crash on my tiger gone. However I
Luming> didn't dig out the true culprit. Maybe this is due to
Luming> excessive optimization.
Luming> BTW, my gcc version is 3.2.3 .
The patch is fine as a temporary workaround, but surely we need to
find out the root cause of this problem. It sounds like there is a
race somewhere. I suppose it's possible that it's a compiler problem,
but I rather doubt it. Are you doing to dig deeper into it? If not,
please let me know.
--david
^ permalink raw reply [flat|nested] 8+ messages in thread* RE: cat /proc/acpi/events bad for your system's health!
2004-03-05 0:16 cat /proc/acpi/events bad for your system's health! David Mosberger
` (5 preceding siblings ...)
2004-03-12 18:02 ` David Mosberger
@ 2004-03-13 2:20 ` Yu, Luming
6 siblings, 0 replies; 8+ messages in thread
From: Yu, Luming @ 2004-03-13 2:20 UTC (permalink / raw)
To: linux-ia64
>
> Luming> I confirm this issue here on tiger with 2.6.4-rc3 kernel ,
> Luming> but 2.4 is ok. Not sure this is a regression.
>
> >> OK. I assume you (or Len) will take care of it?
>
>
> Luming> The following patch make crash on my tiger gone. However I
> Luming> didn't dig out the true culprit. Maybe this is due to
> Luming> excessive optimization.
>
> Luming> BTW, my gcc version is 3.2.3 .
>
> The patch is fine as a temporary workaround, but surely we need to
> find out the root cause of this problem. It sounds like there is a
> race somewhere. I suppose it's possible that it's a compiler problem,
> but I rather doubt it. Are you doing to dig deeper into it? If not,
> please let me know.
>
> --david
>
The main difference with 2.4 is signal_pending. From ACPI log, I also
noticed that
the thread waiting on acpi_bus_event_queue didn't resume proberly,
because there
is no exit message of acpi_bus_receive_event in ACPI log as expected if
you turn on
full debug flag of acpi.
PS. I don't have access to the machine right now.
--Luming
^ permalink raw reply [flat|nested] 8+ messages in thread