On Tue, May 7, 2019 at 7:48 AM Andy Lutomirski luto@amacapital.net wrote:
IOW I think your trick only works if the old and new states are CALL, but we don’t know that until we’ve looked up the record, at which point we can just use the result of the lookup.
It would indeed only work for call instructions. I was thinking we'd know that because we only ever batch up call instructions, though.
But it doesn't matter. I was looking at the ftrace code because I thought there was some subtle timing bug or race or similar. But it turned out my "memmove()" was the problem. See the patch I just sent out.
Linus