Re: [PATCH 0/2] overlayfs lock ordering changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jul 20, 2023 at 6:37 PM Amir Goldstein <amir73il@xxxxxxxxx> wrote:
>
> Hi Miklos,
>
> These prep patches are needed for my start-write-safe series [1].
> This is not urgent and the prep patches don't need to be merged
> for next cycle, but I think these are good changes regardless,
> so wanted to post them for early review - if you like them you can
> queue them for 6.6.
>
> It is quite hard to do the review of the locking reorder patch from the
> diff itself and I couldn't figure out a better way to split this change.
> I've intentionally left some otherwise useless out: goto labels to
> make the patch review a bit simper - they could be removed later.
>
> On the good side, lockdep was very tough with me and it easily detected
> bugs in the earlier versions of the patches.
>
> Going on vacation. will be back round rc6.

Hi Miklos,

Have you had a chance to look at these changes?
I can queue them up in overlayfs-next if they are acceptable.

Thanks,
Amir.

>
> [1] https://github.com/amir73il/linux/commits/start-write-safe
>
> Amir Goldstein (2):
>   ovl: reorder ovl_want_write() after ovl_inode_lock()
>   ovl: avoid lockdep warning with open and llseek of lower file
>
>  fs/overlayfs/copy_up.c | 34 +++++++++++++-------
>  fs/overlayfs/dir.c     | 71 ++++++++++++++++++------------------------
>  fs/overlayfs/export.c  |  7 +----
>  fs/overlayfs/inode.c   | 56 ++++++++++++++++-----------------
>  fs/overlayfs/util.c    | 18 +++++++++--
>  5 files changed, 97 insertions(+), 89 deletions(-)
>
> --
> 2.34.1
>




[Index of Archives]     [Linux Filesystems Devel]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux