Re: [PATCH V2] fs: remove fget_many and fput_many interface

From: Al Viro
Date: Thu May 12 2022 - 20:45:13 EST


On Tue, Nov 02, 2021 at 10:46:48AM +0800, Gou Hao wrote:
> These two interface were added in 091141a42 commit,
> but now there is no place to call them.
>
> The only user of fput/fget_many() was removed in commit
> 62906e89e63b ("io_uring: remove file batch-get optimisation").
>
> A user of get_file_rcu_many() were removed in commit
> f073531070d2 ("init: add an init_dup helper").
>
> And replace atomic_long_sub/add to atomic_long_dec/inc
> can improve performance.
>
> Here are the test results of unixbench:
>
> Cmd: ./Run -c 64 context1
>
> Without patch:
> System Benchmarks Partial Index BASELINE RESULT INDEX
> Pipe-based Context Switching 4000.0 2798407.0 6996.0
> ========
> System Benchmarks Index Score (Partial Only) 6996.0
>
> With patch:
> System Benchmarks Partial Index BASELINE RESULT INDEX
> Pipe-based Context Switching 4000.0 3486268.8 8715.7
> ========
> System Benchmarks Index Score (Partial Only) 8715.7
>
> Signed-off-by: Gou Hao <gouhao@xxxxxxxxxxxxx>

Rebased and applied, with deep apologies for having it fall through cracks
back in November.