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") Thanks, NeilBrown > > Just asking because I did not notice any other dependencies in ovl branch, > so wanted to know which are they. > > Thanks, > Amir. >