Re: [PATCH v2 5/7] x86/mm, tracing: Fix CR2 corruption

From: Eiichi Tsukata
Date: Mon Jul 08 2019 - 04:58:57 EST




On 2019/07/08 16:48, Peter Zijlstra wrote:
...

>
> Or are we going to put the CR2 save/restore on every single tracepoint?
> But then we also need it on the mcount/fentry stubs and we again have
> multiple places.
>
> Whereas if we stick it in the entry path, like I proposed, we fix it in
> one location and we're done.
>

Thanks to your detailed comments and the discussion, I've come to realize
that your solution "save CR2 early in the entry" is the simplest and reasonable.

By the way, is there possibility that the WARNING(#GP in execve(2)) which Steven
previously hit? : https://lore.kernel.org/lkml/20190321095502.47b51356@xxxxxxxxxxxxxxxxxx/

Even if there were, it will *Not* be the bug introduced by this patch series,
but the bug revealed by this series.

Thanks,

Eiichi