[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 06/14] trace: Trace entry point of balloon reque
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 06/14] trace: Trace entry point of balloon request handler |
Date: |
Mon, 23 Aug 2010 11:10:19 +0100 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Sun, Aug 22, 2010 at 04:41:44PM -0500, Anthony Liguori wrote:
> ram_addr_t is not necessarily an unsigned long. It's actually a
> uint64_t. Integer promotion should handle this in most cases but
> having the definition in two different places worries me a bit.
I thought it is unsigned long:
cpu-common.h:typedef unsigned long ram_addr_t;
ram_addr_t cannot be used explicitly because it is only defined for
softmmu targets. User-only builds do not define ram_addr_t and the
generated trace.h header would break the compile.
This is a more general weakness of the tracing system: it is fragile
under #ifdefed builds because it tries to work for softmmu, user-only,
and qemu-tools! Perhaps we should split the trace-events file.
Stefan
- Re: [Qemu-devel] [PATCH 08/14] trace: Support for dynamically enabling/disabling trace events., (continued)
- [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
- Re: [Qemu-devel] [PATCH 00/14] trace: Add static tracing to QEMU, Anthony Liguori, 2010/08/22