Re: [Qemu-devel] [PATCH 01/32] megasas: Replace trace_megasas_dcmd_dump_frame()

[Stefan Hajnoczi]

On Fri, Jul 27, 2012 at 4:02 PM, Paolo Bonzini <address@hidden> wrote:
> From: Hannes Reinecke <address@hidden>
>
> trace_megasas_dcmd_dump_frame() takes 9 arguments, which is
> rather much. Plus the trace infrastructure doesn't support
> it.  As we can get the information via other means it's pointless
> to have it in the driver, so rather use some proper trace
> point here and remove the old one.
>
> Signed-off-by: Hannes Reinecke <address@hidden>
> Cc: Stefan Weil <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  hw/megasas.c |   39 ++++++++++-----------------------------
>  trace-events |    2 +-
>  2 files changed, 11 insertions(+), 30 deletions(-)

The simple trace backend now supports arbitrary argument counts :).

I don't think you need to change this patch but I'm just spreading the
word since this limitation has now been lifted.

Stefan