Commit 629dc870 authored by Andy Adamson's avatar Andy Adamson Committed by Anna Schumaker

NFS store nfs4_deviceid in struct nfs4_filelayout_segment

In preparation for moving the filelayout getdeviceinfo call from
filelayout_alloc_lseg called by pnfs_process_layout
Signed-off-by: default avatarAndy Adamson <andros@netapp.com>
Signed-off-by: default avatarAnna Schumaker <Anna.Schumaker@Netapp.com>
parent 551afbb8
...@@ -572,7 +572,6 @@ static int ...@@ -572,7 +572,6 @@ static int
filelayout_check_layout(struct pnfs_layout_hdr *lo, filelayout_check_layout(struct pnfs_layout_hdr *lo,
struct nfs4_filelayout_segment *fl, struct nfs4_filelayout_segment *fl,
struct nfs4_layoutget_res *lgr, struct nfs4_layoutget_res *lgr,
struct nfs4_deviceid *id,
gfp_t gfp_flags) gfp_t gfp_flags)
{ {
struct nfs4_deviceid_node *d; struct nfs4_deviceid_node *d;
...@@ -602,7 +601,7 @@ filelayout_check_layout(struct pnfs_layout_hdr *lo, ...@@ -602,7 +601,7 @@ filelayout_check_layout(struct pnfs_layout_hdr *lo,
} }
/* find and reference the deviceid */ /* find and reference the deviceid */
d = nfs4_find_get_deviceid(NFS_SERVER(lo->plh_inode), id, d = nfs4_find_get_deviceid(NFS_SERVER(lo->plh_inode), &fl->deviceid,
lo->plh_lc_cred, gfp_flags); lo->plh_lc_cred, gfp_flags);
if (d == NULL) if (d == NULL)
goto out; goto out;
...@@ -657,7 +656,6 @@ static int ...@@ -657,7 +656,6 @@ static int
filelayout_decode_layout(struct pnfs_layout_hdr *flo, filelayout_decode_layout(struct pnfs_layout_hdr *flo,
struct nfs4_filelayout_segment *fl, struct nfs4_filelayout_segment *fl,
struct nfs4_layoutget_res *lgr, struct nfs4_layoutget_res *lgr,
struct nfs4_deviceid *id,
gfp_t gfp_flags) gfp_t gfp_flags)
{ {
struct xdr_stream stream; struct xdr_stream stream;
...@@ -682,9 +680,9 @@ filelayout_decode_layout(struct pnfs_layout_hdr *flo, ...@@ -682,9 +680,9 @@ filelayout_decode_layout(struct pnfs_layout_hdr *flo,
if (unlikely(!p)) if (unlikely(!p))
goto out_err; goto out_err;
memcpy(id, p, sizeof(*id)); memcpy(&fl->deviceid, p, sizeof(fl->deviceid));
p += XDR_QUADLEN(NFS4_DEVICEID4_SIZE); p += XDR_QUADLEN(NFS4_DEVICEID4_SIZE);
nfs4_print_deviceid(id); nfs4_print_deviceid(&fl->deviceid);
nfl_util = be32_to_cpup(p++); nfl_util = be32_to_cpup(p++);
if (nfl_util & NFL4_UFLG_COMMIT_THRU_MDS) if (nfl_util & NFL4_UFLG_COMMIT_THRU_MDS)
...@@ -831,15 +829,14 @@ filelayout_alloc_lseg(struct pnfs_layout_hdr *layoutid, ...@@ -831,15 +829,14 @@ filelayout_alloc_lseg(struct pnfs_layout_hdr *layoutid,
{ {
struct nfs4_filelayout_segment *fl; struct nfs4_filelayout_segment *fl;
int rc; int rc;
struct nfs4_deviceid id;
dprintk("--> %s\n", __func__); dprintk("--> %s\n", __func__);
fl = kzalloc(sizeof(*fl), gfp_flags); fl = kzalloc(sizeof(*fl), gfp_flags);
if (!fl) if (!fl)
return NULL; return NULL;
rc = filelayout_decode_layout(layoutid, fl, lgr, &id, gfp_flags); rc = filelayout_decode_layout(layoutid, fl, lgr, gfp_flags);
if (rc != 0 || filelayout_check_layout(layoutid, fl, lgr, &id, gfp_flags)) { if (rc != 0 || filelayout_check_layout(layoutid, fl, lgr, gfp_flags)) {
_filelayout_free_lseg(fl); _filelayout_free_lseg(fl);
return NULL; return NULL;
} }
......
...@@ -61,6 +61,7 @@ struct nfs4_filelayout_segment { ...@@ -61,6 +61,7 @@ struct nfs4_filelayout_segment {
u32 stripe_unit; u32 stripe_unit;
u32 first_stripe_index; u32 first_stripe_index;
u64 pattern_offset; u64 pattern_offset;
struct nfs4_deviceid deviceid;
struct nfs4_file_layout_dsaddr *dsaddr; /* Point to GETDEVINFO data */ struct nfs4_file_layout_dsaddr *dsaddr; /* Point to GETDEVINFO data */
unsigned int num_fh; unsigned int num_fh;
struct nfs_fh **fh_array; struct nfs_fh **fh_array;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment