[LU-9197] Incorrect return value for llog_cancel_rec()

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.10.0
• Affects Version/s: Lustre 2.8.0, Lustre 2.9.0, Lustre 2.10.0
• Severity: 3
• Assignee: Di Wang
• Reporter: Di Wang

Description

This problem is brought by http://review.whamcloud.com/16333

                rc = llog_trans_destroy(env, loghandle, th);
                if (rc < 0) {
                        /* Sigh, can not destroy the final plain llog, but
                         * the bitmap has been cleared, so the record cannot
                         * be accessed anymore. Let's return 0 for now, and
                         * the orphan will be handled by LFSCK.
                         */
                        CERROR("%s: can't kill empty llog "DFID":%x: rc = %d\n",
                               loghandle->lgh_ctxt->loc_obd->obd_name,
                               PFID(&loghandle->lgh_id.lgl_oi.oi_fid),
                               loghandle->lgh_id.lgl_ogen, rc);
                        GOTO(out_unlock, rc);
                }

If llog cancel is failed, it should still return 0 as commented by the above comments. Initially this returning rc is for restoring the lgh_count if destroy failed, so if this is reverted back to "GOTO(out_unlock, rc = 0);", then the following

        if (rc < 0 && subtract_count) {
                mutex_lock(&loghandle->lgh_hdr_mutex);
                loghandle->lgh_hdr->llh_count++;
                ext2_set_bit(index, LLOG_HDR_BITMAP(llh));
                mutex_unlock(&loghandle->lgh_hdr_mutex);
        }

needs to be resolved without rc < 0.

Issue Links

• is related to: LU-6846 dt_record_write()) ASSERTION( dt->do_body_ops->dbo_write ) failed: