• Michael Ellerman's avatar
    powerpc/vdso: refactor error handling · edb4a8bf
    Michael Ellerman authored
    Linus noticed that the error handling in __arch_setup_additional_pages()
    fails to clear the mm VDSO pointer if _install_special_mapping() fails. 
    In practice there should be no actual bug, because if there's an error the
    VDSO pointer is cleared later in arch_setup_additional_pages().
    
    However it's no longer necessary to set the pointer before installing the
    mapping.  Commit c1bab643 ("powerpc/vdso: Move to
    _install_special_mapping() and remove arch_vma_name()") reworked the code
    so that the VMA name comes from the vm_special_mapping.name, rather than
    relying on arch_vma_name().
    
    So rework the code to only set the VDSO pointer once the mappings have
    been installed correctly, and remove the stale comment.
    
    Link: https://lkml.kernel.org/r/20240812082605.743814-4-mpe@ellerman.id.auSigned-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
    Reviewed-by: default avatarLiam R. Howlett <Liam.Howlett@oracle.com>
    Cc: Christophe Leroy <christophe.leroy@csgroup.eu>
    Cc: Jeff Xu <jeffxu@google.com>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Nicholas Piggin <npiggin@gmail.com>
    Cc: Pedro Falcato <pedro.falcato@gmail.com>
    Cc: David Hildenbrand <david@redhat.com>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    edb4a8bf
vdso.c 10.6 KB