From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-gy0-f177.google.com ([209.85.160.177]) by canuck.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1QuROf-0002XJ-Jn for linux-mtd@lists.infradead.org; Fri, 19 Aug 2011 15:54:14 +0000 Received: by gyh20 with SMTP id 20so2592345gyh.36 for ; Fri, 19 Aug 2011 08:54:10 -0700 (PDT) Subject: Re: [PATCH] ubifs: Do not build debug messages with CONFIG_UBIFS_FS_DEBUG disabled From: Artem Bityutskiy To: Michal Marek Date: Fri, 19 Aug 2011 18:55:56 +0300 In-Reply-To: <1313058586-24239-1-git-send-email-mmarek@suse.cz> References: <1313058586-24239-1-git-send-email-mmarek@suse.cz> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Message-ID: <1313769360.6607.70.camel@sauron> Mime-Version: 1.0 Cc: linux-mtd@lists.infradead.org, adrian.hunter@intel.com, linux-kernel@vger.kernel.org Reply-To: dedekind1@gmail.com List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2011-08-11 at 12:29 +0200, Michal Marek wrote: > With > $ grep -e UBIFS_FS_DEBUG -e DYNAMIC_DEBUG .config > # CONFIG_UBIFS_FS_DEBUG is not set > CONFIG_DYNAMIC_DEBUG=y > > Debug messages are kept in the object files due to the > dynamic_pr_debug() macro, even if they are never going to be printed: > $ make fs/ubifs/super.o > $ strings fs/ubifs/super.o | grep 'compiled on' > compiled on: Aug 11 2011 at 12:21:38 > > Use plain printk to fix this. > > Signed-off-by: Michal Marek Good catch, thanks! Pushed to ubifs-2.6.git, thanks! -- Best Regards, Artem Bityutskiy