* [RFC][PATCH] get rid of passing callbacks to ceph __dentry_leases_walk()
@ 2023-10-29 20:46 Al Viro
2023-10-30 12:18 ` Jeff Layton
0 siblings, 1 reply; 2+ messages in thread
From: Al Viro @ 2023-10-29 20:46 UTC (permalink / raw)
To: ceph-devel; +Cc: linux-fsdevel
__dentry_leases_walk() is gets a callback and calls it for
a bunch of denties; there are exactly two callers and
we already have a flag telling them apart - lwc->dir_lease.
Seeing that indirect calls are costly these days, let's
get rid of the callback and just call the right function
directly. Has a side benefit of saner signatures...
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
---
diff --git a/fs/ceph/dir.c b/fs/ceph/dir.c
index 854cbdd66661..30b06d171a40 100644
--- a/fs/ceph/dir.c
+++ b/fs/ceph/dir.c
@@ -1550,10 +1550,12 @@ struct ceph_lease_walk_control {
unsigned long dir_lease_ttl;
};
+static int __dir_lease_check(const struct dentry *, struct ceph_lease_walk_control *);
+static int __dentry_lease_check(const struct dentry *);
+
static unsigned long
__dentry_leases_walk(struct ceph_mds_client *mdsc,
- struct ceph_lease_walk_control *lwc,
- int (*check)(struct dentry*, void*))
+ struct ceph_lease_walk_control *lwc)
{
struct ceph_dentry_info *di, *tmp;
struct dentry *dentry, *last = NULL;
@@ -1581,7 +1583,10 @@ __dentry_leases_walk(struct ceph_mds_client *mdsc,
goto next;
}
- ret = check(dentry, lwc);
+ if (lwc->dir_lease)
+ ret = __dir_lease_check(dentry, lwc);
+ else
+ ret = __dentry_lease_check(dentry);
if (ret & TOUCH) {
/* move it into tail of dir lease list */
__dentry_dir_lease_touch(mdsc, di);
@@ -1638,7 +1643,7 @@ __dentry_leases_walk(struct ceph_mds_client *mdsc,
return freed;
}
-static int __dentry_lease_check(struct dentry *dentry, void *arg)
+static int __dentry_lease_check(const struct dentry *dentry)
{
struct ceph_dentry_info *di = ceph_dentry(dentry);
int ret;
@@ -1653,9 +1658,9 @@ static int __dentry_lease_check(struct dentry *dentry, void *arg)
return DELETE;
}
-static int __dir_lease_check(struct dentry *dentry, void *arg)
+static int __dir_lease_check(const struct dentry *dentry,
+ struct ceph_lease_walk_control *lwc)
{
- struct ceph_lease_walk_control *lwc = arg;
struct ceph_dentry_info *di = ceph_dentry(dentry);
int ret = __dir_lease_try_check(dentry);
@@ -1694,7 +1699,7 @@ int ceph_trim_dentries(struct ceph_mds_client *mdsc)
lwc.dir_lease = false;
lwc.nr_to_scan = CEPH_CAPS_PER_RELEASE * 2;
- freed = __dentry_leases_walk(mdsc, &lwc, __dentry_lease_check);
+ freed = __dentry_leases_walk(mdsc, &lwc);
if (!lwc.nr_to_scan) /* more invalid leases */
return -EAGAIN;
@@ -1704,7 +1709,7 @@ int ceph_trim_dentries(struct ceph_mds_client *mdsc)
lwc.dir_lease = true;
lwc.expire_dir_lease = freed < count;
lwc.dir_lease_ttl = mdsc->fsc->mount_options->caps_wanted_delay_max * HZ;
- freed +=__dentry_leases_walk(mdsc, &lwc, __dir_lease_check);
+ freed +=__dentry_leases_walk(mdsc, &lwc);
if (!lwc.nr_to_scan) /* more to check */
return -EAGAIN;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [RFC][PATCH] get rid of passing callbacks to ceph __dentry_leases_walk()
2023-10-29 20:46 [RFC][PATCH] get rid of passing callbacks to ceph __dentry_leases_walk() Al Viro
@ 2023-10-30 12:18 ` Jeff Layton
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Layton @ 2023-10-30 12:18 UTC (permalink / raw)
To: Al Viro, ceph-devel; +Cc: linux-fsdevel
On Sun, 2023-10-29 at 20:46 +0000, Al Viro wrote:
> __dentry_leases_walk() is gets a callback and calls it for
> a bunch of denties; there are exactly two callers and
> we already have a flag telling them apart - lwc->dir_lease.
>
> Seeing that indirect calls are costly these days, let's
> get rid of the callback and just call the right function
> directly. Has a side benefit of saner signatures...
>
> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
> ---
> diff --git a/fs/ceph/dir.c b/fs/ceph/dir.c
> index 854cbdd66661..30b06d171a40 100644
> --- a/fs/ceph/dir.c
> +++ b/fs/ceph/dir.c
> @@ -1550,10 +1550,12 @@ struct ceph_lease_walk_control {
> unsigned long dir_lease_ttl;
> };
>
> +static int __dir_lease_check(const struct dentry *, struct ceph_lease_walk_control *);
> +static int __dentry_lease_check(const struct dentry *);
> +
> static unsigned long
> __dentry_leases_walk(struct ceph_mds_client *mdsc,
> - struct ceph_lease_walk_control *lwc,
> - int (*check)(struct dentry*, void*))
> + struct ceph_lease_walk_control *lwc)
> {
> struct ceph_dentry_info *di, *tmp;
> struct dentry *dentry, *last = NULL;
> @@ -1581,7 +1583,10 @@ __dentry_leases_walk(struct ceph_mds_client *mdsc,
> goto next;
> }
>
> - ret = check(dentry, lwc);
> + if (lwc->dir_lease)
> + ret = __dir_lease_check(dentry, lwc);
> + else
> + ret = __dentry_lease_check(dentry);
> if (ret & TOUCH) {
> /* move it into tail of dir lease list */
> __dentry_dir_lease_touch(mdsc, di);
> @@ -1638,7 +1643,7 @@ __dentry_leases_walk(struct ceph_mds_client *mdsc,
> return freed;
> }
>
> -static int __dentry_lease_check(struct dentry *dentry, void *arg)
> +static int __dentry_lease_check(const struct dentry *dentry)
> {
> struct ceph_dentry_info *di = ceph_dentry(dentry);
> int ret;
> @@ -1653,9 +1658,9 @@ static int __dentry_lease_check(struct dentry *dentry, void *arg)
> return DELETE;
> }
>
> -static int __dir_lease_check(struct dentry *dentry, void *arg)
> +static int __dir_lease_check(const struct dentry *dentry,
> + struct ceph_lease_walk_control *lwc)
> {
> - struct ceph_lease_walk_control *lwc = arg;
> struct ceph_dentry_info *di = ceph_dentry(dentry);
>
> int ret = __dir_lease_try_check(dentry);
> @@ -1694,7 +1699,7 @@ int ceph_trim_dentries(struct ceph_mds_client *mdsc)
>
> lwc.dir_lease = false;
> lwc.nr_to_scan = CEPH_CAPS_PER_RELEASE * 2;
> - freed = __dentry_leases_walk(mdsc, &lwc, __dentry_lease_check);
> + freed = __dentry_leases_walk(mdsc, &lwc);
> if (!lwc.nr_to_scan) /* more invalid leases */
> return -EAGAIN;
>
> @@ -1704,7 +1709,7 @@ int ceph_trim_dentries(struct ceph_mds_client *mdsc)
> lwc.dir_lease = true;
> lwc.expire_dir_lease = freed < count;
> lwc.dir_lease_ttl = mdsc->fsc->mount_options->caps_wanted_delay_max * HZ;
> - freed +=__dentry_leases_walk(mdsc, &lwc, __dir_lease_check);
> + freed +=__dentry_leases_walk(mdsc, &lwc);
> if (!lwc.nr_to_scan) /* more to check */
> return -EAGAIN;
>
>
Nice cleanup.
Reviewed-by: Jeff Layton <jlayton@kernel.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-10-30 12:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-29 20:46 [RFC][PATCH] get rid of passing callbacks to ceph __dentry_leases_walk() Al Viro
2023-10-30 12:18 ` Jeff Layton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).