* Re: [PATCH] Fix sysrq emergency thaw
[not found] <5823da614460b8c40809e6e0a6de172a9c4f4611.1468015143.git.cggong@fb.com>
@ 2016-07-19 12:15 ` Chris Mason
0 siblings, 0 replies; 2+ messages in thread
From: Chris Mason @ 2016-07-19 12:15 UTC (permalink / raw)
To: Charles Gong, Alexander Viro
Cc: linux-fsdevel, linux-kernel, kernel-team, Christoph Hellwig
On 07/15/2016 08:38 PM, Charles Gong wrote:
> "SYSRQ + J" triggers a call to emergency_thaw_all(). Currently, this is an
> infinite loop. Once we trigger it, we'll need to do a hard power-cycle. There
> are users reporting this bug from 2012 to 2016, for example, at
> https://bugzilla.kernel.org/show_bug.cgi?id=47741.
>
> This happens because thaw_bdev() fails to return -EINVAL in the non-frozen case,
> and I fixed it so that do_one_thaw() can recognize this case and quit from
> looping. I checked that none of the other thaw_bdev() callers check the return
> value.
Hi Charles,
Can you please resend this without the hunks that change only
whitespace? Looks like your editor was properly killing off trailing
whitespace, but that's not related to your fix.
Also, please reference the commit where the regression was introduced:
commit 4504230a71566785a05d3e6b53fa1ee071b864eb
Author: Christoph Hellwig <hch@lst.de>
Date: Mon Aug 3 23:28:35 2009 +0200
freeze_bdev: grab active reference to frozen superblocks
Then you can add:
Reviewed-by: Chris Mason <clm@fb.com>
Thanks!
-chris
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix sysrq emergency thaw
2016-09-02 22:29 [RESEND PATCH] " Charles Gong
@ 2016-08-05 20:14 ` Pavel Machek
0 siblings, 0 replies; 2+ messages in thread
From: Pavel Machek @ 2016-08-05 20:14 UTC (permalink / raw)
To: Charles Gong
Cc: Alexander Viro, linux-fsdevel, linux-kernel, kernel-team,
Christoph Hellwig
On Wed 2016-07-20 16:11:16, Charles Gong wrote:
> "SYSRQ + J" triggers a call to emergency_thaw_all(). Currently, this
> is an infinite loop. Once we trigger it, we'll need to do a hard
> power-cycle. There are users reporting this bug from 2012 to 2016, for
> example, at https://bugzilla.kernel.org/show_bug.cgi?id=47741.
>
> This happens because thaw_bdev() fails to return -EINVAL in the
> non-frozen case, so fix it so that do_one_thaw() can recognize this case
> and quit from looping. I checked that none of the other thaw_bdev()
> callers check the return value.
>
> The regression was introduced in commit 4504230a7156 ("freeze_bdev: grab
> active reference to frozen superblocks").
>
> Reviewed-by: Chris Mason <clm@fb.com>
> Signed-off-by: Charles Gong <cggong@fb.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
(even magic should be finite...)
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-08-05 20:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <5823da614460b8c40809e6e0a6de172a9c4f4611.1468015143.git.cggong@fb.com>
2016-07-19 12:15 ` [PATCH] Fix sysrq emergency thaw Chris Mason
2016-09-02 22:29 [RESEND PATCH] " Charles Gong
2016-08-05 20:14 ` [PATCH] " Pavel Machek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).