Bug 679452 - WARNING: at fs/proc/generic.c:586 proc_register+0x155/0x192()
Summary: WARNING: at fs/proc/generic.c:586 proc_register+0x155/0x192()
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: 15
Hardware: x86_64
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: John Feeney
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 722848 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-02-22 15:41 UTC by Jaroslav Reznik
Modified: 2013-01-10 07:34 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-09-27 13:47:07 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
acpidump output (162.34 KB, application/octet-stream)
2011-02-25 12:33 UTC, Jaroslav Reznik
no flags Details
Generate unique names for ACPI buttons (1.88 KB, patch)
2011-02-25 15:47 UTC, Matthew Garrett
no flags Details | Diff

Description Jaroslav Reznik 2011-02-22 15:41:11 UTC
Description of problem: kernel oops while booting Lenovo Ideapad S10-3t

[    0.875123] ACPI: Deprecated procfs I/F for AC is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
[    0.885145] ACPI: AC Adapter [ACAD] (on-line)
[    0.885472] input: Power Button as /devices/LNXSYSTM:00/device:00/PNP0A08:00/PNP0C0C:00/input/input0
[    0.885483] ACPI: Power Button [PWRB]
[    0.885657] input: Lid Switch as /devices/LNXSYSTM:00/device:00/PNP0C0D:00/input/input1
[    0.900186] ACPI: Lid Switch [LID]
[    0.900229] ------------[ cut here ]------------
[    0.900246] WARNING: at fs/proc/generic.c:586 proc_register+0x155/0x192()
[    0.900254] Hardware name: 20040M18        
[    0.900261] proc_dir_entry 'power/PWRB' already registered
[    0.900267] Modules linked in:
[    0.900278] Pid: 1, comm: swapper Not tainted 2.6.38-0.rc5.git1.1.fc15.x86_64 #1
[    0.900285] Call Trace:
[    0.900301]  [<ffffffff8105538e>] ? warn_slowpath_common+0x83/0x9b
[    0.900314]  [<ffffffff81055449>] ? warn_slowpath_fmt+0x46/0x48
[    0.900325]  [<ffffffff8116ff0e>] ? proc_register+0x155/0x192
[    0.900334]  [<ffffffff81170629>] ? proc_mkdir_mode+0x3f/0x57
[    0.900343]  [<ffffffff81170657>] ? proc_mkdir+0x16/0x18
[    0.900352]  [<ffffffff8129a5a2>] ? acpi_button_add+0x228/0x3e1
[    0.900362]  [<ffffffff8127754e>] ? acpi_device_probe+0x4e/0x11c
[    0.900372]  [<ffffffff812e2e01>] ? driver_probe_device+0x124/0x1ff
[    0.900380]  [<ffffffff812e2f36>] ? __driver_attach+0x5a/0x7e
[    0.900388]  [<ffffffff812e2edc>] ? __driver_attach+0x0/0x7e
[    0.900396]  [<ffffffff812e1e5d>] ? bus_for_each_dev+0x53/0x87
[    0.900404]  [<ffffffff812e2a0e>] ? driver_attach+0x1e/0x20
[    0.900411]  [<ffffffff812e2623>] ? bus_add_driver+0xe2/0x235
[    0.900421]  [<ffffffff81b8cdfe>] ? acpi_button_init+0x0/0x56
[    0.900429]  [<ffffffff812e311f>] ? driver_register+0x98/0x105
[    0.900436]  [<ffffffff81b8cdfe>] ? acpi_button_init+0x0/0x56
[    0.900445]  [<ffffffff81277d14>] ? acpi_bus_register_driver+0x43/0x45
[    0.900452]  [<ffffffff81b8ce32>] ? acpi_button_init+0x34/0x56
[    0.900462]  [<ffffffff810020a4>] ? do_one_initcall+0x7f/0x137
[    0.900471]  [<ffffffff81b57e88>] ? kernel_init+0x228/0x2a2
[    0.900480]  [<ffffffff8100a9e4>] ? kernel_thread_helper+0x4/0x10
[    0.900489]  [<ffffffff81b57c60>] ? kernel_init+0x0/0x2a2
[    0.900496]  [<ffffffff8100a9e0>] ? kernel_thread_helper+0x0/0x10
[    0.900512] ---[ end trace e93713a9d40cd06c ]---
[    0.900666] input: Power Button as /devices/LNXSYSTM:00/device:00/PNP0C0C:01/input/input2
[    0.900677] ACPI: Power Button [PWRB]
[    0.900856] input: Sleep Button as /devices/LNXSYSTM:00/device:00/PNP0C0E:00/input/input3
[    0.900866] ACPI: Sleep Button [SLPB]
[    0.901061] input: Power Button as /devices/LNXSYSTM:00/LNXPWRBN:00/input/input4
[    0.901071] ACPI: Power Button [PWRF]
[    0.901429] ACPI: acpi_idle yielding to intel_idle
[    0.930101] thermal LNXTHERM:00: registered as thermal_zone0
[    0.930108] ACPI: Thermal Zone [TZ01] (36 C)


How reproducible:
Everytime

  
Actual results:
Kerneloop

Expected results:
No crash, no abrt warning

Additional info: feel free to ask for any additional info!

Comment 1 Matthew Garrett 2011-02-24 19:15:55 UTC
Awesome. Your DSDT appears to define two power buttons. Can you install pmtools, run acpidump as root and attach the output?

Comment 2 Jaroslav Reznik 2011-02-25 12:33:09 UTC
Created attachment 480980 [details]
acpidump output

Comment 3 Matthew Garrett 2011-02-25 15:34:32 UTC
Your ACPI tables are technically correct but practically pathological. I'll figure out a workaround.

Comment 4 Matthew Garrett 2011-02-25 15:47:29 UTC
Created attachment 481022 [details]
Generate unique names for ACPI buttons

Can you test this patch?

Comment 5 Jaroslav Reznik 2011-02-28 16:46:47 UTC
Kernel now exports PWRB, PWRB_1 and PWRF_2 (shouldn't be now PWRF?), no oops.

Comment 6 Matthew Garrett 2011-03-11 20:41:32 UTC
That's odd, I can't find any sign of PWRF in your DSDT. But I'll pass this upstream anyway, since it seems like a good idea.

Comment 7 Josh Boyer 2011-09-27 13:47:07 UTC
Matthew sent this upstream, but they chose to remove this code instead.  See:

http://comments.gmane.org/gmane.linux.acpi.devel/49592

The commit to remove the code was in the 2.6.39 kernel, so we should have that in the latest F15 updates which are 3.0 based.  If you are still seeing problems, please reopen.

Comment 8 Josh Boyer 2011-09-27 16:22:07 UTC
*** Bug 722848 has been marked as a duplicate of this bug. ***


Note You need to log in before you can comment on or make changes to this bug.