pNFS: Remove unwanted optimisation of layoutget
authorTrond Myklebust <trondmy@gmail.com>
Tue, 21 Aug 2018 14:25:34 +0000 (10:25 -0400)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Tue, 21 Aug 2018 17:39:08 +0000 (13:39 -0400)
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>
fs/nfs/pnfs.c

index e11e666ea4774a27c0c47bd7d8adeea091eacd70..e8f232de484f4d591666c6789b9932404e6fc5cb 100644 (file)
@@ -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);