* [PATCH lttng-tools] Print consumerd32, 64, , kernel configuration
@ 2018-05-18 21:16 Jonathan Rajotte
0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Rajotte @ 2018-05-18 21:16 UTC (permalink / raw)
To: lttng-dev; +Cc: jgalar
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
---
src/bin/lttng-sessiond/sessiond-config.c | 49 +++++++++++++++---------
1 file changed, 31 insertions(+), 18 deletions(-)
diff --git a/src/bin/lttng-sessiond/sessiond-config.c b/src/bin/lttng-sessiond/sessiond-config.c
index 587f2f82..c0e0fec3 100644
--- a/src/bin/lttng-sessiond/sessiond-config.c
+++ b/src/bin/lttng-sessiond/sessiond-config.c
@@ -501,22 +501,35 @@ void sessiond_config_log(struct sessiond_config *config)
config->agent_tcp_port.begin,
config->agent_tcp_port.end);
}
- DBG_NO_LOC("\tapplication socket timeout: %i", config->app_socket_timeout);
- DBG_NO_LOC("\tno-kernel: %s", config->no_kernel ? "True" : "False");
- DBG_NO_LOC("\tbackground: %s", config->background ? "True" : "False");
- DBG_NO_LOC("\tdaemonize: %s", config->daemonize ? "True" : "False");
- DBG_NO_LOC("\tsignal parent on start: %s", config->sig_parent ? "True" : "False");
- DBG_NO_LOC("\ttracing group name: %s", config->tracing_group_name.value ? : "Unknown");
- DBG_NO_LOC("\tkmod_probe_list: %s", config->kmod_probes_list.value ? : "None");
- DBG_NO_LOC("\tkmod_extra_probe_list: %s", config->kmod_extra_probes_list.value ? : "None");
- DBG_NO_LOC("\trundir: %s", config->rundir.value ? : "Unknown");
- DBG_NO_LOC("\tapplication socket path: %s", config->apps_unix_sock_path.value ? : "Unknown");
- DBG_NO_LOC("\tclient socket path: %s", config->client_unix_sock_path.value ? : "Unknown");
- DBG_NO_LOC("\twait shm path: %s", config->wait_shm_path.value ? : "Unknown");
- DBG_NO_LOC("\thealth socket path: %s", config->health_unix_sock_path.value ? : "Unknown");
- DBG_NO_LOC("\tLTTNG_UST_CLOCK_PLUGIN: %s", config->lttng_ust_clock_plugin.value ? : "None");
- DBG_NO_LOC("\tpid file path: %s", config->pid_file_path.value ? : "Unknown");
- DBG_NO_LOC("\tlock file path: %s", config->lock_file_path.value ? : "Unknown");
- DBG_NO_LOC("\tsession load path: %s", config->load_session_path.value ? : "None");
- DBG_NO_LOC("\tagent port file path: %s", config->agent_port_file_path.value ? : "Unknown");
+ DBG_NO_LOC("\tapplication socket timeout: %i", config->app_socket_timeout);
+ DBG_NO_LOC("\tno-kernel: %s", config->no_kernel ? "True" : "False");
+ DBG_NO_LOC("\tbackground: %s", config->background ? "True" : "False");
+ DBG_NO_LOC("\tdaemonize: %s", config->daemonize ? "True" : "False");
+ DBG_NO_LOC("\tsignal parent on start: %s", config->sig_parent ? "True" : "False");
+ DBG_NO_LOC("\ttracing group name: %s", config->tracing_group_name.value ? : "Unknown");
+ DBG_NO_LOC("\tkmod_probe_list: %s", config->kmod_probes_list.value ? : "None");
+ DBG_NO_LOC("\tkmod_extra_probe_list: %s", config->kmod_extra_probes_list.value ? : "None");
+ DBG_NO_LOC("\trundir: %s", config->rundir.value ? : "Unknown");
+ DBG_NO_LOC("\tapplication socket path: %s", config->apps_unix_sock_path.value ? : "Unknown");
+ DBG_NO_LOC("\tclient socket path: %s", config->client_unix_sock_path.value ? : "Unknown");
+ DBG_NO_LOC("\twait shm path: %s", config->wait_shm_path.value ? : "Unknown");
+ DBG_NO_LOC("\thealth socket path: %s", config->health_unix_sock_path.value ? : "Unknown");
+ DBG_NO_LOC("\tLTTNG_UST_CLOCK_PLUGIN: %s", config->lttng_ust_clock_plugin.value ? : "None");
+ DBG_NO_LOC("\tpid file path: %s", config->pid_file_path.value ? : "Unknown");
+ DBG_NO_LOC("\tlock file path: %s", config->lock_file_path.value ? : "Unknown");
+ DBG_NO_LOC("\tsession load path: %s", config->load_session_path.value ? : "None");
+ DBG_NO_LOC("\tagent port file path: %s", config->agent_port_file_path.value ? : "Unknown");
+ DBG_NO_LOC("\tconsumerd32 path: %s", config->consumerd32_path.value);
+ DBG_NO_LOC("\tconsumerd32 bin_path: %s", config->consumerd32_bin_path.value);
+ DBG_NO_LOC("\tconsumerd32 lib dir: %s", config->consumerd32_lib_dir.value);
+ DBG_NO_LOC("\tconsumerd32 err unix sock path:%s", config->consumerd32_err_unix_sock_path.value);
+ DBG_NO_LOC("\tconsumerd32 cmd unix sock path:%s", config->consumerd32_cmd_unix_sock_path.value);
+ DBG_NO_LOC("\tconsumerd64 path: %s", config->consumerd64_path.value);
+ DBG_NO_LOC("\tconsumerd64 bin path: %s", config->consumerd64_bin_path.value);
+ DBG_NO_LOC("\tconsumerd64 lib dir: %s", config->consumerd64_lib_dir.value);
+ DBG_NO_LOC("\tconsumerd64 err unix sock path:%s", config->consumerd64_err_unix_sock_path.value);
+ DBG_NO_LOC("\tconsumerd64 cmd unix sock path:%s", config->consumerd64_cmd_unix_sock_path.value);
+ DBG_NO_LOC("\tkconsumerd path: %s", config->kconsumerd_path.value);
+ DBG_NO_LOC("\tkconsumerd err unix sock path: %s", config->kconsumerd_err_unix_sock_path.value);
+ DBG_NO_LOC("\tkconsumerd cmd unix sock path: %s", config->kconsumerd_cmd_unix_sock_path.value);
}
--
2.17.0
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH lttng-tools] Print consumerd32, 64, , kernel configuration
[not found] ` <CA+jJMxs6dOD8cmDn2xygB5uUEMQwGn1LmrsTLhq7Wb9jLkKqBg@mail.gmail.com>
@ 2018-05-28 20:09 ` Jonathan Rajotte-Julien
0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Rajotte-Julien @ 2018-05-28 20:09 UTC (permalink / raw)
To: Jérémie Galarneau; +Cc: lttng-dev, Jeremie Galarneau
On Sat, May 26, 2018 at 04:38:38AM -0400, Jérémie Galarneau wrote:
> The title of this patch has an extra comma.
>
> On 18 May 2018 at 17:16, Jonathan Rajotte <
> jonathan.rajotte-julien@efficios.com> wrote:
>
> > Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
> > ---
> > src/bin/lttng-sessiond/sessiond-config.c | 49 +++++++++++++++---------
> > 1 file changed, 31 insertions(+), 18 deletions(-)
> >
> > diff --git a/src/bin/lttng-sessiond/sessiond-config.c
> > b/src/bin/lttng-sessiond/sessiond-config.c
> > index 587f2f82..c0e0fec3 100644
> > --- a/src/bin/lttng-sessiond/sessiond-config.c
> > +++ b/src/bin/lttng-sessiond/sessiond-config.c
> > @@ -501,22 +501,35 @@ void sessiond_config_log(struct sessiond_config
> > *config)
> > config->agent_tcp_port.begin,
> > config->agent_tcp_port.end);
> > }
> > - DBG_NO_LOC("\tapplication socket timeout: %i",
> > config->app_socket_timeout);
> > - DBG_NO_LOC("\tno-kernel: %s", config->no_kernel
> > ? "True" : "False");
> > - DBG_NO_LOC("\tbackground: %s",
> > config->background ? "True" : "False");
> > - DBG_NO_LOC("\tdaemonize: %s", config->daemonize
> > ? "True" : "False");
> > - DBG_NO_LOC("\tsignal parent on start: %s",
> > config->sig_parent ? "True" : "False");
> > - DBG_NO_LOC("\ttracing group name: %s",
> > config->tracing_group_name.value ? : "Unknown");
> > - DBG_NO_LOC("\tkmod_probe_list: %s",
> > config->kmod_probes_list.value ? : "None");
> > - DBG_NO_LOC("\tkmod_extra_probe_list: %s",
> > config->kmod_extra_probes_list.value ? : "None");
> > - DBG_NO_LOC("\trundir: %s",
> > config->rundir.value ? : "Unknown");
> > - DBG_NO_LOC("\tapplication socket path: %s",
> > config->apps_unix_sock_path.value ? : "Unknown");
> > - DBG_NO_LOC("\tclient socket path: %s",
> > config->client_unix_sock_path.value ? : "Unknown");
> > - DBG_NO_LOC("\twait shm path: %s",
> > config->wait_shm_path.value ? : "Unknown");
> > - DBG_NO_LOC("\thealth socket path: %s",
> > config->health_unix_sock_path.value ? : "Unknown");
> > - DBG_NO_LOC("\tLTTNG_UST_CLOCK_PLUGIN: %s",
> > config->lttng_ust_clock_plugin.value ? : "None");
> > - DBG_NO_LOC("\tpid file path: %s",
> > config->pid_file_path.value ? : "Unknown");
> > - DBG_NO_LOC("\tlock file path: %s",
> > config->lock_file_path.value ? : "Unknown");
> > - DBG_NO_LOC("\tsession load path: %s",
> > config->load_session_path.value ? : "None");
> > - DBG_NO_LOC("\tagent port file path: %s",
> > config->agent_port_file_path.value ? : "Unknown");
> > + DBG_NO_LOC("\tapplication socket timeout: %i",
> > config->app_socket_timeout);
> > + DBG_NO_LOC("\tno-kernel: %s",
> > config->no_kernel ? "True" : "False");
> > + DBG_NO_LOC("\tbackground: %s",
> > config->background ? "True" : "False");
> > + DBG_NO_LOC("\tdaemonize: %s",
> > config->daemonize ? "True" : "False");
> > + DBG_NO_LOC("\tsignal parent on start: %s",
> > config->sig_parent ? "True" : "False");
> > + DBG_NO_LOC("\ttracing group name: %s",
> > config->tracing_group_name.value ? : "Unknown");
> > + DBG_NO_LOC("\tkmod_probe_list: %s",
> > config->kmod_probes_list.value ? : "None");
> > + DBG_NO_LOC("\tkmod_extra_probe_list: %s",
> > config->kmod_extra_probes_list.value ? : "None");
> > + DBG_NO_LOC("\trundir: %s",
> > config->rundir.value ? : "Unknown");
> > + DBG_NO_LOC("\tapplication socket path: %s",
> > config->apps_unix_sock_path.value ? : "Unknown");
> > + DBG_NO_LOC("\tclient socket path: %s",
> > config->client_unix_sock_path.value ? : "Unknown");
> > + DBG_NO_LOC("\twait shm path: %s",
> > config->wait_shm_path.value ? : "Unknown");
> > + DBG_NO_LOC("\thealth socket path: %s",
> > config->health_unix_sock_path.value ? : "Unknown");
> > + DBG_NO_LOC("\tLTTNG_UST_CLOCK_PLUGIN: %s",
> > config->lttng_ust_clock_plugin.value ? : "None");
> > + DBG_NO_LOC("\tpid file path: %s",
> > config->pid_file_path.value ? : "Unknown");
> > + DBG_NO_LOC("\tlock file path: %s",
> > config->lock_file_path.value ? : "Unknown");
> > + DBG_NO_LOC("\tsession load path: %s",
> > config->load_session_path.value ? : "None");
> > + DBG_NO_LOC("\tagent port file path: %s",
> > config->agent_port_file_path.value ? : "Unknown");
> > + DBG_NO_LOC("\tconsumerd32 path: %s",
> > config->consumerd32_path.value);
> > + DBG_NO_LOC("\tconsumerd32 bin_path: %s",
> > config->consumerd32_bin_path.value);
> > + DBG_NO_LOC("\tconsumerd32 lib dir: %s",
> > config->consumerd32_lib_dir.value);
> > + DBG_NO_LOC("\tconsumerd32 err unix sock path:%s",
> > config->consumerd32_err_unix_sock_path.value);
> > + DBG_NO_LOC("\tconsumerd32 cmd unix sock path:%s",
> > config->consumerd32_cmd_unix_sock_path.value);
> > + DBG_NO_LOC("\tconsumerd64 path: %s",
> > config->consumerd64_path.value);
> > + DBG_NO_LOC("\tconsumerd64 bin path: %s",
> > config->consumerd64_bin_path.value);
> > + DBG_NO_LOC("\tconsumerd64 lib dir: %s",
> > config->consumerd64_lib_dir.value);
> > + DBG_NO_LOC("\tconsumerd64 err unix sock path:%s",
> > config->consumerd64_err_unix_sock_path.value);
> > + DBG_NO_LOC("\tconsumerd64 cmd unix sock path:%s",
> > config->consumerd64_cmd_unix_sock_path.value);
> > + DBG_NO_LOC("\tkconsumerd path: %s",
> > config->kconsumerd_path.value);
> > + DBG_NO_LOC("\tkconsumerd err unix sock path: %s",
> > config->kconsumerd_err_unix_sock_path.value);
> > + DBG_NO_LOC("\tkconsumerd cmd unix sock path: %s",
> > config->kconsumerd_cmd_unix_sock_path.value);
> >
>
> Those string values can be NULL. Look at how this is handled for the
Good old glibc that simply print "(null)". Anyway, will fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=25609
Good catch.
> existing string options (ternary + place holder). In this case, using
> "Unknown" seems appropriate.
>
> Thanks,
> Jérémie
>
>
> > }
> > --
> > 2.17.0
> >
> >
>
>
> --
> Jérémie Galarneau
> EfficiOS Inc.
> http://www.efficios.com
--
Jonathan Rajotte-Julien
EfficiOS
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH lttng-tools] Print consumerd32, 64, , kernel configuration
[not found] <20180518211640.5168-1-jonathan.rajotte-julien@efficios.com>
@ 2018-05-26 8:38 ` Jérémie Galarneau
[not found] ` <CA+jJMxs6dOD8cmDn2xygB5uUEMQwGn1LmrsTLhq7Wb9jLkKqBg@mail.gmail.com>
1 sibling, 0 replies; 3+ messages in thread
From: Jérémie Galarneau @ 2018-05-26 8:38 UTC (permalink / raw)
To: Jonathan Rajotte; +Cc: lttng-dev, Jeremie Galarneau
[-- Attachment #1.1: Type: text/plain, Size: 6360 bytes --]
The title of this patch has an extra comma.
On 18 May 2018 at 17:16, Jonathan Rajotte <
jonathan.rajotte-julien@efficios.com> wrote:
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
> ---
> src/bin/lttng-sessiond/sessiond-config.c | 49 +++++++++++++++---------
> 1 file changed, 31 insertions(+), 18 deletions(-)
>
> diff --git a/src/bin/lttng-sessiond/sessiond-config.c
> b/src/bin/lttng-sessiond/sessiond-config.c
> index 587f2f82..c0e0fec3 100644
> --- a/src/bin/lttng-sessiond/sessiond-config.c
> +++ b/src/bin/lttng-sessiond/sessiond-config.c
> @@ -501,22 +501,35 @@ void sessiond_config_log(struct sessiond_config
> *config)
> config->agent_tcp_port.begin,
> config->agent_tcp_port.end);
> }
> - DBG_NO_LOC("\tapplication socket timeout: %i",
> config->app_socket_timeout);
> - DBG_NO_LOC("\tno-kernel: %s", config->no_kernel
> ? "True" : "False");
> - DBG_NO_LOC("\tbackground: %s",
> config->background ? "True" : "False");
> - DBG_NO_LOC("\tdaemonize: %s", config->daemonize
> ? "True" : "False");
> - DBG_NO_LOC("\tsignal parent on start: %s",
> config->sig_parent ? "True" : "False");
> - DBG_NO_LOC("\ttracing group name: %s",
> config->tracing_group_name.value ? : "Unknown");
> - DBG_NO_LOC("\tkmod_probe_list: %s",
> config->kmod_probes_list.value ? : "None");
> - DBG_NO_LOC("\tkmod_extra_probe_list: %s",
> config->kmod_extra_probes_list.value ? : "None");
> - DBG_NO_LOC("\trundir: %s",
> config->rundir.value ? : "Unknown");
> - DBG_NO_LOC("\tapplication socket path: %s",
> config->apps_unix_sock_path.value ? : "Unknown");
> - DBG_NO_LOC("\tclient socket path: %s",
> config->client_unix_sock_path.value ? : "Unknown");
> - DBG_NO_LOC("\twait shm path: %s",
> config->wait_shm_path.value ? : "Unknown");
> - DBG_NO_LOC("\thealth socket path: %s",
> config->health_unix_sock_path.value ? : "Unknown");
> - DBG_NO_LOC("\tLTTNG_UST_CLOCK_PLUGIN: %s",
> config->lttng_ust_clock_plugin.value ? : "None");
> - DBG_NO_LOC("\tpid file path: %s",
> config->pid_file_path.value ? : "Unknown");
> - DBG_NO_LOC("\tlock file path: %s",
> config->lock_file_path.value ? : "Unknown");
> - DBG_NO_LOC("\tsession load path: %s",
> config->load_session_path.value ? : "None");
> - DBG_NO_LOC("\tagent port file path: %s",
> config->agent_port_file_path.value ? : "Unknown");
> + DBG_NO_LOC("\tapplication socket timeout: %i",
> config->app_socket_timeout);
> + DBG_NO_LOC("\tno-kernel: %s",
> config->no_kernel ? "True" : "False");
> + DBG_NO_LOC("\tbackground: %s",
> config->background ? "True" : "False");
> + DBG_NO_LOC("\tdaemonize: %s",
> config->daemonize ? "True" : "False");
> + DBG_NO_LOC("\tsignal parent on start: %s",
> config->sig_parent ? "True" : "False");
> + DBG_NO_LOC("\ttracing group name: %s",
> config->tracing_group_name.value ? : "Unknown");
> + DBG_NO_LOC("\tkmod_probe_list: %s",
> config->kmod_probes_list.value ? : "None");
> + DBG_NO_LOC("\tkmod_extra_probe_list: %s",
> config->kmod_extra_probes_list.value ? : "None");
> + DBG_NO_LOC("\trundir: %s",
> config->rundir.value ? : "Unknown");
> + DBG_NO_LOC("\tapplication socket path: %s",
> config->apps_unix_sock_path.value ? : "Unknown");
> + DBG_NO_LOC("\tclient socket path: %s",
> config->client_unix_sock_path.value ? : "Unknown");
> + DBG_NO_LOC("\twait shm path: %s",
> config->wait_shm_path.value ? : "Unknown");
> + DBG_NO_LOC("\thealth socket path: %s",
> config->health_unix_sock_path.value ? : "Unknown");
> + DBG_NO_LOC("\tLTTNG_UST_CLOCK_PLUGIN: %s",
> config->lttng_ust_clock_plugin.value ? : "None");
> + DBG_NO_LOC("\tpid file path: %s",
> config->pid_file_path.value ? : "Unknown");
> + DBG_NO_LOC("\tlock file path: %s",
> config->lock_file_path.value ? : "Unknown");
> + DBG_NO_LOC("\tsession load path: %s",
> config->load_session_path.value ? : "None");
> + DBG_NO_LOC("\tagent port file path: %s",
> config->agent_port_file_path.value ? : "Unknown");
> + DBG_NO_LOC("\tconsumerd32 path: %s",
> config->consumerd32_path.value);
> + DBG_NO_LOC("\tconsumerd32 bin_path: %s",
> config->consumerd32_bin_path.value);
> + DBG_NO_LOC("\tconsumerd32 lib dir: %s",
> config->consumerd32_lib_dir.value);
> + DBG_NO_LOC("\tconsumerd32 err unix sock path:%s",
> config->consumerd32_err_unix_sock_path.value);
> + DBG_NO_LOC("\tconsumerd32 cmd unix sock path:%s",
> config->consumerd32_cmd_unix_sock_path.value);
> + DBG_NO_LOC("\tconsumerd64 path: %s",
> config->consumerd64_path.value);
> + DBG_NO_LOC("\tconsumerd64 bin path: %s",
> config->consumerd64_bin_path.value);
> + DBG_NO_LOC("\tconsumerd64 lib dir: %s",
> config->consumerd64_lib_dir.value);
> + DBG_NO_LOC("\tconsumerd64 err unix sock path:%s",
> config->consumerd64_err_unix_sock_path.value);
> + DBG_NO_LOC("\tconsumerd64 cmd unix sock path:%s",
> config->consumerd64_cmd_unix_sock_path.value);
> + DBG_NO_LOC("\tkconsumerd path: %s",
> config->kconsumerd_path.value);
> + DBG_NO_LOC("\tkconsumerd err unix sock path: %s",
> config->kconsumerd_err_unix_sock_path.value);
> + DBG_NO_LOC("\tkconsumerd cmd unix sock path: %s",
> config->kconsumerd_cmd_unix_sock_path.value);
>
Those string values can be NULL. Look at how this is handled for the
existing string options (ternary + place holder). In this case, using
"Unknown" seems appropriate.
Thanks,
Jérémie
> }
> --
> 2.17.0
>
>
--
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com
[-- Attachment #1.2: Type: text/html, Size: 8893 bytes --]
[-- Attachment #2: Type: text/plain, Size: 156 bytes --]
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-05-28 20:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-18 21:16 [PATCH lttng-tools] Print consumerd32, 64, , kernel configuration Jonathan Rajotte
[not found] <20180518211640.5168-1-jonathan.rajotte-julien@efficios.com>
2018-05-26 8:38 ` Jérémie Galarneau
[not found] ` <CA+jJMxs6dOD8cmDn2xygB5uUEMQwGn1LmrsTLhq7Wb9jLkKqBg@mail.gmail.com>
2018-05-28 20:09 ` Jonathan Rajotte-Julien
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.