qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [PATCH 01/14] trace: Add trace-events file for declarin


From: Blue Swirl
Subject: [Qemu-devel] Re: [PATCH 01/14] trace: Add trace-events file for declaring trace events
Date: Mon, 30 Aug 2010 20:42:08 +0000

On Mon, Aug 30, 2010 at 8:10 PM, malc <address@hidden> wrote:
> On Mon, 30 Aug 2010, Blue Swirl wrote:
>
>> On Mon, Aug 30, 2010 at 1:27 PM, Stefan Hajnoczi
>> <address@hidden> 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"
>> >
>> > These trace event declarations are processed by a new tool called
>> > tracetool to generate code for the trace events.  Trace event
>> > declarations are independent of the backend tracing system (LTTng User
>> > Space Tracing, ftrace markers, DTrace).
>>
>> I think the tool does not work if 'sh' is not 'bash'. For example, on
>> OpenBSD I got:
>
> Well, it does work with ash.
>
>>
>> config-host.mak is out-of-date, running configure
>>
>> Error: invalid trace backend
>> Please choose a supported trace backend.
>>
>>   GEN   trace.h
>> /src/qemu/tracetool[176]: no closing quote
>>
>> This shows the problem:
>> sh -x ../tracetool --nop --check-backend
>> + set -f
>> ../tracetool[176]: no closing quote
>
> `set -f' is a valid construct according to:
> http://www.opengroup.org/onlinepubs/009695399/utilities/set.html
>
> The problem is likely elsewhere.

Right, the offending lines are:
    echo ${1%%(*}
and
    args=${1#*(}

If I remove both of those, the errors are gone.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]