[LU-11075] suspicious code in ldlm_prepare_lru_list()

Related — Wed, 5 Aug 2020 21:19:14 +0000

There is some suspicious code in ldlm_prepare_lru_list() which should be fixed:

static int ldlm_prepare_lru_list(struct ldlm_namespace *ns,
                                 struct list_head *cancels, int count, int max,
                                 enum ldlm_lru_flags lru_flags)
...
                        if (!ldlm_is_canceling(lock) || /* XXX '||' should be '&&' */
                            !ldlm_is_converting(lock))
                                break;
...
                if (result == LDLM_POLICY_SKIP_LOCK) {
                        lu_ref_del(&lock->l_reference, __func__, current);
                        LDLM_LOCK_RELEASE(lock); /* XXX release should follow if block. */
                        if (no_wait) {
                                spin_lock(&ns->ns_lock);
                                if (!list_empty(&lock->l_lru) &&
                                    lock->l_lru.prev == ns->ns_last_pos)
                                        ns->ns_last_pos = &lock->l_lru;
                                spin_unlock(&ns->ns_lock);
                        }
                        continue;
                }

Whamcloud Community JIRA

[LU-11075] suspicious code in ldlm_prepare_lru_list()