[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/2] trace: [*-user] Add commandline argument
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/2] trace: [*-user] Add commandline arguments to control tracing |
Date: |
Fri, 15 Jul 2016 19:07:26 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Stefan Hajnoczi writes:
> On Wed, Jun 22, 2016 at 12:04:30PM +0200, Lluís Vilanova wrote:
>> Adds three commandline arguments to the main *-user programs, following
>> what's
>> already available in softmmu:
>>
>> * -trace-enable
>> * -trace-events
>> * -trace-file
>>
>>
>> Changes in v2
>> =============
>>
>> * Tell user to use 'help' instead of '?' [Eric Blake].
>> * Remove newlines on argument docs for bsd-user [Eric Blake].
>>
>>
>> Signed-off-by: Lluís Vilanova <address@hidden>
>> ---
>>
>> Lluís Vilanova (2):
>> trace: [linux-user] Commandline arguments to control tracing
>> trace: [bsd-user] Commandline arguments to control tracing
>>
>>
>> bsd-user/main.c | 19 +++++++++++++++++++
>> linux-user/main.c | 28 ++++++++++++++++++++++++++++
>> 2 files changed, 47 insertions(+)
> Hi Lluís,
> Commit e9e0bb2af2248eabafb54402e3127f9f8a8690f5 ("trace: move
> qemu_trace_opts to trace/control.c") made trace_events_init() static.
> This conflicts with your patch series.
> I suggest changing this series to use -trace ... just like
> qemu/qemu-img/qemu-nbd.
I'll send a new version adapted to that.
Thanks,
Lluis