On Thu, Oct 18, 2018 at 5:37 PM, Amir Goldstein <amir73il@xxxxxxxxx> wrote: > Miklos, > > First patch removes pretty big ugliness. > Second patch is just a matter of taste, so as you wish. > > I'll use this opportunity to recap the queue of my proposed patches for > v4.20, so you can see if you missed to consider any of them: > > * 5401b99fcbb9 - (overlayfs-devel) ovl: abstract ovl_inode lock with a helper > * 8edcbcef4e55 - ovl: remove the 'locked' argument of ovl_nlink_{start,end} > * bd2ea818ecca - ovl: relax requirement for non null uuid of lower fs > * 19c221c3a6b5 - ovl: disable new features for deprecated upper fs > * 1c785e83823c - ovl: untangle copy up call chain > * b67871490628 - (ovl-fixes) ovl: fix recursive oi->lock in ovl_link() > * e39a0eb89ae4 - vfs: fix FIGETBSZ ioctl on an overlayfs file > * 431d5170b91d - ovl: fix error handling in ovl_verify_set_fh() > * 388b00daecb8 - kernel/acct.c: fix locking order when switching acct files Pushed an update to overlayfs-next that contains most of these. I've dropped the kernel/acct.c one as I didn't yet have time to review it, and also the option fallback one as that is still under discussion. Passes my tests, but more eyes and testing cannot hurt... Thanks, Miklos