Hi Trond, On Fri, 2012-02-10 at 16:48 +0000, Myklebust, Trond wrote: > On Fri, 2012-02-10 at 19:07 +1100, Harshula wrote: > > Do you see forcedirectio as a sharp object that someone could stab > > themselves with? > > Yes. It does lead to some very subtle POSIX violations. I'm trying out the alternatives. Your list of reasons were convincing. Thanks. > > If the NFS client only does cached async reads of a slowly growing file > > (tail), what's the problem? Is nfs_readpage_sync() gone forever, or > > could it be revived? > > It wouldn't help at all. The problem is the VM's handling of pages vs > the NFS handling of file size. > > The VM basically uses the file size in order to determine how much data > a page contains. If that file size changed between the instance we > finished the READ RPC call, and the instance the VM gets round to > locking the page again, reading the data and then checking the file > size, then the VM may end up copying data beyond the end of that > retrieved by the RPC call. nfs_readpage_sync() keeps doing rsize reads (or PAGE SIZE reads if rsize > PAGE SIZE) till the entire PAGE has been filled or EOF is hit. Since these are synchronous reads, the subsequent READ RPC call is not sent until the previous READ RPC reply arrives. Hence, the READ RPC reply contains the latest metadata about the file, from the NFS server, before deciding whether or not to do more READ RPC calls. That is not the case with the asynchronous READ RPC calls which are queued to be sent before the replies are received. This results in not READing enough data from the NFS server even when the READ RPC reply explicitly states that the file has grown. This mismatch of data and file size is then presented to the VM. If you look at nfs_readpage_sync() code, it does not worry about adjusting the number of bytes to read if it is past the *current* EOF. Only the async code adjusts the number of bytes to read if it is past the *current* EOF. Furthermore, testing showed that using -osync (while nfs_readpage_sync() existed) avoided the NULLs being presented to userspace. cya, # -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
