From: Greg KH <gregkh@linuxfoundation.org>
To: Jens Axboe <axboe@fb.com>
Cc: osandov@fb.com, tj@kernel.org, tom.leiming@gmail.com,
stable@vger.kernel.org, stable-commits@vger.kernel.org
Subject: Re: Patch "loop: fix LO_FLAGS_PARTSCAN hang" has been added to the 4.10-stable tree
Date: Wed, 8 Mar 2017 16:55:23 +0100 [thread overview]
Message-ID: <20170308155523.GA28388@kroah.com> (raw)
In-Reply-To: <39818fd9-a870-cc63-6ba0-0b9f197354eb@fb.com>
On Wed, Mar 08, 2017 at 08:47:39AM -0700, Jens Axboe wrote:
> On 03/08/2017 08:39 AM, gregkh@linuxfoundation.org wrote:
> >
> > This is a note to let you know that I've just added the patch titled
> >
> > loop: fix LO_FLAGS_PARTSCAN hang
> >
> > to the 4.10-stable tree which can be found at:
> > http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> >
> > The filename of the patch is:
> > loop-fix-lo_flags_partscan-hang.patch
> > and it can be found in the queue-4.10 subdirectory.
> >
> > If you, or anyone else, feels it should not be added to the stable tree,
> > please let <stable@vger.kernel.org> know about it.
> >
> >
> > From e02898b423802b1f3a3aaa7f16e896da069ba8f7 Mon Sep 17 00:00:00 2001
> > From: Omar Sandoval <osandov@fb.com>
> > Date: Wed, 1 Mar 2017 10:42:38 -0800
> > Subject: loop: fix LO_FLAGS_PARTSCAN hang
> >
> > From: Omar Sandoval <osandov@fb.com>
> >
> > commit e02898b423802b1f3a3aaa7f16e896da069ba8f7 upstream.
> >
> > loop_reread_partitions() needs to do I/O, but we just froze the queue,
> > so we end up waiting forever. This can easily be reproduced with losetup
> > -P. Fix it by moving the reread to after we unfreeze the queue.
> >
> > Fixes: ecdd09597a57 ("block/loop: fix race between I/O and set_status")
> > Reported-by: Tejun Heo <tj@kernel.org>
> > Signed-off-by: Omar Sandoval <osandov@fb.com>
> > Reviewed-by: Ming Lei <tom.leiming@gmail.com>
> > Signed-off-by: Jens Axboe <axboe@fb.com>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> Why is this being added to a bunch of trees that don't have the commit
> that it fixes? It should not go into any of these.
Because the commit this fixes just got added to all of those trees. Is
that not the correct thing needed here?
thanks,
greg k-h
next prev parent reply other threads:[~2017-03-08 16:08 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-08 15:39 Patch "loop: fix LO_FLAGS_PARTSCAN hang" has been added to the 4.10-stable tree gregkh
2017-03-08 15:47 ` Jens Axboe
2017-03-08 15:52 ` Omar Sandoval
2017-03-08 15:55 ` Jens Axboe
2017-03-08 15:55 ` Greg KH [this message]
2017-03-08 15:58 ` Jens Axboe
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=20170308155523.GA28388@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=axboe@fb.com \
--cc=osandov@fb.com \
--cc=stable-commits@vger.kernel.org \
--cc=stable@vger.kernel.org \
--cc=tj@kernel.org \
--cc=tom.leiming@gmail.com \
/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.