flush_dcache_page does too much?

[anfei]

On Mon, Jan 18, 2010 at 02:44:18PM +0000, Russell King - ARM Linux wrote:
> On Mon, Jan 18, 2010 at 10:15:30PM +0800, anfei wrote:
> > On Mon, Jan 18, 2010 at 02:00:05PM +0000, Russell King - ARM Linux wrote:
> > > On Mon, Jan 18, 2010 at 09:54:31PM +0800, anfei wrote:
> > > > Do you mean this implementation can ensure the coherence between write
> > > > and shared mmapings?  But it's easy to reproduce the alias problem by
> > > > this simple testcase (w/o error handler) on omap2430 with VIPT cache:
> > > 
> > > Your program doesn't do anything to identify any problem.  You don't
> > > even say _what_ problem you see with this program.
> > > 
> > Sorry for that.
> > 
> > > If you have a specific case which fails, please show the problem, please
> > > describe exactly the behaviour that you see, and what you expect to see.
> 
> Are you using a write allocate cache?

I guess not, because this line is neccessary to reproduce the issue:
	tmp = *(addr+0);
If it's write allocate, this line may not be neccessary, since it's just
a read (and cache the data).