[Top][All Lists]

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

Re: [Qemu-devel] [PATCH] trace: add ability to do simple printf logging

From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH] trace: add ability to do simple printf logging via systemtap
Date: Fri, 18 Jan 2019 10:44:16 +0000
User-agent: Mutt/1.10.1 (2018-07-13)

On Thu, Jan 17, 2019 at 02:52:07PM +0000, Daniel P. Berrangé wrote:

Dequeued.  I encountered two issues during testing:

1. s/binary/file/ in qemu-trace-stap's which() function:

diff --git a/scripts/qemu-trace-stap b/scripts/qemu-trace-stap
index b2eb41fbb0..3b4ec5e9b9 100755
--- a/scripts/qemu-trace-stap
+++ b/scripts/qemu-trace-stap
@@ -37,7 +37,7 @@ def which(file):
         if os.path.exists(os.path.join(path, file)):
                 return os.path.join(path, file)
-    print("Unable to find '%s' in $PATH" % binary)
+    print("Unable to find '%s' in $PATH" % file)

2. The tpm_crb_mmio_read trace event's TARGET_FMT_plx format specifier
   macro isn't handled.  stap sees a literal "TARGET_FMT_plx" string and
   reports that the number of format string arguments is incorrect:

   semantic error: Wrong number of args to formatted print operator: identifier 
'printf' at :9189:5
   source:     printf("address@hidden tpm_crb_mmio_read CRB read 0x 
TARGET_FMT_plx  len:%u val: 0x%x\n", pid(), gettimeofday_ns(), addr, size, val)

Attachment: signature.asc
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]