From: ebiederm@xmission.com (Eric W. Biederman)
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Yinghai Lu <yinghai@kernel.org>,
Alexander Beregalov <a.beregalov@gmail.com>,
Greg KH <gregkh@suse.de>,
linux-acpi@vger.kernel.org,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
linux-scsi <linux-scsi@vger.kernel.org>
Subject: Re: 2.6.34-rc1: ACPI: BUG: key f71f0168 not in .data!
Date: Fri, 12 Mar 2010 20:58:36 -0800 [thread overview]
Message-ID: <m1sk843jib.fsf@fess.ebiederm.org> (raw)
In-Reply-To: <1268455576.2802.750.camel@mulgrave.site> (James Bottomley's message of "Fri\, 12 Mar 2010 22\:46\:16 -0600")
James Bottomley <James.Bottomley@HansenPartnership.com> writes:
> On Fri, 2010-03-12 at 19:37 -0800, Yinghai Lu wrote:
>> ---
>> drivers/base/attribute_container.c | 1 +
>> drivers/scsi/lpfc/lpfc_attr.c | 3 +++
>> drivers/scsi/scsi_sysfs.c | 15 ++++++++++++---
>> 3 files changed, 16 insertions(+), 3 deletions(-)
>
> Added linux-scsi, since this seems to be a SCSI fix.
>
> I've no idea what actual problem this is fixing, what's the descriptive
> changelog for this?
The root problem is that we occasionally have unrecognized deadlocks
on the sysfs attributes.
For 2.6.33 I managed to get lockdep mostly working but there the way
I did it generated a lot of false positives.
For 2.6.34 I reworked the sysfs lockdep handling to remove the false
positives by treating each sysfs attribute as it's own class. Most
sysfs attributes are declared statically and don't need any
initialization. Unfortunately there are a handful of dynamically
allocated sysfs attributes that do need initialization.
This patch is a fix to get one of those last handful of dynamically
allocated sysfs attributes that I missed and apparently do not
exist in my test system.
None of this has any effect (for now) when lockdep is disabled.
Eric
next prev parent reply other threads:[~2010-03-13 4:58 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-08 23:47 2.6.34-rc1: ACPI: BUG: key f71f0168 not in .data! Alexander Beregalov
2010-03-09 0:12 ` Yinghai Lu
2010-03-09 0:32 ` Yinghai Lu
2010-03-09 0:40 ` Américo Wang
2010-03-09 0:35 ` Eric W. Biederman
2010-03-09 7:06 ` Yinghai Lu
2010-03-09 7:57 ` Eric W. Biederman
2010-03-09 7:57 ` Eric W. Biederman
2010-03-12 2:29 ` Yinghai Lu
2010-03-12 2:32 ` Yinghai Lu
2010-03-13 3:36 ` Yinghai Lu
2010-03-13 3:54 ` Greg KH
2010-03-13 3:37 ` Yinghai Lu
2010-03-13 4:46 ` James Bottomley
2010-03-13 4:58 ` Eric W. Biederman [this message]
2010-03-13 5:13 ` Matthew Wilcox
2010-03-20 7:17 ` [PATCH] scsi: fix lockdep warning Yinghai Lu
2010-03-20 7:57 ` Wolfram Sang
2010-03-20 12:58 ` Greg KH
2010-03-20 15:41 ` James Bottomley
2010-03-20 19:07 ` [PATCH] driver: fix lockdep with sysfs Yinghai Lu
2010-03-30 15:05 ` [PATCH] scsi: fix lockdep warning Ben Gamari
2010-03-30 17:08 ` Yinghai Lu
2010-03-30 20:33 ` Ben Gamari
2010-03-31 1:55 ` Wolfram Sang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m1sk843jib.fsf@fess.ebiederm.org \
--to=ebiederm@xmission.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=a.beregalov@gmail.com \
--cc=gregkh@suse.de \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=yinghai@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox