linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] [ARM] highmem: fix SMP preemption bug in kmap_high_l1_vipt
Date: Thu, 29 Jul 2010 22:21:17 +0100	[thread overview]
Message-ID: <20100729212117.GB28330@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <alpine.LFD.2.00.1007291058140.20337@xanadu.home>

On Thu, Jul 29, 2010 at 10:59:52AM -0400, Nicolas Pitre wrote:
> On Wed, 28 Jul 2010, gking at nvidia.com wrote:
> 
> > From: Andrew Howe <ahowe@nvidia.com>
> > 
> > smp_processor_id() must not be called from a preemptible context (this
> > is checked by CONFIG_DEBUG_PREEMPT).  kmap_high_l1_vipt() was doing so.
> > This lead to a problem where the wrong per_cpu kmap_high_l1_vipt_depth
> > could be incremented, causing a BUG_ON(*depth <= 0); in
> > kunmap_high_l1_vipt().
> > 
> > The solution is to move the call to smp_processor_id() after the call
> > to preempt_disable().
> > 
> > Signed-off-by: Gary King <gking@nvidia.com>
> 
> Acked-by: Nicolas Pitre <nico.as.pitre@linaro.org>
> 
> Please forward to RMK's patch system.

Unfortunately, I suspect that none of the remaining stuff in my master
branch will make 2.6.35 (which is due this evening) especially as I've
put the fixes for the Versatile/Realview/Versatile Express MMC card
detection in there today, and we still don't know what's right for its
GPIO-based card detection.  (The change to fix the status method may
have broken the GPIO-based method.)

So I'm not sending a pull request this evening.  Hopefully people can
work out what's right for the GPIO based card detection while I'm out
tomorrow, and report back so it can be fixed in the evening if
necessary.

      reply	other threads:[~2010-07-29 21:21 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-29  1:25 [PATCH] [ARM] highmem: fix SMP preemption bug in kmap_high_l1_vipt gking at nvidia.com
2010-07-29 14:59 ` Nicolas Pitre
2010-07-29 21:21   ` Russell King - ARM Linux [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=20100729212117.GB28330@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).