[pnfs] [PATCH 1/7] pnfs: BUG: spinlock held during call to setup_layoutcommit

Fred Isaman iisaman at citi.umich.edu
Mon Jun 9 16:29:02 EDT 2008


This just comments the bug.

Signed-off-by: Fred Isaman <iisaman at citi.umich.edu>
---
 fs/nfs/pnfs.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c
index afb3d71..f910033 100644
--- a/fs/nfs/pnfs.c
+++ b/fs/nfs/pnfs.c
@@ -1894,6 +1894,9 @@ pnfs_layoutcommit_setup(struct pnfs_layoutcommit_data *data, int sync)
 	 * TODO: We may want to avoid memory copies by delay this
 	 * until xdr time.
 	 */
+	/* BUG - a major purpose of this call is to set (and thus allocate)
+	 * data->args.new_layout, but pnfs_spinlock is currently held
+	 */
 	if (nfss->pnfs_curr_ld->ld_io_ops->setup_layoutcommit) {
 		result = nfss->pnfs_curr_ld->ld_io_ops->setup_layoutcommit(
 				nfsi->current_layout,
-- 
1.5.3.3



More information about the pNFS mailing list