Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1685652

Re: Remove __end_entry_SYSENTER_compat?

Show all headers | View raw

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.
--

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Remove __end_entry_SYSENTER_compat? Borislav Petkov <bp@alien8.de> - 2017-07-12 06:20 +0200
  Re: Remove __end_entry_SYSENTER_compat? Ingo Molnar <mingo@kernel.org> - 2017-07-12 10:50 +0200
    Re: Remove __end_entry_SYSENTER_compat? Borislav Petkov <bp@alien8.de> - 2017-07-12 11:30 +0200

csiph-web