Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753560AbbFVRrI (ORCPT ); Mon, 22 Jun 2015 13:47:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43764 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751896AbbFVRqt (ORCPT ); Mon, 22 Jun 2015 13:46:49 -0400 Date: Mon, 22 Jun 2015 13:46:48 -0400 From: Vivek Goyal To: Christoph Hellwig Cc: Mike Snitzer , dm-devel@redhat.com, Andi Kleen , linux-kernel@vger.kernel.org, Tejun Heo Subject: Re: WARNING: at fs/block_dev.c:5 when removing LV on removed device Message-ID: <20150622174648.GA2965@redhat.com> References: <20150618165713.GB19417@two.firstfloor.org> <20150618180408.GD2321@redhat.com> <20150618180804.GC19417@two.firstfloor.org> <20150618181619.GE2321@redhat.com> <20150618190815.GA6390@redhat.com> <20150618192821.GA4027@redhat.com> <20150619064721.GA1183@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150619064721.GA1183@lst.de> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2915 Lines: 81 On Fri, Jun 19, 2015 at 08:47:21AM +0200, Christoph Hellwig wrote: > On Thu, Jun 18, 2015 at 03:28:21PM -0400, Mike Snitzer wrote: > > > WARN_ON_ONCE(write_inode_now(inode, true)) > > > > > > If we failed to write back inode, then warning about it sounds right? > > > > A warning is fine.. not a WARN_ON(). Pretty alarming backtrace spew but > > maybe I'm missing something and DM's blkdev refcount mgmt couldn't > > trigger this WARN_ON()? I fail to see how to avoid it given the device > > isn't thre so write_inode_now() fails. > > > > > What's wrong with that? Should it be just a kernel log of level KERN_WARN > > > instead? > > > > Ideally, but I honestly don't have all the details paged in my head to > > say definitively. First need to answer how vitrio-blk isn't hitting > > this (and DM is). Could it be that __blkdev_put isn't getting called > > for virtio-blk!? > > Just a warnings if fine. In fact we can probably remove that as well > as it will happen after a hot removal all the time. [CC Tejun] Does following patch look good? Thanks Vivek Subject: fs/block_dev.c: Warn on inode writeback failure instead of WARN_ON() If a block device is hot removed and later last reference to devices is put, we try to writeback the dirty inode. But device is gone and that writeback fails. Currently we do a WARN_ON() which does not seem to be the right thing. Convert it to a ratelimited kernel warning. Reported-by: Andi Kleen Signed-off-by: Vivek Goyal --- fs/block_dev.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) Index: rhvgoyal-linux/fs/block_dev.c =================================================================== --- rhvgoyal-linux.orig/fs/block_dev.c 2015-06-18 15:54:52.339383237 -0400 +++ rhvgoyal-linux/fs/block_dev.c 2015-06-22 12:55:47.642504742 -0400 @@ -48,12 +48,17 @@ inline struct block_device *I_BDEV(struc } EXPORT_SYMBOL(I_BDEV); -static void bdev_write_inode(struct inode *inode) +static void bdev_write_inode(struct block_device *bdev) { + struct inode *inode = bdev->bd_inode; + spin_lock(&inode->i_lock); while (inode->i_state & I_DIRTY) { spin_unlock(&inode->i_lock); - WARN_ON_ONCE(write_inode_now(inode, true)); + if (write_inode_now(inode, true)) { + char name[BDEVNAME_SIZE] = ""; + pr_warn_ratelimited("VFS: Dirty inode writeback failed for block device %s.\n", bdevname(bdev, name)); + } spin_lock(&inode->i_lock); } spin_unlock(&inode->i_lock); @@ -1489,7 +1494,7 @@ static void __blkdev_put(struct block_de * ->release can cause the queue to disappear, so flush all * dirty data before. */ - bdev_write_inode(bdev->bd_inode); + bdev_write_inode(bdev); } if (bdev->bd_contains == bdev) { if (disk->fops->release) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/