[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.8 v1 60/60] trace: update docs to reflect
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH for-2.8 v1 60/60] trace: update docs to reflect new code generation approach |
Date: |
Thu, 11 Aug 2016 12:11:06 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Daniel P Berrange writes:
> On Wed, Aug 10, 2016 at 06:11:24PM +0200, Lluís Vilanova wrote:
>> Daniel P Berrange writes:
>>
>> > On Wed, Aug 10, 2016 at 04:33:47PM +0200, Lluís Vilanova wrote:
>> >> Daniel P Berrange writes:
[...]
>> >> For example, first we concatenate /trace-events, /io/trace-events and
>> >> /hw/usb/trace-events into /trace-events-all to produce:
>> >>
>> >> @group core
>> >> @dir /
>> >> # contents of /trace-events
>> >> @group io
>> >> @dir /io
>> >> # contents of /io/trace-events
>> >> @group hw_usb
>> >> @dir /hw/usb
>> >> # contents of /hw/usb/trace-events
>> >>
>> >> Then we can use /trace-events-all to generate all the per-directory
>> >> trace.h
>> >> files while ensuring a perfectly flat event identifier space (a single
>> >> generated-events.c is used).
>> >>
>> >> This should minimize changes to tracing code and avoid further
>> >> restrictions
>> >> (like the "top-level vcpu" one).
>>
>> > That means when any individual trace file is changed, we re-generate the
>> > world, which is exactly what this series is designed to avoid.
>>
>> Nope. We re-generate /trace-events-all, but each individual file generated
>> afterwards (taken from the "@dir" directive) will change only if its new
>> contents do (the Makefile uses cmp to check if files need updating from their
>> corresponding timestamp file).
> The contents will still need to change too if we have a global numbering
> space as your suggestion requires. The event groups are the only way to
> avoid that.
Aha, that's true. I didn't think of that, sorry.
Cheers,
Lluis
- [Qemu-devel] [PATCH for-2.8 v1 54/60] trace: switch target-sparc/ directory to modular trace.h file, (continued)
- [Qemu-devel] [PATCH for-2.8 v1 54/60] trace: switch target-sparc/ directory to modular trace.h file, Daniel P. Berrange, 2016/08/09
- [Qemu-devel] [PATCH for-2.8 v1 56/60] trace: switch target-ppc/ directory to modular trace.h file, Daniel P. Berrange, 2016/08/09
- [Qemu-devel] [PATCH for-2.8 v1 57/60] trace: switch qom/ directory to modular trace.h file, Daniel P. Berrange, 2016/08/09
- [Qemu-devel] [PATCH for-2.8 v1 51/60] trace: switch audio/ directory to modular trace.h file, Daniel P. Berrange, 2016/08/09
- [Qemu-devel] [PATCH for-2.8 v1 53/60] trace: switch target-i386/ directory to modular trace.h file, Daniel P. Berrange, 2016/08/09
- [Qemu-devel] [PATCH for-2.8 v1 60/60] trace: update docs to reflect new code generation approach, Daniel P. Berrange, 2016/08/09
[Qemu-devel] [PATCH for-2.8 v1 59/60] trace: remove the global include/trace.h file, Daniel P. Berrange, 2016/08/09
Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, no-reply, 2016/08/09
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Daniel P. Berrange, 2016/08/09
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Fam Zheng, 2016/08/09
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Daniel P. Berrange, 2016/08/10
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Fam Zheng, 2016/08/10
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Daniel P. Berrange, 2016/08/10
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Fam Zheng, 2016/08/10
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Daniel P. Berrange, 2016/08/10
- Re: [Qemu-devel] [PATCH for-2.8 v1 00/60] Modular build of trace event files, Paolo Bonzini, 2016/08/10