Re: [PATCH v4] x86/msr: Add write msr notrace to avoid the debug codes splash

From: Paolo Bonzini
Date: Wed Oct 26 2016 - 03:56:42 EST




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