General

Profile

Jeff Layton's activity

From 03/24/2019 to 04/22/2019

04/22/2019

06:09 PM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Zheng Yan wrote:
> Jeff Layton wrote:
> > Zheng Yan wrote:
> > > Sorry, I mean we don't need Lx
> >
> > I'm not...
Jeff Layton

04/19/2019

11:37 AM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Zheng Yan wrote:
> Sorry, I mean we don't need Lx
I'm not sure I understand. What if other clients have Ls on the...
Jeff Layton

04/18/2019

05:16 PM Ceph Revision ce0c960a (ceph): Merge pull request #27518 from jtlayton/vstart
vstart: add new option to pass list of block devices to bluestore Jeff Layton

04/17/2019

08:59 PM CephFS Feature #39354: mds: derive wrlock from excl caps
https://github.com/ceph/ceph/pull/27648 Jeff Layton

04/14/2019

11:47 AM Ceph Revision adfe6ccc (ceph): vstart: add new option to pass list of block devices to bluestore
I was doing a bit of testing against a vstart cluster, and ended up
hitting ENOSPC errors. This patch adds a new --bl...
Jeff Layton

04/11/2019

05:07 PM Linux kernel client Bug #39258 (Resolved): add statx support to kcephfs
Now that statx is an official syscall, we should fix kcephfs to take advantage of it. We can avoid requesting conflic... Jeff Layton

04/08/2019

06:07 PM CephFS Feature #38951: client: implement asynchronous unlink/create
Actually it may be sufficient to just wait on any existing dirops to complete before we do a synchronous one. I alrea... Jeff Layton
04:31 PM CephFS Feature #38951: client: implement asynchronous unlink/create
I ran fsstress on these patches today and hit a deadlock:... Jeff Layton
02:00 PM Ceph Revision 91d3d966 (ceph): vstart: add an alias for cephfs-shell to vstart_environment.sh
Signed-off-by: Jeff Layton <jlayton@redhat.com> Jeff Layton

04/06/2019

10:07 AM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Zheng Yan wrote:
> If mds revokes Lx or Fx, client should flush the async unlinks (on the file or under the director...
Jeff Layton

04/05/2019

06:40 PM CephFS Feature #38951: client: implement asynchronous unlink/create
Ok, I have a prototype implementation that depends on a couple of small MDS patches that are discussed here https://t... Jeff Layton
05:41 PM CephFS Feature #39129 (Resolved): create mechanism to delegate ranges of inode numbers to client
Create a mechanism by which we can hand out ranges of inode numbers to MDS clients. The clients can then use those to... Jeff Layton
03:31 PM CephFS Feature #39098: mds: lock caching for asynchronous unlink
This patch seems to do the right thing for me. I'm not sure whether this is the right approach though, and we will ne... Jeff Layton
12:00 PM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Continuing that thought, I see that handle_client_unlink does this unconditionally:... Jeff Layton
10:51 AM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Zheng Yan wrote:
> mds takes xlock on linklock when unlinking file, which always revokes Lsx. why does client want ...
Jeff Layton

04/04/2019

07:33 PM CephFS Feature #39098: mds: lock caching for asynchronous unlink
This does help the MDS to hand out Lx caps on newly created files, but I'm having some trouble getting the cap handli... Jeff Layton
04:38 PM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Now that I think about it, Lx may not be required for rmdir anyway:
We only need that for files that can be hardli...
Jeff Layton
04:06 PM CephFS Feature #39098: mds: lock caching for asynchronous unlink
Thanks Zheng, with both of those patches, I get Lx caps on newly-created files, but not on new directories. This is t... Jeff Layton
10:57 AM CephFS Feature #39098: mds: lock caching for asynchronous unlink
I added that to the userland client and then used cephfs-shell to create a directory and a file inside it. Neither cr... Jeff Layton

04/03/2019

07:53 PM CephFS Feature #39098 (Resolved): mds: lock caching for asynchronous unlink
In order to allow the client to asynchronously delete files, we need Fx caps on the parent, and Lx caps on the inode ... Jeff Layton

04/01/2019

04:35 PM CephFS Feature #38951: client: implement asynchronous unlink/create
I've started going through the kernel client, as I figured this would be more useful there initially (and because I u... Jeff Layton

03/31/2019

12:55 PM Ceph Revision 967d8327 (ceph): vstart: generate environment script suitable for sourcing
At the end, vstart.sh will print out some commands to set up the
environment variables suitably for running things ou...
Jeff Layton

03/29/2019

04:35 PM CephFS Feature #38951: client: implement asynchronous unlink/create
Today, we have no support for asynchronous MDS requests. make_request always blocks on the request. So the first step... Jeff Layton

03/28/2019

01:51 PM Ceph Revision f6ce85d9 (ceph): mds: fix dump routine for session_info_t.used_inos
It says it's dumping used_inos, but it's really dumping prealloc_inos.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Jeff Layton

03/27/2019

07:10 PM CephFS Feature #38951: client: implement asynchronous unlink/create
I'm taking the approach that if we have to contact the server at all, then we probably should just send a synchronous... Jeff Layton
02:53 PM CephFS Feature #38951 (Resolved): client: implement asynchronous unlink/create
We have an open project to teach the client how to buffer creates when it has the right caps (Fx), and a delegated se... Jeff Layton
01:23 PM CephFS Feature #24461: cephfs: improve file create performance buffering file unlink/create operations
I have a couple of starter patches that add a new delegated_inos interval_set to session_info_t.
Questions at this...
Jeff Layton

03/26/2019

04:37 PM Ceph Revision 21339422 (ceph): cephfs-shell: fix mode of script in tree
Make it executable so we can run it inside the tree.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Jeff Layton
02:50 PM Ceph Revision a6587037 (ceph): mds: fix comment over MDRequestImpl
Signed-off-by: Jeff Layton <jlayton@redhat.com> Jeff Layton
02:50 PM Ceph Revision 42cb3a0b (ceph): client: comment fix in _lookup
We're checking for Fs caps on the directory, _not_ a dentry lease.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Jeff Layton
02:50 PM Ceph Revision 6e559f79 (ceph): mds: fix some misleading log messages
There is no process_cap_release function. Change it to print __func__
instead.
Signed-off-by: Jeff Layton <jlayton@r...
Jeff Layton
02:50 PM Ceph Revision 38ebc5d8 (ceph): mds: use stdbool.h instead of hand-rolling our own bool type
Signed-off-by: Jeff Layton <jlayton@redhat.com> Jeff Layton
02:48 PM CephFS Feature #24461: cephfs: improve file create performance buffering file unlink/create operations
Great, so let's do a minor revision on the rules above. In order to buffer creates the client will need:
# an unus...
Jeff Layton
 

Also available in: Atom