* [RFC PATCH lttng-tools] Fix printk instrumentation
@ 2013-07-04 16:58 Mathieu Desnoyers
0 siblings, 0 replies; 5+ messages in thread
From: Mathieu Desnoyers @ 2013-07-04 16:58 UTC (permalink / raw)
To: yannick.brosseau; +Cc: lttng-dev
This patch fixes the printk instrumentation.
- It does not require a 2kB buffer on the stack anymore,
- It introduces __dynamic_array_enc_ext_2() and tp_memcpy_dyn_2(), which
allow copying into a single variable-length buffer from two sources,
useful for kernel < 3.5.
- It adds support for 3.10 kernel printk instrumentation.
Yannick, can you try this out on 3.4, 3.5, 3.9 and 3.10 kernels ? When
you confirm that it works, I'll split it into two patches (one for
lttng-events.h and the other for printk.h), merge, and backport this.
Only built-tested on 3.5 kernel so far.
Thanks,
Mathieu
---
diff --git a/instrumentation/events/lttng-module/printk.h b/instrumentation/events/lttng-module/printk.h
index 4c744f9..e22a2cc 100644
--- a/instrumentation/events/lttng-module/printk.h
+++ b/instrumentation/events/lttng-module/printk.h
@@ -7,7 +7,25 @@
#include <linux/tracepoint.h>
#include <linux/version.h>
-#define MSG_TRACE_MAX_LEN 2048
+#if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,10,0))
+
+TRACE_EVENT(console,
+ TP_PROTO(const char *text, size_t len),
+
+ TP_ARGS(text, len),
+
+ TP_STRUCT__entry(
+ __dynamic_array_text(char, msg, len)
+ ),
+
+ TP_fast_assign(
+ tp_memcpy_dyn(msg, msg)
+ ),
+
+ TP_printk("%s", __get_str(msg))
+)
+
+#elif (LINUX_VERSION_CODE >= KERNEL_VERSION(3,5,0))
TRACE_EVENT_CONDITION(console,
TP_PROTO(const char *log_buf, unsigned start, unsigned end,
@@ -18,46 +36,47 @@ TRACE_EVENT_CONDITION(console,
TP_CONDITION(start != end),
TP_STRUCT__entry(
- __dynamic_array_text(char, msg,
- min_t(unsigned, end - start, MSG_TRACE_MAX_LEN) + 1)
+ __dynamic_array_text(char, msg, end - start)
),
TP_fast_assign(
- tp_memcpy_dyn(msg,
- ({
- char lmsg[MSG_TRACE_MAX_LEN + 1];
-
- if ((end - start) > MSG_TRACE_MAX_LEN)
- start = end - MSG_TRACE_MAX_LEN;
-
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(3,5,0))
- if ((start & (log_buf_len - 1)) >
- (end & (log_buf_len - 1))) {
- memcpy(lmsg,
- log_buf +
- (start & (log_buf_len - 1)),
- log_buf_len -
- (start & (log_buf_len - 1)));
- memcpy(lmsg + log_buf_len -
- (start & (log_buf_len - 1)),
- log_buf,
- end & (log_buf_len - 1));
- } else
- memcpy(lmsg,
- log_buf +
- (start & (log_buf_len - 1)),
- end - start);
-#else
- memcpy(lmsg, log_buf + start, end - start);
-#endif
- lmsg[end - start] = 0;
- lmsg;
- })
- )
+ tp_memcpy_dyn(msg, log_buf + start)
+ ),
+
+ TP_printk("%s", __get_str(msg))
+)
+
+#else /* (LINUX_VERSION_CODE < KERNEL_VERSION(3,5,0)) */
+
+TRACE_EVENT_CONDITION(console,
+ TP_PROTO(const char *log_buf, unsigned start, unsigned end,
+ unsigned log_buf_len),
+
+ TP_ARGS(log_buf, start, end, log_buf_len),
+
+ TP_CONDITION(start != end),
+
+ TP_STRUCT__entry(
+ __dynamic_array_text_2(char, msg,
+ (start & (log_buf_len - 1)) > (end & (log_buf_len - 1))
+ ? log_buf_len - (start & (log_buf_len - 1))
+ : end - start,
+ (start & (log_buf_len - 1)) > (end & (log_buf_len - 1))
+ ? end & (log_buf_len - 1)
+ : 0)
+ ),
+
+ TP_fast_assign(
+ tp_memcpy_dyn_2(msg,
+ log_buf + (start & (log_buf_len - 1)),
+ log_buf)
),
TP_printk("%s", __get_str(msg))
)
+
+#endif
+
#endif /* _TRACE_PRINTK_H */
/* This part must be outside protection */
diff --git a/probes/lttng-events-reset.h b/probes/lttng-events-reset.h
index 44e8ba5..b601e8d 100644
--- a/probes/lttng-events-reset.h
+++ b/probes/lttng-events-reset.h
@@ -29,6 +29,9 @@
#undef __dynamic_array_enc_ext
#define __dynamic_array_enc_ext(_type, _item, _length, _order, _base, _encoding)
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)
+
#undef __dynamic_array_len
#define __dynamic_array_len(_type, _item, _length)
diff --git a/probes/lttng-events.h b/probes/lttng-events.h
index 8a3a886..9606204 100644
--- a/probes/lttng-events.h
+++ b/probes/lttng-events.h
@@ -236,6 +236,10 @@ void trace_##_name(void *__data);
}, \
},
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding) \
+ __dynamic_array_enc_ext(_type, _item, _length1 + _length2, _order, _base, _encoding)
+
#undef __dynamic_array
#define __dynamic_array(_type, _item, _length) \
__dynamic_array_enc_ext(_type, _item, _length, __BYTE_ORDER, 10, none)
@@ -248,6 +252,10 @@ void trace_##_name(void *__data);
#define __dynamic_array_hex(_type, _item, _length) \
__dynamic_array_enc_ext(_type, _item, _length, __BYTE_ORDER, 16, none)
+#undef __dynamic_array_text_2
+#define __dynamic_array_text_2(_type, _item, _length1, _length2) \
+ __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, __BYTE_ORDER, 10, UTF8)
+
#undef __string
#define __string(_item, _src) \
{ \
@@ -407,6 +415,18 @@ static __used struct lttng_probe_desc TP_ID(__probe_desc___, TRACE_SYSTEM) = {
__event_len += sizeof(_type) * __dynamic_len[__dynamic_len_idx]; \
__dynamic_len_idx++;
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ __event_len += lib_ring_buffer_align(__event_len, lttng_alignof(u32)); \
+ __event_len += sizeof(u32); \
+ __event_len += lib_ring_buffer_align(__event_len, lttng_alignof(_type)); \
+ __dynamic_len[__dynamic_len_idx] = (_length1); \
+ __event_len += sizeof(_type) * __dynamic_len[__dynamic_len_idx]; \
+ __dynamic_len_idx++; \
+ __dynamic_len[__dynamic_len_idx] = (_length2); \
+ __event_len += sizeof(_type) * __dynamic_len[__dynamic_len_idx]; \
+ __dynamic_len_idx++;
+
#undef __string
#define __string(_item, _src) \
__event_len += __dynamic_len[__dynamic_len_idx++] = strlen(_src) + 1;
@@ -464,6 +484,10 @@ static inline size_t __event_get_size__##_name(size_t *__dynamic_len, _proto) \
__event_align = max_t(size_t, __event_align, lttng_alignof(u32)); \
__event_align = max_t(size_t, __event_align, lttng_alignof(_type));
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ __dynamic_array_enc_ext(_type, _item, _length1 + _length2, _order, _base, _encoding)
+
#undef __string
#define __string(_item, _src)
@@ -510,6 +534,10 @@ static inline size_t __event_get_align__##_name(_proto) \
#define __dynamic_array_enc_ext(_type, _item, _length, _order, _base, _encoding)\
_type _item;
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ __dynamic_array_enc_ext(_type, _item, _length1 + _length2, _order, _base, _encoding)
+
#undef __string
#define __string(_item, _src) char _item;
@@ -559,6 +587,15 @@ __end_field_##_item##_1: \
goto __assign_##_item##_2; \
__end_field_##_item##_2:
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ goto __assign_##_item##_1; \
+__end_field_##_item##_1: \
+ goto __assign_##_item##_2; \
+__end_field_##_item##_2: \
+ goto __assign_##_item##_3; \
+__end_field_##_item##_3:
+
#undef __string
#define __string(_item, _src) \
goto __assign_##_item; \
@@ -616,10 +653,34 @@ __assign_##dest##_2: \
sizeof(__typemap.dest) * __get_dynamic_array_len(dest));\
goto __end_field_##dest##_2;
+#undef tp_memcpy_dyn_gen_2
+#define tp_memcpy_dyn_gen_2(write_ops, dest, src1, src2) \
+__assign_##dest##_1: \
+ { \
+ u32 __tmpl = __dynamic_len[__dynamic_len_idx] \
+ + __dynamic_len[__dynamic_len_idx + 1]; \
+ lib_ring_buffer_align_ctx(&__ctx, lttng_alignof(u32)); \
+ __chan->ops->event_write(&__ctx, &__tmpl, sizeof(u32)); \
+ } \
+ goto __end_field_##dest##_1; \
+__assign_##dest##_2: \
+ lib_ring_buffer_align_ctx(&__ctx, lttng_alignof(__typemap.dest)); \
+ __chan->ops->write_ops(&__ctx, src1, \
+ sizeof(__typemap.dest) * __get_dynamic_array_len(dest));\
+ goto __end_field_##dest##_2; \
+__assign_##dest##_3: \
+ __chan->ops->write_ops(&__ctx, src2, \
+ sizeof(__typemap.dest) * __get_dynamic_array_len(dest));\
+ goto __end_field_##dest##_3;
+
#undef tp_memcpy_dyn
#define tp_memcpy_dyn(dest, src) \
tp_memcpy_dyn_gen(event_write, dest, src)
+#undef tp_memcpy_dyn_2
+#define tp_memcpy_dyn_2(dest, src1, src2) \
+ tp_memcpy_dyn_gen_2(event_write, dest, src1, src2)
+
#undef tp_memcpy_dyn_from_user
#define tp_memcpy_dyn_from_user(dest, src) \
tp_memcpy_dyn_gen(event_write_from_user, dest, src)
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [RFC PATCH lttng-tools] Fix printk instrumentation
[not found] ` <20130704182228.GA332@Krystal>
@ 2013-07-04 19:19 ` Yannick Brosseau
0 siblings, 0 replies; 5+ messages in thread
From: Yannick Brosseau @ 2013-07-04 19:19 UTC (permalink / raw)
To: Mathieu Desnoyers; +Cc: lttng-dev
On 2013-07-04 14:22, Mathieu Desnoyers wrote:
> * Yannick Brosseau (yannick.brosseau@gmail.com) wrote:
>> 3.9: Works
>> 3.2: Fail to enable with this error msg (will try to get more detailled
>> info or fix)
>> root@debian-wheezy:/home/scientist# lttng enable-event -k console
>> PERROR [6648/6745]: create event ioctl: Invalid argument (in
>> kernel_create_event() at kernel.c:205)
>> Error: Event console: Enable kernel event failed (channel channel0,
>> session auto-20130630-135404)
>> Warning: Some command(s) went wrong
> Hrm, make sure the "condition" trace event actually exist in 3.2 ?
>
Ok, my bad, the printk_console traceport was added only in 3.4, which
works well.
Yannick
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RFC PATCH lttng-tools] Fix printk instrumentation
[not found] ` <51D5B319.6060700@gmail.com>
@ 2013-07-04 18:22 ` Mathieu Desnoyers
[not found] ` <20130704182228.GA332@Krystal>
1 sibling, 0 replies; 5+ messages in thread
From: Mathieu Desnoyers @ 2013-07-04 18:22 UTC (permalink / raw)
To: Yannick Brosseau; +Cc: lttng-dev
* Yannick Brosseau (yannick.brosseau@gmail.com) wrote:
> On 2013-07-04 12:58, Mathieu Desnoyers wrote:
> > This patch fixes the printk instrumentation.
> >
> > - It does not require a 2kB buffer on the stack anymore,
> > - It introduces __dynamic_array_enc_ext_2() and tp_memcpy_dyn_2(), which
> > allow copying into a single variable-length buffer from two sources,
> > useful for kernel < 3.5.
> > - It adds support for 3.10 kernel printk instrumentation.
> >
> > Yannick, can you try this out on 3.4, 3.5, 3.9 and 3.10 kernels ? When
> > you confirm that it works, I'll split it into two patches (one for
> > lttng-events.h and the other for printk.h), merge, and backport this.
> So far:
> 3.10: Works and compile when small typo fixed (see bellow)
ok, fixed.
> 3.9: Works
> 3.2: Fail to enable with this error msg (will try to get more detailled
> info or fix)
> root@debian-wheezy:/home/scientist# lttng enable-event -k console
> PERROR [6648/6745]: create event ioctl: Invalid argument (in
> kernel_create_event() at kernel.c:205)
> Error: Event console: Enable kernel event failed (channel channel0,
> session auto-20130630-135404)
> Warning: Some command(s) went wrong
Hrm, make sure the "condition" trace event actually exist in 3.2 ?
>
>
> > Only built-tested on 3.5 kernel so far.
> >
> > Thanks,
> >
> > Mathieu
> >
> > ---
> > diff --git a/instrumentation/events/lttng-module/printk.h b/instrumentation/events/lttng-module/printk.h
> > index 4c744f9..e22a2cc 100644
> > --- a/instrumentation/events/lttng-module/printk.h
> > +++ b/instrumentation/events/lttng-module/printk.h
> > @@ -7,7 +7,25 @@
> > #include <linux/tracepoint.h>
> > #include <linux/version.h>
> >
> > -#define MSG_TRACE_MAX_LEN 2048
> > +#if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,10,0))
> > +
> > +TRACE_EVENT(console,
> > + TP_PROTO(const char *text, size_t len),
> > +
> > + TP_ARGS(text, len),
> > +
> > + TP_STRUCT__entry(
> > + __dynamic_array_text(char, msg, len)
> > + ),
> > +
> > + TP_fast_assign(
> > + tp_memcpy_dyn(msg, msg)
> >
> second "msg" should be "text"
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RFC PATCH lttng-tools] Fix printk instrumentation
[not found] <20130704165855.GB31483@Krystal>
@ 2013-07-04 17:38 ` Yannick Brosseau
[not found] ` <51D5B319.6060700@gmail.com>
1 sibling, 0 replies; 5+ messages in thread
From: Yannick Brosseau @ 2013-07-04 17:38 UTC (permalink / raw)
To: Mathieu Desnoyers; +Cc: lttng-dev
On 2013-07-04 12:58, Mathieu Desnoyers wrote:
> This patch fixes the printk instrumentation.
>
> - It does not require a 2kB buffer on the stack anymore,
> - It introduces __dynamic_array_enc_ext_2() and tp_memcpy_dyn_2(), which
> allow copying into a single variable-length buffer from two sources,
> useful for kernel < 3.5.
> - It adds support for 3.10 kernel printk instrumentation.
>
> Yannick, can you try this out on 3.4, 3.5, 3.9 and 3.10 kernels ? When
> you confirm that it works, I'll split it into two patches (one for
> lttng-events.h and the other for printk.h), merge, and backport this.
So far:
3.10: Works and compile when small typo fixed (see bellow)
3.9: Works
3.2: Fail to enable with this error msg (will try to get more detailled
info or fix)
root@debian-wheezy:/home/scientist# lttng enable-event -k console
PERROR [6648/6745]: create event ioctl: Invalid argument (in
kernel_create_event() at kernel.c:205)
Error: Event console: Enable kernel event failed (channel channel0,
session auto-20130630-135404)
Warning: Some command(s) went wrong
> Only built-tested on 3.5 kernel so far.
>
> Thanks,
>
> Mathieu
>
> ---
> diff --git a/instrumentation/events/lttng-module/printk.h b/instrumentation/events/lttng-module/printk.h
> index 4c744f9..e22a2cc 100644
> --- a/instrumentation/events/lttng-module/printk.h
> +++ b/instrumentation/events/lttng-module/printk.h
> @@ -7,7 +7,25 @@
> #include <linux/tracepoint.h>
> #include <linux/version.h>
>
> -#define MSG_TRACE_MAX_LEN 2048
> +#if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,10,0))
> +
> +TRACE_EVENT(console,
> + TP_PROTO(const char *text, size_t len),
> +
> + TP_ARGS(text, len),
> +
> + TP_STRUCT__entry(
> + __dynamic_array_text(char, msg, len)
> + ),
> +
> + TP_fast_assign(
> + tp_memcpy_dyn(msg, msg)
>
second "msg" should be "text"
^ permalink raw reply [flat|nested] 5+ messages in thread
* [RFC PATCH lttng-tools] Fix printk instrumentation
@ 2013-07-04 16:57 Mathieu Desnoyers
0 siblings, 0 replies; 5+ messages in thread
From: Mathieu Desnoyers @ 2013-07-04 16:57 UTC (permalink / raw)
To: yannick.brosseau; +Cc: lttng-dev
This patch fixes the printk instrumentation.
- It does not require a 2kB buffer on the stack anymore,
- It introduces __dynamic_array_enc_ext_2() and tp_memcpy_dyn_2(), which
allow copying into a single variable-length buffer from two sources,
useful for kernel < 3.5.
- It adds support for 3.10 kernel printk instrumentation.
Yannick, can you try this out on 3.4, 3.5, 3.9 and 3.10 kernels ? When
you confirm that it works, I'll split it into two patches (one for
lttng-events.h and the other for printk.h), merge, and backport this.
Only built-tested on 3.5 kernel so far.
Thanks,
Mathieu
---
diff --git a/instrumentation/events/lttng-module/printk.h b/instrumentation/events/lttng-module/printk.h
index 4c744f9..e22a2cc 100644
--- a/instrumentation/events/lttng-module/printk.h
+++ b/instrumentation/events/lttng-module/printk.h
@@ -7,7 +7,25 @@
#include <linux/tracepoint.h>
#include <linux/version.h>
-#define MSG_TRACE_MAX_LEN 2048
+#if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,10,0))
+
+TRACE_EVENT(console,
+ TP_PROTO(const char *text, size_t len),
+
+ TP_ARGS(text, len),
+
+ TP_STRUCT__entry(
+ __dynamic_array_text(char, msg, len)
+ ),
+
+ TP_fast_assign(
+ tp_memcpy_dyn(msg, msg)
+ ),
+
+ TP_printk("%s", __get_str(msg))
+)
+
+#elif (LINUX_VERSION_CODE >= KERNEL_VERSION(3,5,0))
TRACE_EVENT_CONDITION(console,
TP_PROTO(const char *log_buf, unsigned start, unsigned end,
@@ -18,46 +36,47 @@ TRACE_EVENT_CONDITION(console,
TP_CONDITION(start != end),
TP_STRUCT__entry(
- __dynamic_array_text(char, msg,
- min_t(unsigned, end - start, MSG_TRACE_MAX_LEN) + 1)
+ __dynamic_array_text(char, msg, end - start)
),
TP_fast_assign(
- tp_memcpy_dyn(msg,
- ({
- char lmsg[MSG_TRACE_MAX_LEN + 1];
-
- if ((end - start) > MSG_TRACE_MAX_LEN)
- start = end - MSG_TRACE_MAX_LEN;
-
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(3,5,0))
- if ((start & (log_buf_len - 1)) >
- (end & (log_buf_len - 1))) {
- memcpy(lmsg,
- log_buf +
- (start & (log_buf_len - 1)),
- log_buf_len -
- (start & (log_buf_len - 1)));
- memcpy(lmsg + log_buf_len -
- (start & (log_buf_len - 1)),
- log_buf,
- end & (log_buf_len - 1));
- } else
- memcpy(lmsg,
- log_buf +
- (start & (log_buf_len - 1)),
- end - start);
-#else
- memcpy(lmsg, log_buf + start, end - start);
-#endif
- lmsg[end - start] = 0;
- lmsg;
- })
- )
+ tp_memcpy_dyn(msg, log_buf + start)
+ ),
+
+ TP_printk("%s", __get_str(msg))
+)
+
+#else /* (LINUX_VERSION_CODE < KERNEL_VERSION(3,5,0)) */
+
+TRACE_EVENT_CONDITION(console,
+ TP_PROTO(const char *log_buf, unsigned start, unsigned end,
+ unsigned log_buf_len),
+
+ TP_ARGS(log_buf, start, end, log_buf_len),
+
+ TP_CONDITION(start != end),
+
+ TP_STRUCT__entry(
+ __dynamic_array_text_2(char, msg,
+ (start & (log_buf_len - 1)) > (end & (log_buf_len - 1))
+ ? log_buf_len - (start & (log_buf_len - 1))
+ : end - start,
+ (start & (log_buf_len - 1)) > (end & (log_buf_len - 1))
+ ? end & (log_buf_len - 1)
+ : 0)
+ ),
+
+ TP_fast_assign(
+ tp_memcpy_dyn_2(msg,
+ log_buf + (start & (log_buf_len - 1)),
+ log_buf)
),
TP_printk("%s", __get_str(msg))
)
+
+#endif
+
#endif /* _TRACE_PRINTK_H */
/* This part must be outside protection */
diff --git a/probes/lttng-events-reset.h b/probes/lttng-events-reset.h
index 44e8ba5..b601e8d 100644
--- a/probes/lttng-events-reset.h
+++ b/probes/lttng-events-reset.h
@@ -29,6 +29,9 @@
#undef __dynamic_array_enc_ext
#define __dynamic_array_enc_ext(_type, _item, _length, _order, _base, _encoding)
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)
+
#undef __dynamic_array_len
#define __dynamic_array_len(_type, _item, _length)
diff --git a/probes/lttng-events.h b/probes/lttng-events.h
index 8a3a886..9606204 100644
--- a/probes/lttng-events.h
+++ b/probes/lttng-events.h
@@ -236,6 +236,10 @@ void trace_##_name(void *__data);
}, \
},
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding) \
+ __dynamic_array_enc_ext(_type, _item, _length1 + _length2, _order, _base, _encoding)
+
#undef __dynamic_array
#define __dynamic_array(_type, _item, _length) \
__dynamic_array_enc_ext(_type, _item, _length, __BYTE_ORDER, 10, none)
@@ -248,6 +252,10 @@ void trace_##_name(void *__data);
#define __dynamic_array_hex(_type, _item, _length) \
__dynamic_array_enc_ext(_type, _item, _length, __BYTE_ORDER, 16, none)
+#undef __dynamic_array_text_2
+#define __dynamic_array_text_2(_type, _item, _length1, _length2) \
+ __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, __BYTE_ORDER, 10, UTF8)
+
#undef __string
#define __string(_item, _src) \
{ \
@@ -407,6 +415,18 @@ static __used struct lttng_probe_desc TP_ID(__probe_desc___, TRACE_SYSTEM) = {
__event_len += sizeof(_type) * __dynamic_len[__dynamic_len_idx]; \
__dynamic_len_idx++;
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ __event_len += lib_ring_buffer_align(__event_len, lttng_alignof(u32)); \
+ __event_len += sizeof(u32); \
+ __event_len += lib_ring_buffer_align(__event_len, lttng_alignof(_type)); \
+ __dynamic_len[__dynamic_len_idx] = (_length1); \
+ __event_len += sizeof(_type) * __dynamic_len[__dynamic_len_idx]; \
+ __dynamic_len_idx++; \
+ __dynamic_len[__dynamic_len_idx] = (_length2); \
+ __event_len += sizeof(_type) * __dynamic_len[__dynamic_len_idx]; \
+ __dynamic_len_idx++;
+
#undef __string
#define __string(_item, _src) \
__event_len += __dynamic_len[__dynamic_len_idx++] = strlen(_src) + 1;
@@ -464,6 +484,10 @@ static inline size_t __event_get_size__##_name(size_t *__dynamic_len, _proto) \
__event_align = max_t(size_t, __event_align, lttng_alignof(u32)); \
__event_align = max_t(size_t, __event_align, lttng_alignof(_type));
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ __dynamic_array_enc_ext(_type, _item, _length1 + _length2, _order, _base, _encoding)
+
#undef __string
#define __string(_item, _src)
@@ -510,6 +534,10 @@ static inline size_t __event_get_align__##_name(_proto) \
#define __dynamic_array_enc_ext(_type, _item, _length, _order, _base, _encoding)\
_type _item;
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ __dynamic_array_enc_ext(_type, _item, _length1 + _length2, _order, _base, _encoding)
+
#undef __string
#define __string(_item, _src) char _item;
@@ -559,6 +587,15 @@ __end_field_##_item##_1: \
goto __assign_##_item##_2; \
__end_field_##_item##_2:
+#undef __dynamic_array_enc_ext_2
+#define __dynamic_array_enc_ext_2(_type, _item, _length1, _length2, _order, _base, _encoding)\
+ goto __assign_##_item##_1; \
+__end_field_##_item##_1: \
+ goto __assign_##_item##_2; \
+__end_field_##_item##_2: \
+ goto __assign_##_item##_3; \
+__end_field_##_item##_3:
+
#undef __string
#define __string(_item, _src) \
goto __assign_##_item; \
@@ -616,10 +653,34 @@ __assign_##dest##_2: \
sizeof(__typemap.dest) * __get_dynamic_array_len(dest));\
goto __end_field_##dest##_2;
+#undef tp_memcpy_dyn_gen_2
+#define tp_memcpy_dyn_gen_2(write_ops, dest, src1, src2) \
+__assign_##dest##_1: \
+ { \
+ u32 __tmpl = __dynamic_len[__dynamic_len_idx] \
+ + __dynamic_len[__dynamic_len_idx + 1]; \
+ lib_ring_buffer_align_ctx(&__ctx, lttng_alignof(u32)); \
+ __chan->ops->event_write(&__ctx, &__tmpl, sizeof(u32)); \
+ } \
+ goto __end_field_##dest##_1; \
+__assign_##dest##_2: \
+ lib_ring_buffer_align_ctx(&__ctx, lttng_alignof(__typemap.dest)); \
+ __chan->ops->write_ops(&__ctx, src1, \
+ sizeof(__typemap.dest) * __get_dynamic_array_len(dest));\
+ goto __end_field_##dest##_2; \
+__assign_##dest##_3: \
+ __chan->ops->write_ops(&__ctx, src2, \
+ sizeof(__typemap.dest) * __get_dynamic_array_len(dest));\
+ goto __end_field_##dest##_3;
+
#undef tp_memcpy_dyn
#define tp_memcpy_dyn(dest, src) \
tp_memcpy_dyn_gen(event_write, dest, src)
+#undef tp_memcpy_dyn_2
+#define tp_memcpy_dyn_2(dest, src1, src2) \
+ tp_memcpy_dyn_gen_2(event_write, dest, src1, src2)
+
#undef tp_memcpy_dyn_from_user
#define tp_memcpy_dyn_from_user(dest, src) \
tp_memcpy_dyn_gen(event_write_from_user, dest, src)
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-07-04 19:19 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-04 16:58 [RFC PATCH lttng-tools] Fix printk instrumentation Mathieu Desnoyers
[not found] <20130704165855.GB31483@Krystal>
2013-07-04 17:38 ` Yannick Brosseau
[not found] ` <51D5B319.6060700@gmail.com>
2013-07-04 18:22 ` Mathieu Desnoyers
[not found] ` <20130704182228.GA332@Krystal>
2013-07-04 19:19 ` Yannick Brosseau
-- strict thread matches above, loose matches on Subject: below --
2013-07-04 16:57 Mathieu Desnoyers
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.