]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
riscv: Remove _dl_skip_args usage
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>
Thu, 26 May 2022 16:12:44 +0000 (13:12 -0300)
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>
Mon, 30 May 2022 19:33:22 +0000 (16:33 -0300)
Since ad43cac44a the generic code already shuffles the argv/envp/auxv
on the stack to remove the ld.so own arguments and thus _dl_skip_args
is always 0.   So there is no need to adjust the argc or argv.

Checked with qemu-user that arguments are correctly passed on both
constructors and main program.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
sysdeps/riscv/dl-machine.h

index 4912735c2b792ac4423886391fd1a90c0d8d63df..9e026ae0110b578856af2ba82d1a2e8cf67c5825 100644 (file)
@@ -107,18 +107,8 @@ elf_machine_dynamic (void)
        " _RTLD_PROLOGUE (_dl_start_user) "\
        # Stash user entry point in s0.\n\
        mv s0, a0\n\
-       # See if we were run as a command with the executable file\n\
-       # name as an extra leading argument.\n\
-       lw a0, _dl_skip_args\n\
-       # Load the original argument count.\n\
+       # Load the adjusted argument count.\n\
        " STRINGXP (REG_L) " a1, 0(sp)\n\
-       # Subtract _dl_skip_args from it.\n\
-       sub a1, a1, a0\n\
-       # Adjust the stack pointer to skip _dl_skip_args words.\n\
-       sll a0, a0, " STRINGXP (PTRLOG) "\n\
-       add sp, sp, a0\n\
-       # Save back the modified argument count.\n\
-       " STRINGXP (REG_S) " a1, 0(sp)\n\
        # Call _dl_init (struct link_map *main_map, int argc, char **argv, char **env) \n\
        " STRINGXP (REG_L) " a0, _rtld_local\n\
        add a2, sp, " STRINGXP (SZREG) "\n\