Hi all, This is v3 of the code to avoid temporary backing file opens in overlayfs, taking into account Al's and Miklos' comments on v2 [1]. If no further comments, this is going for overlayfs-next. Thanks, Amir. Changes since v2: - Simplifications to flow (Al) - Loose backing_file stash in favor of ovl_file (Miklos) Changes since v1: - Use helpers ovl_real_file() and ovl_upper_file() to express that ovl_real_file() cannot return NULL - Fix readability and bug is code to select and store stashed upperfile [1] https://lore.kernel.org/linux-unionfs/20241006082359.263755-1-amir73il@xxxxxxxxx/ Amir Goldstein (5): ovl: do not open non-data lower file for fsync ovl: allocate a container struct ovl_file for ovl private context ovl: store upper real file in ovl_file struct ovl: convert ovl_real_fdget_path() callers to ovl_real_file_path() ovl: convert ovl_real_fdget() callers to ovl_real_file() fs/overlayfs/file.c | 301 +++++++++++++++++++++++++------------------- 1 file changed, 173 insertions(+), 128 deletions(-) -- 2.34.1
