From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756215AbYHQRdp (ORCPT ); Sun, 17 Aug 2008 13:33:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752908AbYHQRdg (ORCPT ); Sun, 17 Aug 2008 13:33:36 -0400 Received: from mtagate4.de.ibm.com ([195.212.29.153]:48664 "EHLO mtagate4.de.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752609AbYHQRdf (ORCPT ); Sun, 17 Aug 2008 13:33:35 -0400 Subject: Re: [patch 1/3] kmsg: Kernel message catalog macros. From: Martin Schwidefsky Reply-To: schwidefsky@de.ibm.com To: Rusty Russell Cc: Tim Hockin , 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 =?ISO-8859-1?Q?Vo=DF?= , Kunai Takashi , Tim Bird In-Reply-To: <200808171511.09346.rusty@rustcorp.com.au> References: <20080730165656.118280544@de.ibm.com> <1218908985.6037.12.camel@localhost> <200808171511.09346.rusty@rustcorp.com.au> Content-Type: text/plain Organization: IBM Corporation Date: Sun, 17 Aug 2008 19:33:00 +0200 Message-Id: <1218994380.20098.10.camel@localhost> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 2008-08-17 at 15:11 +1000, Rusty Russell wrote: > > > Ok, so a specialized version of printk will do the work to calculate > > > the hash. Only, what will we do if there ever is a conflict? The message > > > tag has to be unique. The shorter the hash is, the more likely a > > > collision gets. Don't know if 6 hash digits is enough to just ignore the > > > problem. > > > > And if you ever need to change the text that is in the format string? > > The hash changes. That seems exactly counter to your goal... > > You need to catalogue them all anyway, so you can detect clashes at build > time. Yes, you have to change the new string in that case, but that's easy. Is the suggestion that in case of a clash just change the message string slightly to avoid the clash ? > And the number changing as the message changes is arguably a feature. You > shouldn't (and we generally don't) idly change kernel messages. This is very true. If a message is touched most likely the message description has to be changed as well. If the message hash then changes the tools can warn about it. The small thing now only is the matter of actually implementing the hash and changing the script to work on them.. -- blue skies, Martin. "Reality continues to ruin my life." - Calvin.