Re: [PATCH] ARM: v6: prevent gcc from reordering extended CP15 reads above is_smp() test

2013-07-30 Thread Paul Walmsley
Hi Will On Mon, 29 Jul 2013, Will Deacon wrote: I wouldn't worry about checking for CPU_V6. Besides, we probably need this to be re-evaluated across barrier() when we get CPU migration on a big-little platform anyway (we should probably also drop the __attribute_const__ for that). So you

Re: [PATCH] ARM: v6: prevent gcc from reordering extended CP15 reads above is_smp() test

2013-07-29 Thread Tony Lindgren
Hi, * Paul Walmsley p...@pwsan.com [130728 13:23]: Commit 621a0147d5c921f4cc33636ccd0602ad5d7cbfbc (ARM: 7757/1: mm: don't flush icache in switch_mm with hardware broadcasting) breaks the boot on OMAP2430SDP with omap2plus_defconfig. Tracked to an undefined instruction abort from the CP15

Re: [PATCH] ARM: v6: prevent gcc from reordering extended CP15 reads above is_smp() test

2013-07-29 Thread Will Deacon
Hi Paul, On Sun, Jul 28, 2013 at 09:16:29PM +0100, Paul Walmsley wrote: Commit 621a0147d5c921f4cc33636ccd0602ad5d7cbfbc (ARM: 7757/1: mm: don't flush icache in switch_mm with hardware broadcasting) breaks the boot on OMAP2430SDP with omap2plus_defconfig. Tracked to an undefined instruction

[PATCH] ARM: v6: prevent gcc from reordering extended CP15 reads above is_smp() test

2013-07-28 Thread Paul Walmsley
Commit 621a0147d5c921f4cc33636ccd0602ad5d7cbfbc (ARM: 7757/1: mm: don't flush icache in switch_mm with hardware broadcasting) breaks the boot on OMAP2430SDP with omap2plus_defconfig. Tracked to an undefined instruction abort from the CP15 read in cache_ops_need_broadcast(). It turns out that