[Top][All Lists]
[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 21:01:06 +0000 |
On Mon, Aug 30, 2010 at 8:42 PM, Blue Swirl <address@hidden> wrote:
> 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.
>
This patch fixes the problem. Double quotes do not help.
diff --git a/tracetool b/tracetool
index d640100..01de580 100755
--- a/tracetool
+++ b/tracetool
@@ -29,14 +29,14 @@ EOF
# Get the name of a trace event
get_name()
{
- echo ${1%%(*}
+ echo ${1%%\(*}
}
# Get the argument list of a trace event, including types and names
get_args()
{
local args
- args=${1#*(}
+ args=${1#*\(}
args=${args%)*}
echo "$args"
}
- [Qemu-devel] [PATCH 06/14] trace: Add trace-file command to open/close/flush trace file, (continued)
- [Qemu-devel] [PATCH 06/14] trace: Add trace-file command to open/close/flush trace file, Stefan Hajnoczi, 2010/08/30
- [Qemu-devel] [PATCH 11/14] trace: Trace virtio-blk, multiwrite, and paio_submit, Stefan Hajnoczi, 2010/08/30
- [Qemu-devel] [PATCH 02/14] trace: Add simple built-in tracing backend, Stefan Hajnoczi, 2010/08/30
- [Qemu-devel] [PATCH 03/14] trace: Support for dynamically enabling/disabling trace events, Stefan Hajnoczi, 2010/08/30
- [Qemu-devel] [PATCH 07/14] trace: Add trace file name command-line option, Stefan Hajnoczi, 2010/08/30
- [Qemu-devel] [PATCH 08/14] trace: Add LTTng Userspace Tracer backend, Stefan Hajnoczi, 2010/08/30
- [Qemu-devel] [PATCH 01/14] trace: Add trace-events file for declaring trace events, Stefan Hajnoczi, 2010/08/30
[Qemu-devel] [PATCH 05/14] trace: Specify trace file name, Stefan Hajnoczi, 2010/08/30
[Qemu-devel] [PATCH 09/14] trace: Add user documentation, Stefan Hajnoczi, 2010/08/30
[Qemu-devel] [PATCH 04/14] trace: Support disabled events in trace-events, Stefan Hajnoczi, 2010/08/30
[Qemu-devel] [PATCH 13/14] trace: Trace port IO, Stefan Hajnoczi, 2010/08/30
[Qemu-devel] [PATCH 14/14] trace: Trace entry point of balloon request handler, Stefan Hajnoczi, 2010/08/30