mirror of
				git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
				synced 2025-10-31 08:44:41 +00:00 
			
		
		
		
	nds32: function_graph: Simplify with function_graph_enter()
The function_graph_enter() function does the work of calling the function
graph hook function and the management of the shadow stack, simplifying the
work done in the architecture dependent prepare_ftrace_return().
Have nds32 use the new code, and remove the shadow stack management as well as
having to set up the trace structure.
This is needed to prepare for a fix of a design bug on how the curr_ret_stack
is used.
Cc: Greentime Hu <greentime@andestech.com>
Cc: stable@kernel.org
Fixes: 03274a3ffb ("tracing/fgraph: Adjust fgraph depth before calling trace return callback")
Reviewed-by: Masami Hiramatsu <mhiramat@kernel.org>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
			
			
This commit is contained in:
		
							parent
							
								
									8712b27c57
								
							
						
					
					
						commit
						d48ebb2486
					
				
					 1 changed files with 2 additions and 16 deletions
				
			
		|  | @ -211,29 +211,15 @@ void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr, | |||
| 			   unsigned long frame_pointer) | ||||
| { | ||||
| 	unsigned long return_hooker = (unsigned long)&return_to_handler; | ||||
| 	struct ftrace_graph_ent trace; | ||||
| 	unsigned long old; | ||||
| 	int err; | ||||
| 
 | ||||
| 	if (unlikely(atomic_read(¤t->tracing_graph_pause))) | ||||
| 		return; | ||||
| 
 | ||||
| 	old = *parent; | ||||
| 
 | ||||
| 	trace.func = self_addr; | ||||
| 	trace.depth = current->curr_ret_stack + 1; | ||||
| 
 | ||||
| 	/* Only trace if the calling function expects to */ | ||||
| 	if (!ftrace_graph_entry(&trace)) | ||||
| 		return; | ||||
| 
 | ||||
| 	err = ftrace_push_return_trace(old, self_addr, &trace.depth, | ||||
| 				       frame_pointer, NULL); | ||||
| 
 | ||||
| 	if (err == -EBUSY) | ||||
| 		return; | ||||
| 
 | ||||
| 	*parent = return_hooker; | ||||
| 	if (!function_graph_enter(old, self_addr, frame_pointer, NULL)) | ||||
| 		*parent = return_hooker; | ||||
| } | ||||
| 
 | ||||
| noinline void ftrace_graph_caller(void) | ||||
|  |  | |||
		Loading…
	
	Add table
		
		Reference in a new issue
	
	 Steven Rostedt (VMware)
						Steven Rostedt (VMware)