staging/lustre/utils: missing setting echo seq for getattr/setattr
authorwang di <di.wang@intel.com>
Mon, 3 Jun 2013 13:40:41 +0000 (21:40 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 3 Jun 2013 18:26:34 +0000 (11:26 -0700)
It should set echo seq before do echo getattr/setattr, otherwise
echo_client will regard it as the object with MDT0 sequence.

[picked echo client part for upstream kernel submission]
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3267
Lustre-change: http://review.whamcloud.com/6263
Signed-off-by: wang di <di.wang@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Signed-off-by: Peng Tao <tao.peng@emc.com>
Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/obdecho/echo_client.c

index 0545d1666841172a050619200338261c8a73e3f3..bc58789c3799cc748cefc693f1bb1cfb8800fa36 100644 (file)
@@ -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)