[PATCH 1/4] ARM: Change the mandatory barriers implementation

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Feb 23 06:41:16 EST 2010


On Tue, Feb 23, 2010 at 05:05:31PM +0530, Shilimkar, Santosh wrote:
> OMAP3 and family would be ok with this much needed mandatory barriers
> but for OMAP4, we need platform specified barriers as you mentioned. Will
> have a look at this series and do some testing on OMAP4. Will send
> a follow up patch then.

Wait for v2 - it introduces barriers onto the older CPUs which do not
need it, and therefore is definitely not right.

Eg, ARMv4 CPUs don't need the drain-write-buffer instruction for mb()
and wmb() - we have no relaxed behaviours on those CPUs, so this is
pointless.  As I also pointed out, it breaks ARMv3 because the drain-
write-buffer instruction there is an undefined instruction.

Plus, non-coherent Xscale has the same requirements as ARMv4, so getting
rid of the arch_is_coherent() stuff was completely wrong.

Basically, the patch is not suitable as it currently stands, and is in
need of rework.



More information about the linux-arm-kernel mailing list