Linux-RDMA Archive on lore.kernel.org
 help / color / Atom feed
From: Jeff Layton <jlayton@kernel.org>
To: "J. Bruce Fields" <bfields@fieldses.org>
Cc: Ira Weiny <ira.weiny@intel.com>,
	linux-fsdevel@vger.kernel.org, linux-xfs@vger.kernel.org,
	linux-ext4@vger.kernel.org, linux-rdma@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-nvdimm@lists.01.org,
	linux-mm@kvack.org, Dave Chinner <david@fromorbit.com>,
	Jan Kara <jack@suse.cz>, Theodore Ts'o <tytso@mit.edu>,
	John Hubbard <jhubbard@nvidia.com>,
	Dan Williams <dan.j.williams@intel.com>,
	Jason Gunthorpe <jgg@ziepe.ca>
Subject: Re: Lease semantic proposal
Date: Wed, 02 Oct 2019 16:35:55 -0400
Message-ID: <df9022f0f5d18d71f37ed494a05eaa4509cf0a68.camel@kernel.org> (raw)
In-Reply-To: <20191002192711.GA21386@fieldses.org>

On Wed, 2019-10-02 at 15:27 -0400, J. Bruce Fields wrote:
> On Wed, Oct 02, 2019 at 08:28:40AM -0400, Jeff Layton wrote:
> > On Tue, 2019-10-01 at 11:17 -0700, Ira Weiny wrote:
> > > On Mon, Sep 23, 2019 at 04:17:59PM -0400, Jeff Layton wrote:
> > > > On Mon, 2019-09-23 at 12:08 -0700, Ira Weiny wrote:
> > > > > Since the last RFC patch set[1] much of the discussion of supporting RDMA with
> > > > > FS DAX has been around the semantics of the lease mechanism.[2]  Within that
> > > > > thread it was suggested I try and write some documentation and/or tests for the
> > > > > new mechanism being proposed.  I have created a foundation to test lease
> > > > > functionality within xfstests.[3] This should be close to being accepted.
> > > > > Before writing additional lease tests, or changing lots of kernel code, this
> > > > > email presents documentation for the new proposed "layout lease" semantic.
> > > > > 
> > > > > At Linux Plumbers[4] just over a week ago, I presented the current state of the
> > > > > patch set and the outstanding issues.  Based on the discussion there, well as
> > > > > follow up emails, I propose the following addition to the fcntl() man page.
> > > > > 
> > > > > Thank you,
> > > > > Ira
> > > > > 
> > > > > [1] https://lkml.org/lkml/2019/8/9/1043
> > > > > [2] https://lkml.org/lkml/2019/8/9/1062
> > > > > [3] https://www.spinics.net/lists/fstests/msg12620.html
> > > > > [4] https://linuxplumbersconf.org/event/4/contributions/368/
> > > > > 
> > > > > 
> > > > 
> > > > Thank you so much for doing this, Ira. This allows us to debate the
> > > > user-visible behavior semantics without getting bogged down in the
> > > > implementation details. More comments below:
> > > 
> > > Thanks.  Sorry for the delay in response.  Turns out this email was in my
> > > spam...  :-/  I'll need to work out why.
> > > 
> > > > > <fcntl man page addition>
> > > > > Layout Leases
> > > > > -------------
> > > > > 
> > > > > Layout (F_LAYOUT) leases are special leases which can be used to control and/or
> > > > > be informed about the manipulation of the underlying layout of a file.
> > > > > 
> > > > > A layout is defined as the logical file block -> physical file block mapping
> > > > > including the file size and sharing of physical blocks among files.  Note that
> > > > > the unwritten state of a block is not considered part of file layout.
> > > > > 
> > > > > **Read layout lease F_RDLCK | F_LAYOUT**
> > > > > 
> > > > > Read layout leases can be used to be informed of layout changes by the
> > > > > system or other users.  This lease is similar to the standard read (F_RDLCK)
> > > > > lease in that any attempt to change the _layout_ of the file will be reported to
> > > > > the process through the lease break process.  But this lease is different
> > > > > because the file can be opened for write and data can be read and/or written to
> > > > > the file as long as the underlying layout of the file does not change.
> > > > > Therefore, the lease is not broken if the file is simply open for write, but
> > > > > _may_ be broken if an operation such as, truncate(), fallocate() or write()
> > > > > results in changing the underlying layout.
> > > > > 
> > > > > **Write layout lease (F_WRLCK | F_LAYOUT)**
> > > > > 
> > > > > Write Layout leases can be used to break read layout leases to indicate that
> > > > > the process intends to change the underlying layout lease of the file.
> > > > > 
> > > > > A process which has taken a write layout lease has exclusive ownership of the
> > > > > file layout and can modify that layout as long as the lease is held.
> > > > > Operations which change the layout are allowed by that process.  But operations
> > > > > from other file descriptors which attempt to change the layout will break the
> > > > > lease through the standard lease break process.  The F_LAYOUT flag is used to
> > > > > indicate a difference between a regular F_WRLCK and F_WRLCK with F_LAYOUT.  In
> > > > > the F_LAYOUT case opens for write do not break the lease.  But some operations,
> > > > > if they change the underlying layout, may.
> > > > > 
> > > > > The distinction between read layout leases and write layout leases is that
> > > > > write layout leases can change the layout without breaking the lease within the
> > > > > owning process.  This is useful to guarantee a layout prior to specifying the
> > > > > unbreakable flag described below.
> > > > > 
> > > > > 
> > > > 
> > > > The above sounds totally reasonable. You're essentially exposing the
> > > > behavior of nfsd's layout leases to userland. To be clear, will F_LAYOUT
> > > > leases work the same way as "normal" leases, wrt signals and timeouts?
> > > 
> > > That was my intention, yes.
> > > 
> > > > I do wonder if we're better off not trying to "or" in flags for this,
> > > > and instead have a separate set of commands (maybe F_RDLAYOUT,
> > > > F_WRLAYOUT, F_UNLAYOUT). Maybe I'm just bikeshedding though -- I don't
> > > > feel terribly strongly about it.
> > > 
> > > I'm leaning that was as well.  To make these even more distinct from
> > > F_SETLEASE.
> > > 
> > > > Also, at least in NFSv4, layouts are handed out for a particular byte
> > > > range in a file. Should we consider doing this with an API that allows
> > > > for that in the future? Is this something that would be desirable for
> > > > your RDMA+DAX use-cases?
> > > 
> > > I don't see this.  I've thought it would be a nice thing to have but I don't
> > > know of any hard use case.  But first I'd like to understand how this works for
> > > NFS.
> > > 
> > 
> > The NFSv4.1 spec allows the client to request the layouts for a
> > particular range in the file:
> > 
> > https://tools.ietf.org/html/rfc5661#page-538
> > 
> > The knfsd only hands out whole-file layouts at present. Eventually we
> > may want to make better use of segmented layouts, at which point we'd
> > need something like a byte-range lease.
> > 
> > > > We could add a new F_SETLEASE variant that takes a struct with a byte
> > > > range (something like struct flock).
> > > 
> > > I think this would be another reason to introduce F_[RD|WR|UN]LAYOUT as a
> > > command.  Perhaps supporting smaller byte ranges could be added later?
> > > 
> > 
> > I'd definitely not multiplex this over F_SETLEASE. An F_SETLAYOUT cmd
> > would probably be sufficient, and maybe just reuse
> > F_RDLCK/F_WRLCK/F_UNLCK for the iomode?
> > 
> > For the byte ranges, the catch there is that extending the userland
> > interface for that later will be difficult.
> 
> Why would it be difficult?
> 

