Re: [PATCH v2] flow control for WRITE requests

Trond Myklebust
2009-06-09T23:05:23Z

[-- Attachment #1: Type: text/plain, Size: 932 bytes --]

On Tue, 2009-06-09 at 18:32 -0400, Peter Staubach wrote:
> I still need to move this along.

Sorry, it has been a long week at home (state championships,
graduation...).

I did promise to send a dump of the state of the fstatat() stuff from
LSF (see attachments).

As for the patch you posted, I did have comments that haven't really
been addressed. As I said, I certainly don't see the need to have
write() wait for writebacks to complete. I also don't accept that we
need to treat random writes as fundamentally different from serial
writes.
I'm currently inclining towards adding a switch to turn off strict posix
behaviour. There weren't too many people asking for it earlier, and
there aren't that many applications out there that are sensitive to the
exact mtime. Samba and backup applications are the major exceptions to
that rule, but you don't really run those on top of NFS clients if you
can avoid it...

Cheers
Trond

[-- Attachment #2: 1 --]
[-- Type: application/mbox, Size: 19263 bytes --]

[-- Attachment #3: 2 --]
[-- Type: application/mbox, Size: 16084 bytes --]

[-- Attachment #4: 3 --]
[-- Type: application/mbox, Size: 1953 bytes --]