File 0028-dx_dirs-stop-dx-insert-iteration-for-callb... of Package ocfs2-tools

Overview Repositories Revisions Requests Users Attributes Meta

File 0028-dx_dirs-stop-dx-insert-iteration-for-callback-error.patch of Package ocfs2-tools (Revision ceb0573c5b577ed31c10eaa51a7b3cb7)

Currently displaying revision ceb0573c5b577ed31c10eaa51a7b3cb7 , Show latest

From 1f49857cc5fa914ac0d3577f841b398421ca01a0 Mon Sep 17 00:00:00 2001
From: Coly Li <coly.li@suse.de>
Date: Mon, 26 Apr 2010 22:37:32 +0800
Subject: [PATCH 28/30] dx_dirs: stop dx insert iteration for callback error

Callback ocfs2_dx_dir_insert() may encounter memory alloc or I/O error.
These kind of errors can not return to caller of the callback directly,
so the dir block iteration for dx insert can not stop immediately when
such errors occure.

This patch adds a member 'errcode_t err' to struct dx_insert_ctxt, which
can catch the error and permit ocfs2_dx_dir_insert() returns
OCFS2_EXTENT_ERROR to it's caller to stop the iteration immediately.

Signed-off-by: Coly Li <coly.li@suse.de>
Cc: Mark Fasheh <mfasheh@suse.com>
Cc: Tao Ma <tao.ma@oracle.com>
---
 libocfs2/dir_indexed.c |   61 +++++++++++++++++++++++++++++------------------
 1 files changed, 38 insertions(+), 23 deletions(-)

diff --git a/libocfs2/dir_indexed.c b/libocfs2/dir_indexed.c
index 5f3db95..08c43b7 100644
--- a/libocfs2/dir_indexed.c
+++ b/libocfs2/dir_indexed.c
@@ -269,6 +269,7 @@ struct dx_insert_ctxt {
 	uint64_t dir_blkno;
 	uint64_t dx_root_blkno;
 	ocfs2_filesys *fs;
+	errcode_t err;
 };
 
 
@@ -1069,7 +1070,8 @@ static int ocfs2_dx_dir_insert(struct ocfs2_dir_entry *dentry,
 				char *buf,
 				void *priv_data)
 {
-	errcode_t ret = 0;
+	int ret = 0;
+	errcode_t err;
 	char *dx_buf = NULL;
 	char *dx_leaf_buf = NULL;
 	struct ocfs2_dx_root_block *dx_root = NULL;
@@ -1081,17 +1083,17 @@ static int ocfs2_dx_dir_insert(struct ocfs2_dir_entry *dentry,
 	uint64_t dx_root_blkno = ctxt->dx_root_blkno;
 	int write_dx_leaf = 0;
 
-	ret = ocfs2_malloc_block(fs->fs_io, &dx_buf);
-	if (ret)
-		goto out;
+	err = ocfs2_malloc_block(fs->fs_io, &dx_buf);
+	if (err)
+		goto set_err;
 
-	ret = ocfs2_malloc_block(fs->fs_io, &dx_leaf_buf);
-	if (ret)
-		goto out;
+	err = ocfs2_malloc_block(fs->fs_io, &dx_leaf_buf);
+	if (err)
+		goto set_err;
 
-	ret = ocfs2_read_dx_root(fs, dx_root_blkno, dx_buf);
-	if (ret)
-		goto out;
+	err = ocfs2_read_dx_root(fs, dx_root_blkno, dx_buf);
+	if (err)
+		goto set_err;
 
 	dx_root = (struct ocfs2_dx_root_block *)dx_buf;
 	memset(&lookup, 0, sizeof(struct ocfs2_dir_lookup_result));
@@ -1104,19 +1106,21 @@ static int ocfs2_dx_dir_insert(struct ocfs2_dir_entry *dentry,
 			goto insert_into_entries;
 		} else {
 			/* root block is full, expand it to an extent */
-			ret = ocfs2_expand_inline_dx_root(fs, dx_root);
-			if (ret)
-				goto out;
+			err = ocfs2_expand_inline_dx_root(fs, dx_root);
+			if (err)
+				goto set_err;
 		}
 	}
 
-	ret = ocfs2_find_dir_space_dx(fs, dx_root,
+	err = ocfs2_find_dir_space_dx(fs, dx_root,
 				dentry->name, dentry->name_len, &lookup);
-	if (ret)
-		goto out;
-	ret = ocfs2_read_dx_leaf(fs, lookup.dl_dx_leaf_blkno, dx_leaf_buf);
-	if (ret)
-		goto out;
+	if (err)
+		goto set_err;
+
+	err = ocfs2_read_dx_leaf(fs, lookup.dl_dx_leaf_blkno, dx_leaf_buf);
+	if (err)
+		goto set_err;
+
 	dx_leaf = (struct ocfs2_dx_leaf *)dx_leaf_buf;
 	entry_list = &dx_leaf->dl_list;
 	write_dx_leaf = 1;
@@ -1124,12 +1128,18 @@ static int ocfs2_dx_dir_insert(struct ocfs2_dir_entry *dentry,
 insert_into_entries:
 	ocfs2_dx_entry_list_insert(entry_list, &lookup.dl_hinfo, blocknr);
 	if (write_dx_leaf) {
-		ret = ocfs2_write_dx_leaf(fs, dx_leaf->dl_blkno, dx_leaf);
-		if (ret)
-			goto out;
+		err = ocfs2_write_dx_leaf(fs, dx_leaf->dl_blkno, dx_leaf);
+		if (err)
+			goto set_err;
 	}
 	dx_root->dr_num_entries += 1;
-	ret = ocfs2_write_dx_root(fs, dx_root_blkno, dx_buf);
+	err = ocfs2_write_dx_root(fs, dx_root_blkno, dx_buf);
+	if (!err)
+       		goto out;
+
+set_err:
+	ctxt->err = err;
+	ret = OCFS2_EXTENT_ERROR;
 out:
 	if (dx_leaf_buf)
 		ocfs2_free(&dx_leaf_buf);
@@ -1256,8 +1266,13 @@ errcode_t ocfs2_dx_dir_build(ocfs2_filesys *fs,
 	ctxt.dir_blkno = dir;
 	ctxt.dx_root_blkno = dr_blkno;
 	ctxt.fs = fs;
+	ctxt.err = 0;
 	ret = ocfs2_dir_iterate(fs, dir, 0, NULL,
 				ocfs2_dx_dir_insert,  &ctxt);
+	if (ctxt.err)
+		ret = ctxt.err;
+	if (ret)
+		goto out;
 
 	/* check quota for dx_leaf */
 	ret = ocfs2_read_dx_root(fs, dr_blkno, dx_buf);
-- 
1.7.0.2

Places

File 0028-dx_dirs-stop-dx-insert-iteration-for-callback-error.patch of Package ocfs2-tools (Revision ceb0573c5b577ed31c10eaa51a7b3cb7)

Places