LU-15025 quota: stale edquot after clearing limits

When hard and soft limit set to 0, lqe enforced flag is also
set to false. As qmt_adjust_qunit does not handle not enforced
lqes, edquot set to the pool continues to be true and a user
gets -EDQUOT even if all pool limits are cleared. This was ok
for global pool lqe as since it turned off, zero limits are
sent to OSTs causing OSTs to release all granted space and
avoid EDQUOT. Fix this for PQ - set edquot and qunit to zero,
since appropriate lqe becomes "not enforced".

HPE-bug-id: LUS-10146
Change-Id: I1e08929bae7e1b37b1e8cbbc44859a786b5fb090
Reviewed-on: https://es-gerrit.dev.cray.com/158915/
Reviewed-by: Alexander Boyko <alexander.boyko@hpe.com>
Tested-by: Elena Gryaznova <c17455@cray.com>
Tested-by: Jenkins Build User <nssreleng@cray.com>
Signed-off-by: Sergey Cheremencev <sergey.cheremencev@hpe.com>
Reviewed-on: https://review.whamcloud.com/45000
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Shaun Tancheff <shaun.tancheff@hpe.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>

diff --git a/lustre/quota/qmt_handler.c b/lustre/quota/qmt_handler.c
index 28b860e..2ed9051 100644 (file)
--- a/lustre/quota/qmt_handler.c
+++ b/lustre/quota/qmt_handler.c
@@ -220,10 +220,18 @@ quota_set:
 
                /* change enforced status based on new parameters */
                if (lqe->lqe_id.qid_uid == 0 || (lqe->lqe_hardlimit == 0 &&
-                   lqe->lqe_softlimit == 0))
-                       lqe->lqe_enforced = false;
-               else
+                   lqe->lqe_softlimit == 0)) {
+                       if (lqe->lqe_enforced) {
+                               lqe->lqe_enforced = false;
+                               /* Clear qunit and edquot as lqe_adjust_edquot
+                                * does not handle not enforced lqes */
+                               lqe->lqe_edquot = 0;
+                               lqe->lqe_qunit = 0;
+                               need_id_notify = true;
+                       }
+               } else {
                        lqe->lqe_enforced = true;
+               }
 
                dirtied = true;
        }