From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752475AbaKCOdm (ORCPT ); Mon, 3 Nov 2014 09:33:42 -0500 Received: from smtprelay0167.hostedemail.com ([216.40.44.167]:53306 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751584AbaKCOdk (ORCPT ); Mon, 3 Nov 2014 09:33:40 -0500 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,:::::::::,RULES_HIT:41:355:379:541:599:973:982:988:989:1260:1277:1311:1313:1314:1345:1359:1373:1437:1515:1516:1518:1534:1541:1593:1594:1711:1730:1747:1777:1792:2110:2393:2559:2562:2828:3138:3139:3140:3141:3142:3353:3622:3865:3866:3867:3868:3870:3871:3872:3873:3874:4321:5007:6119:6261:6691:10004:10400:10848:11232:11658:11914:12296:12517:12519:12740:13069:13071:13161:13229:13311:13357:14096:14097:21060:21080,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0 X-HE-Tag: back58_1fa7cfebb2247 X-Filterd-Recvd-Size: 2657 Message-ID: <1415025215.17743.12.camel@perches.com> Subject: Re: [PATCH 04/13] autofs4 - change printks AUTOFS defined prints From: Joe Perches To: Ian Kent Cc: Andrew Morton , linux-fsdevel , autofs mailing list , Kernel Mailing List Date: Mon, 03 Nov 2014 06:33:35 -0800 In-Reply-To: <1415006418.6573.6.camel@perseus.themaw.net> 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> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.12.7-0ubuntu1 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 17:20 +0800, Ian Kent wrote: > On Mon, 2014-11-03 at 00:25 -0800, Joe Perches wrote: > > On Mon, 2014-11-03 at 16:12 +0800, Ian Kent wrote: > > > Use the AUTOFS_*() print defines instead of raw printks. Hi again Ian > > It's probably better to simply use > > #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > > or > > #define pr_fmt(fmt) KBUILD_MODNAME ":%d:%s" fmt, current->pid, __func__ > > if you _really_ want pid/func in the output > > I do, yes. 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. cheers, Joe