Commit 3132e49e authored by Jeff Layton's avatar Jeff Layton Committed by Anna Schumaker

pnfs: track multiple layout types in fsinfo structure

Current NFSv4.1/pNFS client assumes that MDS supports only one layout
type. While it's true for most existing servers, nevertheless, this can
be change in the near future.

For now, this patch just plumbs in the ability to track a list of
layouts in the fsinfo structure. The existing behavior of the client
is preserved, by having it just select the first entry in the list.
Signed-off-by: default avatarTigran Mkrtchyan <tigran.mkrtchyan@desy.de>
Signed-off-by: default avatarJeff Layton <jlayton@poochiereds.net>
Reviewed-by: default avatarJ. Bruce Fields <bfields@fieldses.org>
Signed-off-by: default avatarAnna Schumaker <Anna.Schumaker@Netapp.com>
parent 2813b626
...@@ -785,7 +785,7 @@ int nfs_probe_fsinfo(struct nfs_server *server, struct nfs_fh *mntfh, struct nfs ...@@ -785,7 +785,7 @@ int nfs_probe_fsinfo(struct nfs_server *server, struct nfs_fh *mntfh, struct nfs
} }
fsinfo.fattr = fattr; fsinfo.fattr = fattr;
fsinfo.layouttype = 0; memset(fsinfo.layouttype, 0, sizeof(fsinfo.layouttype));
error = clp->rpc_ops->fsinfo(server, mntfh, &fsinfo); error = clp->rpc_ops->fsinfo(server, mntfh, &fsinfo);
if (error < 0) if (error < 0)
goto out_error; goto out_error;
......
...@@ -4725,14 +4725,13 @@ static int decode_getfattr(struct xdr_stream *xdr, struct nfs_fattr *fattr, ...@@ -4725,14 +4725,13 @@ static int decode_getfattr(struct xdr_stream *xdr, struct nfs_fattr *fattr,
} }
/* /*
* Decode potentially multiple layout types. Currently we only support * Decode potentially multiple layout types.
* one layout driver per file system.
*/ */
static int decode_first_pnfs_layout_type(struct xdr_stream *xdr, static int decode_pnfs_layout_types(struct xdr_stream *xdr,
uint32_t *layouttype) uint32_t *layouttype)
{ {
__be32 *p; __be32 *p;
int num; uint32_t num, i;
p = xdr_inline_decode(xdr, 4); p = xdr_inline_decode(xdr, 4);
if (unlikely(!p)) if (unlikely(!p))
...@@ -4741,18 +4740,17 @@ static int decode_first_pnfs_layout_type(struct xdr_stream *xdr, ...@@ -4741,18 +4740,17 @@ static int decode_first_pnfs_layout_type(struct xdr_stream *xdr,
/* pNFS is not supported by the underlying file system */ /* pNFS is not supported by the underlying file system */
if (num == 0) { if (num == 0) {
*layouttype = 0;
return 0; return 0;
} }
if (num > 1) if (num > NFS_MAX_LAYOUT_TYPES)
printk(KERN_INFO "NFS: %s: Warning: Multiple pNFS layout " printk(KERN_INFO "NFS: %s: Warning: Too many (%d) pNFS layout types\n", __func__, num);
"drivers per filesystem not supported\n", __func__);
/* Decode and set first layout type, move xdr->p past unused types */ /* Decode and set first layout type, move xdr->p past unused types */
p = xdr_inline_decode(xdr, num * 4); p = xdr_inline_decode(xdr, num * 4);
if (unlikely(!p)) if (unlikely(!p))
goto out_overflow; goto out_overflow;
*layouttype = be32_to_cpup(p); for(i = 0; i < num && i < NFS_MAX_LAYOUT_TYPES; i++)
layouttype[i] = be32_to_cpup(p++);
return 0; return 0;
out_overflow: out_overflow:
print_overflow_msg(__func__, xdr); print_overflow_msg(__func__, xdr);
...@@ -4772,10 +4770,9 @@ static int decode_attr_pnfstype(struct xdr_stream *xdr, uint32_t *bitmap, ...@@ -4772,10 +4770,9 @@ static int decode_attr_pnfstype(struct xdr_stream *xdr, uint32_t *bitmap,
if (unlikely(bitmap[1] & (FATTR4_WORD1_FS_LAYOUT_TYPES - 1U))) if (unlikely(bitmap[1] & (FATTR4_WORD1_FS_LAYOUT_TYPES - 1U)))
return -EIO; return -EIO;
if (bitmap[1] & FATTR4_WORD1_FS_LAYOUT_TYPES) { if (bitmap[1] & FATTR4_WORD1_FS_LAYOUT_TYPES) {
status = decode_first_pnfs_layout_type(xdr, layouttype); status = decode_pnfs_layout_types(xdr, layouttype);
bitmap[1] &= ~FATTR4_WORD1_FS_LAYOUT_TYPES; bitmap[1] &= ~FATTR4_WORD1_FS_LAYOUT_TYPES;
} else }
*layouttype = 0;
return status; return status;
} }
...@@ -4856,7 +4853,7 @@ static int decode_fsinfo(struct xdr_stream *xdr, struct nfs_fsinfo *fsinfo) ...@@ -4856,7 +4853,7 @@ static int decode_fsinfo(struct xdr_stream *xdr, struct nfs_fsinfo *fsinfo)
status = decode_attr_time_delta(xdr, bitmap, &fsinfo->time_delta); status = decode_attr_time_delta(xdr, bitmap, &fsinfo->time_delta);
if (status != 0) if (status != 0)
goto xdr_error; goto xdr_error;
status = decode_attr_pnfstype(xdr, bitmap, &fsinfo->layouttype); status = decode_attr_pnfstype(xdr, bitmap, fsinfo->layouttype);
if (status != 0) if (status != 0)
goto xdr_error; goto xdr_error;
......
...@@ -102,32 +102,37 @@ unset_pnfs_layoutdriver(struct nfs_server *nfss) ...@@ -102,32 +102,37 @@ unset_pnfs_layoutdriver(struct nfs_server *nfss)
* Try to set the server's pnfs module to the pnfs layout type specified by id. * Try to set the server's pnfs module to the pnfs layout type specified by id.
* Currently only one pNFS layout driver per filesystem is supported. * Currently only one pNFS layout driver per filesystem is supported.
* *
* @id layout type. Zero (illegal layout type) indicates pNFS not in use. * @ids array of layout types supported by MDS.
*/ */
void void
set_pnfs_layoutdriver(struct nfs_server *server, const struct nfs_fh *mntfh, set_pnfs_layoutdriver(struct nfs_server *server, const struct nfs_fh *mntfh,
u32 id) u32 *ids)
{ {
struct pnfs_layoutdriver_type *ld_type = NULL; struct pnfs_layoutdriver_type *ld_type = NULL;
u32 id;
if (id == 0)
goto out_no_driver;
if (!(server->nfs_client->cl_exchange_flags & if (!(server->nfs_client->cl_exchange_flags &
(EXCHGID4_FLAG_USE_NON_PNFS | EXCHGID4_FLAG_USE_PNFS_MDS))) { (EXCHGID4_FLAG_USE_NON_PNFS | EXCHGID4_FLAG_USE_PNFS_MDS))) {
printk(KERN_ERR "NFS: %s: id %u cl_exchange_flags 0x%x\n", printk(KERN_ERR "NFS: %s: cl_exchange_flags 0x%x\n",
__func__, id, server->nfs_client->cl_exchange_flags); __func__, server->nfs_client->cl_exchange_flags);
goto out_no_driver; goto out_no_driver;
} }
id = ids[0];
if (!id)
goto out_no_driver;
ld_type = find_pnfs_driver(id); ld_type = find_pnfs_driver(id);
if (!ld_type) { if (!ld_type) {
request_module("%s-%u", LAYOUT_NFSV4_1_MODULE_PREFIX, id); request_module("%s-%u", LAYOUT_NFSV4_1_MODULE_PREFIX, id);
ld_type = find_pnfs_driver(id); ld_type = find_pnfs_driver(id);
}
if (!ld_type) { if (!ld_type) {
dprintk("%s: No pNFS module found for %u.\n", dprintk("%s: No pNFS module found for %u.\n", __func__, id);
__func__, id);
goto out_no_driver; goto out_no_driver;
} }
}
server->pnfs_curr_ld = ld_type; server->pnfs_curr_ld = ld_type;
if (ld_type->set_layoutdriver if (ld_type->set_layoutdriver
&& ld_type->set_layoutdriver(server, mntfh)) { && ld_type->set_layoutdriver(server, mntfh)) {
......
...@@ -236,7 +236,7 @@ void pnfs_get_layout_hdr(struct pnfs_layout_hdr *lo); ...@@ -236,7 +236,7 @@ void pnfs_get_layout_hdr(struct pnfs_layout_hdr *lo);
void pnfs_put_lseg(struct pnfs_layout_segment *lseg); void pnfs_put_lseg(struct pnfs_layout_segment *lseg);
void pnfs_put_lseg_locked(struct pnfs_layout_segment *lseg); void pnfs_put_lseg_locked(struct pnfs_layout_segment *lseg);
void set_pnfs_layoutdriver(struct nfs_server *, const struct nfs_fh *, u32); void set_pnfs_layoutdriver(struct nfs_server *, const struct nfs_fh *, u32 *);
void unset_pnfs_layoutdriver(struct nfs_server *); void unset_pnfs_layoutdriver(struct nfs_server *);
void pnfs_generic_pg_init_read(struct nfs_pageio_descriptor *, struct nfs_page *); void pnfs_generic_pg_init_read(struct nfs_pageio_descriptor *, struct nfs_page *);
int pnfs_generic_pg_readpages(struct nfs_pageio_descriptor *desc); int pnfs_generic_pg_readpages(struct nfs_pageio_descriptor *desc);
...@@ -657,7 +657,7 @@ pnfs_wait_on_layoutreturn(struct inode *ino, struct rpc_task *task) ...@@ -657,7 +657,7 @@ pnfs_wait_on_layoutreturn(struct inode *ino, struct rpc_task *task)
} }
static inline void set_pnfs_layoutdriver(struct nfs_server *s, static inline void set_pnfs_layoutdriver(struct nfs_server *s,
const struct nfs_fh *mntfh, u32 id) const struct nfs_fh *mntfh, u32 *ids)
{ {
} }
......
...@@ -124,6 +124,11 @@ struct nfs_fattr { ...@@ -124,6 +124,11 @@ struct nfs_fattr {
| NFS_ATTR_FATTR_SPACE_USED \ | NFS_ATTR_FATTR_SPACE_USED \
| NFS_ATTR_FATTR_V4_SECURITY_LABEL) | NFS_ATTR_FATTR_V4_SECURITY_LABEL)
/*
* Maximal number of supported layout drivers.
*/
#define NFS_MAX_LAYOUT_TYPES 8
/* /*
* Info on the file system * Info on the file system
*/ */
...@@ -139,7 +144,7 @@ struct nfs_fsinfo { ...@@ -139,7 +144,7 @@ struct nfs_fsinfo {
__u64 maxfilesize; __u64 maxfilesize;
struct timespec time_delta; /* server time granularity */ struct timespec time_delta; /* server time granularity */
__u32 lease_time; /* in seconds */ __u32 lease_time; /* in seconds */
__u32 layouttype; /* supported pnfs layout driver */ __u32 layouttype[NFS_MAX_LAYOUT_TYPES]; /* supported pnfs layout driver */
__u32 blksize; /* preferred pnfs io block size */ __u32 blksize; /* preferred pnfs io block size */
__u32 clone_blksize; /* granularity of a CLONE operation */ __u32 clone_blksize; /* granularity of a CLONE operation */
}; };
......
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