lkml.org 
[lkml]   [2019]   [Jul]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v2 5/7] x86/mm, tracing: Fix CR2 corruption
    From
    Date


    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

    \
     
     \ /
      Last update: 2019-07-08 10:59    [W:5.259 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site