lkml.org 
[lkml]   [2018]   [Oct]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 07/11] x86/pkeys: Drop the preempt-disable section
    Date
    From: Rik van Riel <riel@surriel.com>

    The fpu->initialized flag should not be changed underneath us. This might be a
    fallout during the removal of the LazyFPU support. The FPU is marked
    initialized as soon as the state has been set to an initial value. It does not
    signal if the CPU's FPU registers are loaded.

    Signed-off-by: Rik van Riel <riel@surriel.com>
    Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
    ---
    arch/x86/mm/pkeys.c | 6 +-----
    1 file changed, 1 insertion(+), 5 deletions(-)

    diff --git a/arch/x86/mm/pkeys.c b/arch/x86/mm/pkeys.c
    index a150984171684..eeb03b0f0f514 100644
    --- a/arch/x86/mm/pkeys.c
    +++ b/arch/x86/mm/pkeys.c
    @@ -40,17 +40,13 @@ int __execute_only_pkey(struct mm_struct *mm)
    * dance to set PKRU if we do not need to. Check it
    * first and assume that if the execute-only pkey is
    * write-disabled that we do not have to set it
    - * ourselves. We need preempt off so that nobody
    - * can make fpregs inactive.
    + * ourselves.
    */
    - preempt_disable();
    if (!need_to_set_mm_pkey &&
    current->thread.fpu.initialized &&
    !__pkru_allows_read(read_pkru(), execute_only_pkey)) {
    - preempt_enable();
    return execute_only_pkey;
    }
    - preempt_enable();

    /*
    * Set up PKRU so that it denies access for everything
    --
    2.19.0
    \
     
     \ /
      Last update: 2018-10-04 16:07    [W:4.302 / U:0.172 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site