Re: Remove __end_entry_SYSENTER_compat?

[Borislav Petkov]

On Wed, Jul 12, 2017 at 10:48:03AM +0200, Ingo Molnar wrote:
> Hm, I'd argue that the old code is much clearer: we need both the start and the 
> end of a function and have the properly named symbols for that.
> 
> That entry_SYSCALL_compat() happens to start just where 
> __end_entry_SYSENTER_compat is an accident of placement.

Yeah, probably not worth the effort of actually making it less reliable
this way...

> Is it even true - doesn't ENTRY() imply an .align, in which case it might be that 
> __end_entry_SYSENTER_compat != entry_SYSCALL_compat?

Yes, ENTRY does .p2align:

.globl __end_entry_SYSENTER_compat; __end_entry_SYSENTER_compat:
.type entry_SYSENTER_compat, @function ; .size entry_SYSENTER_compat, .-entry_SYSENTER_compat
# 184 "arch/x86/entry/entry_64_compat.S"
.globl entry_SYSCALL_compat ; .p2align 4, 0x90 ; entry_SYSCALL_compat:
				^^^^^^
Pads to an alignment of 4 with NOPs.

> In fact that appears to be the case for my defconfig:
> 
>   ffffffff81942f90 T entry_SYSENTER_compat
>   ffffffff81942feb T __end_entry_SYSENTER_compat
>   ffffffff81942ff0 T entry_SYSCALL_compat
> 
> So unless there's some disadvantage beyond having one more symbol, I'd favor the 
> old approach.

Yeah, my only concern was getting rid of the global symbol but it
doesn't work in this case.

Thanks.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--