[PATCH] [preloader] Add proper CFI instructions to _start functions

Alexandre Julliard julliard at winehq.org
Mon Aug 15 08:46:07 CDT 2016


Keno Fischer <keno at juliacomputing.com> writes:

> diff --git a/loader/preloader.c b/loader/preloader.c
> index db0e998..8a86311 100644
> --- a/loader/preloader.c
> +++ b/loader/preloader.c
> @@ -212,6 +212,7 @@ struct
>  void _start(void);
>  extern char _end[];
>  __ASM_GLOBAL_FUNC(_start,
> +                  "\t.cfi_undefined %eip\n"
>                    "\tmovl $243,%eax\n"        /* SYS_set_thread_area */
>                    "\tmovl $thread_ldt,%ebx\n"
>                    "\tint $0x80\n"             /* allocate gs segment */

Looks OK, but it would need to be wrapped with the __ASM_CFI macro.
Also could you please make sure to include your signoff?

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list