linux-kernel - Re: 2.6.30 oops with acpi/button

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <1247194664.3568.13.camel@localhost.localdomain>
Date:	Fri, 10 Jul 2009 10:57:44 +0800
From:	ykzhao <yakui.zhao@...el.com>
To:	"Stephen J. Gowdy" <gowdy@...n.ch>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>
Subject: Re: 2.6.30 oops with acpi/button

On Thu, 2009-07-09 at 16:29 +0800, Stephen J. Gowdy wrote:
> Hi Len & Bjorn,
>  	I've not had time to go back to it but with 2.6.30 I get an oops 
> when I close the lid on my HP Compaq 6910p laptop. I see you removed some 
> null pointer checks.. perhaps tha tis why? Here is the oops extracted from 
> my log file. If you want me to test a patch let me know. I'm about to try 
> 2.6.30.1 but I don't see anything in the ChangeLog that is likely to make 
> a difference.
>  	I've gone back to 2.6.29.4 where it is fine. I tried unloading the 
> "button" module first but in that case my laptop locked up without an 
> oops. No flashing num/caps lock or anything, just froze when I close the 
> lid. The version of button.c from 2.6.29.4  doesn't compile in the 2.6.30 
> tree;
> 
> drivers/acpi/button.c: In function ‘acpi_button_add_fs’:
> drivers/acpi/button.c:203: error: ‘struct proc_dir_entry’ has no member 
> named ‘owner’
> drivers/acpi/button.c:208: error: ‘struct proc_dir_entry’ has no member 
> named ‘owner’
> drivers/acpi/button.c: In function ‘acpi_button_init’:
> drivers/acpi/button.c:525: error: ‘struct proc_dir_entry’ has no member 
> named ‘owner’
The proc_dir_entry::owner member is removed after the following commit
is shipped:
    commit 99b76233803beab302123d243eea9e41149804f3
Author: Alexey Dobriyan <adobriyan@...il.com>
Date:   Wed Mar 25 22:48:06 2009 +0300

    proc 2/2: remove struct proc_dir_entry::owner

It seems that the owner member is already removed in the button driver.
Why does the above issue still exist?

Will you please try the latest kernel and see whether the issue still
exists?

For the LID oops issue on the HP6910P, will you please enable
"CONFIG_DRM_I915_KMS" in kernel configuration and see whether the issue
still exists? This had better be done in the latest kernel(for example:
2.6.31-rc1/2)

thanks.
   Yakui

> 
>  	Any ideas?
> 
> 
>  						regards,
> 
>  						Stephen.
> 
> -- 
>   /------------------------------------+-------------------------\
> |Stephen J. Gowdy                     | CERN       Office: 8-1-11|
> |http://cern.ch/gowdy/                | CH-1211 Geneva 23        |
> |                                     | Switzerland              |
> |EMail: gowdy@...n.ch                 | Tel: +41 76 487 2215     |
>   \------------------------------------+-------------------------/

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/