xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Keir Fraser <keir.fraser@eu.citrix.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: New release candidate for Xen 4.0.0 (RC9)
Date: Wed, 19 May 2010 13:10:36 +0100	[thread overview]
Message-ID: <C81993CC.149D7%keir.fraser@eu.citrix.com> (raw)
In-Reply-To: <20100331210716.GA31475@phenom.dumpdata.com>

On 31/03/2010 22:07, "Konrad Rzeszutek Wilk" <konrad.wilk@oracle.com> wrote:

>> I'm only interested in fixing regressions right now for 4.0.0. If this bug
>> has existed since at least 3.4 then the fix (when we have one!) can wait a
>> bit longer, for 4.0.1. My guess is that the MTRR interface is getting little
> 
> Yeah. I get the same failure with 3.4 from xen-3.4-testing:

I didn't reproduce this with my setup, but the crash below is because
spin_unlock() is finding that the lock is not locked on entry. This is the
set_atomicity_lock as acquired/released by prepare_set/post_set. Looking at
the code they seem to be used in matched pairs and I can't see any memory
corruption issues or anything. If you want to get this fixed then you'll
have to add some tracing to find out what's going on (e.g, dump the lock
state at the end of prepare_set and start of post_set). If you can reproduce
this deterministically with your setup then should be pretty easy to nail
this bug.

 -- Keir

> (XEN)    [<ffff828c8011bd14>] _spin_unlock+0xc/0x22
> (XEN)    [<ffff828c80184265>] post_set+0x70/0x72
> (XEN)    [<ffff828c801843d7>] generic_set_mtrr+0xd9/0xf3
> (XEN)    [<0000000000000002>] ???
> (XEN)    
> (XEN) 
> (XEN) ****************************************
> (XEN) Panic on CPU 0:
> (XEN) Assertion '((lock)->lock <= 0)' failed at
> /home/konrad/git/tip/nex-VI/xen/xen/include/asm/sp:18
> (XEN) ****************************************
> (XEN) 

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

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-31  9:27 New release candidate for Xen 4.0.0 (RC9) Keir Fraser
2010-03-31 10:07 ` Joanna Rutkowska
2010-03-31 10:54   ` Keir Fraser
2010-03-31 11:00     ` Joanna Rutkowska
2010-03-31 11:13       ` Keir Fraser
2010-03-31 12:38       ` Ian Pratt
2010-03-31 14:20         ` Dan Magenheimer
2010-03-31 14:32           ` Keir Fraser
2010-03-31 14:44           ` Ian Pratt
2010-03-31 19:55           ` Grant McWilliams
2010-03-31 15:02 ` Bastian Blank
2010-03-31 15:09   ` Keir Fraser
2010-03-31 15:44     ` Bastian Blank
2010-03-31 16:38       ` Keir Fraser
2010-03-31 18:03         ` Konrad Rzeszutek Wilk
2010-03-31 18:18           ` Keir Fraser
2010-03-31 20:40             ` Jeremy Fitzhardinge
2010-03-31 19:10 ` Konrad Rzeszutek Wilk
2010-03-31 19:33   ` Keir Fraser
2010-03-31 19:52     ` Konrad Rzeszutek Wilk
2010-03-31 20:07       ` Keir Fraser
2010-03-31 21:07         ` Konrad Rzeszutek Wilk
2010-05-19 12:10           ` Keir Fraser [this message]
2010-05-26 15:02             ` Konrad Rzeszutek Wilk

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=C81993CC.149D7%keir.fraser@eu.citrix.com \
    --to=keir.fraser@eu.citrix.com \
    --cc=konrad.wilk@oracle.com \
    --cc=xen-devel@lists.xensource.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).