diff options
| author | Keir Fraser <keir.fraser@citrix.com> | 2008-02-11 09:47:19 +0000 |
|---|---|---|
| committer | Keir Fraser <keir.fraser@citrix.com> | 2008-02-11 09:47:19 +0000 |
| commit | e1b02119e639a9e13f79c99d145bf8a3c535593a (patch) | |
| tree | 6218d66cc914d3462ada18ff5fb72f643c3624ba /xen/arch/x86/trace.c | |
| parent | 1c8d4c1040bcfda0be3402025efe15c360c2c52d (diff) | |
| download | xen-e1b02119e639a9e13f79c99d145bf8a3c535593a.tar.gz xen-e1b02119e639a9e13f79c99d145bf8a3c535593a.tar.bz2 xen-e1b02119e639a9e13f79c99d145bf8a3c535593a.zip | |
xentrace: Remove redundant tb_done_init checks, and add missing ones.
Hand inspection of gcc -02 output confirms significantly shorter
codepaths for inactive (i.e. normal case) tracing.
Signed-off-by: Michael A Fetterman <Michael.Fetterman@cl.cam.ac.uk>
Diffstat (limited to 'xen/arch/x86/trace.c')
| -rw-r--r-- | xen/arch/x86/trace.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/xen/arch/x86/trace.c b/xen/arch/x86/trace.c index ddb32202bd..65e66a3280 100644 --- a/xen/arch/x86/trace.c +++ b/xen/arch/x86/trace.c @@ -15,9 +15,6 @@ asmlinkage void trace_hypercall(void) { struct cpu_user_regs *regs = guest_cpu_user_regs(); - if ( !tb_init_done ) - return; - #ifdef __x86_64__ if ( is_pv_32on64_vcpu(current) ) { @@ -52,9 +49,6 @@ asmlinkage void trace_hypercall(void) void __trace_pv_trap(int trapnr, unsigned long eip, int use_error_code, unsigned error_code) { - if ( !tb_init_done ) - return; - #ifdef __x86_64__ if ( is_pv_32on64_vcpu(current) ) { @@ -99,9 +93,6 @@ void __trace_pv_page_fault(unsigned long addr, unsigned error_code) { unsigned long eip = guest_cpu_user_regs()->eip; - if ( !tb_init_done ) - return; - #ifdef __x86_64__ if ( is_pv_32on64_vcpu(current) ) { @@ -135,9 +126,6 @@ void __trace_pv_page_fault(unsigned long addr, unsigned error_code) void __trace_trap_one_addr(unsigned event, unsigned long va) { - if ( !tb_init_done ) - return; - #ifdef __x86_64__ if ( is_pv_32on64_vcpu(current) ) { @@ -155,9 +143,6 @@ void __trace_trap_one_addr(unsigned event, unsigned long va) void __trace_trap_two_addr(unsigned event, unsigned long va1, unsigned long va2) { - if ( !tb_init_done ) - return; - #ifdef __x86_64__ if ( is_pv_32on64_vcpu(current) ) { @@ -185,9 +170,6 @@ void __trace_ptwr_emulation(unsigned long addr, l1_pgentry_t npte) { unsigned long eip = guest_cpu_user_regs()->eip; - if ( !tb_init_done ) - return; - /* We have a couple of different modes to worry about: * - 32-on-32: 32-bit pte, 32-bit virtual addresses * - pae-on-pae, pae-on-64: 64-bit pte, 32-bit virtual addresses |