Re: 2.4.19pre7aa1

[Andrew Morton <akpm@zip.com.au>]

Andrea Arcangeli wrote:
> 
> ...
> Only in 2.4.19pre6aa1: 00_prepare-write-fixes-2
> Only in 2.4.19pre7aa1: 00_prepare-write-fixes-3
> 
>         Add a missing flush_dcache_page() to the prepare write corruption
>         fixes. Noticed by Andrew Morton.
> 

Why do we perform those "flushes"[1] at all?  The memsets should
never occur when the page is mapped into any process tables.

I seem to be missing something here.


[1] Can we please not that term?  A "flush" is something which you
    do to a "dunny" after taking a "crap". [2]

    Caches are either "written back" or are "invalidated".  Nothing
    else.

    This is not just semantics.  This stuff is hard.  90% of kernel
    developers are on x86, and 90% of those do not understand the
    nuances of all this.  The careful choice and use of terminology
    will aid other platforms.

[2] And a "sync" is something which you wash your hands in after the
    "flush".

    Dirty disk data is either "written out" or is "waited upon".  The
    kernel has real performance bugs due to this confusion.

-