From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754811AbYLLU5l (ORCPT ); Fri, 12 Dec 2008 15:57:41 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752555AbYLLU5e (ORCPT ); Fri, 12 Dec 2008 15:57:34 -0500 Received: from fg-out-1718.google.com ([72.14.220.159]:58714 "EHLO fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751464AbYLLU5d (ORCPT ); Fri, 12 Dec 2008 15:57:33 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=fQ1n3uds+ahbZMQe/9RX0fQBfolo/lcGHLNLA5yE4gwIYWvA9PWyBleDQmH7Y8GREJ 6t8A4NOO2/p91gNotSYRI517jTvcbA6RaBeuBzJNGZ9VBciJxPYDZ/6UM5Mc69u2lAp1 fcqDKOlMkXOcRGmsHxhHt7Lo9UKcGbkdgC2OM= Date: Fri, 12 Dec 2008 23:57:29 +0300 From: Cyrill Gorcunov To: Steven Rostedt Cc: Steven Rostedt , Ingo Molnar , "H. Peter Anvin" , Thomas Gleixner , LKML , Alexander van Heukelum , fweisbec Subject: Re: [RFC] x86: entry_64 - introduce FTRACE_ frame macro Message-ID: <20081212205729.GA28554@localhost> References: <20081212184136.GB14192@localhost> <1229113167.32623.12.camel@localhost.localdomain> <20081212204724.GA26194@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [Steven Rostedt - Fri, Dec 12, 2008 at 03:52:56PM -0500] | ... | You can put them in arch/x86/include/asm/ftrace.h That's assembly safe. | | -- Steve | Yep, while you were typing the message I was testing the change :) Do you like it? - Cyrill - --- arch/x86/include/asm/ftrace.h | 29 ++++++++++++++++++++- arch/x86/kernel/entry_64.S | 57 +++++------------------------------------- 2 files changed, 35 insertions(+), 51 deletions(-) Index: linux-2.6.git/arch/x86/include/asm/ftrace.h =================================================================== --- linux-2.6.git.orig/arch/x86/include/asm/ftrace.h +++ linux-2.6.git/arch/x86/include/asm/ftrace.h @@ -1,6 +1,33 @@ #ifndef _ASM_X86_FTRACE_H #define _ASM_X86_FTRACE_H +#ifdef __ASSEMBLY__ + + .macro FTRACE_SAVE_FRAME + /* taken from glibc */ + subq $0x38, %rsp + movq %rax, (%rsp) + movq %rcx, 8(%rsp) + movq %rdx, 16(%rsp) + movq %rsi, 24(%rsp) + movq %rdi, 32(%rsp) + movq %r8, 40(%rsp) + movq %r9, 48(%rsp) + .endm + + .macro FTRACE_RESTORE_FRAME + movq 48(%rsp), %r9 + movq 40(%rsp), %r8 + movq 32(%rsp), %rdi + movq 24(%rsp), %rsi + movq 16(%rsp), %rdx + movq 8(%rsp), %rcx + movq (%rsp), %rax + addq $0x38, %rsp + .endm + +#endif + #ifdef CONFIG_FUNCTION_TRACER #define MCOUNT_ADDR ((long)(mcount)) #define MCOUNT_INSN_SIZE 5 /* sizeof mcount call */ @@ -46,7 +73,7 @@ struct ftrace_ret_stack { /* * Primary handler of a function return. * It relays on ftrace_return_to_handler. - * Defined in entry32.S + * Defined in entry_32/64.S */ extern void return_to_handler(void); Index: linux-2.6.git/arch/x86/kernel/entry_64.S =================================================================== --- linux-2.6.git.orig/arch/x86/kernel/entry_64.S +++ linux-2.6.git/arch/x86/kernel/entry_64.S @@ -70,15 +70,7 @@ ENTRY(ftrace_caller) cmpl $0, function_trace_stop jne ftrace_stub - /* taken from glibc */ - subq $0x38, %rsp - movq %rax, (%rsp) - movq %rcx, 8(%rsp) - movq %rdx, 16(%rsp) - movq %rsi, 24(%rsp) - movq %rdi, 32(%rsp) - movq %r8, 40(%rsp) - movq %r9, 48(%rsp) + FTRACE_SAVE_FRAME movq 0x38(%rsp), %rdi movq 8(%rbp), %rsi @@ -88,14 +80,7 @@ ENTRY(ftrace_caller) ftrace_call: call ftrace_stub - movq 48(%rsp), %r9 - movq 40(%rsp), %r8 - movq 32(%rsp), %rdi - movq 24(%rsp), %rsi - movq 16(%rsp), %rdx - movq 8(%rsp), %rcx - movq (%rsp), %rax - addq $0x38, %rsp + FTRACE_RESTORE_FRAME #ifdef CONFIG_FUNCTION_GRAPH_TRACER .globl ftrace_graph_call @@ -129,15 +114,7 @@ ftrace_stub: retq trace: - /* taken from glibc */ - subq $0x38, %rsp - movq %rax, (%rsp) - movq %rcx, 8(%rsp) - movq %rdx, 16(%rsp) - movq %rsi, 24(%rsp) - movq %rdi, 32(%rsp) - movq %r8, 40(%rsp) - movq %r9, 48(%rsp) + FTRACE_SAVE_FRAME movq 0x38(%rsp), %rdi movq 8(%rbp), %rsi @@ -145,14 +122,7 @@ trace: call *ftrace_trace_function - movq 48(%rsp), %r9 - movq 40(%rsp), %r8 - movq 32(%rsp), %rdi - movq 24(%rsp), %rsi - movq 16(%rsp), %rdx - movq 8(%rsp), %rcx - movq (%rsp), %rax - addq $0x38, %rsp + FTRACE_RESTORE_FRAME jmp ftrace_stub END(mcount) @@ -164,14 +134,7 @@ ENTRY(ftrace_graph_caller) cmpl $0, function_trace_stop jne ftrace_stub - subq $0x38, %rsp - movq %rax, (%rsp) - movq %rcx, 8(%rsp) - movq %rdx, 16(%rsp) - movq %rsi, 24(%rsp) - movq %rdi, 32(%rsp) - movq %r8, 40(%rsp) - movq %r9, 48(%rsp) + FTRACE_SAVE_FRAME leaq 8(%rbp), %rdi movq 0x38(%rsp), %rsi @@ -179,14 +142,8 @@ ENTRY(ftrace_graph_caller) call prepare_ftrace_return - movq 48(%rsp), %r9 - movq 40(%rsp), %r8 - movq 32(%rsp), %rdi - movq 24(%rsp), %rsi - movq 16(%rsp), %rdx - movq 8(%rsp), %rcx - movq (%rsp), %rax - addq $0x38, %rsp + FTRACE_RESTORE_FRAME + retq END(ftrace_graph_caller)