2016-10-26 15:56 GMT+08:00 Paolo Bonzini <[email protected]>: > > > On 26/10/2016 09:55, Wanpeng Li wrote: >>>> >> -static void kvm_guest_apic_eoi_write(u32 reg, u32 val) >>>> >> +static void kvm_guest_apic_eoi_write_notrace(u32 reg, u32 val) >>> > >>> > Huh, this is not notrace?!? >> Sorry, maybe I misunderstand you. >> >> ack_APIC_irq() >> -> apic_eoi() >> -> apic->eoi_write() >> -> kvm_guest_apic_eoi_write() >> -> apic->native_eoi_write() >> -> native_apic_msr_eoi_write() >> -> wrmsr_notrace() >> -> native_write_msr_notrace() >> -> native_apic_mem_write() >> >> The call path is notrace. Where I miss? > > kvm_guest_apic_eoi_write should be marked as notrace, shouldn't it?
I see. notrace static void kvm_guest_apic_eoi_write_notrace(u32 reg, u32 val) and notrace static void kvm_guest_apic_eoi_write(u32 reg, u32 val), which one you prefer? Regards, Wanpeng Li
