<div dir="ltr"><div><div><div><div><div><div>Thanks David! I've had the impression that LTTng follows kernel coding style but it diverts on the if/else case.<br></div>Personaly I don't buy the {} is better argument, but I'll try to remember your preference :)<br>
<br></div>If you could help me a bit with the config file syntax I'll be happy to add support for it.<br></div>What would help me is a working example with all the required values. As I said<br></div>an empty config file did not work for me, so there seems to be some required options and I was not able to<br>
</div>figure them out.<br><br></div>Cheers,<br>Jan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-05-14 22:47 GMT+02:00 David Goulet <span dir="ltr"><<a href="mailto:dgoulet@efficios.com" target="_blank">dgoulet@efficios.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Merged!<br>
<br>
I merged it with some minor syntax fixes mostly in modprobe.c. There is<br>
one thing I don't accept at all is if/else without {}. Remember the goto<br>
fail; bug :).<br>
<br>
Anyhow, I fixed them to fit my OCD if/else issue. ;)<br>
<br>
Big thanks for that, this is really great! I would love to improve that<br>
with support in the "lttng" command line that could take a list and<br>
spawn the sessiond with it. Also, the configuration file coming in 2.5<br>
(currently in master).<br>
<br>
Cheers!<br>
<span class="HOEnZb"><font color="#888888">David<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 14 May (16:26:28), Jan Glauber wrote:<br>
> Hi David,<br>
><br>
> here are the rebased patches (onto 88564da). There are no functional<br>
> changes to the previous version.<br>
><br>
> Jan Glauber (4):<br>
>   Use common functions for module loading and unloading<br>
>   Seperate probe modules from control modules<br>
>   Remove required parameter from module parameters<br>
>   Configurable kernel module probes support<br>
><br>
>  doc/man/lttng-sessiond.8              |    7 +<br>
>  src/bin/lttng-sessiond/kern-modules.h |    3 +-<br>
>  src/bin/lttng-sessiond/main.c         |    5 +<br>
>  src/bin/lttng-sessiond/modprobe.c     |  307 ++++++++++++++++++---------------<br>
>  src/bin/lttng-sessiond/modprobe.h     |    2 +<br>
>  src/common/defaults.h                 |    3 +<br>
>  src/common/utils.c                    |   10 ++<br>
>  src/common/utils.h                    |    1 +<br>
>  8 files changed, 200 insertions(+), 138 deletions(-)<br>
><br>
> --<br>
> 1.7.9.5<br>
><br>
</div></div></blockquote></div><br></div>