From: Lai Siyao <lai.siyao@intel.com>
Date: Fri, 4 Aug 2017 15:16:46 +0000 (+0800)
Subject: LU-9725 quota: always deregister lwp
X-Git-Tag: 2.10.1-RC1~28
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=e53c0fbeefc1c29d7b5256c6a4cc6ead96ae41e8;p=fs%2Flustre-release.git

LU-9725 quota: always deregister lwp

qsd should always deregiter lwp upon finish no matter qsd_exp was
set before, otherwise the item will stay on the list, but qsd has
been freed.

Lustre-change: https://review.whamcloud.com/#/c/28356/
Lustre-commit: ce8ca7d3564439285a56982430f380354b697f68

Signed-off-by: Lai Siyao <lai.siyao@intel.com>
Change-Id: I0d6206f2f2bc8177d0aa35b350f534d85eab1c03
Reviewed-on: https://review.whamcloud.com/28357
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
---

diff --git a/lustre/quota/qsd_lib.c b/lustre/quota/qsd_lib.c
index 5478683..f4c37e6 100644
--- a/lustre/quota/qsd_lib.c
+++ b/lustre/quota/qsd_lib.c
@@ -550,11 +550,9 @@ void qsd_fini(const struct lu_env *env, struct qsd_instance *qsd)
 	for (qtype = USRQUOTA; qtype < LL_MAXQUOTAS; qtype++)
 		qsd_qtype_fini(env, qsd, qtype);
 
-	if (qsd->qsd_exp) {
-		/* deregister connection to the quota master */
-		qsd->qsd_exp_valid = false;
-		lustre_deregister_lwp_item(&qsd->qsd_exp);
-	}
+	/* deregister connection to the quota master */
+	qsd->qsd_exp_valid = false;
+	lustre_deregister_lwp_item(&qsd->qsd_exp);
 
 	/* release per-filesystem information */
 	if (qsd->qsd_fsinfo != NULL) {