[LU-6515] Always true check in mdd_migrate_create

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.12.0
• Labels: easy
• Severity: 3
• Assignee: WC Triage
• Reporter: Oleg Drokin

Description

smatch highlights this case in mdd_migrate_create():

        handle = mdd_trans_create(env, mdd);
        if (IS_ERR(handle))
                GOTO(out_free, rc = PTR_ERR(handle));

        /* Note: this transaction is part of migration, and it is not
         * the last step of migration, so we set th_local = 1 to avoid
         * update last rcvd for this transaction */
        handle->th_local = 1;
....
stop_trans:
        if (handle != NULL)
                mdd_trans_stop(env, mdd, rc, handle);
out_free:
...

handle, once allocated, is never freed or reassigned, so I feel like we should just drop that redundant check to avoid some extra confusion?

Issue Links

• is related to: LU-4684 DNE3: allow migrating DNE striped directory