diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-10-15 14:58:04 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-11-04 14:43:39 -0500 |
commit | bc5a89b337ee4b2fa6f577e7e1220d8c1ece71fc (patch) | |
tree | 6880127e805e1739dd00b2f8b5e65dbcff699567 /fs/nfs/pnfs.c | |
parent | eba24e1fe57df4e4cdee58af940f762eb336a113 (diff) |
NFSv4.1: Remove assertion BUG_ON()s from the files and generic layout code
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/pnfs.c')
-rw-r--r-- | fs/nfs/pnfs.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c index dcbc9b20474b..e7165d915362 100644 --- a/fs/nfs/pnfs.c +++ b/fs/nfs/pnfs.c | |||
@@ -634,7 +634,6 @@ send_layoutget(struct pnfs_layout_hdr *lo, | |||
634 | 634 | ||
635 | dprintk("--> %s\n", __func__); | 635 | dprintk("--> %s\n", __func__); |
636 | 636 | ||
637 | BUG_ON(ctx == NULL); | ||
638 | lgp = kzalloc(sizeof(*lgp), gfp_flags); | 637 | lgp = kzalloc(sizeof(*lgp), gfp_flags); |
639 | if (lgp == NULL) | 638 | if (lgp == NULL) |
640 | return NULL; | 639 | return NULL; |
@@ -1115,7 +1114,6 @@ pnfs_update_layout(struct inode *ino, | |||
1115 | * chance of a CB_LAYOUTRECALL(FILE) coming in. | 1114 | * chance of a CB_LAYOUTRECALL(FILE) coming in. |
1116 | */ | 1115 | */ |
1117 | spin_lock(&clp->cl_lock); | 1116 | spin_lock(&clp->cl_lock); |
1118 | BUG_ON(!list_empty(&lo->plh_layouts)); | ||
1119 | list_add_tail(&lo->plh_layouts, &server->layouts); | 1117 | list_add_tail(&lo->plh_layouts, &server->layouts); |
1120 | spin_unlock(&clp->cl_lock); | 1118 | spin_unlock(&clp->cl_lock); |
1121 | } | 1119 | } |
@@ -1211,7 +1209,7 @@ pnfs_generic_pg_init_read(struct nfs_pageio_descriptor *pgio, struct nfs_page *r | |||
1211 | { | 1209 | { |
1212 | u64 rd_size = req->wb_bytes; | 1210 | u64 rd_size = req->wb_bytes; |
1213 | 1211 | ||
1214 | BUG_ON(pgio->pg_lseg != NULL); | 1212 | WARN_ON_ONCE(pgio->pg_lseg != NULL); |
1215 | 1213 | ||
1216 | if (req->wb_offset != req->wb_pgbase) { | 1214 | if (req->wb_offset != req->wb_pgbase) { |
1217 | nfs_pageio_reset_read_mds(pgio); | 1215 | nfs_pageio_reset_read_mds(pgio); |
@@ -1240,7 +1238,7 @@ void | |||
1240 | pnfs_generic_pg_init_write(struct nfs_pageio_descriptor *pgio, | 1238 | pnfs_generic_pg_init_write(struct nfs_pageio_descriptor *pgio, |
1241 | struct nfs_page *req, u64 wb_size) | 1239 | struct nfs_page *req, u64 wb_size) |
1242 | { | 1240 | { |
1243 | BUG_ON(pgio->pg_lseg != NULL); | 1241 | WARN_ON_ONCE(pgio->pg_lseg != NULL); |
1244 | 1242 | ||
1245 | if (req->wb_offset != req->wb_pgbase) { | 1243 | if (req->wb_offset != req->wb_pgbase) { |
1246 | nfs_pageio_reset_write_mds(pgio); | 1244 | nfs_pageio_reset_write_mds(pgio); |