perf sched: Remove dead code
Several variables are not used at all, cut'n'paste leftovers. Also check if the sample_type is RAW earlier, to avoid needless searches. Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: Frédéric Weisbecker <fweisbec@gmail.com> Cc: "H. Peter Anvin" <hpa@zytor.com> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Mike Galbraith <efault@gmx.de> Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
		
					parent
					
						
							
								1b46cddfcc
							
						
					
				
			
			
				commit
				
					
						a80deb622d
					
				
			
		
					 1 changed files with 4 additions and 30 deletions
				
			
		| 
						 | 
					@ -1544,16 +1544,15 @@ process_raw_event(event_t *raw_event __used, void *more_data,
 | 
				
			||||||
static int
 | 
					static int
 | 
				
			||||||
process_sample_event(event_t *event, unsigned long offset, unsigned long head)
 | 
					process_sample_event(event_t *event, unsigned long offset, unsigned long head)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	char level;
 | 
					 | 
				
			||||||
	int show = 0;
 | 
					 | 
				
			||||||
	struct dso *dso = NULL;
 | 
					 | 
				
			||||||
	struct thread *thread;
 | 
						struct thread *thread;
 | 
				
			||||||
	u64 ip = event->ip.ip;
 | 
						u64 ip = event->ip.ip;
 | 
				
			||||||
	u64 timestamp = -1;
 | 
						u64 timestamp = -1;
 | 
				
			||||||
	u32 cpu = -1;
 | 
						u32 cpu = -1;
 | 
				
			||||||
	u64 period = 1;
 | 
						u64 period = 1;
 | 
				
			||||||
	void *more_data = event->ip.__more_data;
 | 
						void *more_data = event->ip.__more_data;
 | 
				
			||||||
	int cpumode;
 | 
					
 | 
				
			||||||
 | 
						if (!(sample_type & PERF_SAMPLE_RAW))
 | 
				
			||||||
 | 
							return 0;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	thread = threads__findnew(event->ip.pid, &threads, &last_match);
 | 
						thread = threads__findnew(event->ip.pid, &threads, &last_match);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1589,32 +1588,7 @@ process_sample_event(event_t *event, unsigned long offset, unsigned long head)
 | 
				
			||||||
		return -1;
 | 
							return -1;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	cpumode = event->header.misc & PERF_RECORD_MISC_CPUMODE_MASK;
 | 
						process_raw_event(event, more_data, cpu, timestamp, thread);
 | 
				
			||||||
 | 
					 | 
				
			||||||
	if (cpumode == PERF_RECORD_MISC_KERNEL) {
 | 
					 | 
				
			||||||
		show = SHOW_KERNEL;
 | 
					 | 
				
			||||||
		level = 'k';
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
		dso = kernel_dso;
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
		dump_printf(" ...... dso: %s\n", dso->name);
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	} else if (cpumode == PERF_RECORD_MISC_USER) {
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
		show = SHOW_USER;
 | 
					 | 
				
			||||||
		level = '.';
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	} else {
 | 
					 | 
				
			||||||
		show = SHOW_HV;
 | 
					 | 
				
			||||||
		level = 'H';
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
		dso = hypervisor_dso;
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
		dump_printf(" ...... dso: [hypervisor]\n");
 | 
					 | 
				
			||||||
	}
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	if (sample_type & PERF_SAMPLE_RAW)
 | 
					 | 
				
			||||||
		process_raw_event(event, more_data, cpu, timestamp, thread);
 | 
					 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return 0;
 | 
						return 0;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue