[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPXgP11HzstSBt1PQUgJeGC59yXc=DSYQuruB40LOsm=RZQBZA@mail.gmail.com>
Date: Fri, 26 Jul 2013 14:43:21 +0200
From: Kay Sievers <kay@...y.org>
To: Hidehiro Kawai <hidehiro.kawai.ez@...achi.com>
Cc: linux-kernel@...r.kernel.org, yrl.pp-manager.tt@...achi.com,
akpm@...ux-foundation.org, gregkh@...uxfoundation.org,
davem@...emloft.net
Subject: Re: [RFC PATCH 0/5] Add a hash value for each line in /dev/kmsg
On Wed, Jul 3, 2013 at 3:46 AM, Hidehiro Kawai
<hidehiro.kawai.ez@...achi.com> wrote:
> This patch series adds hash values of printk format strings into
> each line of /dev/kmsg outputs as follows:
>
> 6,154,325061,-,b7db707c@...nel/smp.c:554;Brought up 4 CPUs
/dev/kmsg is to a certain degree a kernel ABI. Having source code
locations in exported log records might cause people / userspace tools
to rely on these strings and expect stability here. The kernel though
cannot make any promises of its source code layout.
The hash is supposed to identify the content of a message, but what if
someone fixes the string? Maybe someone just fixes a one char typo,
the hash will change and the message will not be recognizable any
more.
As much as "automated" hash creation sounds simple; I really think
adding explicit "manually" created random message ids to the bunch of
messages that are interesting is the better option long-term. It
shouldn't be that many messages, most of the printk output is not
really useful for automated inspection or to trigger specific actions.
Messages ideally should not have any direct context to the code that
emits them, they should just identify the message and add a few
structured properties to the message.
This is how userspace identifies log messages and maintains abstract
descriptions of the specific messages to act when they appear:
http://www.freedesktop.org/wiki/Software/systemd/catalog/
Connecting kernel log message texts and source code locations with
message identifiers looks quite dangerous, hard to keep stable if
things are evolving. It might cause serious problems over time.
Thanks,
Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists