[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/9] linux-user: Split loader-related prototypes into load
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 4/9] linux-user: Split loader-related prototypes into loader.h |
Date: |
Wed, 8 Sep 2021 18:00:39 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 9/8/21 5:44 PM, Peter Maydell wrote:
> Split guest-binary loader prototypes out into a new header
> loader.h which we include only where required.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> linux-user/loader.h | 59 ++++++++++++++++++++++++++++++++++++++++++
> linux-user/qemu.h | 40 ----------------------------
> linux-user/elfload.c | 1 +
> linux-user/flatload.c | 1 +
> linux-user/linuxload.c | 1 +
> linux-user/main.c | 1 +
> linux-user/signal.c | 1 +
> linux-user/syscall.c | 1 +
> 8 files changed, 65 insertions(+), 40 deletions(-)
> create mode 100644 linux-user/loader.h
>
> diff --git a/linux-user/loader.h b/linux-user/loader.h
> +void do_init_thread(struct target_pt_regs *regs, struct image_info *infop);
> +abi_ulong loader_build_argptr(int envc, int argc, abi_ulong sp,
> + abi_ulong stringp, int push_ptr);
> +int loader_exec(int fdexec, const char *filename, char **argv, char **envp,
> + struct target_pt_regs *regs, struct image_info *infop,
> + struct linux_binprm *);
Pre-existing invalid style alignment, otherwise:
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
- [PATCH v2 0/9] linux-user: split internals out of qemu.h, Peter Maydell, 2021/09/08
- [PATCH v2 1/9] linux-user: Fix coding style nits in qemu.h, Peter Maydell, 2021/09/08
- [PATCH v2 5/9] linux-user: Split mmap prototypes into user-mmap.h, Peter Maydell, 2021/09/08
- [PATCH v2 2/9] linux-user: Split strace prototypes into strace.h, Peter Maydell, 2021/09/08
- [PATCH v2 4/9] linux-user: Split loader-related prototypes into loader.h, Peter Maydell, 2021/09/08
- [PATCH v2 6/9] linux-user: Split safe-syscall macro into its own header, Peter Maydell, 2021/09/08
- [PATCH v2 8/9] linux-user: Don't include gdbstub.h in qemu.h, Peter Maydell, 2021/09/08
- [PATCH v2 9/9] linux-user: Drop unneeded includes from qemu.h, Peter Maydell, 2021/09/08
- [PATCH v2 7/9] linux-user: Split linux-user internals out of qemu.h, Peter Maydell, 2021/09/08