[ltt-dev] [UST PATCH 1/3] Change code for consistency
David Goulet
david.goulet at polymtl.ca
Fri Sep 17 17:40:24 EDT 2010
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 10-09-17 05:07 PM, Mathieu Desnoyers wrote:
> * David Goulet (david.goulet at polymtl.ca) wrote:
>> Frankly, I don't know what is CPP. I'll be happy if you can show me what it is.
>>
>> However, every patch that I submit, I tested them with the tests/runtests and my
>> benchmark test.
>
> You know it, just just don't know it's called "CPP". It stands for "C
> Preprocessor". gcc -E runs the preprocessor stage only.
>
Awwwwwwww :) ... of course i'm using that... it's the only way I can decrypt the
UST Macros :P
David
> Mathieu
>
>>
>> Thanks
>> David
>>
>> Selon Nils Carlson <nils.carlson at ludd.ltu.se>:
>>
>>> Looks good!
>>>
>>> Have you tried piping the old and new one through CPP with a file
>>> containing just one marker and running diff aftwerwards? This is sort
>>> of a good "test" when playing with the macros to make sure you've got
>>> it all right. Also very educational as you get to see what everything
>>> expands into in the end.
>>>
>>> /Nils
>>> On Sep 17, 2010, at 9:41 PM, David Goulet wrote:
>>>
>>>> Use the kcompat stringify header for double level
>>>> stringification for markers section.
>>>>
>>>> Also use the GET_MARKER macro for uniformity across the code.
>>>>
>>>> Signed-off-by: David Goulet <david.goulet at polymtl.ca>
>>>> ---
>>>> include/ust/marker.h | 33 +++++++++++++++------------------
>>>> 1 files changed, 15 insertions(+), 18 deletions(-)
>>>>
>>>> diff --git a/include/ust/marker.h b/include/ust/marker.h
>>>> index 2751f58..287f20f 100644
>>>> --- a/include/ust/marker.h
>>>> +++ b/include/ust/marker.h
>>>> @@ -30,15 +30,12 @@
>>>> #include <urcu/list.h>
>>>> #include <ust/processor.h>
>>>> #include <ust/kcompat/kcompat.h>
>>>> +#include <ust/kcompat/stringify.h>
>>>>
>>>> #include <bits/wordsize.h>
>>>>
>>>> struct marker;
>>>>
>>>> -/* To stringify the expansion of a define */
>>>> -#define XSTR(d) STR(d)
>>>> -#define STR(s) #s
>>>> -
>>>> /**
>>>> * marker_probe_func - Type of a marker probe function
>>>> * @mdata: marker data
>>>> @@ -83,6 +80,8 @@ struct marker {
>>>> #define CONFIG_MARKERS
>>>> #ifdef CONFIG_MARKERS
>>>>
>>>> +#define GET_MARKER(channel, name) (__mark_##channel##_##name)
>>>> +
>>>> #define _DEFINE_MARKER(channel, name, tp_name_str, tp_cb, format,
>>>> unique, m) \
>>>> struct registers regs; \
>>>> \
>>>> @@ -96,14 +95,14 @@ struct marker {
>>>> * will try to create the same symbols, resulting in a
>>>> conflict. This \
>>>> * is not unusual as it can be the result of function
>>>> inlining. \
>>>> */ \
>>>> - ".ifndef __mstrtab_" XSTR(channel) "_" XSTR(name)
>>>> "_channel_" XSTR(unique) "\n\t" \
>>>> + ".ifndef __mstrtab_" __stringify(channel) "_"
>>>> __stringify(name) "_channel_" __stringify(unique) "\n\t" \
>>>> /*".section __markers_strings\n\t"*/ \
>>>> ".section __markers_strings,\"aw\", at progbits\n\t" \
>>>> - "__mstrtab_" XSTR(channel) "_" XSTR(name) "_channel_"
>>>> XSTR(unique) ":\n\t" \
>>>> - ".string \"" XSTR(channel) "\"\n\t" \
>>>> - "__mstrtab_" XSTR(channel) "_" XSTR(name) "_name_"
>>>> XSTR(unique) ":\n\t" \
>>>> - ".string \"" XSTR(name) "\"\n\t" \
>>>> - "__mstrtab_" XSTR(channel) "_" XSTR(name) "_format_"
>>>> XSTR(unique) ":\n\t" \
>>>> + "__mstrtab_" __stringify(channel) "_" __stringify(name)
>>>> "_channel_" __stringify(unique) ":\n\t" \
>>>> + ".string \"" __stringify(channel) "\"\n\t" \
>>>> + "__mstrtab_" __stringify(channel) "_" __stringify(name)
>>>> "_name_" __stringify(unique) ":\n\t" \
>>>> + ".string \"" __stringify(name) "\"\n\t" \
>>>> + "__mstrtab_" __stringify(channel) "_" __stringify(name)
>>>> "_format_" __stringify(unique) ":\n\t" \
>>>> ".string " "\"" format "\"" "\n\t" \
>>>> ".previous\n\t" \
>>>> ".endif\n\t" \
>>>> @@ -113,14 +112,14 @@ struct marker {
>>>> ".section __markers,\"aw\", at progbits\n\t" \
>>>> ".balign 8\n\t" \
>>>> "2:\n\t" \
>>>> - _ASM_PTR "(__mstrtab_" XSTR(channel) "_" XSTR(name)
>>>> "_channel_" XSTR(unique) ")\n\t" /* channel string */ \
>>>> - _ASM_PTR "(__mstrtab_" XSTR(channel) "_" XSTR(name) "_name_"
>>>> XSTR(unique) ")\n\t" /* name string */ \
>>>> - _ASM_PTR "(__mstrtab_" XSTR(channel) "_" XSTR(name)
>>>> "_format_" XSTR(unique) ")\n\t" /* format string */ \
>>>> + _ASM_PTR "(__mstrtab_" __stringify(channel) "_"
>>>> __stringify(name) "_channel_" __stringify(unique) ")\n\t" /* channel
>>>> string */ \
>>>> + _ASM_PTR "(__mstrtab_" __stringify(channel) "_"
>>>> __stringify(name) "_name_" __stringify(unique) ")\n\t" /* name
>>>> string */ \
>>>> + _ASM_PTR "(__mstrtab_" __stringify(channel) "_"
>>>> __stringify(name) "_format_" __stringify(unique) ")\n\t" /* format
>>>> string */ \
>>>> ".byte 0\n\t" /* state imv */ \
>>>> ".byte 0\n\t" /* ptype */ \
>>>> ".word 0\n\t" /* channel_id */ \
>>>> ".word 0\n\t" /* event_id */ \
>>>> - ".balign " XSTR(__WORDSIZE) " / 8\n\t" /* alignment */ \
>>>> + ".balign " __stringify(__WORDSIZE) " / 8\n\t" /* alignment
>>>> */ \
>>>> _ASM_PTR "(marker_probe_cb)\n\t" /* call */ \
>>>> _ASM_PTR "(__mark_empty_function)\n\t" /*
>>>> marker_probe_closure single.field1 */ \
>>>> _ASM_PTR "0\n\t" /* marker_probe_closure single.field2 */ \
>>>> @@ -146,7 +145,7 @@ struct marker {
>>>> static const char __mstrtab_##channel##_##name[] \
>>>> __attribute__((section("__markers_strings"))) \
>>>> = #channel "\0" #name "\0" format; \
>>>> - static struct marker __mark_##channel##_##name \
>>>> + static struct marker GET_MARKER(channel, name) \
>>>> __attribute__((section("__markers"), aligned(8))) = \
>>>> { __mstrtab_##channel##_##name, \
>>>> &__mstrtab_##channel##_##name[sizeof(#channel)], \
>>>> @@ -195,15 +194,13 @@ struct marker {
>>>> } \
>>>> DEFINE_MARKER_TP(channel, name, tp_name, tp_cb, format);\
>>>> __mark_check_format(format, ## args); \
>>>> - (*__mark_##channel##_##name.call)(&__mark_##channel##_##name, \
>>>> + (*GET_MARKER(channel, name).call)(&GET_MARKER(channel, name), \
>>>> call_private, ®s, ## args); \
>>>> } while (0)
>>>>
>>>> extern void marker_update_probe_range(struct marker *begin,
>>>> struct marker *end);
>>>>
>>>> -#define GET_MARKER(channel, name) (__mark_##channel##_##name)
>>>> -
>>>> #else /* !CONFIG_MARKERS */
>>>> #define DEFINE_MARKER(channel, name, tp_name, tp_cb, format, m)
>>>> #define __trace_mark(generic, channel, name, call_private, format,
>>>> args...) \
>>>> --
>>>> 1.7.2.3
>>>>
>>>>
>>>> _______________________________________________
>>>> ltt-dev mailing list
>>>> ltt-dev at lists.casi.polymtl.ca
>>>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>>
>>>
>>
>>
>>
>
- --
David Goulet
LTTng project, DORSAL Lab.
1024D/16BD8563
BE3C 672B 9331 9796 291A 14C6 4AF7 C14B 16BD 8563
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
iEYEARECAAYFAkyT4EgACgkQSvfBSxa9hWNuswCgnbat1TuV0xkGqtlrj+SZSPac
7LAAoLO0LHtIZgxcKZ07NHIuIEnE7XeK
=IB+h
-----END PGP SIGNATURE-----
More information about the lttng-dev
mailing list