From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758076AbYHZLzm (ORCPT ); Tue, 26 Aug 2008 07:55:42 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754099AbYHZLzO (ORCPT ); Tue, 26 Aug 2008 07:55:14 -0400 Received: from ozlabs.org ([203.10.76.45]:45572 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753520AbYHZLzL (ORCPT ); Tue, 26 Aug 2008 07:55:11 -0400 From: Rusty Russell To: schwidefsky@de.ibm.com Subject: Re: [patch 1/3] kmsg: Kernel message catalog macros. Date: Tue, 26 Aug 2008 11:38:26 +1000 User-Agent: KMail/1.9.9 Cc: linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, lf_kernel_messages@lists.linux-foundation.org, Andrew Morton , Michael Holzheu , Gerrit Huizenga , Greg Kroah-Hartman , Randy Dunlap , Jan Kara , Pavel Machek , Sam Ravnborg , Joe Perches , Jochen =?utf-8?q?Vo=C3=9F?= , Kunai Takashi , Tim Bird References: <20080730165656.118280544@de.ibm.com> <200808131433.02966.rusty@rustcorp.com.au> <1219679790.16131.28.camel@localhost> In-Reply-To: <1219679790.16131.28.camel@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200808261138.28076.rusty@rustcorp.com.au> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 26 August 2008 01:56:30 Martin Schwidefsky wrote: > On Wed, 2008-08-13 at 14:33 +1000, Rusty Russell wrote: > > Can you hash the format string to generate the id? 6 hex digits should > > be enough, and your tool can check for clashes. As it's bad form to have > > identical strings for different semantics anyway, this seems to make > > sense. > > If we go with hashes there is one more thing: kmsg(0, ) > The variant where we manually assign the message ids knows about the > "special" id 0. There is no documentation required for id 0 and none is > wanted. If we replace the manual ids with hashes this will get lost. You > could argue that a kmsg with id 0 is a normal printk so why not just use > printk? What is lost is the information that this printk has been found > to be not important enough to be documented. Hmm, #define KERN_IGNORE KERN_DEBUG? Rusty.