* Denys Vlasenko <dvlas...@redhat.com> wrote:

>       +8b 74 24 68            mov    0x68(%rsp),%esi
>       +8b 7c 24 70            mov    0x70(%rsp),%edi
>       +8b 54 24 60            mov    0x60(%rsp),%edx

Btw., could you (in another patch) order the restoration properly, by pt_regs 
memory order, where possible?

So this:

> +     movl    RSI(%rsp), %esi
> +     movl    RDI(%rsp), %edi
> +     movl    RDX(%rsp), %edx
>       movl    RIP(%rsp), %ecx
>       movl    EFLAGS(%rsp), %r11d

would become:

        movl    RDX(%rsp), %edx
        movl    RSI(%rsp), %esi
        movl    RDI(%rsp), %edi
        movl    RIP(%rsp), %ecx
        movl    EFLAGS(%rsp), %r11d

... or so.

In fact I'd suggest we structure movl based restoration precisely after the 
field 
order in the structure, and comment on cases where we depart from what's in 
pt_regs - to make it all easier to verify.

Thanks,

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

Reply via email to