Messages in this thread | | | Subject | Re: [PATCH v2 5/7] x86/mm, tracing: Fix CR2 corruption | From | Eiichi Tsukata <> | Date | Mon, 8 Jul 2019 17:58:49 +0900 |
| |
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@gandalf.local.home/
Even if there were, it will *Not* be the bug introduced by this patch series, but the bug revealed by this series.
Thanks,
Eiichi
| |