On Sun, Feb 4, 2024 at 8:33 AM Oliver Sang <oliver.sang@xxxxxxxxx> wrote:
>
> hi, Amir,
>
> On Fri, Feb 02, 2024 at 11:13:56AM +0200, Amir Goldstein wrote:
> > On Wed, Jan 31, 2024 at 5:47 PM Amir Goldstein <amir73il@xxxxxxxxx> wrote:
> > >
> > > On Wed, Jan 31, 2024 at 4:13 PM kenel test robot <oliver.sang@xxxxxxxxx> wrote:
> > > >
> > > >
> > > >
> > > > Hello,
> > > >
> > > > kernel test robot noticed a -11.2% regression of stress-ng.file-ioctl.ops_per_sec on:
> > > >
> > > >
> > > > commit: dfad37051ade6ac0d404ef4913f3bd01954ee51c ("remap_range: move permission hooks out of do_clone_file_range()")
> > > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git master
> > > >
> > >
> > > Can you please try this fix:
> > >
> > > 7d4213664bda remap_range: move sanity checks out of do_clone_file_range()
> > >
> > > from:
> > >
> > > https://github.com/amir73il/linux ovl-fixes
> > >
> >
> > Sorry, Oliver, this was a buggy commit.
> > I pushed this fixes version to ovl-fixes branch:
> >
> > 1c5e7db8e1b2 remap_range: merge do_clone_file_range() into
> > vfs_clone_file_range()
> >
> > Can you please test.
>
> the regression disappeared by above commit in our tests.
>
> I noticed this branch is based on v6.8-rc2, so I directly tested upon it and its
> parent (3f01e53bf6). I found 3f01e53bf6 has same data as dfad37051a we reported.
>
> and on 1c5e7db8e1b2, the performance back to the same level before dfad37051a.
>
Thanks for testing!
Christian, can you please amend the fix commit to
Reported-and-tested-by: kernel test robot <oliver.sang@xxxxxxxxx>
Thanks,
Amir.
