diff options
| author |   Trond Myklebust <trondmy@gmail.com> | 2018-08-21 10:25:34 -0400 |
|---|---|---|
| committer |   Anna Schumaker <Anna.Schumaker@Netapp.com> | 2018-08-21 13:39:08 -0400 |
| commit | 0af4c8be97a14d1df8a78b4993a09e8dff545a18 (patch) | |
| tree | 0a4917fa2cd8ea7b3ddfac84f0cc8730af19a757 /fs/nfs/pnfs.c | |
| parent | pNFS/flexfiles: ff_layout_pg_init_read should exit on error (diff) | |
| download | linux-dev-0af4c8be97a14d1df8a78b4993a09e8dff545a18.tar.xz linux-dev-0af4c8be97a14d1df8a78b4993a09e8dff545a18.zip | |
pNFS: Remove unwanted optimisation of layoutget
If we knew that the file was empty, we wouldn't be asking for a layout.
Any optimisation here is already done before calling pnfs_update_layout().
As it stands, we sometimes end up doing an unnecessary inband read to
the MDS even when holding a layout.
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/nfs/pnfs.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c index e11e666ea477..e8f232de484f 100644 --- a/fs/nfs/pnfs.c +++ b/fs/nfs/pnfs.c @@ -1823,12 +1823,6 @@ pnfs_update_layout(struct inode *ino, goto out; } - if (iomode == IOMODE_READ && i_size_read(ino) == 0) { - trace_pnfs_update_layout(ino, pos, count, iomode, lo, lseg, - PNFS_UPDATE_LAYOUT_RD_ZEROLEN); - goto out; - } - if (pnfs_within_mdsthreshold(ctx, ino, iomode)) { trace_pnfs_update_layout(ino, pos, count, iomode, lo, lseg, PNFS_UPDATE_LAYOUT_MDSTHRESH); |