Skip to content
  • Luc Van Oostenryck's avatar
    riscv: split the declaration of __copy_user · 86406d51
    Luc Van Oostenryck authored
    
    
    We use a single __copy_user assembly function to copy memory both from
    and to userspace. While this works, it triggers sparse errors because
    we're implicitly casting between the kernel and user address spaces by
    calling __copy_user.
    
    This patch splits the C declaration into a pair of functions,
    __asm_copy_{to,from}_user, that have sane semantics WRT __user. This
    split make things fine from sparse's point of view. The assembly
    implementation keeps a single definition but add a double ENTRY() for it,
    one for __asm_copy_to_user and another one for __asm_copy_from_user.
    The result is a spare-safe implementation that pays no performance
    or code size penalty.
    
    Signed-off-by: default avatarLuc Van Oostenryck <luc.vanoostenryck@gmail.com>
    Signed-off-by: default avatarPalmer Dabbelt <palmer@sifive.com>
    86406d51