From c7f1a96010fd2243fa0b301543862e70784f7b35 Mon Sep 17 00:00:00 2001 From: "John L. Hammond" Date: Tue, 10 Mar 2015 09:35:56 -0500 Subject: [PATCH] LU-6354 mdd: use cr_tfid on error in mdd_changelog_ns_store() In mdd_changelog_ns_store() use the target FID in the the changelog record for the CERROR() message rather than the sfid parameter (which may be NULL). Signed-off-by: John L. Hammond Change-Id: If493f982c4ec6ae36dc9703353c35a9f7b6b0d43 Reviewed-on: http://review.whamcloud.com/14027 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Niu Yawei Reviewed-by: Henri Doreau Reviewed-by: Oleg Drokin --- lustre/mdd/mdd_dir.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lustre/mdd/mdd_dir.c b/lustre/mdd/mdd_dir.c index 9d03d17..b472596 100644 --- a/lustre/mdd/mdd_dir.c +++ b/lustre/mdd/mdd_dir.c @@ -851,8 +851,10 @@ int mdd_changelog_ns_store(const struct lu_env *env, rc = mdd_changelog_store(env, mdd, rec, handle); if (rc < 0) { - CERROR("changelog failed: rc=%d, op%d %s c"DFID" p"DFID"\n", - rc, type, tname->ln_name, PFID(sfid), PFID(tpfid)); + CERROR("%s: cannot store changelog record: type = %d, " + "name = '%s', t = "DFID", p = "DFID": rc = %d\n", + mdd2obd_dev(mdd)->obd_name, type, tname->ln_name, + PFID(&rec->cr.cr_tfid), PFID(&rec->cr.cr_pfid), rc); return -EFAULT; } -- 1.8.3.1