Bug 8010

Summary:	inconsistent lid button state
Product:	ACPI	Reporter:	S.Caglar Onur (caglar)
Component:	Power-Other	Assignee:	Zhang Rui (rui.zhang)
Status:	REJECTED INSUFFICIENT_DATA
Severity:	normal	CC:	acpi-bugzilla
Priority:	P2
Hardware:	i386
OS:	Linux
Kernel Version:	2.6.16, 2.6.18., 2.6.20	Subsystem:
Regression:	---	Bisected commit-id:
Attachments:	.config dmidecode acpidump dmesg dmesg check if acpi_bus_generate_event succeeds dmesg.2.6.22_rc4

Description S.Caglar Onur 2007-02-14 03:01:11 UTC

Most recent kernel where this bug did *NOT* occur:
Distribution: Pardus
Hardware Environment: SONY FS-215B
Software Environment:
Problem Description:

After closing the screen, lid button didn't generates needed open event but all
close events can be monitored, so all userspace application relays on that fails.

For 3 open/close session, only close events reported.

zangetsu ~ # cat /proc/acpi/event
button/lid LID0 00000080 00000005
button/lid LID0 00000080 00000006
button/lid LID0 00000080 00000007

* Power button works without a problem. 
* Battery/Adapter info updated without a problem.

And According to iasl my dsdt is valid


zangetsu ~ # cat /proc/acpi/dsdt > dsdt
zangetsu ~ # iasl -d dsdt

Intel ACPI Component Architecture
AML Disassembler version 20061109 [Jan 17 2007]
Copyright (C) 2000 - 2006 Intel Corporation
Supports ACPI Specification Revision 3.0a

Loading Acpi table from file dsdt
Acpi table [DSDT] successfully installed and loaded
Pass 1 parse of [DSDT]
Pass 2 parse of [DSDT]
Parsing Deferred Opcodes (Methods/Buffers/Packages/Regions)
..................................................................................................................................................................................................................................................................................................................................
Parsing completed
Disassembly completed, written to "dsdt.dsl"
zangetsu ~ # iasl dsdt.dsl

Intel ACPI Component Architecture
ASL Optimizing Compiler version 20061109 [Jan 17 2007]
Copyright (C) 2000 - 2006 Intel Corporation
Supports ACPI Specification Revision 3.0a

ASL Input:  dsdt.dsl - 4600 lines, 156540 bytes, 1656 keywords
AML Output: dsdt.aml - 15180 bytes 473 named objects 1183 executable opcodes

Compilation complete. 0 Errors, 0 Warnings, 0 Remarks, 31 Optimizations

Comment 1 S.Caglar Onur 2007-02-14 03:05:10 UTC

Created attachment 10413 [details]
.config

Comment 2 S.Caglar Onur 2007-02-14 03:06:28 UTC

Created attachment 10414 [details]
dmidecode

Comment 3 S.Caglar Onur 2007-02-14 03:07:11 UTC

Created attachment 10415 [details]
acpidump

Comment 4 S.Caglar Onur 2007-02-14 03:11:40 UTC

Ahh forget to add, /proc updated without a problem;

[caglar@zangetsu][~]> cat /proc/acpi/button/lid/LID0/state
state:      open
[caglar@zangetsu][~]> cat /proc/acpi/button/lid/LID0/state
state:      closed

Comment 5 S.Caglar Onur 2007-02-14 03:17:08 UTC

Created attachment 10416 [details]
dmesg

Comment 6 S.Caglar Onur 2007-02-14 03:34:19 UTC

Created attachment 10417 [details]
dmesg

Whoops, i attached wrong dmesg

Comment 7 Zhang Rui 2007-06-05 01:29:39 UTC

Created attachment 11681 [details]
check if acpi_bus_generate_event succeeds

You can add a printk at the beginning of acpi_button_notify and apply this
patch as well.
Open/close the lid and attach the _dmesg_ output, so that we can make sure:
1.if BIOS send a notification to OS when Lid is opened.
2.if OS generate the ACPI event successfully after receiving the notification.

Comment 8 S.Caglar Onur 2007-06-05 02:15:10 UTC

Created attachment 11682 [details]
dmesg.2.6.22_rc4

dmesg.2.6.22_rc4 with #11681 applied

Comment 9 Zhang Rui 2007-09-28 18:56:25 UTC

Hmm, this is similar to the bug 6767.
can you do the same test in comment #10 in bug 6767.

Comment 10 Fu Michael 2007-10-29 18:21:37 UTC

mark bug as unreproducible due to no response from bug reporter. if reopen, please come with needed data. thanks.