[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v3 0/8] Rewrite tracetool using python modul
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [RFC PATCH v3 0/8] Rewrite tracetool using python modules |
Date: |
Mon, 02 Apr 2012 14:52:48 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.94 (gnu/linux) |
Stefan Hajnoczi writes:
> On Fri, Mar 30, 2012 at 07:12:46PM +0200, Lluís Vilanova wrote:
>> Stefan Hajnoczi writes:
>> > Only one point: please don't introduce PUBLIC yet. Let's add it when
>> > it's needed. At the moment nothing uses it.
>>
>> > I have tested this series with all backends and looked at the diff
>> > between the old tracetool and tracetool.py.
>>
>> > I'm happy with this series. We need to test it hard for 1.1 to make
>> > sure there are no hickups for tracing users.
>>
>> Sure. I'll send v4 without the PUBLIC-related contents and turning "kwargs"
>> into
>> specific arguments.
>>
>> Once accepted, I'll send the first two batches of changes in my queue, which
>> are
>> orthogonal to instrumentation (basically tracing cleanups in both the build
>> and
>> the API).
>>
>> I suppose everyone prefers to wait until after the 1.1 release to start
>> having a
>> look at instrumentation (which I still have to port to this new tracetool).
> Great. I'd like to merge the Python tracetool for 1.1. I've diffed the
> generated code and it appears compatible with the shell tracetool
> output. Therefore I'm pretty confident with this series.
> Future series will not make it into 1.1 just because we're running out
> of time to review and test.
Do you mean future versions of this series (e.g., v5) or just other series?
Thanks,
Lluis
--
"And it's much the same thing with knowledge, for whenever you learn
something new, the whole world becomes that much richer."
-- The Princess of Pure Reason, as told by Norton Juster in The Phantom
Tollbooth