Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

riscv: hwprobe: do not produce frtace relocation #725

Closed
wants to merge 1 commit into from

Commits on Mar 25, 2024

  1. riscv: hwprobe: do not produce frtace relocation

    Such relocation causes crash of android linker similar to one
    described in commit e05d57d
    ("riscv: Fixup __vdso_gettimeofday broke dynamic ftrace").
    
    Looks like this relocation is added by CONFIG_DYNAMIC_FTRACE which is
    disabled in the default android kernel.
    
    Before:
    
    readelf -rW arch/riscv/kernel/vdso/vdso.so:
    
    Relocation section '.rela.dyn' at offset 0xd00 contains 1 entry:
        Offset             Info             Type
    0000000000000d20  0000000000000003 R_RISCV_RELATIVE
    
    objdump:
    0000000000000c86 <__vdso_riscv_hwprobe@@LINUX_4.15>:
     c86:   0001                    nop
     c88:   0001                    nop
     c8a:   0001                    nop
     c8c:   0001                    nop
     c8e:   e211                    bnez    a2,c92 <__vdso_riscv_hwprobe...
    
    After:
    readelf -rW arch/riscv/kernel/vdso/vdso.so:
    
    There are no relocations in this file.
    
    objdump:
    0000000000000c86 <__vdso_riscv_hwprobe@@LINUX_4.15>:
     c86:   e211                    bnez    a2,c8a <__vdso_riscv_hwprobe...
     c88:   c6b9                    beqz    a3,cd6 <__vdso_riscv_hwprobe...
     c8a:   e739                    bnez    a4,cd8 <__vdso_riscv_hwprobe...
     c8c:   ffffd797                auipc   a5,0xffffd
    
    Also disable SCS since it also should not be available in vdso.
    
    Fixes: aa5af0a ("RISC-V: Add hwprobe vDSO function and data")
    Signed-off-by: Roman Artemev <roman.artemev@syntacore.com>
    Signed-off-by: Vladimir Isaev <vladimir.isaev@syntacore.com>
    Reviewed-by: Guo Ren <guoren@kernel.org>
    Signed-off-by: Björn Töpel <bjorn@rivosinc.com>
    visaev-sc authored and Björn Töpel committed Mar 25, 2024
    Configuration menu
    Copy the full SHA
    db73bba View commit details
    Browse the repository at this point in the history