From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "oleksandr@natalenko.name" <oleksandr@natalenko.name>
Cc: "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
"hch@lst.de" <hch@lst.de>,
"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
"martin@lichtvoll.de" <martin@lichtvoll.de>,
"martin.petersen@oracle.com" <martin.petersen@oracle.com>,
"axboe@kernel.dk" <axboe@kernel.dk>,
"ming.lei@redhat.com" <ming.lei@redhat.com>
Subject: Re: [PATCH v11 0/7] block, scsi, md: Improve suspend and resume
Date: Thu, 9 Nov 2017 16:54:32 +0000 [thread overview]
Message-ID: <1510246471.2608.11.camel@wdc.com> (raw)
In-Reply-To: <15255980.N0RxqQ3jgM@natalenko.name>
T24gVGh1LCAyMDE3LTExLTA5IGF0IDA3OjE2ICswMTAwLCBPbGVrc2FuZHIgTmF0YWxlbmtvIHdy
b3RlOg0KPiBpcyB0aGlzIHNvbWV0aGluZyBrbm93biB0byB5b3UsIG9yIGl0IGlzIGp1c3QgbXkg
ZmF1bHQgYXBwbHlpbmcgdGhpcyBzZXJpZXMgdG8gDQo+IHY0LjEzPyBFeGNlcHQgaGF2aW5nIHRo
aXMgd2FybmluZywgc3VzcGVuZC9yZXN1bWUgd29ya3MgZm9yIG1lOg0KPiANCj4gWyAgIDI3LjM4
Mzg0Nl0gc2QgMDowOjA6MDogW3NkYV0gU3RhcnRpbmcgZGlzaw0KPiBbICAgMjcuMzgzOTc2XSBz
ZCAxOjA6MDowOiBbc2RiXSBTdGFydGluZyBkaXNrDQo+IFsgICAyNy40NTEyMThdIHNkYjogQXR0
ZW1wdCB0byBhbGxvY2F0ZSBub24tcHJlZW1wdCByZXF1ZXN0IGluIHByZWVtcHQtb25seSANCj4g
bW9kZS4NCj4gWyAgIDI3LjQ1OTY0MF0gLS0tLS0tLS0tLS0tWyBjdXQgaGVyZSBdLS0tLS0tLS0t
LS0tDQo+IFsgICAyNy40NjQ1MjFdIFdBUk5JTkc6IENQVTogMCBQSUQ6IDE3MiBhdCBibG9jay9i
bGstY29yZS5jOjgyMyBibGtfcXVldWVfZW50ZXIrMHgyMjIvMHgyODANCg0KSGVsbG8gT2xla3Nh
bmRyLA0KDQpUaGFua3MgZm9yIHRoZSB0ZXN0aW5nLiBUaGUgd2FybmluZyB0aGF0IHlvdSByZXBv
cnRlZCBpcyBleHBlY3RlZC4gTWF5YmUgaXQNCnNob3VsZCBiZSBsZWZ0IG91dCB0byBhdm9pZCB0
aGF0IHVzZXJzIGdldCBjb25mdXNlZC4NCg0KSmVucywgdGhpcyBwYXRjaCBzZXJpZXMgc3RpbGwg
YXBwbGllcyBjbGVhbmx5IG9uIHRvcCBvZiB5b3VyIGZvci00LjE1L2Jsb2NrDQpicmFuY2guIEFy
ZSB5b3UgZmluZSB3aXRoIHRoaXMgcGF0Y2ggc2VyaWVzIG9yIGRvIHlvdSBwZXJoYXBzIHdhbnQg
bWUgdG8NCnJlcG9zdCBpdCB3aXRoIE9sZWtzYW5kcidzIFRlc3RlZC1ieSB0YWcgYWRkZWQgdG8g
ZWFjaCBwYXRjaD8NCg0KVGhhbmtzLA0KDQpCYXJ0Lg==
WARNING: multiple messages have this Message-ID (diff)
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "oleksandr@natalenko.name" <oleksandr@natalenko.name>
Cc: "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
"hch@lst.de" <hch@lst.de>,
"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
"martin@lichtvoll.de" <martin@lichtvoll.de>,
"martin.petersen@oracle.com" <martin.petersen@oracle.com>,
"axboe@kernel.dk" <axboe@kernel.dk>,
"ming.lei@redhat.com" <ming.lei@redhat.com>
Subject: Re: [PATCH v11 0/7] block, scsi, md: Improve suspend and resume
Date: Thu, 9 Nov 2017 16:54:32 +0000 [thread overview]
Message-ID: <1510246471.2608.11.camel@wdc.com> (raw)
In-Reply-To: <15255980.N0RxqQ3jgM@natalenko.name>
On Thu, 2017-11-09 at 07:16 +0100, Oleksandr Natalenko wrote:
> is this something known to you, or it is just my fault applying this series to
> v4.13? Except having this warning, suspend/resume works for me:
>
> [ 27.383846] sd 0:0:0:0: [sda] Starting disk
> [ 27.383976] sd 1:0:0:0: [sdb] Starting disk
> [ 27.451218] sdb: Attempt to allocate non-preempt request in preempt-only
> mode.
> [ 27.459640] ------------[ cut here ]------------
> [ 27.464521] WARNING: CPU: 0 PID: 172 at block/blk-core.c:823 blk_queue_enter+0x222/0x280
Hello Oleksandr,
Thanks for the testing. The warning that you reported is expected. Maybe it
should be left out to avoid that users get confused.
Jens, this patch series still applies cleanly on top of your for-4.15/block
branch. Are you fine with this patch series or do you perhaps want me to
repost it with Oleksandr's Tested-by tag added to each patch?
Thanks,
Bart.
next prev parent reply other threads:[~2017-11-09 16:54 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-30 22:41 [PATCH v11 0/7] block, scsi, md: Improve suspend and resume Bart Van Assche
2017-10-30 22:41 ` Bart Van Assche
2017-10-30 22:41 ` [PATCH v11 1/7] block: Make q_usage_counter also track legacy requests Bart Van Assche
2017-10-30 22:41 ` Bart Van Assche
2017-10-30 22:42 ` [PATCH v11 2/7] block: Introduce blk_get_request_flags() Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-10-30 22:42 ` [PATCH v11 3/7] block: Introduce BLK_MQ_REQ_PREEMPT Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-10-30 22:42 ` [PATCH v11 4/7] ide, scsi: Tell the block layer at request allocation time about preempt requests Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-10-30 22:42 ` [PATCH v11 5/7] block: Add the QUEUE_FLAG_PREEMPT_ONLY request queue flag Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-10-30 22:42 ` [PATCH v11 6/7] block, scsi: Make SCSI quiesce and resume work reliably Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-10-30 22:42 ` [PATCH v11 7/7] block, nvme: Introduce blk_mq_req_flags_t Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-10-30 22:42 ` Bart Van Assche
2017-11-09 6:16 ` [PATCH v11 0/7] block, scsi, md: Improve suspend and resume Oleksandr Natalenko
2017-11-09 6:16 ` Oleksandr Natalenko
2017-11-09 16:54 ` Bart Van Assche [this message]
2017-11-09 16:54 ` Bart Van Assche
2017-11-09 16:55 ` Jens Axboe
2017-11-09 17:10 ` Oleksandr Natalenko
2017-11-09 17:10 ` Oleksandr Natalenko
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=1510246471.2608.11.camel@wdc.com \
--to=bart.vanassche@wdc.com \
--cc=axboe@kernel.dk \
--cc=hch@lst.de \
--cc=linux-block@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=martin@lichtvoll.de \
--cc=ming.lei@redhat.com \
--cc=oleksandr@natalenko.name \
/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.