On Wed, Jul 16, 2025 at 10:05 AM NeilBrown <neil@xxxxxxxxxx> wrote:
>
> On Wed, 16 Jul 2025, Amir Goldstein wrote:
> > On Wed, Jul 16, 2025 at 9:19 AM NeilBrown <neil@xxxxxxxxxx> wrote:
> > >
> > > On Wed, 16 Jul 2025, Amir Goldstein wrote:
> > > > On Wed, Jul 16, 2025 at 2:47 AM NeilBrown <neil@xxxxxxxxxx> wrote:
> > > > >
> > > > > More excellent review feedback - more patches :-)
> > > > >
> > > > > I've chosen to use ovl_parent_lock() here as a temporary and leave the
> > > > > debate over naming for the VFS version of the function until all the new
> > > > > names are introduced later.
> > > >
> > > > Perfect.
> > > >
> > > > Please push v3 patches to branch pdirops, or to a clean branch
> > > > based on vfs-6.17.file, so I can test them.
> > >
> > > There is a branch "ovl" which is based on vfs.all as I depend on a
> > > couple of other vfs changes.
> >
> > ok I will test this one.
> >
> > Do you mean that ovl branch depends on other vfs changes or that pdirops
> > which is based on ovl branch depends on other vfs changes?
>
> ovl branch depends on
>
> Commit bc9241367aac ("VFS: change old_dir and new_dir in struct renamedata to dentrys")
I see.
Anyway, testing looks good on your branch.
Christian,
