* Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
* Use is subject to license terms.
*
- * Copyright (c) 2011, 2012, Intel, Inc.
+ * Copyright (c) 2012, Intel Corporation.
*/
/*
* This file is part of Lustre, http://www.lustre.org/
if (index == 0)
index = 1;
- cfs_spin_lock(&loghandle->lgh_hdr_lock);
+ spin_lock(&loghandle->lgh_hdr_lock);
llh->llh_count++;
- if (ext2_set_bit(index, llh->llh_bitmap)) {
- CERROR("argh, index %u already set in log bitmap?\n",
- index);
- cfs_spin_unlock(&loghandle->lgh_hdr_lock);
- LBUG(); /* should never happen */
- }
- cfs_spin_unlock(&loghandle->lgh_hdr_lock);
+ if (ext2_set_bit(index, llh->llh_bitmap)) {
+ CERROR("argh, index %u already set in log bitmap?\n",
+ index);
+ spin_unlock(&loghandle->lgh_hdr_lock);
+ LBUG(); /* should never happen */
+ }
+ spin_unlock(&loghandle->lgh_hdr_lock);
cathandle->lgh_last_idx = index;
llh->llh_tail.lrt_index = index;
*
* Assumes caller has already pushed us into the kernel context and is locking.
* We return a lock on the handle to ensure nobody yanks it from us.
+ *
+ * This takes extra reference on llog_handle via llog_handle_get() and require
+ * this reference to be put by caller using llog_handle_put()
*/
int llog_cat_id2handle(const struct lu_env *env, struct llog_handle *cathandle,
struct llog_handle **res, struct llog_logid *logid)
if (cathandle == NULL)
RETURN(-EBADF);
- cfs_down_write(&cathandle->lgh_lock);
+ down_write(&cathandle->lgh_lock);
cfs_list_for_each_entry(loghandle, &cathandle->u.chd.chd_head,
u.phd.phd_entry) {
struct llog_logid *cgl = &loghandle->lgh_id;
continue;
}
loghandle->u.phd.phd_cat_handle = cathandle;
- cfs_up_write(&cathandle->lgh_lock);
+ up_write(&cathandle->lgh_lock);
GOTO(out, rc = 0);
}
}
- cfs_up_write(&cathandle->lgh_lock);
+ up_write(&cathandle->lgh_lock);
rc = llog_open(env, cathandle->lgh_ctxt, &loghandle, logid, NULL,
LLOG_OPEN_EXISTS);
CERROR("%s: error opening log id "LPX64":%x: rc = %d\n",
cathandle->lgh_ctxt->loc_obd->obd_name,
logid->lgl_oid, logid->lgl_ogen, rc);
- GOTO(out, rc);
+ RETURN(rc);
}
rc = llog_init_handle(env, loghandle, LLOG_F_IS_PLAIN, NULL);
if (rc < 0) {
llog_close(env, loghandle);
- GOTO(out, rc);
+ loghandle = NULL;
+ RETURN(rc);
}
- cfs_down_write(&cathandle->lgh_lock);
+ down_write(&cathandle->lgh_lock);
cfs_list_add(&loghandle->u.phd.phd_entry, &cathandle->u.chd.chd_head);
- cfs_up_write(&cathandle->lgh_lock);
+ up_write(&cathandle->lgh_lock);
loghandle->u.phd.phd_cat_handle = cathandle;
loghandle->u.phd.phd_cookie.lgc_lgl = cathandle->lgh_id;
loghandle->lgh_hdr->llh_cat_idx;
EXIT;
out:
+ llog_handle_get(loghandle);
*res = loghandle;
- return rc;
+ return 0;
}
int llog_cat_close(const struct lu_env *env, struct llog_handle *cathandle)
rc);
index = loghandle->u.phd.phd_cookie.lgc_index;
-
- LASSERT(index);
- llog_cat_set_first_idx(cathandle, index);
- rc = llog_cancel_rec(env, cathandle, index);
- if (rc == 0)
- CDEBUG(D_RPCTRACE,
- "cancel plain log at index %u of "
- "catalog "LPX64"\n",
- index, cathandle->lgh_id.lgl_oid);
+ llog_cat_cleanup(env, cathandle, NULL, index);
}
llog_close(env, loghandle);
}
struct llog_handle *loghandle = NULL;
ENTRY;
- cfs_down_read_nested(&cathandle->lgh_lock, LLOGH_CAT);
+ down_read_nested(&cathandle->lgh_lock, LLOGH_CAT);
loghandle = cathandle->u.chd.chd_current_log;
if (loghandle) {
struct llog_log_hdr *llh;
- cfs_down_write_nested(&loghandle->lgh_lock, LLOGH_LOG);
+ down_write_nested(&loghandle->lgh_lock, LLOGH_LOG);
llh = loghandle->lgh_hdr;
if (llh == NULL ||
loghandle->lgh_last_idx < LLOG_BITMAP_SIZE(llh) - 1) {
- cfs_up_read(&cathandle->lgh_lock);
+ up_read(&cathandle->lgh_lock);
RETURN(loghandle);
} else {
- cfs_up_write(&loghandle->lgh_lock);
+ up_write(&loghandle->lgh_lock);
}
}
- cfs_up_read(&cathandle->lgh_lock);
+ up_read(&cathandle->lgh_lock);
/* time to use next log */
/* first, we have to make sure the state hasn't changed */
- cfs_down_write_nested(&cathandle->lgh_lock, LLOGH_CAT);
+ down_write_nested(&cathandle->lgh_lock, LLOGH_CAT);
loghandle = cathandle->u.chd.chd_current_log;
if (loghandle) {
struct llog_log_hdr *llh;
- cfs_down_write_nested(&loghandle->lgh_lock, LLOGH_LOG);
+ down_write_nested(&loghandle->lgh_lock, LLOGH_LOG);
llh = loghandle->lgh_hdr;
LASSERT(llh);
if (loghandle->lgh_last_idx < LLOG_BITMAP_SIZE(llh) - 1) {
- cfs_up_write(&cathandle->lgh_lock);
+ up_write(&cathandle->lgh_lock);
RETURN(loghandle);
} else {
- cfs_up_write(&loghandle->lgh_lock);
+ up_write(&loghandle->lgh_lock);
}
}
loghandle = cathandle->u.chd.chd_next_log;
cathandle->u.chd.chd_current_log = loghandle;
cathandle->u.chd.chd_next_log = NULL;
- cfs_down_write_nested(&loghandle->lgh_lock, LLOGH_LOG);
- cfs_up_write(&cathandle->lgh_lock);
+ down_write_nested(&loghandle->lgh_lock, LLOGH_LOG);
+ up_write(&cathandle->lgh_lock);
LASSERT(loghandle);
RETURN(loghandle);
}
if (!llog_exist(loghandle)) {
rc = llog_cat_new_log(env, cathandle, loghandle, th);
if (rc < 0) {
- cfs_up_write(&loghandle->lgh_lock);
+ up_write(&loghandle->lgh_lock);
RETURN(rc);
}
}
/* now let's try to add the record */
rc = llog_write_rec(env, loghandle, rec, reccookie, 1, buf, -1, th);
- if (rc < 0)
- CERROR("llog_write_rec %d: lh=%p\n", rc, loghandle);
- cfs_up_write(&loghandle->lgh_lock);
+ if (rc < 0)
+ CDEBUG_LIMIT(rc == -ENOSPC ? D_HA : D_ERROR,
+ "llog_write_rec %d: lh=%p\n", rc, loghandle);
+ up_write(&loghandle->lgh_lock);
if (rc == -ENOSPC) {
/* try to use next log */
loghandle = llog_cat_current_log(cathandle, th);
if (!llog_exist(loghandle)) {
rc = llog_cat_new_log(env, cathandle, loghandle, th);
if (rc < 0) {
- cfs_up_write(&loghandle->lgh_lock);
+ up_write(&loghandle->lgh_lock);
RETURN(rc);
}
}
-1, th);
if (rc < 0)
CERROR("llog_write_rec %d: lh=%p\n", rc, loghandle);
- cfs_up_write(&loghandle->lgh_lock);
+ up_write(&loghandle->lgh_lock);
}
RETURN(rc);
if (cathandle->u.chd.chd_current_log == NULL) {
/* declare new plain llog */
- cfs_down_write(&cathandle->lgh_lock);
+ down_write(&cathandle->lgh_lock);
if (cathandle->u.chd.chd_current_log == NULL) {
rc = llog_open(env, cathandle->lgh_ctxt, &loghandle,
NULL, NULL, LLOG_OPEN_NEW);
&cathandle->u.chd.chd_head);
}
}
- cfs_up_write(&cathandle->lgh_lock);
+ up_write(&cathandle->lgh_lock);
} else if (cathandle->u.chd.chd_next_log == NULL) {
/* declare next plain llog */
- cfs_down_write(&cathandle->lgh_lock);
+ down_write(&cathandle->lgh_lock);
if (cathandle->u.chd.chd_next_log == NULL) {
rc = llog_open(env, cathandle->lgh_ctxt, &loghandle,
NULL, NULL, LLOG_OPEN_NEW);
&cathandle->u.chd.chd_head);
}
}
- cfs_up_write(&cathandle->lgh_lock);
+ up_write(&cathandle->lgh_lock);
}
if (rc)
GOTO(out, rc);
rc = llog_cat_id2handle(env, cathandle, &loghandle, lgl);
if (rc) {
- CERROR("Cannot find log "LPX64"\n", lgl->lgl_oid);
- break;
+ CERROR("%s: cannot find handle for llog "LPX64": %d\n",
+ cathandle->lgh_ctxt->loc_obd->obd_name,
+ lgl->lgl_oid, rc);
+ failed++;
+ continue;
}
lrc = llog_cancel_rec(env, loghandle, cookies->lgc_index);
if (lrc == 1) { /* log has been destroyed */
index = loghandle->u.phd.phd_cookie.lgc_index;
- cfs_down_write(&cathandle->lgh_lock);
- if (cathandle->u.chd.chd_current_log == loghandle)
- cathandle->u.chd.chd_current_log = NULL;
- cfs_up_write(&cathandle->lgh_lock);
- llog_close(env, loghandle);
-
- LASSERT(index);
- llog_cat_set_first_idx(cathandle, index);
- lrc = llog_cancel_rec(env, cathandle, index);
- if (lrc == 0)
- CDEBUG(D_RPCTRACE, "cancel plain log at index"
- " %u of catalog "LPX64"\n",
- index, cathandle->lgh_id.lgl_oid);
+ rc = llog_cat_cleanup(env, cathandle, loghandle,
+ index);
} else if (lrc == -ENOENT) {
if (rc == 0) /* ENOENT shouldn't rewrite any error */
rc = lrc;
failed++;
rc = lrc;
}
+ llog_handle_put(loghandle);
}
if (rc)
CERROR("%s: fail to cancel %d of %d llog-records: rc = %d\n",
rec->lrh_index, cat_llh->lgh_id.lgl_oid);
rc = llog_cat_id2handle(env, cat_llh, &llh, &lir->lid_id);
- if (rc) {
- CERROR("Cannot find handle for log "LPX64"\n",
- lir->lid_id.lgl_oid);
- RETURN(rc);
- }
+ if (rc) {
+ CERROR("%s: cannot find handle for llog "LPX64": %d\n",
+ cat_llh->lgh_ctxt->loc_obd->obd_name,
+ lir->lid_id.lgl_oid, rc);
+ RETURN(rc);
+ }
if (rec->lrh_index < d->lpd_startcat)
/* Skip processing of the logs until startcat */
cd.lpcd_last_idx = 0;
rc = llog_process_or_fork(env, llh, d->lpd_cb, d->lpd_data,
&cd, false);
- /* Continue processing the next log from idx 0 */
- d->lpd_startidx = 0;
- } else {
+ /* Continue processing the next log from idx 0 */
+ d->lpd_startidx = 0;
+ } else {
rc = llog_process_or_fork(env, llh, d->lpd_cb, d->lpd_data,
NULL, false);
- }
+ }
+ llog_handle_put(llh);
- RETURN(rc);
+ RETURN(rc);
}
int llog_cat_process_or_fork(const struct lu_env *env,
le32_to_cpu(rec->lrh_index), cat_llh->lgh_id.lgl_oid);
rc = llog_cat_id2handle(env, cat_llh, &llh, &lir->lid_id);
- if (rc) {
- CERROR("Cannot find handle for log "LPX64"\n",
- lir->lid_id.lgl_oid);
- RETURN(rc);
- }
+ if (rc) {
+ CERROR("%s: cannot find handle for llog "LPX64": %d\n",
+ cat_llh->lgh_ctxt->loc_obd->obd_name,
+ lir->lid_id.lgl_oid, rc);
+ RETURN(rc);
+ }
rc = llog_reverse_process(env, llh, d->lpd_cb, d->lpd_data, NULL);
- RETURN(rc);
+ llog_handle_put(llh);
+ RETURN(rc);
}
int llog_cat_reverse_process(const struct lu_env *env,
RETURN(0);
}
+/* Cleanup deleted plain llog traces from catalog */
+int llog_cat_cleanup(const struct lu_env *env, struct llog_handle *cathandle,
+ struct llog_handle *loghandle, int index)
+{
+ int rc;
+
+ LASSERT(index);
+ if (loghandle != NULL) {
+ /* remove destroyed llog from catalog list and
+ * chd_current_log variable */
+ down_write(&cathandle->lgh_lock);
+ if (cathandle->u.chd.chd_current_log == loghandle)
+ cathandle->u.chd.chd_current_log = NULL;
+ cfs_list_del_init(&loghandle->u.phd.phd_entry);
+ up_write(&cathandle->lgh_lock);
+ LASSERT(index == loghandle->u.phd.phd_cookie.lgc_index);
+ /* llog was opened and keep in a list, close it now */
+ llog_close(env, loghandle);
+ }
+ /* remove plain llog entry from catalog by index */
+ llog_cat_set_first_idx(cathandle, index);
+ rc = llog_cancel_rec(env, cathandle, index);
+ if (rc == 0)
+ CDEBUG(D_HA, "cancel plain log at index"
+ " %u of catalog "LPX64"\n",
+ index, cathandle->lgh_id.lgl_oid);
+ return rc;
+}
+
int cat_cancel_cb(const struct lu_env *env, struct llog_handle *cathandle,
struct llog_rec_hdr *rec, void *data)
{
struct llog_logid_rec *lir = (struct llog_logid_rec *)rec;
struct llog_handle *loghandle;
struct llog_log_hdr *llh;
- int rc, index;
+ int rc;
ENTRY;
if (rec->lrh_type != LLOG_LOGID_MAGIC) {
- CERROR("%s: invalid record in catalog\n",
- loghandle->lgh_ctxt->loc_obd->obd_name);
+ CERROR("invalid record in catalog\n");
RETURN(-EINVAL);
}
CDEBUG(D_HA, "processing log "LPX64":%x at index %u of catalog "
rc = llog_cat_id2handle(env, cathandle, &loghandle, &lir->lid_id);
if (rc) {
- CERROR("%s: cannot find handle for llog "LPX64"\n: %d",
- loghandle->lgh_ctxt->loc_obd->obd_name,
+ CERROR("%s: cannot find handle for llog "LPX64": %d\n",
+ cathandle->lgh_ctxt->loc_obd->obd_name,
lir->lid_id.lgl_oid, rc);
if (rc == -ENOENT || rc == -ESTALE) {
- index = rec->lrh_index;
- goto cat_cleanup;
+ /* remove index from catalog */
+ llog_cat_cleanup(env, cathandle, NULL, rec->lrh_index);
}
RETURN(rc);
}
CERROR("%s: fail to destroy empty log: rc = %d\n",
loghandle->lgh_ctxt->loc_obd->obd_name, rc);
- index = loghandle->u.phd.phd_cookie.lgc_index;
- llog_close(env, loghandle);
-
-cat_cleanup:
- LASSERT(index);
- llog_cat_set_first_idx(cathandle, index);
- rc = llog_cancel_rec(env, cathandle, index);
- if (rc == 0)
- CDEBUG(D_HA,
- "cancel log "LPX64":%x at index %u of catalog "
- LPX64"\n", lir->lid_id.lgl_oid,
- lir->lid_id.lgl_ogen, rec->lrh_index,
- cathandle->lgh_id.lgl_oid);
+ llog_cat_cleanup(env, cathandle, loghandle,
+ loghandle->u.phd.phd_cookie.lgc_index);
}
+ llog_handle_put(loghandle);
RETURN(rc);
}
if (rc)
RETURN(rc);
- rc = llog_process(env, llh, cat_cancel_cb, NULL, NULL);
+ rc = llog_process_or_fork(env, llh, cat_cancel_cb, NULL, NULL, false);
if (rc)
CERROR("%s: llog_process() with cat_cancel_cb failed: rc = "
"%d\n", llh->lgh_ctxt->loc_obd->obd_name, rc);
git://git.whamcloud.com / fs / lustre-release.git / blobdiff