From: leif.lindholm@arm.com (Leif Lindholm)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: Add SWP/SWPB emulation for ARMv7 processors
Date: Fri, 18 Dec 2009 14:15:24 -0000 [thread overview]
Message-ID: <000001ca7fec$8aca3ec0$a05ebc40$@lindholm@arm.com> (raw)
In-Reply-To: <13B9B4C6EF24D648824FF11BE896716203A7FA69CC@dlee02.ent.ti.com>
Thanks for the feedback.
> From: Woodruff, Richard [mailto:r-woodruff2 at ti.com]
> Sent: 17 December 2009 18:20
> Still, the exclusive operations for some systems don't extend beyond an
> ARM smp-cluster. Not all cores and interconnect implementations support
> the range lock. As such from a SOC point of view SWP still has value
> as it may exist where lwrex/swrex would fail. It is also much lighter
> than using some kind of mailbox alternative.
>
> While the ARM is important to many SOCs its not the whole thing. I've
> seen some side threads where people are busy blindly removing SWP and
> quoting half information.
There is nothing half about SWP/SWPB being deprecated from ARMv6 onwards.
Also, not all systems properly implement the bus locking support required
for SWP to be properly atomic externally.
> My main comment on patch in this regard is to make the point you won't
> be able to fully emulate a SWP with the sequence you posted if you
> consider other cores outside the ARM cluster. Some note to that effect
> would be positive in the description.
I would argue that this is not a property of the patch, but rather the
architectural deprecation of the instruction(s).
However, I take your point. What if I add a comment that this emulation
will not work on uncached memory regions where no global exclusive
monitor is implemented?
Regards
/
Leif
next prev parent reply other threads:[~2009-12-18 14:15 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-17 17:54 [PATCH] ARM: Add SWP/SWPB emulation for ARMv7 processors Leif Lindholm
2009-12-17 18:19 ` Woodruff, Richard
2009-12-17 19:16 ` Jamie Lokier
2009-12-17 20:43 ` Woodruff, Richard
2009-12-18 20:31 ` Jamie Lokier
2010-01-08 23:20 ` Woodruff, Richard
2009-12-18 14:15 ` Leif Lindholm [this message]
2009-12-17 19:32 ` Jamie Lokier
2009-12-18 10:00 ` Catalin Marinas
2009-12-18 10:06 ` Catalin Marinas
2009-12-18 17:01 ` Leif Lindholm
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='000001ca7fec$8aca3ec0$a05ebc40$@lindholm@arm.com' \
--to=leif.lindholm@arm.com \
--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.