<div dir="ltr"><div>Le jeu. 16 juil. 2015 à 12:41, Mathieu Desnoyers <<a href="mailto:mathieu.desnoyers@efficios.com">mathieu.desnoyers@efficios.com</a>> a écrit :<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I cannot reproduce this issue with a kernel 4.1.1 and with<br>
lttng-modules master and stable-2.6 HEAD.<br>
<br>
Can you double-check ?<br></blockquote><div><br></div><div>Thanks for this follow-up. I confirm that the bug is reproducible with lttng-modules master on linux 4.1. I opened an issue in the bug tracker for further details. I added a script to reproduce the bug easily. <div><br></div><div><a href="https://bugs.lttng.org/issues/895">https://bugs.lttng.org/issues/895</a><br></div></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Moreover, just changing the TRACE_SYSTEM without changing<br>
the tracepoint namespacing prefix from kvm_ to kvm_x86 in the<br>
whole file is wrong.</blockquote><div><br></div><div>I do not understand. In lttng-ust, there is a TRACEPOINT_PROVIDER string that must be repeated into each TRACEPOINT_EVENT declaration, but this is not the case for the kernel LTTNG_TRACEPOINT_EVENT macro. Can you explain what you mean by "namespace"?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">And if we do this, we would need the _MAP<br>
flavor of the macros to map to the original kernel names.<br></blockquote><div><br></div><div>In fact, event names are not modified by the patch, such that backward compatibility seems preserved AFAIK. Do I miss something?</div><div><br></div><div>Francis<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
----- On Jul 14, 2015, at 2:43 PM, Francis Giraldeau <a href="mailto:francis.giraldeau@gmail.com" target="_blank">francis.giraldeau@gmail.com</a> wrote:<br>
<br>
> From: Cédric Biancheri <<a href="mailto:cedric.biancheri@polymtl.ca" target="_blank">cedric.biancheri@polymtl.ca</a>><br>
><br>
> The lttng-probe-kvm and lttng-probe-kvm-x86 had the same TRACE_SYSTEM string.<br>
> Because of that, both modules can't be loaded at the same time. The second<br>
> module fails to load silently. This patch rename the TRACE_SYSTEM of<br>
> lttng-probe-kvm-x86 to avoid the clash.<br>
><br>
> Signed-off-by: Francis Giraldeau <<a href="mailto:francis.giraldeau@gmail.com" target="_blank">francis.giraldeau@gmail.com</a>><br>
> ---<br>
> instrumentation/events/lttng-module/arch/x86/kvm/trace.h | 2 +-<br>
> 1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/instrumentation/events/lttng-module/arch/x86/kvm/trace.h<br>
> b/instrumentation/events/lttng-module/arch/x86/kvm/trace.h<br>
> index 32c4743..ddbfda7 100644<br>
> --- a/instrumentation/events/lttng-module/arch/x86/kvm/trace.h<br>
> +++ b/instrumentation/events/lttng-module/arch/x86/kvm/trace.h<br>
> @@ -12,7 +12,7 @@<br>
> #include <../arch/x86/kvm/kvm_cache_regs.h><br>
><br>
> #undef TRACE_SYSTEM<br>
> -#define TRACE_SYSTEM kvm<br>
> +#define TRACE_SYSTEM kvm_x86<br>
><br>
> /*<br>
>  * Tracepoint for guest mode entry.<br>
> --<br>
> 2.1.4<br>
><br>
><br>
> _______________________________________________<br>
> lttng-dev mailing list<br>
> <a href="mailto:lttng-dev@lists.lttng.org" target="_blank">lttng-dev@lists.lttng.org</a><br>
> <a href="http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev" rel="noreferrer" target="_blank">http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev</a><br>
<br>
--<br>
Mathieu Desnoyers<br>
EfficiOS Inc.<br>
<a href="http://www.efficios.com" rel="noreferrer" target="_blank">http://www.efficios.com</a><br>
</blockquote></div></div>