Skip to content
  • Sam bobroff's avatar
    powerpc: Correct DSCR during TM context switch · 96d01610
    Sam bobroff authored
    
    
    Correct the DSCR SPR becoming temporarily corrupted if a task is
    context switched during a transaction.
    
    The problem occurs while suspending the task and is caused by saving
    the DSCR to thread.dscr after it has already been set to the CPU's
    default value:
    
    __switch_to() calls __switch_to_tm()
    	which calls tm_reclaim_task()
    	which calls tm_reclaim_thread()
    	which calls tm_reclaim()
    		where the DSCR is set to the CPU's default
    __switch_to() calls _switch()
    		where thread.dscr is set to the DSCR
    
    When the task is resumed, it's transaction will be doomed (as usual)
    and the DSCR SPR will be corrupted, although the checkpointed value
    will be correct. Therefore the DSCR will be immediately corrected by
    the transaction aborting, unless it has been suspended. In that case
    the incorrect value can be seen by the task until it resumes the
    transaction.
    
    The fix is to treat the DSCR similarly to the TAR and save it early
    in __switch_to().
    
    A program exposing the problem is added to the kernel self tests as:
    tools/testing/selftests/powerpc/tm/tm-resched-dscr.
    
    Signed-off-by: default avatarSam Bobroff <sam.bobroff@au1.ibm.com>
    CC: <stable@vger.kernel.org> [v3.10+]
    Signed-off-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
    96d01610