All of lore.kernel.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: clean-up common multi-platform kconfig options
Date: Sun, 8 Dec 2013 04:02:41 +0100	[thread overview]
Message-ID: <201312080402.42101.arnd@arndb.de> (raw)
In-Reply-To: <alpine.LFD.2.10.1312072111410.15948@knanqh.ubzr>

On Sunday 08 December 2013, Nicolas Pitre wrote:
> The right fix is probably to leave it as is and optimize your user space 
> for ARMv6k instead, meaning that this operation would then be 
> implemented inline rather than relying on the kernel helper.

That would not make the program run on ARMv6 though. The bug we're trying
to fix is for running a kernel with ARMv6k support built-in on an
OMAP2 with ARMv6. The kuser helper already supports cmpxchg for pure
v6 builds, but that code doesn't get built (and isn't SMP safe) with
CONFIG_CPU_V6K or CONFIG_CPU_V7 enabled.
 
> Otherwise if you really want to optimize the kernel helper at runtime 
> for ARMv6k when ARMv6 is also configured in, then you could do some 
> instruction patching in kuser_init() as we already do for getting the 
> TLS value.

This does sound like the cleanest solution.

	Arnd

  reply	other threads:[~2013-12-08  3:02 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-05 16:58 [PATCH] ARM: clean-up common multi-platform kconfig options Rob Herring
2013-12-05 20:25 ` Arnd Bergmann
2013-12-05 21:34   ` Rob Herring
2013-12-05 21:50     ` Arnd Bergmann
2013-12-06  4:10       ` Rob Herring
2013-12-06 16:41         ` Arnd Bergmann
2013-12-06 16:59           ` Nicolas Pitre
2013-12-06 20:01           ` Rob Herring
2013-12-07  4:52             ` Arnd Bergmann
2013-12-07 17:41               ` Tony Lindgren
2013-12-07 18:10             ` Russell King - ARM Linux
2013-12-07 20:49               ` Rob Herring
2013-12-08  2:21                 ` Nicolas Pitre
2013-12-08  3:02                   ` Arnd Bergmann [this message]
2013-12-08  3:39               ` Arnd Bergmann
2013-12-07 18:02           ` Russell King - ARM Linux
2013-12-07 18:31             ` Arnd Bergmann
2013-12-11 16:41 ` Michal Simek

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=201312080402.42101.arnd@arndb.de \
    --to=arnd@arndb.de \
    --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 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.