[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/14] trace: Add trace-events file for declarin
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 01/14] trace: Add trace-events file for declaring trace events |
Date: |
Thu, 12 Aug 2010 20:15:44 +0100 |
On Thu, Aug 12, 2010 at 5:07 PM, malc <address@hidden> wrote:
> On Thu, 12 Aug 2010, Stefan Hajnoczi wrote:
>
>> This patch introduces the trace-events file where trace events can be
>> declared like so:
>>
>> qemu_malloc(size_t size) "size %zu"
>> qemu_free(void *ptr) "ptr %p"
>>
>
> [..snip..]
>
>> + echo -n "$name, "
>
> Once again: standard echo doesn't have any command line arguments, please
> use printf instead.
Sorry about this. The s/echo -n/printf "%s"/ fix is in the patchset
in "[PATCH 07/14] trace: Add simple built-in tracing backend". I
didn't push the change back far enough in the commit history.
Will fix for v2.
Stefan
- [Qemu-devel] [PATCH 14/14] trace: Add user documentation, (continued)
- [Qemu-devel] [PATCH 14/14] trace: Add user documentation, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 03/14] trace: Trace virtio-blk, multiwrite, and paio_submit, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 08/14] trace: Support for dynamically enabling/disabling trace events., Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 11/14] trace: Add trace-file command to open/close/flush trace file, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 01/14] trace: Add trace-events file for declaring trace events, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 04/14] trace: Trace virtqueue operations, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 06/14] trace: Trace entry point of balloon request handler, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 10/14] trace: Specify trace file name, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 05/14] trace: Trace port IO, Stefan Hajnoczi, 2010/08/12
- [Qemu-devel] [PATCH 09/14] trace: Support disabled events in trace-events, Stefan Hajnoczi, 2010/08/12