 80dd5c4918
			
		
	
	
		80dd5c4918
		
	
	
	
	
		
			
			The declarations in the generated-tracers.h file are assuming there's only ever going to be one instance of this header, as they are not namespaced. When we have one header per event group, if a single source file needs to include multiple sets of trace events, the symbols will all clash. This change thus introduces a '--group NAME' arg to the 'tracetool' program. This will cause all the symbols in the generated header files to be given a unique namespace. If no group is given, the group name 'common' is used, which is suitable for the current usage where there is only one global trace-events file used for code generation. Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Daniel P. Berrange <berrange@redhat.com> Reviewed-by: Lluís Vilanova <vilanova@ac.upc.edu> Message-id: 1475588159-30598-21-git-send-email-berrange@redhat.com Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
		
			
				
	
	
		
			71 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			71 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
| #!/usr/bin/env python
 | |
| # -*- coding: utf-8 -*-
 | |
| 
 | |
| """
 | |
| trace/generated-tracers.c
 | |
| """
 | |
| 
 | |
| __author__     = "Lluís Vilanova <vilanova@ac.upc.edu>"
 | |
| __copyright__  = "Copyright 2012-2014, Lluís Vilanova <vilanova@ac.upc.edu>"
 | |
| __license__    = "GPL version 2 or (at your option) any later version"
 | |
| 
 | |
| __maintainer__ = "Stefan Hajnoczi"
 | |
| __email__      = "stefanha@linux.vnet.ibm.com"
 | |
| 
 | |
| 
 | |
| from tracetool import out
 | |
| 
 | |
| 
 | |
| def generate(events, backend, group):
 | |
|     active_events = [e for e in events
 | |
|                      if "disable" not in e.properties]
 | |
| 
 | |
|     out('/* This file is autogenerated by tracetool, do not edit. */',
 | |
|         '',
 | |
|         '#include "qemu/osdep.h"',
 | |
|         '#include "trace.h"',
 | |
|         '')
 | |
| 
 | |
|     for e in events:
 | |
|         out('uint16_t %s;' % e.api(e.QEMU_DSTATE))
 | |
| 
 | |
|     for e in events:
 | |
|         if "vcpu" in e.properties:
 | |
|             vcpu_id = 0
 | |
|         else:
 | |
|             vcpu_id = "TRACE_VCPU_EVENT_NONE"
 | |
|         out('TraceEvent %(event)s = {',
 | |
|             '    .id = 0,',
 | |
|             '    .vcpu_id = %(vcpu_id)s,',
 | |
|             '    .name = \"%(name)s\",',
 | |
|             '    .sstate = %(sstate)s,',
 | |
|             '    .dstate = &%(dstate)s ',
 | |
|             '};',
 | |
|             event = e.api(e.QEMU_EVENT),
 | |
|             vcpu_id = vcpu_id,
 | |
|             name = e.name,
 | |
|             sstate = "TRACE_%s_ENABLED" % e.name.upper(),
 | |
|             dstate = e.api(e.QEMU_DSTATE))
 | |
| 
 | |
|     out('TraceEvent *%(group)s_trace_events[] = {',
 | |
|         group = group.lower())
 | |
| 
 | |
|     for e in events:
 | |
|         out('    &%(event)s,', event = e.api(e.QEMU_EVENT))
 | |
| 
 | |
|     out('  NULL,',
 | |
|         '};',
 | |
|         '')
 | |
| 
 | |
|     out('static void trace_%(group)s_register_events(void)',
 | |
|         '{',
 | |
|         '    trace_event_register_group(%(group)s_trace_events);',
 | |
|         '}',
 | |
|         'trace_init(trace_%(group)s_register_events)',
 | |
|         group = group.lower())
 | |
| 
 | |
|     backend.generate_begin(active_events, group)
 | |
|     for event in active_events:
 | |
|         backend.generate(event, group)
 | |
|     backend.generate_end(active_events, group)
 |