All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Vrabel <david.vrabel@citrix.com>
To: Sander Eikelenboom <linux@eikelenboom.it>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Wei Liu <wei.liu2@citrix.com>
Subject: Re: BUG: using smp_processor_id() in preemptible [00000000] code: blkback.1.xvdb/9138 caller is decrease_reservation
Date: Wed, 11 Sep 2013 11:09:05 +0100	[thread overview]
Message-ID: <52304141.7040309@citrix.com> (raw)
In-Reply-To: <1323058712.20130910201353@eikelenboom.it>

On 10/09/13 19:13, Sander Eikelenboom wrote:
> Hi Wei,
> 
> Just back from holiday i tried running a new xen-unstable and linux
> kernel (current Linus his tree + Konrad's last pull request merged on top).
> I saw a thread and patch about a bug_on in increase_reservation ...
> i'm seeing the splat below in dom0 when guests get started.

Yes, the use of __per_cpu() in decrease_reservation is not safe.

Stefano, what testing did you give "xen/balloon: set a mapping for
ballooned out pages" (cd9151e2).  The number of critical problems it's
had suggests not a lot?

I'm also becoming less happy with the inconsistency between the p2m
updates between the different (non-)auto_translated_physmap guest types.

I think it (and all the attempts to fix it) should be reverted at this
stage and we should try again for 3.13 which some more through testing
and a more careful look at what updates to the p2m are needed.

David

  parent reply	other threads:[~2013-09-11 10:09 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-10 18:13 BUG: using smp_processor_id() in preemptible [00000000] code: blkback.1.xvdb/9138 caller is decrease_reservation Sander Eikelenboom
2013-09-11  9:25 ` Wei Liu
2013-09-11 10:09 ` David Vrabel [this message]
2013-09-11 12:08   ` Stefano Stabellini
2013-09-11 12:39     ` David Vrabel
2013-09-11 16:25       ` Sander Eikelenboom
2013-09-11 17:04         ` Sander Eikelenboom
2013-09-11 17:15           ` Stefano Stabellini
2013-09-11 17:29             ` Wei Liu
2013-09-11 17:35               ` Stefano Stabellini
2013-09-11 17:42                 ` Konrad Rzeszutek Wilk
2013-09-11 17:44                   ` Stefano Stabellini
2013-09-11 17:55                 ` Stefano Stabellini
2013-09-11 19:16                   ` Wei Liu
2013-09-11 17:58             ` Sander Eikelenboom
2013-09-11 16:31       ` Stefano Stabellini

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=52304141.7040309@citrix.com \
    --to=david.vrabel@citrix.com \
    --cc=linux@eikelenboom.it \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.