lkml.org 
[lkml]   [2014]   [Feb]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v2 1/4] x86: Use the default ABI for the 32-bit vDSO
    From
    Date
    How many internal function calls are there? It seems we should try to avoid those as much as possible by suitable inlining.

    On February 27, 2014 11:28:25 PM PST, Stefani Seibold <stefani@seibold.net> wrote:
    >Am Donnerstag, den 27.02.2014, 16:18 -0800 schrieb Andy Lutomirski:
    >> There's no reason for the vDSO to use a special function call ABI.
    >Use
    >> the platform defaults.
    >>
    >
    >The only reason was performance. What is good for the kernel should be
    >also good for the VDSO. Now all functions inside the VDSO will pass the
    >arguments by stack.
    >
    >> Signed-off-by: Andy Lutomirski <luto@amacapital.net>
    >> ---
    >> arch/x86/vdso/Makefile | 2 +-
    >> 1 file changed, 1 insertion(+), 1 deletion(-)
    >>
    >> diff --git a/arch/x86/vdso/Makefile b/arch/x86/vdso/Makefile
    >> index 92daaa6..80584f5 100644
    >> --- a/arch/x86/vdso/Makefile
    >> +++ b/arch/x86/vdso/Makefile
    >> @@ -149,7 +149,7 @@ KBUILD_CFLAGS_32 := $(filter-out
    >-m64,$(KBUILD_CFLAGS))
    >> KBUILD_CFLAGS_32 := $(filter-out
    >-mcmodel=kernel,$(KBUILD_CFLAGS_32))
    >> KBUILD_CFLAGS_32 := $(filter-out -fno-pic,$(KBUILD_CFLAGS_32))
    >> KBUILD_CFLAGS_32 := $(filter-out -mfentry,$(KBUILD_CFLAGS_32))
    >> -KBUILD_CFLAGS_32 += -m32 -msoft-float -mregparm=3
    >-freg-struct-return -fpic
    >> +KBUILD_CFLAGS_32 += -m32 -msoft-float -mregparm=0 -fpic
    >> $(vdso32-images:%=$(obj)/%.dbg): KBUILD_CFLAGS = $(KBUILD_CFLAGS_32)
    >>
    >> $(vdso32-images:%=$(obj)/%.dbg): $(obj)/vdso32-%.so.dbg: FORCE \

    --
    Sent from my mobile phone. Please pardon brevity and lack of formatting.


    \
     
     \ /
      Last update: 2014-02-28 17:01    [W:2.176 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site