Messages in this thread | | | Subject | Re: [PATCH] x86/signal: Deinline get_sigframe, save 240 bytes | From | "H. Peter Anvin" <> | Date | Mon, 28 Sep 2015 05:33:53 -0700 |
| |
This one makes me slightly nervous, because it isn't clear that these aren't potentially performance sensitive.
On September 28, 2015 5:23:57 AM PDT, Denys Vlasenko <dvlasenk@redhat.com> wrote: >This function compiles to 277 bytes of machine code and has 4 >callsites. > >Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com> >CC: Ingo Molnar <mingo@kernel.org> >CC: Andy Lutomirski <luto@amacapital.net> >CC: "H. Peter Anvin" <hpa@zytor.com> >CC: Borislav Petkov <bp@alien8.de> >CC: Brian Gerst <brgerst@gmail.com> >CC: x86@kernel.org >CC: linux-kernel@vger.kernel.org >--- > arch/x86/kernel/signal.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/arch/x86/kernel/signal.c b/arch/x86/kernel/signal.c >index da52e6b..054a4d0 100644 >--- a/arch/x86/kernel/signal.c >+++ b/arch/x86/kernel/signal.c >@@ -196,7 +196,7 @@ static unsigned long align_sigframe(unsigned long >sp) > return sp; > } > >-static inline void __user * >+static void __user * >get_sigframe(struct k_sigaction *ka, struct pt_regs *regs, size_t >frame_size, > void __user **fpstate) > {
-- Sent from my Android device with K-9 Mail. Please excuse my brevity.
| |