]> git.baikalelectronics.ru Git - kernel.git/commit
powerpc: do_notify_resume can be called with bad thread_info flags argument
authorAnton Blanchard <anton@samba.org>
Fri, 31 Oct 2014 05:50:57 +0000 (16:50 +1100)
committerMichael Ellerman <mpe@ellerman.id.au>
Fri, 31 Oct 2014 05:52:46 +0000 (16:52 +1100)
commit70b0d53d1d6b95aed62a295f5a26f61e38f1a691
tree2230d3d8d813bada4f9ffe54fe6ffbcb947c9bd6
parent6309ca934b59ef3d70465de1bd12be73db0c94a6
powerpc: do_notify_resume can be called with bad thread_info flags argument

Back in 9c090eefaa51 ("powerpc: Rework lazy-interrupt handling") we
added a call out to restore_interrupts() (written in c) before calling
do_notify_resume:

        bl      restore_interrupts
        addi    r3,r1,STACK_FRAME_OVERHEAD
        bl      do_notify_resume

Unfortunately do_notify_resume takes two arguments, the second one
being the thread_info flags:

void do_notify_resume(struct pt_regs *regs, unsigned long thread_info_flags)

We do populate r4 (the second argument) earlier, but
restore_interrupts() is free to muck it up all it wants. My guess is
the gcc compiler gods shone down on us and its register allocator
never used r4. Sometimes, rarely, luck is on our side.

LLVM on the other hand did trample r4.

Signed-off-by: Anton Blanchard <anton@samba.org>
Cc: stable@vger.kernel.org
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/entry_64.S