lkml.org 
[lkml]   [2018]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.16 062/410] x86/speculation: Use Indirect Branch Prediction Barrier in context switch
    3.16.57-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Tim Chen <tim.c.chen@linux.intel.com>

    commit 18bf3c3ea8ece8f03b6fc58508f2dfd23c7711c7 upstream.

    Flush indirect branches when switching into a process that marked itself
    non dumpable. This protects high value processes like gpg better,
    without having too high performance overhead.

    If done naïvely, we could switch to a kernel idle thread and then back
    to the original process, such as:

    process A -> idle -> process A

    In such scenario, we do not have to do IBPB here even though the process
    is non-dumpable, as we are switching back to the same process after a
    hiatus.

    To avoid the redundant IBPB, which is expensive, we track the last mm
    user context ID. The cost is to have an extra u64 mm context id to track
    the last mm we were using before switching to the init_mm used by idle.
    Avoiding the extra IBPB is probably worth the extra memory for this
    common scenario.

    For those cases where tlb_defer_switch_to_init_mm() returns true (non
    PCID), lazy tlb will defer switch to init_mm, so we will not be changing
    the mm for the process A -> idle -> process A switch. So IBPB will be
    skipped for this case.

    Thanks to the reviewers and Andy Lutomirski for the suggestion of
    using ctx_id which got rid of the problem of mm pointer recycling.

    Signed-off-by: Tim Chen <tim.c.chen@linux.intel.com>
    Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
    Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
    Cc: ak@linux.intel.com
    Cc: karahmed@amazon.de
    Cc: arjan@linux.intel.com
    Cc: torvalds@linux-foundation.org
    Cc: linux@dominikbrodowski.net
    Cc: peterz@infradead.org
    Cc: bp@alien8.de
    Cc: luto@kernel.org
    Cc: pbonzini@redhat.com
    Cc: gregkh@linux-foundation.org
    Link: https://lkml.kernel.org/r/1517263487-3708-1-git-send-email-dwmw@amazon.co.uk
    [bwh: Backported to 3.16: Drop the optimisation for switching via the idle
    task, since we don't have mm_context_t::ctx_id here]
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    --- a/arch/x86/mm/tlb.c
    +++ b/arch/x86/mm/tlb.c
    @@ -10,6 +10,7 @@

    #include <asm/tlbflush.h>
    #include <asm/mmu_context.h>
    +#include <asm/nospec-branch.h>
    #include <asm/cache.h>
    #include <asm/apic.h>
    #include <asm/uv/uv.h>
    @@ -100,6 +101,24 @@ void switch_mm_irqs_off(struct mm_struct
    unsigned cpu = smp_processor_id();

    if (likely(prev != next)) {
    + /*
    + * Avoid user/user BTB poisoning by flushing the branch
    + * predictor when switching between processes. This stops
    + * one process from doing Spectre-v2 attacks on another.
    + *
    + * As an optimization, flush indirect branches only when
    + * switching into processes that disable dumping. This
    + * protects high value processes like gpg, without having
    + * too high performance overhead. IBPB is *expensive*!
    + *
    + * This will not flush branches when switching into kernel
    + * threads. It will flush if we switch to a different non-
    + * dumpable process.
    + */
    + if (tsk && tsk->mm &&
    + get_dumpable(tsk->mm) != SUID_DUMP_USER)
    + indirect_branch_prediction_barrier();
    +
    this_cpu_write(cpu_tlbstate.state, TLBSTATE_OK);
    this_cpu_write(cpu_tlbstate.active_mm, next);
    cpumask_set_cpu(cpu, mm_cpumask(next));
    \
     
     \ /
      Last update: 2018-06-07 16:44    [W:4.167 / U:0.884 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site