Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.

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

To: David Laight <David.Laight@xxxxxxxxxx>
Subject: Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
From: Chuck Lever III <chuck.lever@xxxxxxxxxx>
Date: Fri, 15 Dec 2023 19:35:57 +0000
Accept-language: en-US
Cc: Neil Brown <neilb@xxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>, Christian Brauner <brauner@xxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>, Oleg Nesterov <oleg@xxxxxxxxxx>, Jeff Layton <jlayton@xxxxxxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Linux NFS Mailing List <linux-nfs@xxxxxxxxxxxxxxx>
In-reply-to: <ac74bdb82e114d71b26864fe51f6433b@AcuMS.aculab.com>
Thread-index: AQHaKYg6Xsdfrw6D/0iqk7wx2wl73LCfe+UAgAOmxoCAAVXmgIAANc2AgAAN4ACAAAJCgIABpOgAgARSNwCAABMcAA==
Thread-topic: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.

> On Dec 15, 2023, at 1:27 PM, David Laight <David.Laight@xxxxxxxxxx> wrote:
> 
> ...
> 
> I'm also sure I remember that nfs wasn't supposed to respond to a write
> until it had issued the actual disk write - but maybe no one do that
> any more because it really is too slow.
> (Especially if the 'disk' is a USB stick.)

That rule applies only to NFSv2, which no-one should use any more.

NFSv3 and later use a two-phase write. Clients send a COMMIT after
an UNSTABLE WRITE to get the persistence guarantee.

--
Chuck Lever

References:
- [PATCH 0/3] nfsd: fully close all files in the nfsd threads
  - From: NeilBrown
- [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: NeilBrown
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: Chuck Lever
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: NeilBrown
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: Al Viro
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: NeilBrown
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: Al Viro
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: Al Viro
- Re: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: NeilBrown
- RE: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
  - From: David Laight

Prev by Date: RE: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
Next by Date: Re: [RFC] HACK: overlayfs: Optimize overlay/restore creds
Previous by thread: RE: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
Next by thread: RE: [PATCH 1/3] nfsd: use __fput_sync() to avoid delayed closing of files.
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]