Re: [PATCH 1/2] Store fdata with timestamp and pid in name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks!

On Mon, Jun 4, 2012 at 7:22 PM, Jan Friesse <jfriesse@xxxxxxxxxx> wrote:
> This should allow easier handling of various blackbox dumps. Original
> fdata name is now symlink to latest created dump.
>
> Signed-off-by: Jan Friesse <jfriesse@xxxxxxxxxx>
> ---
>  exec/main.c |   28 ++++++++++++++++++++++++----
>  1 files changed, 24 insertions(+), 4 deletions(-)
>
> diff --git a/exec/main.c b/exec/main.c
> index 46fab92..78aa522 100644
> --- a/exec/main.c
> +++ b/exec/main.c
> @@ -188,6 +188,27 @@ void corosync_state_dump (void)
>        }
>  }
>
> +static void corosync_blackbox_write_to_file (void)
> +{
> +       char fname[PATH_MAX];
> +        struct tm cur_time_tm;
> +        time_t cur_time_t;
> +
> +       cur_time_t = time(NULL);
> +       localtime_r(&cur_time_t, &cur_time_tm);
> +
> +       snprintf(fname, PATH_MAX, "%s/fdata-%04u-%02u-%02uT%02u:%02u:%02u-%lld",
> +           LOCALSTATEDIR "/lib/corosync",
> +           cur_time_tm.tm_year + 1900, cur_time_tm.tm_mon, cur_time_tm.tm_mday,
> +           cur_time_tm.tm_hour, cur_time_tm.tm_min, cur_time_tm.tm_sec,
> +           (long long int)getpid());
> +
> +       qb_log_blackbox_write_to_file(fname);
> +
> +       unlink(LOCALSTATEDIR "/lib/corosync/fdata");
> +       symlink(fname, LOCALSTATEDIR "/lib/corosync/fdata");
> +}
> +
>  static void unlink_all_completed (void)
>  {
>        api->timer_delete (corosync_stats_timer_handle);
> @@ -203,7 +224,6 @@ void corosync_shutdown_request (void)
>  static int32_t sig_diag_handler (int num, void *data)
>  {
>        corosync_state_dump ();
> -       qb_log_blackbox_write_to_file(LOCALSTATEDIR "/lib/corosync/fdata");
>        return 0;
>  }
>
> @@ -216,7 +236,7 @@ static int32_t sig_exit_handler (int num, void *data)
>  static void sigsegv_handler (int num)
>  {
>        (void)signal (SIGSEGV, SIG_DFL);
> -       qb_log_blackbox_write_to_file(LOCALSTATEDIR "/lib/corosync/fdata");
> +       corosync_blackbox_write_to_file ();
>        qb_log_fini();
>        raise (SIGSEGV);
>  }
> @@ -224,7 +244,7 @@ static void sigsegv_handler (int num)
>  static void sigabrt_handler (int num)
>  {
>        (void)signal (SIGABRT, SIG_DFL);
> -       qb_log_blackbox_write_to_file(LOCALSTATEDIR "/lib/corosync/fdata");
> +       corosync_blackbox_write_to_file ();
>        qb_log_fini();
>        raise (SIGABRT);
>  }
> @@ -827,7 +847,7 @@ static void fplay_key_change_notify_fn (
>  {
>        if (strcmp(key_name, "runtime.blackbox.dump_flight_data") == 0) {
>                fprintf(stderr,"Writetofile\n");
> -               qb_log_blackbox_write_to_file (LOCALSTATEDIR "/lib/corosync/fdata");
> +               corosync_blackbox_write_to_file ();
>        }
>        if (strcmp(key_name, "runtime.blackbox.dump_state") == 0) {
>                fprintf(stderr,"statefump\n");
> --
> 1.7.1
>
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss
_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss



[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux