From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756210Ab0CIAgN (ORCPT ); Mon, 8 Mar 2010 19:36:13 -0500 Received: from out02.mta.xmission.com ([166.70.13.232]:32885 "EHLO out02.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753884Ab0CIAgH convert rfc822-to-8bit (ORCPT ); Mon, 8 Mar 2010 19:36:07 -0500 To: Yinghai Lu Cc: Alexander Beregalov , Greg KH , linux-acpi@vger.kernel.org, Linux Kernel Mailing List Subject: Re: 2.6.34-rc1: ACPI: BUG: key f71f0168 not in .data! References: <86802c441003081612r2f44efd6n9664b4b7c98a636a@mail.gmail.com> From: ebiederm@xmission.com (Eric W. Biederman) Date: Mon, 08 Mar 2010 16:35:59 -0800 In-Reply-To: <86802c441003081612r2f44efd6n9664b4b7c98a636a@mail.gmail.com> (Yinghai Lu's message of "Mon\, 8 Mar 2010 16\:12\:38 -0800") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT X-XM-SPF: eid=;;;mid=;;;hst=in01.mta.xmission.com;;;ip=76.21.114.89;;;frm=ebiederm@xmission.com;;;spf=neutral X-SA-Exim-Connect-IP: 76.21.114.89 X-SA-Exim-Mail-From: ebiederm@xmission.com X-SA-Exim-Scanned: No (on in01.mta.xmission.com); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Yinghai Lu writes: Yep this looks like a couple more cases of drivers that don't statically declare and allocate their sysfs attributes. The ipmi driver looks like it just wants to waste memory. I haven't made sense of the thermal driver yet. Something like the untested patch below should fix the impi driver. Eric > On Mon, Mar 8, 2010 at 3:47 PM, Alexander Beregalov > wrote: >> Hi >> >> Kernel version is 2.6.34-rc1 >> >> bus: 'acpi': add driver processor >> bus: 'acpi': driver_probe_device: matched device LNXCPU:00 with driver processor >> bus: 'acpi': really_probe: probing driver processor with device LNXCPU:00 >> device: 'cooling_device0': device_add >> driver: 'LNXCPU:00': driver_bound: bound to device 'processor' >> bus: 'acpi': really_probe: bound device LNXCPU:00 to driver processor >> bus: 'acpi': add driver thermal >> bus: 'acpi': driver_probe_device: matched device LNXTHERM:00 with driver thermal >> bus: 'acpi': really_probe: probing driver thermal with device LNXTHERM:00 >> bus: 'acpi': driver_probe_device: matched device LNXTHERM:01 with driver thermal >> bus: 'acpi': really_probe: probing driver thermal with device LNXTHERM:01 >> device: 'thermal_zone0': device_add >> BUG: key f71f0168 not in .data! >> ------------[ cut here ]------------ >> WARNING: at kernel/lockdep.c:2706 lockdep_init_map+0x16f/0x4f0() >> Hardware name: >> Modules linked in: >> Pid: 1, comm: swapper Not tainted 2.6.34-rc1 #1 >> Call Trace: >>  [] ? printk+0x18/0x20 >>  [] warn_slowpath_common+0x6d/0xa0 >>  [] ? lockdep_init_map+0x16f/0x4f0 >>  [] ? lockdep_init_map+0x16f/0x4f0 >>  [] ? do_one_initcall+0x0/0x190 >>  [] warn_slowpath_null+0x15/0x20 >>  [] lockdep_init_map+0x16f/0x4f0 >>  [] sysfs_add_file_mode+0x58/0xb0 >>  [] sysfs_add_file+0x13/0x20 >> Switched to NOHz mode on CPU #0 >>  [] sysfs_create_file+0x26/0x30 >>  [] device_create_file+0x13/0x20 >>  [] thermal_zone_bind_cooling_device+0x1e4/0x2f0 >>  [] ? acpi_bus_data_handler+0x0/0x5 >>  [] acpi_thermal_cooling_device_cb+0x70/0x11e >>  [] ? thermal_zone_device_register+0x260/0x400 >>  [] ? thermal_zone_bind_cooling_device+0x0/0x2f0 >>  [] acpi_thermal_bind_cooling_device+0xd/0xf >>  [] thermal_zone_device_register+0x2d9/0x400 >>  [] acpi_thermal_add+0x176/0x398 >>  [] ? sysfs_do_create_link+0xa1/0x140 >>  [] acpi_device_probe+0x3a/0xf4 >>  [] driver_probe_device+0xb7/0x1b0 >>  [] __driver_attach+0x89/0x90 >>  [] bus_for_each_dev+0x48/0x70 >>  [] driver_attach+0x19/0x20 >>  [] ? __driver_attach+0x0/0x90 >>  [] bus_add_driver+0xcb/0x230 >>  [] ? acpi_device_remove+0x0/0x90 >>  [] driver_register+0x65/0x110 >>  [] ? kernel_init+0x0/0x120 >>  [] ? proc_mkdir_mode+0x33/0x50 >>  [] ? kernel_init+0x0/0x120 >>  [] acpi_bus_register_driver+0x3a/0x3f >>  [] acpi_thermal_init+0x50/0x6f >>  [] do_one_initcall+0x2d/0x190 >>  [] ? kernel_init+0x0/0x120 >>  [] ? acpi_thermal_init+0x0/0x6f >>  [] ? kernel_init+0x0/0x120 >>  [] kernel_init+0x87/0x120 >>  [] kernel_thread_helper+0x6/0x1c >> ---[ end trace a7919e7f17c0a725 ]--- > > > got > > [ 145.239329] BUG: key ffff88206fe99c58 not in .data! > [ 145.244994] ------------[ cut here ]------------ > [ 145.250949] WARNING: at kernel/lockdep.c:2706 lockdep_init_map+0xf1/0x53c() > [ 145.262424] Hardware name: Sun Fire x4800 > [ 145.267543] Modules linked in: > [ 145.275078] Pid: 1, comm: swapper Not tainted 2.6.34-rc1-yh-dirty #221 > [ 145.285140] Call Trace: > [ 145.290143] [] warn_slowpath_common+0x7c/0x94 > [ 145.304624] [] warn_slowpath_null+0x14/0x16 > [ 145.314632] [] lockdep_init_map+0xf1/0x53c > [ 145.325228] [] ? sysfs_new_dirent+0x8f/0x104 > [ 145.334331] [] sysfs_add_file_mode+0x66/0xac > [ 145.339850] [] sysfs_add_file+0x11/0x13 > [ 145.355513] [] sysfs_create_file+0x2a/0x2c > [ 145.360637] [] device_create_file+0x19/0x1b > [ 145.375187] [] ipmi_register_smi+0x862/0xd72 > [ 145.384606] [] ? put_device+0x17/0x19 > [ 145.395142] [] ? device_add+0x44b/0x52a > [ 145.403666] [] ? autoremove_wake_function+0x0/0x39 > [ 145.416309] [] try_smi_init+0x6d0/0x8a0 > [ 145.424534] [] init_ipmi_si+0x4cf/0x907 > [ 145.434799] [] ? init_ipmi_si+0x0/0x907 > [ 145.439677] [] do_one_initcall+0x72/0x17f > [ 145.455681] [] kernel_init+0x166/0x1f0 > [ 145.463319] [] kernel_thread_helper+0x4/0x10 > [ 145.476497] [] ? restore_args+0x0/0x30 > [ 145.485014] [] ? kernel_init+0x0/0x1f0 > [ 145.495859] [] ? kernel_thread_helper+0x0/0x10 > [ 145.501541] ---[ end trace 0ec12b1482c47368 ]--- diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c index ec5e3f8..16e9073 100644 --- a/drivers/char/ipmi/ipmi_msghandler.c +++ b/drivers/char/ipmi/ipmi_msghandler.c @@ -2272,42 +2272,52 @@ static int create_files(struct bmc_device *bmc) bmc->device_id_attr.attr.name = "device_id"; bmc->device_id_attr.attr.mode = S_IRUGO; bmc->device_id_attr.show = device_id_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->provides_dev_sdrs_attr.attr.name = "provides_device_sdrs"; bmc->provides_dev_sdrs_attr.attr.mode = S_IRUGO; bmc->provides_dev_sdrs_attr.show = provides_dev_sdrs_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->revision_attr.attr.name = "revision"; bmc->revision_attr.attr.mode = S_IRUGO; bmc->revision_attr.show = revision_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->firmware_rev_attr.attr.name = "firmware_revision"; bmc->firmware_rev_attr.attr.mode = S_IRUGO; bmc->firmware_rev_attr.show = firmware_rev_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->version_attr.attr.name = "ipmi_version"; bmc->version_attr.attr.mode = S_IRUGO; bmc->version_attr.show = ipmi_version_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->add_dev_support_attr.attr.name = "additional_device_support"; bmc->add_dev_support_attr.attr.mode = S_IRUGO; bmc->add_dev_support_attr.show = add_dev_support_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->manufacturer_id_attr.attr.name = "manufacturer_id"; bmc->manufacturer_id_attr.attr.mode = S_IRUGO; bmc->manufacturer_id_attr.show = manufacturer_id_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->product_id_attr.attr.name = "product_id"; bmc->product_id_attr.attr.mode = S_IRUGO; bmc->product_id_attr.show = product_id_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->guid_attr.attr.name = "guid"; bmc->guid_attr.attr.mode = S_IRUGO; bmc->guid_attr.show = guid_show; + sysfs_attr_init(&bmc->device_id_attr.attr); bmc->aux_firmware_rev_attr.attr.name = "aux_firmware_revision"; bmc->aux_firmware_rev_attr.attr.mode = S_IRUGO; bmc->aux_firmware_rev_attr.show = aux_firmware_rev_show; + sysfs_attr_init(&bmc->device_id_attr.attr); err = device_create_file(&bmc->dev->dev, &bmc->device_id_attr);