From mboxrd@z Thu Jan 1 00:00:00 1970 From: Artem Bityutskiy Subject: [RFC][PATCH 15/16] writeback: clean-up the warning about non-registered bdi Date: Fri, 16 Jul 2010 15:45:11 +0300 Message-ID: <1279284312-2411-16-git-send-email-dedekind1@gmail.com> References: <1279284312-2411-1-git-send-email-dedekind1@gmail.com> Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Jens Axboe Return-path: In-Reply-To: <1279284312-2411-1-git-send-email-dedekind1@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org From: Artem Bityutskiy In '__mark_inode_dirty()', when the corresponding bdi was not properly registered, we print a warning. But the corresponding code is a bit untidy, it used if and 'WARN_ON(1)' and printk. This patch turns it into one multi-line WARN() statement which looks tidier and also uses 'unlikely()' for the condition which might matter a tiny bit for this hot-path function. Signed-off-by: Artem Bityutskiy --- fs/fs-writeback.c | 9 +++------ 1 files changed, 3 insertions(+), 6 deletions(-) diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c index 559092d..83662fb 100644 --- a/fs/fs-writeback.c +++ b/fs/fs-writeback.c @@ -981,12 +981,9 @@ void __mark_inode_dirty(struct inode *inode, int flags) struct bdi_writeback *wb = &inode_to_bdi(inode)->wb; struct backing_dev_info *bdi = wb->bdi; - if (bdi_cap_writeback_dirty(bdi) && - !test_bit(BDI_registered, &bdi->state)) { - WARN_ON(1); - printk(KERN_ERR "bdi-%s not registered\n", - bdi->name); - } + WARN(bdi_cap_writeback_dirty(bdi) && + !test_bit(BDI_registered, &bdi->state), + "bdi-%s not registered\n", bdi->name); inode->dirtied_when = jiffies; list_move(&inode->i_list, &wb->b_dirty); -- 1.7.1.1