All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "snitzer@redhat.com" <snitzer@redhat.com>,
	"mcgrof@kernel.org" <mcgrof@kernel.org>
Cc: "mwilck@suse.com" <mwilck@suse.com>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"agk@redhat.com" <agk@redhat.com>,
	Damien Le Moal <Damien.LeMoal@wdc.com>,
	"hare@suse.de" <hare@suse.de>,
	"axboe@kernel.dk" <axboe@kernel.dk>,
	"yuchao0@huawei.com" <yuchao0@huawei.com>,
	"ghe@suse.com" <ghe@suse.com>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	"jaegeuk@kernel.org" <jaegeuk@kernel.org>,
	"tchvatal@suse.com" <tchvatal@suse.com>,
	"zren@suse.com" <zren@suse.com>
Subject: Re: dm-zoned-tools: add zoned disk udev rules for scheduler / dmsetup
Date: Thu, 14 Jun 2018 16:23:39 +0000	[thread overview]
Message-ID: <4bca9936ccc0cc67404e8da8f7a3c5948ba7e518.camel@wdc.com> (raw)
In-Reply-To: <20180614123806.GA32162@redhat.com>

T24gVGh1LCAyMDE4LTA2LTE0IGF0IDA4OjM4IC0wNDAwLCBNaWtlIFNuaXR6ZXIgd3JvdGU6DQo+
IE9uIFdlZCwgSnVuIDEzIDIwMTggYXQgIDg6MTFwbSAtMDQwMCwNCj4gTHVpcyBSLiBSb2RyaWd1
ZXogPG1jZ3JvZkBrZXJuZWwub3JnPiB3cm90ZToNCj4gPiAxKSBzY2hlZHVsZXIgc2V0dXANCj4g
DQo+IFRoaXMgaXMgd3JvbmcuLiBpZiB6b25lZCBkZXZpY2VzIGFyZSBzbyBkZXBlbmRlbnQgb24g
ZGVhZGxpbmUgb3INCj4gbXEtZGVhZGxpbmUgdGhlbiB0aGUga2VybmVsIHNob3VsZCBhbGxvdyB0
aGVtIHRvIGJlIGhhcmRjb2RlZC4gIEkga25vdw0KPiBKZW5zIHJlbW92ZWQgdGhlIEFQSSB0byBk
byBzbyBidXQgdGhlIGZhY3QgdGhhdCBkcml2ZXJzIG5lZWQgdG8gcmVseSBvbg0KPiBoYWNrcyBs
aWtlIHRoaXMgdWRldiBydWxlIHRvIGdldCBhIGZ1bmN0aW9uYWwgZGV2aWNlIGlzIHByb29mIHdl
IG5lZWQgdG8NCj4gYWxsb3cgZHJpdmVycyB0byBpbXBvc2UgdGhlIHNjaGVkdWxlciB1c2VkLg0K
DQpIZWxsbyBNaWtlLA0KDQpBcyB5b3Uga25vdyB0aGUgTGludXgga2VybmVsIGJsb2NrIGxheWVy
IHN0YWNrIGNhbiByZW9yZGVyIHJlcXVlc3RzLiBIb3dldmVyLA0KZm9yIHpvbmVkIGJsb2NrIGRl
dmljZXMgaXQgaXMgZXNzZW50aWFsIHRoYXQgdGhlIGJsb2NrIGRldmljZSByZWNlaXZlcyB3cml0
ZQ0KcmVxdWVzdHMgaW4gdGhlIHNhbWUgb3JkZXIgYXMgdGhlc2Ugd2VyZSBzdWJtaXR0ZWQgYnkg
dGhlICh1c2VyIHNwYWNlKQ0KYXBwbGljYXRpb24gb3IgYnkgdGhlIChrZXJuZWwpIGZpbGVzeXN0
ZW0uIEFmdGVyIGEgbG9uZyBkZWJhdGUgdGhlIGNob2ljZSB3YXMNCm1hZGUgdG8gbWFrZSBJL08g
c2NoZWR1bGVycyByZXNwb25zaWJsZSBmb3IgZ3VhcmFudGVlaW5nIHRvIHByZXNlcnZlIHRoZSB3
cml0ZQ0Kb3JkZXIuIFRvZGF5IG9ubHkgdGhlIGRlYWRsaW5lIHNjaGVkdWxlciBndWFyYW50ZWVz
IHRoYXQgdGhlIHdyaXRlIG9yZGVyIGlzDQpwcmVzZXJ2ZWQuIEhlbmNlIHRoZSB1ZGV2IHJ1bGUg
dGhhdCBzZXRzIHRoZSBkZWFkbGluZSBzY2hlZHVsZXIgZm9yIHpvbmVkDQpibG9jayBkZXZpY2Vz
Lg0KDQpCYXJ0Lg0KDQoNCg0KDQo=

