Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S938941AbcKXLDZ (ORCPT ); Thu, 24 Nov 2016 06:03:25 -0500 Received: from mail-wm0-f49.google.com ([74.125.82.49]:33282 "EHLO mail-wm0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S936026AbcKXLDZ (ORCPT ); Thu, 24 Nov 2016 06:03:25 -0500 From: Miklos Szeredi To: linux-unionfs@vger.kernel.org Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 7/7] ovl: intercept fsync Date: Thu, 24 Nov 2016 11:55:44 +0100 Message-Id: <1479984944-1017-9-git-send-email-mszeredi@redhat.com> X-Mailer: git-send-email 2.5.5 In-Reply-To: <1479984944-1017-1-git-send-email-mszeredi@redhat.com> References: <1479984944-1017-1-git-send-email-mszeredi@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1632 Lines: 55 Fsync is really an inode operation (AFAICS) so a doing it on a O_RDONLY file descriptor should flush any data written through an O_WRONLY file descriptor for example. To make this work correctly in case the file is copied up after being opened for read intercept the fsync fop, similarly to read_iter and mmap. Signed-off-by: Miklos Szeredi --- fs/overlayfs/inode.c | 23 ++++++++++++++++++++--- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/fs/overlayfs/inode.c b/fs/overlayfs/inode.c index 25b31c6ebe9e..f522cb350f3c 100644 --- a/fs/overlayfs/inode.c +++ b/fs/overlayfs/inode.c @@ -412,6 +412,24 @@ static int ovl_mmap(struct file *file, struct vm_area_struct *vma) return file->f_op->mmap(file, vma); } +static int ovl_fsync(struct file *file, loff_t start, loff_t end, int datasync) +{ + int ret; + + if (likely(ovl_file_is_lower(file))) { + return OVL_CALL_REAL_FOP(file, + fsync(file, start, end, datasync)); + } + file = filp_clone_open(file); + if (IS_ERR(file)) + return PTR_ERR(file); + + ret = vfs_fsync_range(file, start, end, datasync); + fput(file); + + return ret; +} + static struct ovl_fops *ovl_fops_find(const struct file_operations *orig) { struct ovl_fops *ofop; @@ -453,9 +471,8 @@ static struct ovl_fops *ovl_fops_get(struct file *file) ofop->fops.read_iter = ovl_read_iter; if (orig->mmap) ofop->fops.mmap = ovl_mmap; - - /* These will need to be intercepted: */ - ofop->fops.fsync = orig->fsync; + if (orig->fsync) + ofop->fops.fsync = ovl_fsync; /* * These should be intercepted, but they are very unlikely to be -- 2.5.5