Legacy userland code that wanted to use byte range enabled layouts would
have to be rebuilt to take advantage of them. If we require a range from
the get-go, then they will get the benefit of them once they're
available.
 
> > What I'd probably suggest
> > (and what would jive with the way pNFS works) would be to go ahead and
> > add an offset and length to the arguments and result (maybe also
> > whence?).
> 
> Why not add new commands with range arguments later if it turns out to
> be necessary?

We could do that. It'd be a little ugly, IMO, simply because then we'd
end up with two interfaces that do almost the exact same thing.

Should byte-range layouts at that point conflict with non-byte range
layouts, or should they be in different "spaces" (a'la POSIX and flock
locks)? When it's all one interface, those sorts of questions sort of
answer themselves. When they aren't we'll have to document them clearly
and I think the result will be more confusing for userland programmers.

If you felt strongly about leaving those out for now, you could just do
something similar to what Aleksa is planning for openat2 -- have a
struct pointer and length as arguments for this cmd, and only have a
single iomode member in there for now.

The kernel would have to know how to deal with "legacy" and byte-range-
enabled variants if we ever extend it, but that's not too hard to
handle.
-- 
Jeff Layton <jlayton@kernel.org>


  reply index

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-23 19:08 Ira Weiny
2019-09-23 20:17 ` Jeff Layton
2019-10-01 18:17   ` Ira Weiny
2019-10-02 12:28     ` Jeff Layton
2019-10-02 19:27       ` bfields
2019-10-02 20:35         ` Jeff Layton [this message]
2019-10-03  8:43           ` Jan Kara
2019-10-03 15:37           ` J. Bruce Fields
2019-10-03  9:01     ` Jan Kara
2019-10-03 17:05       ` Ira Weiny
2019-09-23 22:26 ` Dave Chinner
2019-09-25 23:46   ` Ira Weiny
2019-09-26 11:29     ` Jeff Layton
2019-09-30  8:42     ` Dave Chinner
2019-10-01 21:01       ` Ira Weiny
2019-10-02 13:07         ` Dan Williams
2019-10-10 10:39         ` Dave Chinner
2019-10-04  7:51       ` Jan Kara

Reply instructions:

You may reply publically 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=df9022f0f5d18d71f37ed494a05eaa4509cf0a68.camel@kernel.org \
    --to=jlayton@kernel.org \
    --cc=bfields@fieldses.org \
    --cc=dan.j.williams@intel.com \
    --cc=david@fromorbit.com \
    --cc=ira.weiny@intel.com \
    --cc=jack@suse.cz \
    --cc=jgg@ziepe.ca \
    --cc=jhubbard@nvidia.com \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-nvdimm@lists.01.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=linux-xfs@vger.kernel.org \
    --cc=tytso@mit.edu \
    /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

Linux-RDMA Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-rdma/0 linux-rdma/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-rdma linux-rdma/ https://lore.kernel.org/linux-rdma \
		linux-rdma@vger.kernel.org
	public-inbox-index linux-rdma

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-rdma


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git