[PATCH 04/23] staging/lustre/utils: missing setting echo seq for getattr/setattr

Peng Tao bergwolf at gmail.com
Fri May 31 17:22:54 UTC 2013


From: wang di <di.wang at intel.com>

It should set echo seq before do echo getattr/setattr, otherwise
echo_client will regard it as the object with MDT0 sequence.

Intel-bug-id: LU-3267
Lustre-commit: 3e2d4d2d7477dd338a87de103027d37865285e44
Lustre-change: http://review.whamcloud.com/6263
Signed-off-by: wang di <di.wang at intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger at intel.com>
Reviewed-by: Lai Siyao <lai.siyao at intel.com>

[picked client side change and updated for upstream kernel submission]
Signed-off-by: Peng Tao <tao.peng at emc.com>
Signed-off-by: Andreas Dilger <andreas.dilger at intel.com>
---
 .../staging/lustre/lustre/obdecho/echo_client.c    |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/lustre/lustre/obdecho/echo_client.c b/drivers/staging/lustre/lustre/obdecho/echo_client.c
index 0545d16..bc58789 100644
--- a/drivers/staging/lustre/lustre/obdecho/echo_client.c
+++ b/drivers/staging/lustre/lustre/obdecho/echo_client.c
@@ -1081,8 +1081,9 @@ static struct echo_object *cl_echo_object_find(struct echo_device *d,
 	LASSERT(lsmp);
 	lsm = *lsmp;
 	LASSERT(lsm);
-	LASSERT(ostid_id(&lsm->lsm_oi) != 0);
-	LASSERT(ostid_seq(&lsm->lsm_oi) == FID_SEQ_ECHO);
+	LASSERTF(ostid_id(&lsm->lsm_oi) != 0, DOSTID"\n", POSTID(&lsm->lsm_oi));
+	LASSERTF(ostid_seq(&lsm->lsm_oi) == FID_SEQ_ECHO, DOSTID"\n",
+		 POSTID(&lsm->lsm_oi));
 
 	/* Never return an object if the obd is to be freed. */
 	if (echo_dev2cl(d)->cd_lu_dev.ld_obd->obd_stopping)
-- 
1.7.9.5




More information about the devel mailing list