aboutsummaryrefslogtreecommitdiffstats
path: root/fs/nfs/pnfs.c
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@lst.de>2014-08-21 12:09:25 -0400
committerTrond Myklebust <trond.myklebust@primarydata.com>2014-09-10 15:47:01 -0400
commit5f919c9f10c1cf821ee5f414683214a361a1b98c (patch)
tree401fab5447d465efa72b750905ea31868a7d4a5b /fs/nfs/pnfs.c
parent47abadefad213bb7de9592d2e09a8bd282ddc3de (diff)
pnfs: allow splicing pre-encoded pages into the layoutcommit args
Currently there is no XDR buffer space allocated for the per-layout driver layoutcommit payload, which leads to server buffer overflows in the blocklayout driver even under simple workloads. As we can't do per-layout sizes for XDR operations we'll have to splice a previously encoded list of pages into the XDR stream, similar to how we handle ACL buffers. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'fs/nfs/pnfs.c')
-rw-r--r--fs/nfs/pnfs.c15
1 files changed, 15 insertions, 0 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c
index 57b5728e0b8e..8827ab130ed3 100644
--- a/fs/nfs/pnfs.c
+++ b/fs/nfs/pnfs.c
@@ -1854,6 +1854,7 @@ void pnfs_cleanup_layoutcommit(struct nfs4_layoutcommit_data *data)
1854int 1854int
1855pnfs_layoutcommit_inode(struct inode *inode, bool sync) 1855pnfs_layoutcommit_inode(struct inode *inode, bool sync)
1856{ 1856{
1857 struct pnfs_layoutdriver_type *ld = NFS_SERVER(inode)->pnfs_curr_ld;
1857 struct nfs4_layoutcommit_data *data; 1858 struct nfs4_layoutcommit_data *data;
1858 struct nfs_inode *nfsi = NFS_I(inode); 1859 struct nfs_inode *nfsi = NFS_I(inode);
1859 loff_t end_pos; 1860 loff_t end_pos;
@@ -1904,6 +1905,20 @@ pnfs_layoutcommit_inode(struct inode *inode, bool sync)
1904 data->args.lastbytewritten = end_pos - 1; 1905 data->args.lastbytewritten = end_pos - 1;
1905 data->res.server = NFS_SERVER(inode); 1906 data->res.server = NFS_SERVER(inode);
1906 1907
1908 if (ld->prepare_layoutcommit) {
1909 status = ld->prepare_layoutcommit(&data->args);
1910 if (status) {
1911 spin_lock(&inode->i_lock);
1912 if (end_pos < nfsi->layout->plh_lwb)
1913 nfsi->layout->plh_lwb = end_pos;
1914 spin_unlock(&inode->i_lock);
1915 put_rpccred(data->cred);
1916 set_bit(NFS_INO_LAYOUTCOMMIT, &nfsi->flags);
1917 goto clear_layoutcommitting;
1918 }
1919 }
1920
1921
1907 status = nfs4_proc_layoutcommit(data, sync); 1922 status = nfs4_proc_layoutcommit(data, sync);
1908out: 1923out:
1909 if (status) 1924 if (status)