Re: [PATCH] ARM: KVM: Simplify tracepoint text

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

 



On Sat, Aug 10, 2013 at 11:47:25PM +0400, Sergei Shtylyov wrote:
> Hello.
> 
> On 08/09/2013 07:41 AM, Christoffer Dall wrote:
> 
> >The tracepoint for kvm_guest_fault was extremely long, make it a
> >slightly bit shorter.
> 
> >Signed-off-by: Christoffer Dall <christoffer.dall@xxxxxxxxxx>
> >---
> >  arch/arm/kvm/trace.h |    7 +++----
> >  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> >diff --git a/arch/arm/kvm/trace.h b/arch/arm/kvm/trace.h
> >index a8e73ed..8cbaea2 100644
> >--- a/arch/arm/kvm/trace.h
> >+++ b/arch/arm/kvm/trace.h
> >@@ -59,10 +59,9 @@ TRACE_EVENT(kvm_guest_fault,
> >  		__entry->ipa			= ipa;
> >  	),
> >
> >-	TP_printk("guest fault at PC %#08lx (hxfar %#08lx, "
> >-		  "ipa %#16llx, hsr %#08lx",
> >-		  __entry->vcpu_pc, __entry->hxfar,
> >-		  __entry->ipa, __entry->hsr)
> >+	TP_printk("ipa 0x%llx, hsr 0x%#08lx, hxfar 0x%#08lx, pc 0x%#08lx",
> 
>    "0x" and "#" are redundant, they mean the same thing, hexadecimal prefix.
> 
Duh, indeed, nicely spotted.  I'll respin.

Thanks,
-Christoffer
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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