lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 10 Oct 2014 14:27:27 +0300 From: Vadim Kochan <vadim4j@...il.com> To: Stephen Hemminger <stephen@...workplumber.org> Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org> Subject: Re: [PATCH iproute2] ip lib: Change timestamp to be shorter and on the same line Hi, I was thinking about new option but can't find out nothing better than "-tshort" matched option. Regards, On Wed, Sep 17, 2014 at 9:06 AM, Vadim Kochan <vadim4j@...il.com> wrote: > Sure, I can add new option, currently timestamp is printed on separated > line so this solution I think can be easier to parse when it will be > on the same line as message, > and easier to detect to which message timestamp belongs. > > On Wed, Sep 17, 2014 at 5:38 AM, Stephen Hemminger > <stephen@...workplumber.org> wrote: >> On Tue, 16 Sep 2014 12:58:06 +0300 >> Vadim Kochan <vadim4j@...il.com> wrote: >> >>> Changed timestamp format to look like more logging info: >>> >>> [Sep 01 20:56:11.853146]2: enp0s25: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default >>> link/ether 3c:97:0e:a3:86:2e brd ff:ff:ff:ff:ff:ff >>> >>> Signed-off-by: Vadim Kochan <vadim4j@...il.com> >>> --- >>> lib/utils.c | 7 +++---- >>> 1 file changed, 3 insertions(+), 4 deletions(-) >>> >>> diff --git a/lib/utils.c b/lib/utils.c >>> index dc21567..03092ba 100644 >>> --- a/lib/utils.c >>> +++ b/lib/utils.c >>> @@ -772,14 +772,13 @@ __u8* hexstring_a2n(const char *str, __u8 *buf, int blen) >>> int print_timestamp(FILE *fp) >>> { >>> struct timeval tv; >>> - char *tstr; >>> + char tstr[40] = {}; >>> >>> memset(&tv, 0, sizeof(tv)); >>> gettimeofday(&tv, NULL); >>> >>> - tstr = asctime(localtime(&tv.tv_sec)); >>> - tstr[strlen(tstr)-1] = 0; >>> - fprintf(fp, "Timestamp: %s %ld usec\n", tstr, (long)tv.tv_usec); >>> + strftime(tstr, sizeof(tstr), "%b %d %H:%M:%S", localtime(&tv.tv_sec)); >>> + fprintf(fp, "[%s.%ld]", tstr, (long)tv.tv_usec); >>> return 0; >>> } >>> >> >> I am loath to change the output format since people write scripts parsing >> output. Maybe add a new flag option to choose format? -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists