Re: [PATCH 1/3] Fix COW D-cache aliasing on fork

From: Martin Schwidefsky
Date: Mon Oct 23 2006 - 04:50:34 EST


On Fri, 2006-10-20 at 13:10 -0700, Linus Torvalds wrote:
>
> On Fri, 20 Oct 2006, David Miller wrote:
> >
> > I did some more digging, here's what I think the hardware actually
> > does:
>
> Ok, this sounds sane.
>
> What should we do about this? How does this patch look to people?
>
> (Totally untested, and I'm not sure we should even do that whole
> "oldmm->mm_users" test, but I'm throwing it out here for discussion, in
> case it matters for performance. The second D$ flush should obviously be
> unnecessary for the common unthreaded case, which is why none of this has
> mattered historically, I think).
>
> Comments? We need ARM, MIPS, sparc and S390 at the very least to sign off
> on this, and somebody to write a nice explanation for the changelog (and
> preferably do this through -mm too).

On s390 you never have to worry about cache flushing. It is not stated
anywhere in the principles of operation but the architecture has to be
PIPT, otherwise it couldn't possibly work. The best indication for it is
that there is no cache flush instruction. The view on data in memory is
always consistent.

--
blue skies,
Martin.

Martin Schwidefsky
Linux for zSeries Development & Services
IBM Deutschland Entwicklung GmbH

"Reality continues to ruin my life." - Calvin.


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