• Dmitry Safonov's avatar
    x86/vdso: Set vDSO pointer only after success · 67dece7d
    Dmitry Safonov authored
    Those pointers were initialized before call to _install_special_mapping()
    after the commit:
    
      f7b6eb3f ("x86: Set context.vdso before installing the mapping")
    
    This is not required anymore as special mappings have their vma name and
    don't use arch_vma_name() after commit:
    
      a62c34bd ("x86, mm: Improve _install_special_mapping and fix x86 vdso naming")
    
    So, this way to init looks less entangled.
    
    I even belive that we can remove NULL initializers:
    
     - on failure load_elf_binary() will not start a new thread;
     - arch_prctl will have the same pointers as before syscall.
    Signed-off-by: default avatarDmitry Safonov <dsafonov@virtuozzo.com>
    Acked-by: default avatarAndy Lutomirski <luto@kernel.org>
    Cc: 0x7f454c46@gmail.com
    Cc: Borislav Petkov <bp@alien8.de>
    Cc: Brian Gerst <brgerst@gmail.com>
    Cc: Cyrill Gorcunov <gorcunov@openvz.org>
    Cc: Denys Vlasenko <dvlasenk@redhat.com>
    Cc: H. Peter Anvin <hpa@zytor.com>
    Cc: Josh Poimboeuf <jpoimboe@redhat.com>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: linux-mm@kvack.org
    Cc: oleg@redhat.com
    Link: http://lkml.kernel.org/r/20161027141516.28447-3-dsafonov@virtuozzo.comSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
    67dece7d
vma.c 9.27 KB