public inbox for linux-next@vger.kernel.org
 help / color / mirror / Atom feed
From: Mike Snitzer <snitzer@redhat.com>
To: Mikulas Patocka <mpatocka@redhat.com>
Cc: Benjamin Block <bblock@linux.ibm.com>,
	Heiko Carstens <heiko.carstens@de.ibm.com>,
	dm-devel@redhat.com, linux-next@vger.kernel.org,
	Steffen Maier <maier@linux.ibm.com>,
	Michael Holzheu <holzheu@linux.ibm.com>
Subject: Re: linux-next: Boot hangs 3 minutes with device mapper on s390
Date: Fri, 1 Mar 2019 15:30:08 -0500	[thread overview]
Message-ID: <20190301203008.GB30080@redhat.com> (raw)
In-Reply-To: <alpine.LRH.2.02.1903011516160.20592@file01.intranet.prod.int.rdu2.redhat.com>

On Fri, Mar 01 2019 at  3:19pm -0500,
Mikulas Patocka <mpatocka@redhat.com> wrote:

> 
> 
> On Fri, 1 Mar 2019, Michael Holzheu wrote:
> 
> > Hi Mike,
> > 
> > On Fedora 29, the following "linux-next" commit introduced a regression on s390:
> > 
> >   commit 1efa3bb79d3de8ca1b7f6770313a1fc0bebe25c7
> >   Author: Mike Snitzer <snitzer@redhat.com>
> >   Date:   Fri Feb 22 11:23:01 2019 -0500
> > 
> >     dm: must allocate dm_noclone for stacked noclone devices
> >     
> >     Otherwise various lvm2 testsuite tests fail because the lower layers of
> >     the stacked noclone device aren't updated to allocate a new 'struct
> >     dm_clone' that reflects the upper layer bio that was issued to it.
> >     
> >     Fixes: 97a89458020b38 ("dm: improve noclone bio support")
> >     Reported-by: Mikulas Patocka <mpatocka@redhat.com>
> >     Signed-off-by: Mike Snitzer <snitzer@redhat.com>
> 
> Should we just drop 97a89458020b388b910160c4f4aa5e24318d2460 and 
> 1efa3bb79d3de8ca1b7f6770313a1fc0bebe25c7?

No.

> 97a89458020b388b910160c4f4aa5e24318d2460 claims to be an improvement, but 
> it broke the tests twice.

This report from Michael needs proper review and triage.

It literally makes _zero_ sense that all 5 mpath devices are suffering
from a 3 minute boot stall (rooted in udev action) when I'm pretty
certain they are using request-based multipath for 4 of the 5 devices.
The 5th likely having a dm-linear ontop of request-based multipath.
And I've tested stacking bio-based linear on request-based multipath.

Mike

      parent reply	other threads:[~2019-03-01 20:30 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20190301183350.5ff697d4@TP-holzheu>
2019-03-01 18:30 ` linux-next: Boot hangs 3 minutes with device mapper on s390 Mike Snitzer
2019-03-04 10:01   ` Steffen Maier
2019-03-04 10:03   ` Michael Holzheu
2019-03-05  4:03     ` Mike Snitzer
2019-03-05  9:46       ` Mikulas Patocka
2019-03-05 13:29         ` Mike Snitzer
2019-03-05 18:02       ` Alexander Duyck
2019-03-05 22:21         ` bio-based DM's "noclone" changes have been dropped from 5.1 [was: Re: linux-next: Boot hangs 3 minutes with device mapper on s390] Mike Snitzer
2019-03-07 10:18           ` Michael Holzheu
     [not found] ` <alpine.LRH.2.02.1903011516160.20592@file01.intranet.prod.int.rdu2.redhat.com>
2019-03-01 20:30   ` Mike Snitzer [this message]

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=20190301203008.GB30080@redhat.com \
    --to=snitzer@redhat.com \
    --cc=bblock@linux.ibm.com \
    --cc=dm-devel@redhat.com \
    --cc=heiko.carstens@de.ibm.com \
    --cc=holzheu@linux.ibm.com \
    --cc=linux-next@vger.kernel.org \
    --cc=maier@linux.ibm.com \
    --cc=mpatocka@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox