From: "Yan, Zheng" <zyan@redhat.com>
To: Jeff Layton <jlayton@kernel.org>, ceph-devel@vger.kernel.org
Cc: idryomov@gmail.com, sage@redhat.com, pdonnell@redhat.com
Subject: Re: [PATCH v6 00/13] ceph: async directory operations support
Date: Tue, 3 Mar 2020 00:22:51 +0800 [thread overview]
Message-ID: <544d98ca-6dc9-bcda-3f99-ff87f646265c@redhat.com> (raw)
In-Reply-To: <20200302141434.59825-1-jlayton@kernel.org>
On 3/2/20 10:14 PM, Jeff Layton wrote:
> v6: move handling of CEPH_I_ASYNC_CREATE from __send_cap into callers
> also issue ceph_mdsc_release_dir_caps() in complete_request
> properly handle -EJUKEBOX return in async callbacks
>
> I previously pulled the async unlink patch from ceph-client/testing, so
> this set includes a revised version of that as well, and orders it
> some other changes.
>
> The main change from v5 is to rework the callers of __send_cap to either
> skip sending or wait if the create reply hasn't come in yet.
>
> We may not actually need patch #7 here. Zheng had that delta in one
> of the earlier patches, but I'm not sure it's really needed now. It
> may make sense to just take it on its own merits though.
>
> Jeff Layton (12):
> ceph: make kick_flushing_inode_caps non-static
> ceph: add flag to designate that a request is asynchronous
> ceph: track primary dentry link
> ceph: add infrastructure for waiting for async create to complete
> ceph: make __take_cap_refs non-static
> ceph: cap tracking for async directory operations
> ceph: perform asynchronous unlink if we have sufficient caps
> ceph: make ceph_fill_inode non-static
> ceph: decode interval_sets for delegated inos
> ceph: add new MDS req field to hold delegated inode number
> ceph: cache layout in parent dir on first sync create
> ceph: attempt to do async create when possible
>
> Yan, Zheng (1):
> ceph: don't take refs to want mask unless we have all bits
>
> fs/ceph/caps.c | 91 ++++++++----
> fs/ceph/dir.c | 111 ++++++++++++++-
> fs/ceph/file.c | 269 +++++++++++++++++++++++++++++++++--
> fs/ceph/inode.c | 58 ++++----
> fs/ceph/mds_client.c | 196 ++++++++++++++++++++++---
> fs/ceph/mds_client.h | 24 +++-
> fs/ceph/super.c | 20 +++
> fs/ceph/super.h | 23 ++-
> include/linux/ceph/ceph_fs.h | 17 ++-
> 9 files changed, 724 insertions(+), 85 deletions(-)
>
series
Reviewed-by: "Yan, Zheng" <zyan@redhat.com>
next prev parent reply other threads:[~2020-03-02 16:23 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-02 14:14 [PATCH v6 00/13] ceph: async directory operations support Jeff Layton
2020-03-02 14:14 ` [PATCH v6 01/13] ceph: make kick_flushing_inode_caps non-static Jeff Layton
2020-03-02 14:14 ` [PATCH v6 02/13] ceph: add flag to designate that a request is asynchronous Jeff Layton
2020-03-02 14:14 ` [PATCH v6 03/13] ceph: track primary dentry link Jeff Layton
2020-03-02 14:14 ` [PATCH v6 04/13] ceph: add infrastructure for waiting for async create to complete Jeff Layton
2020-03-02 14:14 ` [PATCH v6 05/13] ceph: make __take_cap_refs non-static Jeff Layton
2020-03-02 14:14 ` [PATCH v6 06/13] ceph: cap tracking for async directory operations Jeff Layton
2020-03-02 14:14 ` [PATCH v6 07/13] ceph: don't take refs to want mask unless we have all bits Jeff Layton
2020-03-02 14:14 ` [PATCH v6 08/13] ceph: perform asynchronous unlink if we have sufficient caps Jeff Layton
2020-03-02 14:14 ` [PATCH v6 09/13] ceph: make ceph_fill_inode non-static Jeff Layton
2020-03-02 14:14 ` [PATCH v6 10/13] ceph: decode interval_sets for delegated inos Jeff Layton
2020-03-05 11:45 ` Luis Henriques
2020-03-05 12:02 ` Jeff Layton
2020-03-05 12:20 ` Luis Henriques
2020-03-05 13:36 ` Ilya Dryomov
2020-03-05 13:44 ` Jeff Layton
2020-03-02 14:14 ` [PATCH v6 11/13] ceph: add new MDS req field to hold delegated inode number Jeff Layton
2020-03-02 14:14 ` [PATCH v6 12/13] ceph: cache layout in parent dir on first sync create Jeff Layton
2020-03-02 14:14 ` [PATCH v6 13/13] ceph: attempt to do async create when possible Jeff Layton
2020-03-02 16:22 ` Yan, Zheng [this message]
2020-03-02 21:07 ` [PATCH v6 00/13] ceph: async directory operations support Jeff Layton
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=544d98ca-6dc9-bcda-3f99-ff87f646265c@redhat.com \
--to=zyan@redhat.com \
--cc=ceph-devel@vger.kernel.org \
--cc=idryomov@gmail.com \
--cc=jlayton@kernel.org \
--cc=pdonnell@redhat.com \
--cc=sage@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox