From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932094Ab3COTBm (ORCPT ); Fri, 15 Mar 2013 15:01:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42280 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753986Ab3COTBk (ORCPT ); Fri, 15 Mar 2013 15:01:40 -0400 Date: Fri, 15 Mar 2013 15:01:36 -0400 From: Jeff Layton To: Joe Perches Cc: Steve French , linux-cifs@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] cifs: Rename cERROR and cFYI to cifs_dbg Message-ID: <20130315150136.7f7a262a@tlielax.poochiereds.net> In-Reply-To: <1363289077.2146.19.camel@joe-AO722> References: <1363128287.23052.21.camel@joe-AO722> <1363174614.2031.9.camel@joe-AO722> <20130313075155.7464a34a@tlielax.poochiereds.net> <1363178421.2031.29.camel@joe-AO722> <1363289077.2146.19.camel@joe-AO722> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 14 Mar 2013 12:24:37 -0700 Joe Perches wrote: > It's not obvious from reading the macro names that these macros > are for debugging. Convert the names to a single more typical > kernel style cifs_dbg macro. > > cERROR(1, ...) -> cifs_dbg(VFS, ...) > cFYI(1, ...) -> cifs_dbg(FYI, ...) > cFYI(DBG2, ...) -> cifs_dbg(NOISY, ...) > > Move the terminating format newline from the macro to the call site. > > Add CONFIG_CIFS_DEBUG function cifs_vfs_err to emit the > "CIFS VFS: " prefix for VFS messages. > > Size is reduced ~ 1% when CONFIG_CIFS_DEBUG is set (default y) > > $ size fs/cifs/cifs.ko* > text data bss dec hex filename > 265245 2525 132 267902 4167e fs/cifs/cifs.ko.new > 268359 2525 132 271016 422a8 fs/cifs/cifs.ko.old > (my apologies -- my MUA has a mind of its own sometimes) This all looks like good stuff. I am a bit concerned about mashing all of these cleanups into the same patch though. > Other miscellaneous changes around these conversions: > > o Miscellaneous typo fixes > o Add terminating \n's to almost all formats and remove them > from the macros to be more kernel style like. A few formats > previously had defective \n's > o Remove unnecessary OOM messages as kmalloc() calls dump_stack > o Coalesce formats to make grep easier, > added missing spaces when coalescing formats > o Use %s, __func__ instead of embedded function name > o Removed unnecessary "cifs: " prefixes > o Convert kzalloc with multiply to kcalloc ^^^ Things like this really ought to be a separate patch, even though it is a trivial change. That's a minor nit though... > o Remove unused cifswarn macro > I think we ought to go ahead and take this for 3.10. I do have some minor concern about having to deal with backports of later patches to kernels that don't have these changes, but hey, that's the price of dealing with old kernels. The sooner Steve merges this into his for-next tree, the better. This bound to give us all sorts of merge conflicts for the 3.10 window, so we want to make sure that people know what to base their work on. Acked-by: Jeff Layton