linux-block.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stefan Haberland <sth@linux.vnet.ibm.com>
To: Christoph Hellwig <hch@lst.de>,
	Christian Borntraeger <borntraeger@de.ibm.com>
Cc: Jens Axboe <axboe@kernel.dk>,
	Bart Van Assche <Bart.VanAssche@wdc.com>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux-s390 <linux-s390@vger.kernel.org>,
	Martin Schwidefsky <schwidefsky@de.ibm.com>
Subject: Re: 4.14: WARNING: CPU: 4 PID: 2895 at block/blk-mq.c:1144 with virtio-blk (also 4.12 stable)
Date: Mon, 18 Dec 2017 14:56:04 +0100	[thread overview]
Message-ID: <0520e469-563b-486c-9ab8-00d8944ffa9d@linux.vnet.ibm.com> (raw)
In-Reply-To: <20171206232924.GA16584@lst.de>

On 07.12.2017 00:29, Christoph Hellwig wrote:
> On Wed, Dec 06, 2017 at 01:25:11PM +0100, Christian Borntraeger wrote:
> t > commit 11b2025c3326f7096ceb588c3117c7883850c068    -> bad
>>      blk-mq: create a blk_mq_ctx for each possible CPU
>> does not boot on DASD and
>> commit 9c6ae239e01ae9a9f8657f05c55c4372e9fc8bcc    -> good
>>     genirq/affinity: assign vectors to all possible CPUs
>> does boot with DASD disks.
>>
>> Also adding Stefan Haberland if he has an idea why this fails on DASD and adding Martin (for the
>> s390 irq handling code).
> That is interesting as it really isn't related to interrupts at all,
> it just ensures that possible CPUs are set in ->cpumask.
>
> I guess we'd really want:
>
> e005655c389e3d25bf3e43f71611ec12f3012de0
> "blk-mq: only select online CPUs in blk_mq_hctx_next_cpu"
>
> before this commit, but it seems like the whole stack didn't work for
> your either.
>
> I wonder if there is some weird thing about nr_cpu_ids in s390?
> --
> To unsubscribe from this list: send the line "unsubscribe linux-s390" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

I tried this on my system and the blk-mq-hotplug-fix branch does not 
boot for me as well.
The disks get up and running and I/O works fine. At least the partition 
detection and EXT4-fs mount works.

But at some point in time the disk do not get any requests.

I currently have no clue why.
I took a dump and had a look at the disk states and they are fine. No 
error in the logs or in our debug entrys. Just empty DASD devices 
waiting to be called for I/O requests.

Do you have anything I could have a look at?

  parent reply	other threads:[~2017-12-18 13:56 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-17 14:42 4.14: WARNING: CPU: 4 PID: 2895 at block/blk-mq.c:1144 with virtio-blk Christian Borntraeger
2017-11-20 19:20 ` Bart Van Assche
2017-11-20 19:29   ` Christian Borntraeger
2017-11-20 19:42     ` Jens Axboe
2017-11-20 20:49       ` Christian Borntraeger
2017-11-20 20:52         ` Jens Axboe
2017-11-21  8:35           ` Christian Borntraeger
2017-11-21  9:50             ` Christian Borntraeger
2017-11-21 10:14               ` 4.14: WARNING: CPU: 4 PID: 2895 at block/blk-mq.c:1144 with virtio-blk (also 4.12 stable) Christian Borntraeger
2017-11-21 17:27                 ` Jens Axboe
2017-11-21 18:09                   ` Jens Axboe
2017-11-21 18:12                     ` Christian Borntraeger
2017-11-21 18:27                       ` Jens Axboe
2017-11-21 18:39                         ` Jens Axboe
2017-11-21 19:15                           ` Christian Borntraeger
2017-11-21 19:30                             ` Jens Axboe
2017-11-21 20:12                               ` Christian Borntraeger
2017-11-21 20:14                                 ` Jens Axboe
2017-11-21 20:19                                   ` Christian Borntraeger
2017-11-21 20:21                                     ` Jens Axboe
2017-11-21 20:31                                       ` Christian Borntraeger
2017-11-21 20:39                                         ` Jens Axboe
2017-11-22  7:28                                           ` Christoph Hellwig
2017-11-22 14:46                                             ` Jens Axboe
2017-11-23 14:34                                               ` Christoph Hellwig
2017-11-23 14:42                                                 ` Hannes Reinecke
2017-11-23 14:47                                                   ` Christoph Hellwig
2017-11-23 15:05                                                 ` Christian Borntraeger
2017-11-23 18:17                                                 ` Christian Borntraeger
2017-11-23 18:25                                                   ` Christoph Hellwig
2017-11-23 18:28                                                     ` Christian Borntraeger
2017-11-23 18:32                                                       ` Christoph Hellwig
2017-11-23 18:59                                                         ` Christian Borntraeger
2017-11-24 13:09                                                           ` Christian Borntraeger
2017-11-27 15:54                                                             ` Christoph Hellwig
2017-11-29 19:18                                                               ` Christian Borntraeger
2017-11-29 19:36                                                                 ` Christian Borntraeger
2017-12-04 16:21                                                                 ` Christoph Hellwig
2017-12-06 12:25                                                                   ` Christian Borntraeger
2017-12-06 23:29                                                                     ` Christoph Hellwig
2017-12-07  9:20                                                                       ` Christian Borntraeger
2017-12-14 17:32                                                                         ` Christian Borntraeger
2017-12-18 13:56                                                                       ` Stefan Haberland [this message]
2017-12-20 15:47                                                                         ` Christian Borntraeger
2018-01-11  9:13                                                                           ` Ming Lei
2018-01-11  9:26                                                                             ` Stefan Haberland
2018-01-11 11:44                                                                             ` Christian Borntraeger
2018-01-11 13:17                                                                               ` Stefan Haberland
2018-01-11 17:46                                                                             ` Christoph Hellwig
2018-01-12  1:16                                                                               ` Ming Lei
2017-11-23 14:02                       ` Christoph Hellwig
2017-11-23 14:08                         ` Christoph Hellwig

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=0520e469-563b-486c-9ab8-00d8944ffa9d@linux.vnet.ibm.com \
    --to=sth@linux.vnet.ibm.com \
    --cc=Bart.VanAssche@wdc.com \
    --cc=axboe@kernel.dk \
    --cc=borntraeger@de.ibm.com \
    --cc=hch@lst.de \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=schwidefsky@de.ibm.com \
    --cc=tglx@linutronix.de \
    /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;
as well as URLs for NNTP newsgroup(s).