[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH v4 10/11] vl.c: log system invocation when ena
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-trivial] [PATCH v4 10/11] vl.c: log system invocation when enabled |
Date: |
Tue, 4 Aug 2015 14:30:43 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On 2015-08-03 10:14, Alex Bennée wrote:
> This makes it a little easier to remember how you generated that 100Mb
> trace log you saved for a future date.
>
> Signed-off-by: Alex Bennée <address@hidden>
> ---
> configure | 2 +-
> vl.c | 18 ++++++++++++++++++
> 2 files changed, 19 insertions(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index 704b34c..9cc6a48 100755
> --- a/configure
> +++ b/configure
> @@ -1445,7 +1445,7 @@ else
> fi
>
> gcc_flags="-Wold-style-declaration -Wold-style-definition -Wtype-limits"
> -gcc_flags="-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers
> $gcc_flags"
> +gcc_flags="-Wformat-security -Wno-format-y2k -Winit-self
> -Wignored-qualifiers $gcc_flags"
> gcc_flags="-Wmissing-include-dirs -Wempty-body -Wnested-externs $gcc_flags"
> gcc_flags="-Wendif-labels $gcc_flags"
> gcc_flags="-Wno-initializer-overrides $gcc_flags"
> diff --git a/vl.c b/vl.c
> index 05211cf..6f0ae74 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4094,12 +4094,30 @@ int main(int argc, char **argv, char **envp)
>
> if (log_mask) {
> int mask;
> + char fmt_time[512];
> + time_t start_time = time(NULL);
> + struct tm *local_start = localtime(&start_time);
> +
> +
> + if (log_file) {
> + qemu_set_log_filename(log_file);
> + }
> +
> mask = qemu_str_to_log_mask(log_mask);
> if (!mask) {
> qemu_print_log_usage(stdout);
> exit(1);
> }
> qemu_set_log(mask);
> +
> + if (strftime(fmt_time, sizeof(fmt_time), "%c", local_start) > 0) {
Given we don't allow translation in qemu_log, shouldn't we just use a
fixed date/time format? It looks like other parts of QEMU has more or
less standardized to %Y-%m-%d %H:%M:%S, with some changes at the
separator level.
> + qemu_log("System Emulation started at %s\n", fmt_time);
> + qemu_log("Invocation:");
> + for (i = 0; i < argc; i++) {
> + qemu_log("%s ", argv[i]);
> + }
> + qemu_log("\n");
> + }
> }
>
> if (!is_daemonized()) {
Otherwise:
Reviewed-by: Aurelien Jarno <address@hidden>
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net
- Re: [Qemu-trivial] [PATCH v4 09/11] target-arm: dfilter support for in_asm, op, opt_op, (continued)
- [Qemu-trivial] [PATCH v4 07/11] qemu-log: new option -dfilter to limit output, Alex Bennée, 2015/08/03
- [Qemu-trivial] [PATCH v4 06/11] qemu-log: support simple pid substitution in logfile, Alex Bennée, 2015/08/03
- [Qemu-trivial] [PATCH v4 08/11] qemu-log: dfilter-ise exec, out_asm, and op_opt, Alex Bennée, 2015/08/03
- [Qemu-trivial] [PATCH v4 10/11] vl.c: log system invocation when enabled, Alex Bennée, 2015/08/03
[Qemu-trivial] [PATCH v4 03/11] qemu-log: correct help text for -d cpu, Alex Bennée, 2015/08/03