WARNING: multiple messages have this Message-ID (diff)
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "snitzer@redhat.com" <snitzer@redhat.com>,
	"mcgrof@kernel.org" <mcgrof@kernel.org>
Cc: "mwilck@suse.com" <mwilck@suse.com>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"agk@redhat.com" <agk@redhat.com>,
	Damien Le Moal <Damien.LeMoal@wdc.com>,
	"hare@suse.de" <hare@suse.de>,
	"axboe@kernel.dk" <axboe@kernel.dk>,
	"yuchao0@huawei.com" <yuchao0@huawei.com>,
	"ghe@suse.com" <ghe@suse.com>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	"jaegeuk@kernel.org" <jaegeuk@kernel.org>,
	"tchvatal@suse.com" <tchvatal@suse.com>,
	"zren@suse.com" <zren@suse.com>
Subject: Re: dm-zoned-tools: add zoned disk udev rules for scheduler / dmsetup
Date: Thu, 14 Jun 2018 16:23:39 +0000	[thread overview]
Message-ID: <4bca9936ccc0cc67404e8da8f7a3c5948ba7e518.camel@wdc.com> (raw)
In-Reply-To: <20180614123806.GA32162@redhat.com>

On Thu, 2018-06-14 at 08:38 -0400, Mike Snitzer wrote:
> On Wed, Jun 13 2018 at  8:11pm -0400,
> Luis R. Rodriguez <mcgrof@kernel.org> wrote:
> > 1) scheduler setup
> 
> This is wrong.. if zoned devices are so dependent on deadline or
> mq-deadline then the kernel should allow them to be hardcoded.  I know
> Jens removed the API to do so but the fact that drivers need to rely on
> hacks like this udev rule to get a functional device is proof we need to
> allow drivers to impose the scheduler used.

Hello Mike,

As you know the Linux kernel block layer stack can reorder requests. However,
for zoned block devices it is essential that the block device receives write
requests in the same order as these were submitted by the (user space)
application or by the (kernel) filesystem. After a long debate the choice was
made to make I/O schedulers responsible for guaranteeing to preserve the write
order. Today only the deadline scheduler guarantees that the write order is
preserved. Hence the udev rule that sets the deadline scheduler for zoned
block devices.

Bart.





  reply	other threads:[~2018-06-14 16:23 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-14  0:11 [PATCH] dm-zoned-tools: add zoned disk udev rules for scheduler / dmsetup Luis R. Rodriguez
2018-06-14 10:01 ` Damien Le Moal
2018-06-14 10:01   ` Damien Le Moal
2018-06-14 13:39   ` Bart Van Assche
2018-06-14 13:39     ` Bart Van Assche
2018-06-14 13:42     ` Christoph Hellwig
2018-06-15 11:07       ` Martin Wilck
2018-06-15 11:07         ` Martin Wilck
2018-06-14 12:38 ` Mike Snitzer
2018-06-14 16:23   ` Bart Van Assche [this message]
2018-06-14 16:23     ` Bart Van Assche
2018-06-14 17:37   ` Luis R. Rodriguez
2018-06-14 17:46     ` Luis R. Rodriguez
2018-06-14 17:58     ` Mike Snitzer
2018-06-15  9:59       ` Damien Le Moal
2018-06-15  9:59         ` Damien Le Moal
2018-06-15 14:50         ` Mike Snitzer
2018-06-15  9:00   ` Damien Le Moal
2018-06-15  9:00     ` Damien Le Moal
2018-06-14 16:19 ` [PATCH] " Bart Van Assche
2018-06-14 16:19   ` Bart Van Assche
2018-06-14 17:44   ` Luis R. Rodriguez

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=4bca9936ccc0cc67404e8da8f7a3c5948ba7e518.camel@wdc.com \
    --to=bart.vanassche@wdc.com \
    --cc=Damien.LeMoal@wdc.com \
    --cc=agk@redhat.com \
    --cc=axboe@kernel.dk \
    --cc=ghe@suse.com \
    --cc=hare@suse.de \
    --cc=jaegeuk@kernel.org \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=mcgrof@kernel.org \
    --cc=mwilck@suse.com \
    --cc=snitzer@redhat.com \
    --cc=tchvatal@suse.com \
    --cc=yuchao0@huawei.com \
    --cc=zren@suse.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.