From: Vivek Goyal <vgoyal@redhat.com>
To: Tejun Heo <tj@kernel.org>
Cc: Christoph Hellwig <hch@lst.de>, Mike Snitzer <snitzer@redhat.com>,
dm-devel@redhat.com, Andi Kleen <andi@firstfloor.org>,
linux-kernel@vger.kernel.org
Subject: Re: WARNING: at fs/block_dev.c:5 when removing LV on removed device
Date: Mon, 22 Jun 2015 13:55:41 -0400 [thread overview]
Message-ID: <20150622175541.GB2965@redhat.com> (raw)
In-Reply-To: <20150622175255.GB3710@mtj.duckdns.org>
On Mon, Jun 22, 2015 at 01:52:55PM -0400, Tejun Heo wrote:
> Hello,
>
> On Mon, Jun 22, 2015 at 01:46:48PM -0400, Vivek Goyal wrote:
> > 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.
>
> Yeah, looks good to me. Just one nit.
>
> ...
> > + 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));
>
> This wasn't reported before either but maybe we wanna report the errno
> too? Also, don't we usually break the line for parameters?
>
> pr_..("long format string going over 80 col...\n",
> param0, param1, ...);
Hi Tejun,
Will do. I have been doing some go programming in docker recently and this
has been a side effect of that. :-) Will format it correctly.
Thanks
Vivek
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
WARNING: multiple messages have this Message-ID (diff)
From: Vivek Goyal <vgoyal@redhat.com>
To: Tejun Heo <tj@kernel.org>
Cc: Christoph Hellwig <hch@lst.de>, Mike Snitzer <snitzer@redhat.com>,
dm-devel@redhat.com, Andi Kleen <andi@firstfloor.org>,
linux-kernel@vger.kernel.org
Subject: Re: WARNING: at fs/block_dev.c:5 when removing LV on removed device
Date: Mon, 22 Jun 2015 13:55:41 -0400 [thread overview]
Message-ID: <20150622175541.GB2965@redhat.com> (raw)
In-Reply-To: <20150622175255.GB3710@mtj.duckdns.org>
On Mon, Jun 22, 2015 at 01:52:55PM -0400, Tejun Heo wrote:
> Hello,
>
> On Mon, Jun 22, 2015 at 01:46:48PM -0400, Vivek Goyal wrote:
> > 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.
>
> Yeah, looks good to me. Just one nit.
>
> ...
> > + 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));
>
> This wasn't reported before either but maybe we wanna report the errno
> too? Also, don't we usually break the line for parameters?
>
> pr_..("long format string going over 80 col...\n",
> param0, param1, ...);
Hi Tejun,
Will do. I have been doing some go programming in docker recently and this
has been a side effect of that. :-) Will format it correctly.
Thanks
Vivek
next prev parent reply other threads:[~2015-06-22 17:55 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-18 16:57 WARNING: at fs/block_dev.c:5 when removing LV on removed device Andi Kleen
2015-06-18 18:04 ` Mike Snitzer
2015-06-18 18:08 ` Andi Kleen
2015-06-18 18:16 ` Mike Snitzer
2015-06-18 19:08 ` [dm-devel] " Vivek Goyal
2015-06-18 19:28 ` Mike Snitzer
2015-06-19 6:47 ` Christoph Hellwig
2015-06-22 17:46 ` Vivek Goyal
2015-06-22 17:46 ` Vivek Goyal
2015-06-22 17:52 ` Tejun Heo
2015-06-22 17:52 ` Tejun Heo
2015-06-22 17:55 ` Vivek Goyal [this message]
2015-06-22 17:55 ` Vivek Goyal
2015-06-23 10:08 ` Christoph Hellwig
2015-06-18 19:53 ` [dm-devel] " Andi Kleen
2015-06-18 21:01 ` Vivek Goyal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150622175541.GB2965@redhat.com \
--to=vgoyal@redhat.com \
--cc=andi@firstfloor.org \
--cc=dm-devel@redhat.com \
--cc=hch@lst.de \
--cc=linux-kernel@vger.kernel.org \
--cc=snitzer@redhat.com \
--cc=tj@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.