From: Tilman Schmidt <tilman@imap.cc>
To: Greg KH <greg@kroah.com>
Cc: linux-kernel@vger.kernel.org, hjlipp@web.de
Subject: Re: [PATCH] reduce syslog clutter
Date: Fri, 24 Feb 2006 10:29:35 +0100 [thread overview]
Message-ID: <43FED1FF.8050103@imap.cc> (raw)
In-Reply-To: <20060224060505.GA19111@kroah.com>
[-- Attachment #1: Type: text/plain, Size: 1384 bytes --]
Hello Greg,
thanks for your comments.
Greg KH wrote:
> On Fri, Feb 24, 2006 at 12:10:05AM +0100, Tilman Schmidt wrote:
>> The current versions of the err() / info() / warn() syslog macros
>> insert __FILE__ at the beginning of the message, which expands to
>> the complete path name of the source file within the kernel tree.
>
> Note, this is the usb usage, you might want to post this on the
> linux-usb-devel mailing list :)
Ok.
>> With the following patch, when used in a module, they'll insert the
>> module name instead, which is significantly shorter and also tends to
>> be more useful to users trying to make sense of a particular message.
>>
>> The patch also adds macros for the KERN_NOTICE severity level which
>> was so far uncatered for.
>
> Does anyone want to use it?
Yes, me. :-)
In fact, they are being used in the Gigaset drivers which are currently
in the process of being submitted for inclusion in the kernel tree, but
their definitions are in the driver specific header file
drivers/isdn/gigaset/gigaset.h, although they really belong with
their brothers and sisters in usb.h / device.h.
> I suggest splitting this up into two different patches.
Ok.
Regards
Tilman
--
Tilman Schmidt E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 250 bytes --]
next prev parent reply other threads:[~2006-02-24 9:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-23 23:10 [PATCH] reduce syslog clutter Tilman Schmidt
2006-02-24 6:05 ` Greg KH
2006-02-24 9:29 ` Tilman Schmidt [this message]
-- strict thread matches above, loose matches on Subject: below --
2006-02-24 9:50 Tilman Schmidt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=43FED1FF.8050103@imap.cc \
--to=tilman@imap.cc \
--cc=greg@kroah.com \
--cc=hjlipp@web.de \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox