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>
		
			
				
	
	
		
			79 lines
		
	
	
		
			2.1 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			79 lines
		
	
	
		
			2.1 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
#!/usr/bin/env python
 | 
						|
# -*- coding: utf-8 -*-
 | 
						|
 | 
						|
"""
 | 
						|
trace/generated-tracers.h
 | 
						|
"""
 | 
						|
 | 
						|
__author__     = "Lluís Vilanova <vilanova@ac.upc.edu>"
 | 
						|
__copyright__  = "Copyright 2012-2016, 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):
 | 
						|
    out('/* This file is autogenerated by tracetool, do not edit. */',
 | 
						|
        '',
 | 
						|
        '#ifndef TRACE_%s_GENERATED_TRACERS_H' % group.upper(),
 | 
						|
        '#define TRACE_%s_GENERATED_TRACERS_H' % group.upper(),
 | 
						|
        '',
 | 
						|
        '#include "qemu-common.h"',
 | 
						|
        '#include "trace/control.h"',
 | 
						|
        '')
 | 
						|
 | 
						|
    for e in events:
 | 
						|
        out('extern TraceEvent %(event)s;',
 | 
						|
            event = e.api(e.QEMU_EVENT))
 | 
						|
 | 
						|
    for e in events:
 | 
						|
        out('extern uint16_t %s;' % e.api(e.QEMU_DSTATE))
 | 
						|
 | 
						|
    # static state
 | 
						|
    for e in events:
 | 
						|
        if 'disable' in e.properties:
 | 
						|
            enabled = 0
 | 
						|
        else:
 | 
						|
            enabled = 1
 | 
						|
        if "tcg-exec" in e.properties:
 | 
						|
            # a single define for the two "sub-events"
 | 
						|
            out('#define TRACE_%(name)s_ENABLED %(enabled)d',
 | 
						|
                name=e.original.name.upper(),
 | 
						|
                enabled=enabled)
 | 
						|
        out('#define TRACE_%s_ENABLED %d' % (e.name.upper(), enabled))
 | 
						|
 | 
						|
    backend.generate_begin(events, group)
 | 
						|
 | 
						|
    for e in events:
 | 
						|
        if "vcpu" in e.properties:
 | 
						|
            trace_cpu = next(iter(e.args))[1]
 | 
						|
            cond = "trace_event_get_vcpu_state(%(cpu)s,"\
 | 
						|
                   " TRACE_%(id)s)"\
 | 
						|
                   % dict(
 | 
						|
                       cpu=trace_cpu,
 | 
						|
                       id=e.name.upper())
 | 
						|
        else:
 | 
						|
            cond = "true"
 | 
						|
 | 
						|
        out('',
 | 
						|
            'static inline void %(api)s(%(args)s)',
 | 
						|
            '{',
 | 
						|
            '    if (%(cond)s) {',
 | 
						|
            api=e.api(),
 | 
						|
            args=e.args,
 | 
						|
            cond=cond)
 | 
						|
 | 
						|
        if "disable" not in e.properties:
 | 
						|
            backend.generate(e, group)
 | 
						|
 | 
						|
        out('    }',
 | 
						|
            '}')
 | 
						|
 | 
						|
    backend.generate_end(events, group)
 | 
						|
 | 
						|
    out('#endif /* TRACE_%s_GENERATED_TRACERS_H */' % group.upper())
 |