Re: [PATCH 2/2] x86/vdso: set vdso pointer only after success

[Andy Lutomirski]

On Thu, Oct 27, 2016 at 7:15 AM, Dmitry Safonov <dsafonov@xxxxxxxxxxxxx> wrote:
> Those pointers were initialized before call to _install_special_mapping
> after the commit f7b6eb3fa072 ("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 a62c34bd2a8a
> ("x86, mm: Improve _install_special_mapping and fix x86 vdso naming").
> So, this way to init looks less entangled.
> I even belive, 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.

Acked-by: Andy Lutomirski <luto@xxxxxxxxxx>