Re: [PATCH] ceph: set FMODE_CAN_ODIRECT instead of a dummy direct_IO method

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

 




On 6/12/23 13:35, Christoph Hellwig wrote:
Since commit a2ad63daa88b ("VFS: add FMODE_CAN_ODIRECT file flag") file
systems can just set the FMODE_CAN_ODIRECT flag at open time instead of
wiring up a dummy direct_IO method to indicate support for direct I/O.
Do that for ceph so that noop_direct_IO can eventually be removed.

Signed-off-by: Christoph Hellwig <hch@xxxxxx>
---
  fs/ceph/addr.c | 1 -
  fs/ceph/file.c | 2 ++
  2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c
index 6bb251a4d613eb..19c0f42540b600 100644
--- a/fs/ceph/addr.c
+++ b/fs/ceph/addr.c
@@ -1401,7 +1401,6 @@ const struct address_space_operations ceph_aops = {
  	.dirty_folio = ceph_dirty_folio,
  	.invalidate_folio = ceph_invalidate_folio,
  	.release_folio = ceph_release_folio,
-	.direct_IO = noop_direct_IO,
  };
static void ceph_block_sigs(sigset_t *oldset)
diff --git a/fs/ceph/file.c b/fs/ceph/file.c
index f4d8bf7dec88a8..314c5d5971bf4a 100644
--- a/fs/ceph/file.c
+++ b/fs/ceph/file.c
@@ -368,6 +368,8 @@ int ceph_open(struct inode *inode, struct file *file)
  	flags = file->f_flags & ~(O_CREAT|O_EXCL);
  	if (S_ISDIR(inode->i_mode))
  		flags = O_DIRECTORY;  /* mds likes to know */
+	if (S_ISREG(inode->i_mode))

BTW, the commit a2ad63daa88b ("VFS: add FMODE_CAN_ODIRECT file flag") doesn't check the S_ISREG, and I couldn't see this commit and NFS confine it to regular files, is that okay ?

Thanks

- Xiubo


+		file->f_mode |= FMODE_CAN_ODIRECT;
dout("open inode %p ino %llx.%llx file %p flags %d (%d)\n", inode,
  	     ceph_vinop(inode), file, flags, file->f_flags);




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Ceph Dev]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux