Re: 2.4.19pre7aa1

Andrew Morton (akpm@zip.com.au)
Sat, 20 Apr 2002 12:37:51 -0700


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.

-
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/