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)
next prev parent 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).