Commit c7314d74 authored by Al Viro's avatar Al Viro

nfsd regression since delayed fput()

Background: nfsd v[23] had throughput regression since delayed fput
went in; every read or write ends up doing fput() and we get a pair
of extra context switches out of that (plus quite a bit of work
in queue_work itselfi, apparently).  Use of schedule_delayed_work()
gives it a chance to accumulate a bit before we do __fput() on all
of them.  I'm not too happy about that solution, but... on at least
one real-world setup it reverts about 10% throughput loss we got from
switch to delayed fput.
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 43ae9e3f
...@@ -297,7 +297,7 @@ void flush_delayed_fput(void) ...@@ -297,7 +297,7 @@ void flush_delayed_fput(void)
delayed_fput(NULL); delayed_fput(NULL);
} }
static DECLARE_WORK(delayed_fput_work, delayed_fput); static DECLARE_DELAYED_WORK(delayed_fput_work, delayed_fput);
void fput(struct file *file) void fput(struct file *file)
{ {
...@@ -317,7 +317,7 @@ void fput(struct file *file) ...@@ -317,7 +317,7 @@ void fput(struct file *file)
} }
if (llist_add(&file->f_u.fu_llist, &delayed_fput_list)) if (llist_add(&file->f_u.fu_llist, &delayed_fput_list))
schedule_work(&delayed_fput_work); schedule_delayed_work(&delayed_fput_work, 1);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment