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? Paolo
