NFSv4.1: remove GETATTR from ds writes
Any WRITE compound directed to a data server needs to have the GETATTR calls suppressed. Signed-off-by: Fred Isaman <iisaman@netapp.com> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
This commit is contained in:
committed by
Trond Myklebust
parent
0382b74409
commit
7ffd10640d
@@ -3149,7 +3149,11 @@ static void nfs4_proc_write_setup(struct nfs_write_data *data, struct rpc_messag
|
|||||||
{
|
{
|
||||||
struct nfs_server *server = NFS_SERVER(data->inode);
|
struct nfs_server *server = NFS_SERVER(data->inode);
|
||||||
|
|
||||||
data->args.bitmask = server->cache_consistency_bitmask;
|
if (data->lseg) {
|
||||||
|
data->args.bitmask = NULL;
|
||||||
|
data->res.fattr = NULL;
|
||||||
|
} else
|
||||||
|
data->args.bitmask = server->cache_consistency_bitmask;
|
||||||
if (!data->write_done_cb)
|
if (!data->write_done_cb)
|
||||||
data->write_done_cb = nfs4_write_done_cb;
|
data->write_done_cb = nfs4_write_done_cb;
|
||||||
data->res.server = server;
|
data->res.server = server;
|
||||||
|
@@ -2275,7 +2275,8 @@ static void nfs4_xdr_enc_write(struct rpc_rqst *req, struct xdr_stream *xdr,
|
|||||||
encode_putfh(xdr, args->fh, &hdr);
|
encode_putfh(xdr, args->fh, &hdr);
|
||||||
encode_write(xdr, args, &hdr);
|
encode_write(xdr, args, &hdr);
|
||||||
req->rq_snd_buf.flags |= XDRBUF_WRITE;
|
req->rq_snd_buf.flags |= XDRBUF_WRITE;
|
||||||
encode_getfattr(xdr, args->bitmask, &hdr);
|
if (args->bitmask)
|
||||||
|
encode_getfattr(xdr, args->bitmask, &hdr);
|
||||||
encode_nops(&hdr);
|
encode_nops(&hdr);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -5694,8 +5695,9 @@ static int nfs4_xdr_dec_write(struct rpc_rqst *rqstp, struct xdr_stream *xdr,
|
|||||||
status = decode_write(xdr, res);
|
status = decode_write(xdr, res);
|
||||||
if (status)
|
if (status)
|
||||||
goto out;
|
goto out;
|
||||||
decode_getfattr(xdr, res->fattr, res->server,
|
if (res->fattr)
|
||||||
!RPC_IS_ASYNC(rqstp->rq_task));
|
decode_getfattr(xdr, res->fattr, res->server,
|
||||||
|
!RPC_IS_ASYNC(rqstp->rq_task));
|
||||||
if (!status)
|
if (!status)
|
||||||
status = res->count;
|
status = res->count;
|
||||||
out:
|
out:
|
||||||
|
Reference in New Issue
Block a user