Re: [PATCH 2/3] KVM: X86: Remove tailing newline for tracepoints

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 29 Jul 2019 at 13:35, Peter Xu <zhexu@xxxxxxxxxx> wrote:
>
> It's done by TP_printk() already.
>
> Signed-off-by: Peter Xu <peterx@xxxxxxxxxx>
> ---
>  arch/x86/kvm/trace.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/x86/kvm/trace.h b/arch/x86/kvm/trace.h
> index 26423d2e45df..76a39bc25b95 100644
> --- a/arch/x86/kvm/trace.h
> +++ b/arch/x86/kvm/trace.h
> @@ -1323,7 +1323,7 @@ TRACE_EVENT(kvm_avic_incomplete_ipi,
>                 __entry->index = index;
>         ),
>
> -       TP_printk("vcpu=%u, icrh:icrl=%#010x:%08x, id=%u, index=%u\n",
> +       TP_printk("vcpu=%u, icrh:icrl=%#010x:%08x, id=%u, index=%u",
>                   __entry->vcpu, __entry->icrh, __entry->icrl,
>                   __entry->id, __entry->index)
>  );
> @@ -1348,7 +1348,7 @@ TRACE_EVENT(kvm_avic_unaccelerated_access,
>                 __entry->vec = vec;
>         ),
>
> -       TP_printk("vcpu=%u, offset=%#x(%s), %s, %s, vec=%#x\n",
> +       TP_printk("vcpu=%u, offset=%#x(%s), %s, %s, vec=%#x",
>                   __entry->vcpu,
>                   __entry->offset,
>                   __print_symbolic(__entry->offset, kvm_trace_symbol_apic),
> @@ -1368,7 +1368,7 @@ TRACE_EVENT(kvm_hv_timer_state,
>                         __entry->vcpu_id = vcpu_id;
>                         __entry->hv_timer_in_use = hv_timer_in_use;
>                         ),
> -               TP_printk("vcpu_id %x hv_timer %x\n",
> +               TP_printk("vcpu_id %x hv_timer %x",
>                         __entry->vcpu_id,
>                         __entry->hv_timer_in_use)

The last one is handled by commit 7be373b6de503 .

Regards,
Wanpeng Li



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux