| Message ID | 20231106065045.895874-1-amir73il@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [GIT,PULL] overlayfs update for 6.7 | expand |
On Sun, 5 Nov 2023 at 22:50, Amir Goldstein <amir73il@gmail.com> wrote: > > - Overlayfs aio cleanups and fixes [1] > > - Overlayfs lock ordering changes [2] > > - Add support for nesting overlayfs private xattrs [3] > > - Add new mount options for appending lowerdirs [4] > > [1] https://lore.kernel.org/r/20230912173653.3317828-1-amir73il@gmail.com/ > [2] https://lore.kernel.org/r/20230816152334.924960-1-amir73il@gmail.com/ > [3] https://lore.kernel.org/r/cover.1694512044.git.alexl@redhat.com/ > [4] https://lore.kernel.org/r/20231030120419.478228-1-amir73il@gmail.com/ *Please* don't make me have to follow links just to see basic details. Merge messages should stand on their own, not just point to "look, here are the details in our lore archives". Yes, even when having internet access is much more common, there are situations where it's not there or is slow. Or maybe lore has issues. Or maybe people just don't want to switch to a browser to look up details that may or may not be relevant to them. I copied the relevant stuff into my merge, but please don't make me do this next time. Just give me a merge message with the details spelled out, not some link to a cover letter for a patch series. Linus
The pull request you sent on Mon, 6 Nov 2023 08:50:45 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/overlayfs/vfs.git ovl-update-6.7
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/7f851936a0ca4b231224ee296cba28f9b1bc555e
Thank you!
On Tue, Nov 7, 2023 at 9:54 PM Linus Torvalds <torvalds@linux-foundation.org> wrote: > > On Sun, 5 Nov 2023 at 22:50, Amir Goldstein <amir73il@gmail.com> wrote: > > > > - Overlayfs aio cleanups and fixes [1] > > > > - Overlayfs lock ordering changes [2] > > > > - Add support for nesting overlayfs private xattrs [3] > > > > - Add new mount options for appending lowerdirs [4] > > > > [1] https://lore.kernel.org/r/20230912173653.3317828-1-amir73il@gmail.com/ > > [2] https://lore.kernel.org/r/20230816152334.924960-1-amir73il@gmail.com/ > > [3] https://lore.kernel.org/r/cover.1694512044.git.alexl@redhat.com/ > > [4] https://lore.kernel.org/r/20231030120419.478228-1-amir73il@gmail.com/ > > *Please* don't make me have to follow links just to see basic details. > No problem. > Merge messages should stand on their own, not just point to "look, > here are the details in our lore archives". Yes, even when having > internet access is much more common, there are situations where it's > not there or is slow. Or maybe lore has issues. Or maybe people just > don't want to switch to a browser to look up details that may or may > not be relevant to them. > > I copied the relevant stuff into my merge, but please don't make me do > this next time. Thanks for adding this information - I certainly didn't intend for you to do this extra work. > Just give me a merge message with the details spelled > out, not some link to a cover letter for a patch series. > OK. To explain the reason why I add those links: A while ago, I was helping out with backporting many xfs patches to stable tree after a long period of neglection. I found the need to point back from fix commits to patch series because the cover letters often contained important information relevant for backporting and testing. So I hacked up a small b4 gadget to disassemble a PR into lore links to patch series: (*) you don't have to follow this link unless you are curious ;) https://github.com/amir73il/b4/commit/f5966362a524182206d5c5e8a4f96fba5d4c92ca At the time, I had wished that the links to the patch series composing the PR would have been in the PR email (doesn't really matter if you take them into the merge commit). This is why I included this "debug info" of the PR. Next time, if I include the lore links, I will separate them to a different section, so it is clear that I do not require them to be in the merge commit. Thanks, Amir.
Hi Linus, Please pull overlayfs update for 6.7. Most of the changes in this update have been sitting in linux-next for several weeks, based on two prerequisite vfs topic branches. Last week, after the vfs branches were merged, I rebased overlayfs-next to reduce noise. Last week, I also added patches for new mount options to replace the lowerdir append syntax that we disabled in v6.6-rc6 and v6.5.8. This branch has gone through the usual overlayfs test routines and it merges cleanly with master branch of the moment. Thanks, Amir. ---------------------------------------------------------------- The following changes since commit 14ab6d425e80674b6a0145f05719b11e82e64824: Merge tag 'vfs-6.7.ctime' of gitolite.kernel.org:pub/scm/linux/kernel/git/vfs/vfs (2023-10-30 09:47:13 -1000) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/overlayfs/vfs.git ovl-update-6.7 for you to fetch changes up to 24e16e385f2272b1a9df51337a5c32d28a29c7ad: ovl: add support for appending lowerdirs one by one (2023-10-31 00:13:02 +0200) ---------------------------------------------------------------- overlayfs update for 6.7 Contains the following patch sets: - Overlayfs aio cleanups and fixes [1] - Overlayfs lock ordering changes [2] - Add support for nesting overlayfs private xattrs [3] - Add new mount options for appending lowerdirs [4] [1] https://lore.kernel.org/r/20230912173653.3317828-1-amir73il@gmail.com/ [2] https://lore.kernel.org/r/20230816152334.924960-1-amir73il@gmail.com/ [3] https://lore.kernel.org/r/cover.1694512044.git.alexl@redhat.com/ [4] https://lore.kernel.org/r/20231030120419.478228-1-amir73il@gmail.com/ ---------------------------------------------------------------- Alexander Larsson (4): ovl: Add OVL_XATTR_TRUSTED/USER_PREFIX_LEN macros ovl: Support escaped overlay.* xattrs ovl: Add an alternative type of whiteout ovl: Add documentation on nesting of overlayfs mounts Amir Goldstein (14): ovl: use simpler function to convert iocb to rw flags ovl: propagate IOCB_APPEND flag on writes to realfile ovl: punt write aio completion to workqueue ovl: protect copying of realinode attributes to ovl inode ovl: add helper ovl_file_modified() ovl: split ovl_want_write() into two helpers ovl: reorder ovl_want_write() after ovl_inode_lock() ovl: do not open/llseek lower file with upper sb_writers held ovl: do not encode lower fh with upper sb_writers held ovl: Move xattr support to new xattrs.c file ovl: remove unused code in lowerdir param parsing ovl: store and show the user provided lowerdir mount option ovl: refactor layer parsing helpers ovl: add support for appending lowerdirs one by one Documentation/filesystems/overlayfs.rst | 40 +++- fs/overlayfs/Makefile | 2 +- fs/overlayfs/copy_up.c | 142 +++++++++----- fs/overlayfs/dir.c | 64 +++---- fs/overlayfs/export.c | 7 +- fs/overlayfs/file.c | 88 ++++++--- fs/overlayfs/inode.c | 165 ++-------------- fs/overlayfs/namei.c | 52 ++++-- fs/overlayfs/overlayfs.h | 72 +++++-- fs/overlayfs/params.c | 322 +++++++++++++++++--------------- fs/overlayfs/params.h | 1 + fs/overlayfs/readdir.c | 27 ++- fs/overlayfs/super.c | 92 ++------- fs/overlayfs/util.c | 115 +++++++++++- fs/overlayfs/xattrs.c | 271 +++++++++++++++++++++++++++ fs/super.c | 1 + 16 files changed, 929 insertions(+), 532 deletions(-) create mode 100644 fs/overlayfs/xattrs.c