[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: |
Mon, 23 Aug 2010 11:00:12 +0100 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Sun, Aug 22, 2010 at 04:38:36PM -0500, Anthony Liguori wrote:
> On 08/12/2010 05:36 AM, Stefan Hajnoczi wrote:
> >+linetoh_nop()
> >+{
> >+ local name args
> >+ name=$(get_name "$1")
> >+ args=$(get_args "$1")
> >+
> >+ # Define an empty function for the trace event
> >+ cat<<EOF
> >+static inline void trace_$name($args)
> >+{
> >+}
>
> This seems like it can potentially create a single header that ends
> up needed to know about every type of structure QEMU implements, no?
> If so, this would be challenging for improving modularity.
>
> Or would we restrict trace points to not allow passing of anything
> but void *s?
Trace events use void *. It would be nicer to use the specific struct
pointer type but that is not possible for the reason you mentioned.
Stefan
- Re: [Qemu-devel] [PATCH 14/14] trace: Add user documentation, (continued)
- [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
- [Qemu-devel] [PATCH 02/14] trace: Trace qemu_malloc() and qemu_vmalloc(), Stefan Hajnoczi, 2010/08/12
- Re: [Qemu-devel] [PATCH 00/14] trace: Add static tracing to QEMU, Blue Swirl, 2010/08/12