On Mon, May 09 2016 at 11:50am -0400, hei...@redhat.com <hei...@redhat.com> wrote:
> From: Heinz Mauelshagen <hei...@redhat.com> > > Introduced by upstream commit 70d9798b95562abac005d4ba71d28820f9a201eb > > The raid0 personality does not create mddev->thread as oposed to > other personalities leading to its unconditional access in > mddev_suspend() causing an oops. > > Patch checks for mddev->thread in order to keep the > intention of aforementioned commit. > > Signed-off-by: Heinz Mauelshagen <hei...@redhat.com> Seems this should be marked with: Fixes: 70d9798b9556 ("MD: warn for potential deadlock") Cc: sta...@vger.kernel.org # 4.5+ > > > --- > drivers/md/md.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/md/md.c b/drivers/md/md.c > index 194580f..d91920d 100644 > --- a/drivers/md/md.c > +++ b/drivers/md/md.c > @@ -305,7 +305,7 @@ static blk_qc_t md_make_request(struct request_queue *q, > struct bio *bio) > */ > void mddev_suspend(struct mddev *mddev) > { > - WARN_ON_ONCE(current == mddev->thread->tsk); > + WARN_ON_ONCE(mddev->thread && current == mddev->thread->tsk); > if (mddev->suspended++) > return; > synchronize_rcu(); > -- > 2.5.5 > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel -- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel