Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-30 Thread Po-Yu Chuang
On Thu, May 30, 2013 at 5:12 PM, Will Deacon wrote: > On Thu, May 30, 2013 at 09:15:26AM +0100, Po-Yu Chuang wrote: >> On Wed, May 29, 2013 at 5:34 PM, Po-Yu Chuang >> wrote: >> > Hi Will, >> > >> > On Wed, May 29, 2013 at 4:54 PM, Will Deacon wrote: >> >> On Wed, May 29, 2013 at 03:14:58AM +01

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-30 Thread Will Deacon
On Thu, May 30, 2013 at 09:15:26AM +0100, Po-Yu Chuang wrote: > On Wed, May 29, 2013 at 5:34 PM, Po-Yu Chuang > wrote: > > Hi Will, > > > > On Wed, May 29, 2013 at 4:54 PM, Will Deacon wrote: > >> On Wed, May 29, 2013 at 03:14:58AM +0100, Po-Yu Chuang wrote: > >>> Will, > >>> I guess nobody noti

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-30 Thread Po-Yu Chuang
On Wed, May 29, 2013 at 5:34 PM, Po-Yu Chuang wrote: > Hi Will, > > On Wed, May 29, 2013 at 4:54 PM, Will Deacon wrote: >> On Wed, May 29, 2013 at 03:14:58AM +0100, Po-Yu Chuang wrote: >>> Will, >>> I guess nobody noticed this because the MMU of later v7 processors >>> fetches page table >>> from

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-29 Thread Po-Yu Chuang
Hi Will, On Wed, May 29, 2013 at 4:54 PM, Will Deacon wrote: > On Wed, May 29, 2013 at 03:14:58AM +0100, Po-Yu Chuang wrote: >> Will, >> I guess nobody noticed this because the MMU of later v7 processors >> fetches page table >> from D-cache. It even doesn't need to clean pmd to PoU. > > It does

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-29 Thread Will Deacon
On Wed, May 29, 2013 at 03:14:58AM +0100, Po-Yu Chuang wrote: > Will, > I guess nobody noticed this because the MMU of later v7 processors > fetches page table > from D-cache. It even doesn't need to clean pmd to PoU. It does if it's UP. The walker is only guaranteed to read from L1 if you have th

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Po-Yu Chuang
On Wed, May 29, 2013 at 2:48 AM, Sricharan R wrote: > > On Tuesday 28 May 2013 07:37 PM, Will Deacon wrote: > > On Tue, May 28, 2013 at 03:03:36PM +0100, Sricharan R wrote: > >> On Tuesday 28 May 2013 06:35 PM, Will Deacon wrote: > >>> On Tue, May 28, 2013 at 11:48:20AM +0100, Po-Yu Chuang wrote:

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Sricharan R
On Tuesday 28 May 2013 07:37 PM, Will Deacon wrote: > On Tue, May 28, 2013 at 03:03:36PM +0100, Sricharan R wrote: >> On Tuesday 28 May 2013 06:35 PM, Will Deacon wrote: >>> On Tue, May 28, 2013 at 11:48:20AM +0100, Po-Yu Chuang wrote: This bug was introduced in commit e651eab0. Some v4/v

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Catalin Marinas
On Tue, May 28, 2013 at 02:05:02PM +0100, Will Deacon wrote: > On Tue, May 28, 2013 at 11:48:20AM +0100, Po-Yu Chuang wrote: > > This bug was introduced in commit e651eab0. > > Some v4/v5 platforms failed to boot due to this. > > > > Signed-off-by: Po-Yu Chuang > > --- > > arch/arm/mm/mmu.c |

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Will Deacon
On Tue, May 28, 2013 at 03:03:36PM +0100, Sricharan R wrote: > On Tuesday 28 May 2013 06:35 PM, Will Deacon wrote: > > On Tue, May 28, 2013 at 11:48:20AM +0100, Po-Yu Chuang wrote: > >> This bug was introduced in commit e651eab0. > >> Some v4/v5 platforms failed to boot due to this. > >> > >> Signe

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Sricharan R
On Tuesday 28 May 2013 06:35 PM, Will Deacon wrote: > On Tue, May 28, 2013 at 11:48:20AM +0100, Po-Yu Chuang wrote: >> This bug was introduced in commit e651eab0. >> Some v4/v5 platforms failed to boot due to this. >> >> Signed-off-by: Po-Yu Chuang >> --- >> arch/arm/mm/mmu.c |4 +++- >> 1 fi

Re: [PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Will Deacon
On Tue, May 28, 2013 at 11:48:20AM +0100, Po-Yu Chuang wrote: > This bug was introduced in commit e651eab0. > Some v4/v5 platforms failed to boot due to this. > > Signed-off-by: Po-Yu Chuang > --- > arch/arm/mm/mmu.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a

[PATCH] ARM: map_init_section flushes incorrect pmd

2013-05-28 Thread Po-Yu Chuang
This bug was introduced in commit e651eab0. Some v4/v5 platforms failed to boot due to this. Signed-off-by: Po-Yu Chuang --- arch/arm/mm/mmu.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c index e0d8565..19a43f8 100644 --- a/arch/