lkml.org 
[lkml]   [2018]   [May]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH -resend 02/27] x86: assembly, use SYM_DATA for data

* Ingo Molnar <mingo@kernel.org> wrote:

>
> * Jiri Slaby <jslaby@suse.cz> wrote:
>
> > We make proper use of SYM_DATA on global data instead of marking them as
> > ENTRY. ENTRY is intended for functions and shall be paired with ENDPROC.
> > ENTRY also aligns symbols which creates unnecessary holes here between
> > data.
> >
> > ENTRY from saved_eip in wakeup_32 and many saved_* in wakeup_64 is
> > simply dropped, as these symbols are local only.
>
> Yeah, so this commit talks about ENTRY() and ENDPROC(), while the new methods are
> SYM_FUNC_START()/END().

BTW., I like the series otherwise: for example in my comment above I wrote up the
SYM_FUNC_START()/END() names from memory, without having to look up the
primitives.

[ Then I double checked it from the series to make sure I got it right. ;-) ]

Before this I could never remember all these random, idiosyncratic, path dependent
macro names - now there's at least some good high level structure to them. So it's
a good step forward.

Thanks,

Ingo

\
 
 \ /
  Last update: 2018-05-15 08:40    [W:0.064 / U:0.604 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site