kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <gregkh@suse.de>
To: "Américo Wang" <xiyou.wangcong@gmail.com>
Cc: CaT <cat@zip.com.au>, Yong Zhang <yong.zhang@windriver.com>,
	lkml@zip.com.au, mtosatti@redhat.com, kvm@vger.kernel.org,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Avi Kivity <avi@redhat.com>
Subject: Re: 2.6.33.3: possible recursive locking detected
Date: Wed, 12 May 2010 12:46:22 -0700	[thread overview]
Message-ID: <20100512194622.GA31418@suse.de> (raw)
In-Reply-To: <20100512043420.GC5358@cr0.nay.redhat.com>

On Wed, May 12, 2010 at 12:34:20PM +0800, Américo Wang wrote:
> On Tue, May 11, 2010 at 08:03:20AM -0700, Greg KH wrote:
> >On Tue, May 11, 2010 at 09:33:50PM +1000, CaT wrote:
> >> On Wed, May 05, 2010 at 10:52:50AM +0800, Américo Wang wrote:
> >> > On Wed, May 5, 2010 at 10:32 AM, Yong Zhang <yong.zhang@windriver.com> wrote:
> >> > > On Tue, May 04, 2010 at 11:37:37AM +0300, Avi Kivity wrote:
> >> > >> On 05/04/2010 10:03 AM, CaT wrote:
> >> > >> >I'm currently running 2.6.33.3 in a KVM instance emulating a core2duo
> >> > >> >on 1 cpu with virtio HDs running on top of a core2duo host running 2.6.33.3.
> >> > >> >qemu-kvm version 0.12.3.
> >> > >
> >> > > Can you try commit 6992f5334995af474c2b58d010d08bc597f0f2fe in the latest
> >> > > kernel?
> >> > >
> >> > 
> >> > Hmm, 2.6.33 -stable has commit 846f99749ab68bbc7f75c74fec305de675b1a1bf?
> >> > 
> >> > Actually, these 3 commits fixed it:
> >> > 
> >> > 6992f5334995af474c2b58d010d08bc597f0f2fe sysfs: Use one lockdep class
> >> > per sysfs ttribute.
> >> > a2db6842873c8e5a70652f278d469128cb52db70 sysfs: Only take active
> >> > references on attributes.
> >> > e72ceb8ccac5f770b3e696e09bb673dca7024b20 sysfs: Remove sysfs_get/put_active_two
> >> > 
> >> > However, there are many other patches needed to amend these, so I think
> >> > it's not suitable for -stable to include, perhaps a revert of
> >> > 846f99749ab68bbc7f75c74fec305de675b1a1bf is better.
> >> 
> >> Slightly at a loss as to what to do, now. It's a virt instance so I can
> >> apply patches at will but, well, clarity is good. :)
> >
> >Just ignore the lockdep warnings as they are bogus, or turn them off, or
> >use .34-rc7, as they are resolved there.
> >
> 
> How about reverting that patch for 2.6.33 stable tree?

No, as that patch is not reverted in Linus's tree, right?  Just turn off
lockdep if this is bothering you.

thanks,

greg k-h

      reply	other threads:[~2010-05-12 19:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-04  7:03 2.6.33.3: possible recursive locking detected CaT
2010-05-04  8:37 ` Avi Kivity
2010-05-05  2:32   ` Yong Zhang
2010-05-05  2:52     ` Américo Wang
2010-05-11 11:33       ` CaT
2010-05-11 15:03         ` Greg KH
2010-05-12  4:34           ` Américo Wang
2010-05-12 19:46             ` Greg KH [this message]

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=20100512194622.GA31418@suse.de \
    --to=gregkh@suse.de \
    --cc=avi@redhat.com \
    --cc=cat@zip.com.au \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkml@zip.com.au \
    --cc=mtosatti@redhat.com \
    --cc=xiyou.wangcong@gmail.com \
    --cc=yong.zhang@windriver.com \
    /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;
as well as URLs for NNTP newsgroup(s).