From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx3-rdu2.redhat.com ([66.187.233.73]:43856 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754526AbeFNR6z (ORCPT ); Thu, 14 Jun 2018 13:58:55 -0400 Date: Thu, 14 Jun 2018 13:58:52 -0400 From: Mike Snitzer To: "Luis R. Rodriguez" Cc: damien.lemoal@wdc.com, hare@suse.de, axboe@kernel.dk, jaegeuk@kernel.org, yuchao0@huawei.com, ghe@suse.com, mwilck@suse.com, tchvatal@suse.com, zren@suse.com, agk@redhat.com, linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org Subject: Re: dm-zoned-tools: add zoned disk udev rules for scheduler / dmsetup Message-ID: <20180614175852.GA1034@redhat.com> References: <20180614001147.1545-1-mcgrof@kernel.org> <20180614123806.GA32162@redhat.com> <20180614173719.GI5527@wotan.suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180614173719.GI5527@wotan.suse.de> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, Jun 14 2018 at 1:37pm -0400, Luis R. Rodriguez wrote: > On Thu, Jun 14, 2018 at 08:38:06AM -0400, Mike Snitzer wrote: > > On Wed, Jun 13 2018 at 8:11pm -0400, > > Luis R. Rodriguez wrote: > > > > > Setting up a zoned disks in a generic form is not so trivial. There > > > is also quite a bit of tribal knowledge with these devices which is not > > > easy to find. > > > > > > The currently supplied demo script works but it is not generic enough to be > > > practical for Linux distributions or even developers which often move > > > from one kernel to another. > > > > > > This tries to put a bit of this tribal knowledge into an initial udev > > > rule for development with the hopes Linux distributions can later > > > deploy. Three rule are added. One rule is optional for now, it should be > > > extended later to be more distribution-friendly and then I think this > > > may be ready for consideration for integration on distributions. > > > > > > 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. > > This is the point to the patch as well, I actually tend to agree with you, > and I had tried to draw up a patch to do just that, however its *not* possible > today to do this and would require some consensus. So from what I can tell > we *have* to live with this one or a form of it. Ie a file describing which > disk serial gets deadline and which one gets mq-deadline. > > Jens? > > Anyway, let's assume this is done in the kernel, which one would use deadline, > which one would use mq-deadline? The zoned storage driver needs to make that call based on what mode it is in. If it is using blk-mq then it selects mq-deadline, otherwise deadline. > > > 2) backlist f2fs devices > > > > There should porbably be support in dm-zoned for detecting whether a > > zoned device was formatted with f2fs (assuming there is a known f2fs > > superblock)? > > Not sure what you mean. Are you suggesting we always setup dm-zoned for > all zoned disks and just make an excemption on dm-zone code to somehow > use the disk directly if a filesystem supports zoned disks directly somehow? No, I'm saying that a udev rule wouldn't be needed if dm-zoned just errored out if asked to consume disks that already have an f2fs superblock. And existing filesystems should get conflicting superblock awareness "for free" if blkid or whatever is trained to be aware of f2fs's superblock. > f2fs does not require dm-zoned. What would be required is a bit more complex > given one could dedicate portions of the disk to f2fs and other portions to > another filesystem, which would require dm-zoned. > > Also filesystems which *do not* support zoned disks should *not* be allowing > direct setup. Today that's all filesystems other than f2fs, in the future > that may change. Those are bullets we are allowing to trigger for users > just waiting to shot themselves on the foot with. > > So who's going to work on all the above? It should take care of itself if existing tools are trained to be aware of new signatures. E.g. ext4 and xfs already are aware of one another so that you cannot reformat a device with the other unless force is given. Same kind of mutual exclussion needs to happen for zoned devices. So the zoned device tools, dm-zoned, f2fs, whatever.. they need to be updated to not step on each others toes. And other filesystems' tools need to be updated to be zoned device aware. > The point of the udev script is to illustrate the pains to properly deploy > zoned disks on distributions today and without a roadmap... this is what > at least I need on my systems today to reasonably deploy these disks for > my own development. > > Consensus is indeed needed for a broader picture. Yeap. > > > 3) run dmsetup for the rest of devices > > > > automagically running dmsetup directly from udev to create a dm-zoned > > target is very much wrong. It just gets in the way of proper support > > that should be add to appropriate tools that admins use to setup their > > zoned devices. For instance, persistent use of dm-zoned target should > > be made reliable with a volume manager.. > > Ah yes, but who's working on that? How long will it take? No idea, as is (from my vantage point) there is close to zero demand for zoned devices. It won't be a priority until enough customers are asking for it. > I agree it is odd to expect one to use dmsetup and then use a volume manager on > top of it, if we can just add proper support onto the volume manager... then > that's a reasonable way to go. > > But *we're not there* yet, and as-is today, what is described in the udev > script is the best we can do for a generic setup. Just because doing things right takes work doesn't mean it makes sense to elevate this udev script to be packaged in some upstream project like udev or whatever. But if SUSE or some other distro wants to ship it that is fine. > > In general this udev script is unwelcome and makes things way worse for > > the long-term success of zoned devices. > > dm-zoned-tools does not acknowledge in any way a roadmap, and just provides > a script, which IMHO is less generic and less distribution friendly. Having > a udev rule in place to demonstrate the current state of affairs IMHO is > more scalable demonstrates the issues better than the script. > > If we have an agreed upon long term strategy lets document that. But from > what I gather we are not even in consensus with regards to the scheduler > stuff. If we have consensus on the other stuff lets document that as > dm-zoned-tools is the only place I think folks could find to reasonably > deploy these things. I'm sure Damien and others will have something to say here. > > I don't dispute there is an obvious void for how to properly setup zoned > > devices, but this script is _not_ what should fill that void. > > Good to know! Again, consider it as an alternative to the script. > > I'm happy to adapt the language and supply it only as an example script > developers can use, but we can't leave users hanging as well. Let's at > least come up with a plan which we seem to agree on and document that. Best to try to get Damien and others more invested in zoned devices to help you take up your cause. I think it is worthwhile to develop a strategy. But it needs to be done in terms of the norms of the existing infrastructure we all make use of today. So first step is making existing tools zoned device aware (even if to reject such devices). Mike