From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] nilfs2: rework error message subsystem Date: Tue, 3 Dec 2013 16:29:44 -0800 Message-ID: <20131203162944.8f9ef3a08a3292a1959dd025@linux-foundation.org> References: <1385993333.3945.3.camel@slavad-ubuntu> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Ryusuke Konishi , Michael Conrad , linux-nilfs@vger.kernel.org, Linux FS Devel To: slava@dubeyko.com Return-path: Received: from mail.linuxfoundation.org ([140.211.169.12]:55473 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751206Ab3LDA3q (ORCPT ); Tue, 3 Dec 2013 19:29:46 -0500 In-Reply-To: <1385993333.3945.3.camel@slavad-ubuntu> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Mon, 02 Dec 2013 18:08:53 +0400 Vyacheslav Dubeyko wrote: > There are situations when real error can emit huge amount of > messages. As a result, system log contains many identical > error messages. This patch changes printk on > pr_[err|warn]_ratelimited version in proper places. Also > printk was changed on pr_[err|warning|info] version in > other places. It converts every printk in nilfs2 into pr_foo_ratelimited (and bloats nilfs2.ko by 5k in the process). Isn't this rather overkill?