From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751328AbaKEXDG (ORCPT ); Wed, 5 Nov 2014 18:03:06 -0500 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:33599 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751187AbaKEXC6 (ORCPT ); Wed, 5 Nov 2014 18:02:58 -0500 X-Sasl-enc: aqUDmO1sysoUDNM2J121+W4GAerWoLs9FqdQBlk8G6f3 1415228577 Message-ID: <1415228573.2365.24.camel@perseus.themaw.net> Subject: Re: [PATCH 04/13] autofs4 - change printks AUTOFS defined prints From: Ian Kent To: Joe Perches Cc: Andrew Morton , linux-fsdevel , autofs mailing list , Kernel Mailing List Date: Thu, 06 Nov 2014 07:02:53 +0800 In-Reply-To: <1415025215.17743.12.camel@perches.com> References: <20141103081218.6215.85903.stgit@perseus.themaw.net> <20141103081234.6215.82821.stgit@perseus.themaw.net> <1415003136.17743.9.camel@perches.com> <1415006418.6573.6.camel@perseus.themaw.net> <1415025215.17743.12.camel@perches.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-4.fc20) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2014-11-03 at 06:33 -0800, Joe Perches wrote: > > That's fine. I left out the trailing semicolon/space. > The pr_fmt could be something like: > #define pr_fmt(fmt) KBUILD_MODNAME ":%d:%s: " fmt, current->pid, __func__ > or add a "pid:" descriptor prefix if you like too: > #define pr_fmt(fmt) KBUILD_MODNAME ":pid:%d:%s: " fmt, current->pid, __func__ > > > > it's better to use a consistent style for > > > these logging functions ideally with terminating > > > newlines so there isn't a mix of code with > > > and without those newlines. That inconsistency > > > leads to unintended defects. > > > > The idea here was to make the logging consistent throughout. > > Mine too. > > > I have become used of not using the new-line terminator in logging over > > the years and tend to favour that myself. You recommend not doing that > > probably from a kernel wide consistency perspective? Maybe that is > > better ... > > Yes, kernel style consistency is the rationale. > > Over time, people come along and add messages > while not reading the code very closely so using > the kernel style with newlines can help avoid > these trivial defects. I can see how not including the trailing newline in the macros is a good thing and I'll forward a couple more patches to Andrew for this and fix the inconsistencies. But idea of using pr_xxx() and pr_fmt() (actually that's too open to name clashes so it would need to be named something like autofs_pr_fmt() anyway) looks like it results in less readable code so I'd really prefer not to do that. But thanks for pointing out the problems with the logging calls and for you constructive suggestions. Ian