[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 7/8] trace: [simple] Bump up log version number
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 7/8] trace: [simple] Bump up log version number |
Date: |
Fri, 14 Mar 2014 10:02:54 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Mar 12, 2014 at 03:42:19PM +0100, Lluís Vilanova wrote:
> Stefan Hajnoczi writes:
>
> > On Sun, Feb 23, 2014 at 08:37:35PM +0100, Lluís Vilanova wrote:
> >> The following tracetool cleanup changes the event numbering policy.
> >>
> >> Signed-off-by: Lluís Vilanova <address@hidden>
> >> ---
> >> scripts/simpletrace.py | 10 +++++-----
> >> trace/simple.c | 2 +-
> >> 2 files changed, 6 insertions(+), 6 deletions(-)
>
> > I would also like to add a change to the simpletrace binary format. The
> > 32-bit reserved field will take the pid of QEMU. I will also submit a
> > SystemTap script that produces simpletrace binary format output - this
> > allows us to produce simpletrace-compatible data with a QEMU binary that
> > only has SystemTap support. Since SystemTap is global it sees all QEMU
> > processes and we need to distinguish between them using the pid.
>
> > Your series will be merged into qemu.git at the start of the QEMU 2.1
> > release cycle. My patch can follow during the release cycle. End-users
> > will not see the changes until QEMU 2.1 is released.
>
> Sounds excellent, but note that I also wrote a patch for multi-backend tracing
> based on Kazuya's idea (must be applied on top of this series). Not sure if
> that
> would influence your decision about SystemTap's format (since you could then
> have both compiled into QEMU).
>
> I will later send the latest version (rebased on this series).
Thanks, I see you sent the patch. The SystemTap -> simpletrace bridge
is for a slightly different use case. I'm putting the patches together
and will CC you.
Stefan