This patch fixes issues reported by checkpatch for file
lustre/obdclass/llog_test.c
Change-Id: I7b148b3db29f374421bad43764ca40bf7e8d6a9f
Test-Parameters: trivial
Signed-off-by: Arshad Hussain <arshad.super@gmail.com>
Reviewed-on: https://review.whamcloud.com/33501
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
static struct llog_logid cat_logid;
struct llog_mini_rec {
static struct llog_logid cat_logid;
struct llog_mini_rec {
- struct llog_rec_hdr lmr_hdr;
- struct llog_rec_tail lmr_tail;
+ struct llog_rec_hdr lmr_hdr;
+ struct llog_rec_tail lmr_tail;
} __attribute__((packed));
static int verify_handle(char *test, struct llog_handle *llh, int num_recs)
} __attribute__((packed));
static int verify_handle(char *test, struct llog_handle *llh, int num_recs)
static int llog_test_1(const struct lu_env *env,
struct obd_device *obd, char *name)
{
static int llog_test_1(const struct lu_env *env,
struct obd_device *obd, char *name)
{
- struct llog_handle *llh;
- struct llog_ctxt *ctxt;
+ struct llog_handle *llh;
+ struct llog_ctxt *ctxt;
static int llog_test_2(const struct lu_env *env, struct obd_device *obd,
char *name, struct llog_handle **llh)
{
static int llog_test_2(const struct lu_env *env, struct obd_device *obd,
char *name, struct llog_handle **llh)
{
- struct llog_ctxt *ctxt;
- struct llog_handle *lgh;
- struct llog_logid logid;
- int rc;
- struct llog_mini_rec lmr;
+ struct llog_ctxt *ctxt;
+ struct llog_handle *lgh;
+ struct llog_logid logid;
+ int rc;
+ struct llog_mini_rec lmr;
} else {
size_t chunk_size = lgh->lgh_hdr->llh_hdr.lrh_len;
} else {
size_t chunk_size = lgh->lgh_hdr->llh_hdr.lrh_len;
- /* For variable size records the start offset is unknown, trust
- * the first value and check others are consistent with it. */
+ /*
+ * For variable size records the start offset is unknown, trust
+ * the first value and check others are consistent with it.
+ */
if (test_3_rec_off == 0)
test_3_rec_off = lgh->lgh_cur_offset;
if (test_3_rec_off == 0)
test_3_rec_off = lgh->lgh_cur_offset;
if (rc < 0)
CERROR("cb_test_3: cannot modify record while processing\n");
if (rc < 0)
CERROR("cb_test_3: cannot modify record while processing\n");
- /* Add new record to the llog at *last_rec position one by one to
- * check that last block is re-read during processing */
+ /*
+ * Add new record to the llog at *last_rec position one by one to
+ * check that last block is re-read during processing
+ */
if (cur_idx == *last_rec || cur_idx == (*last_rec + 1)) {
rc = llog_write(env, lgh, rec, LLOG_NEXT_IDX);
if (rc < 0)
if (cur_idx == *last_rec || cur_idx == (*last_rec + 1)) {
rc = llog_write(env, lgh, rec, LLOG_NEXT_IDX);
if (rc < 0)
llh->lgh_hdr->llh_size = sizeof(struct llog_gen_rec);
llh->lgh_hdr->llh_flags |= LLOG_F_IS_FIXSIZE;
llh->lgh_hdr->llh_size = sizeof(struct llog_gen_rec);
llh->lgh_hdr->llh_flags |= LLOG_F_IS_FIXSIZE;
- /* Fill the llog with 64-bytes records, use 1023 records,
+ /*
+ * Fill the llog with 64-bytes records, use 1023 records,
* so last chunk will be partially full. Don't change this
* value until record size is changed.
*/
* so last chunk will be partially full. Don't change this
* value until record size is changed.
*/
CWARN("3b: write 566 variable size llog records\n");
CWARN("3b: write 566 variable size llog records\n");
- /* Drop llh_size to 0 to mark llog as variable-size and write
- * header to make this change permanent. */
+ /*
+ * Drop llh_size to 0 to mark llog as variable-size and write
+ * header to make this change permanent.
+ */
llh->lgh_hdr->llh_flags &= ~LLOG_F_IS_FIXSIZE;
llog_write(env, llh, &llh->lgh_hdr->llh_hdr, LLOG_HEADER_IDX);
hdr->lrh_type = OBD_CFG_REC;
llh->lgh_hdr->llh_flags &= ~LLOG_F_IS_FIXSIZE;
llog_write(env, llh, &llh->lgh_hdr->llh_hdr, LLOG_HEADER_IDX);
hdr->lrh_type = OBD_CFG_REC;
- /* there are 1025 64-bytes records in llog already,
+ /*
+ * there are 1025 64-bytes records in llog already,
* the last chunk contains single record, i.e. 64 bytes.
* Each pair of variable size records is 200 bytes, so
* we will have the following distribution per chunks:
* the last chunk contains single record, i.e. 64 bytes.
* Each pair of variable size records is 200 bytes, so
* we will have the following distribution per chunks:
/* Test catalogue additions */
static int llog_test_4(const struct lu_env *env, struct obd_device *obd)
{
/* Test catalogue additions */
static int llog_test_4(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_handle *cath;
- char name[10];
- int rc, rc2, i, buflen;
- struct llog_mini_rec lmr;
- struct llog_cookie cookie;
- struct llog_ctxt *ctxt;
- int num_recs = 0;
- char *buf;
- struct llog_rec_hdr *rec;
+ struct llog_handle *cath;
+ char name[10];
+ int rc, rc2, i, buflen;
+ struct llog_mini_rec lmr;
+ struct llog_cookie cookie;
+ struct llog_ctxt *ctxt;
+ int num_recs = 0;
+ char *buf;
+ struct llog_rec_hdr *rec;
static int cat_print_cb(const struct lu_env *env, struct llog_handle *llh,
struct llog_rec_hdr *rec, void *data)
{
static int cat_print_cb(const struct lu_env *env, struct llog_handle *llh,
struct llog_rec_hdr *rec, void *data)
{
- struct llog_logid_rec *lir = (struct llog_logid_rec *)rec;
- struct lu_fid fid = {0};
+ struct llog_logid_rec *lir = (struct llog_logid_rec *)rec;
+ struct lu_fid fid = {0};
if (rec->lrh_type != LLOG_LOGID_MAGIC) {
CERROR("invalid record in catalog\n");
if (rec->lrh_type != LLOG_LOGID_MAGIC) {
CERROR("invalid record in catalog\n");
/* Test log and catalogue processing */
static int llog_test_5(const struct lu_env *env, struct obd_device *obd)
{
/* Test log and catalogue processing */
static int llog_test_5(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_handle *llh = NULL;
- char name[10];
- int rc, rc2;
- struct llog_mini_rec lmr;
- struct llog_ctxt *ctxt;
+ struct llog_handle *llh = NULL;
+ char name[10];
+ int rc, rc2;
+ struct llog_mini_rec lmr;
+ struct llog_ctxt *ctxt;
static int llog_test_6(const struct lu_env *env, struct obd_device *obd,
char *name)
{
static int llog_test_6(const struct lu_env *env, struct obd_device *obd,
char *name)
{
- struct obd_device *mgc_obd;
- struct llog_ctxt *ctxt;
- struct obd_uuid *mgs_uuid;
- struct obd_export *exp;
- struct obd_uuid uuid = { "LLOG_TEST6_UUID" };
- struct llog_handle *llh = NULL;
- struct llog_ctxt *nctxt;
- int rc, rc2;
+ struct obd_device *mgc_obd;
+ struct llog_ctxt *ctxt;
+ struct obd_uuid *mgs_uuid;
+ struct obd_export *exp;
+ struct obd_uuid uuid = { "LLOG_TEST6_UUID" };
+ struct llog_handle *llh = NULL;
+ struct llog_ctxt *nctxt;
+ int rc, rc2;
ctxt = llog_get_context(obd, LLOG_TEST_ORIG_CTXT);
LASSERT(ctxt);
ctxt = llog_get_context(obd, LLOG_TEST_ORIG_CTXT);
LASSERT(ctxt);
static int llog_test_7_sub(const struct lu_env *env, struct llog_ctxt *ctxt)
{
static int llog_test_7_sub(const struct lu_env *env, struct llog_ctxt *ctxt)
{
- struct llog_handle *llh;
- int rc = 0, i, process_count;
- int num_recs = 0;
+ struct llog_handle *llh;
+ int rc = 0, i, process_count;
+ int num_recs = 0;
/* Test all llog records writing and processing */
static int llog_test_7(const struct lu_env *env, struct obd_device *obd)
{
/* Test all llog records writing and processing */
static int llog_test_7(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_ctxt *ctxt;
- int rc;
+ struct llog_ctxt *ctxt;
+ int rc;
static int llog_truncate(const struct lu_env *env, struct dt_object *o)
{
static int llog_truncate(const struct lu_env *env, struct dt_object *o)
{
- struct lu_attr la;
- struct thandle *th;
- struct dt_device *d;
- int rc;
+ struct lu_attr la;
+ struct thandle *th;
+ struct dt_device *d;
+ int rc;
static int llog_test_8(const struct lu_env *env, struct obd_device *obd)
{
static int llog_test_8(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_handle *llh = NULL;
- char name[10];
- int rc, rc2, i;
- int orig_counter;
- struct llog_mini_rec lmr;
- struct llog_ctxt *ctxt;
- struct dt_object *obj = NULL;
+ struct llog_handle *llh = NULL;
+ char name[10];
+ int rc, rc2, i;
+ int orig_counter;
+ struct llog_mini_rec lmr;
+ struct llog_ctxt *ctxt;
+ struct dt_object *obj = NULL;
}
}
CWARN("8b: second llog "DFID"\n",
}
}
CWARN("8b: second llog "DFID"\n",
- PFID(lu_object_fid(&llh->u.chd.chd_current_log->lgh_obj->do_lu)));
+ PFID(lu_object_fid(&llh->u.chd.chd_current_log->lgh_obj->do_lu)));
rc2 = llog_cat_close(env, llh);
if (rc2) {
rc2 = llog_cat_close(env, llh);
if (rc2) {
static int llog_test_9_sub(const struct lu_env *env, struct llog_ctxt *ctxt)
{
static int llog_test_9_sub(const struct lu_env *env, struct llog_ctxt *ctxt)
{
- struct llog_handle *llh;
- struct lu_fid fid;
- int rc = 0;
+ struct llog_handle *llh;
+ struct lu_fid fid;
+ int rc = 0;
/* Prepare different types of llog records for llog_reader test*/
static int llog_test_9(const struct lu_env *env, struct obd_device *obd)
{
/* Prepare different types of llog records for llog_reader test*/
static int llog_test_9(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_ctxt *ctxt;
- int rc;
+ struct llog_ctxt *ctxt;
+ int rc;
}
struct llog_process_info {
}
struct llog_process_info {
- struct llog_handle *lpi_loghandle;
- llog_cb_t lpi_cb;
- void *lpi_cbdata;
- void *lpi_catdata;
- int lpi_rc;
- struct completion lpi_completion;
- const struct lu_env *lpi_env;
- struct task_struct *lpi_reftask;
+ struct llog_handle *lpi_loghandle;
+ llog_cb_t lpi_cb;
+ void *lpi_cbdata;
+ void *lpi_catdata;
+ int lpi_rc;
+ struct completion lpi_completion;
+ const struct lu_env *lpi_env;
+ struct task_struct *lpi_reftask;
};
static int llog_test_process_thread(void *arg)
{
};
static int llog_test_process_thread(void *arg)
{
- struct llog_process_info *lpi = arg;
+ struct llog_process_info *lpi = arg;
llog_process(NULL, lpi->lpi_loghandle, lpi->lpi_cb, lpi->lpi_cbdata,
lpi->lpi_catdata);
llog_process(NULL, lpi->lpi_loghandle, lpi->lpi_cb, lpi->lpi_cbdata,
lpi->lpi_catdata);
static int cat_check_old_cb(const struct lu_env *env, struct llog_handle *llh,
struct llog_rec_hdr *rec, void *data)
{
static int cat_check_old_cb(const struct lu_env *env, struct llog_handle *llh,
struct llog_rec_hdr *rec, void *data)
{
- struct llog_logid_rec *lir = (struct llog_logid_rec *)rec;
- struct lu_fid fid = {0};
- struct lu_fid *prev_fid = data;
+ struct llog_logid_rec *lir = (struct llog_logid_rec *)rec;
+ struct lu_fid fid = {0};
+ struct lu_fid *prev_fid = data;
if (rec->lrh_type != LLOG_LOGID_MAGIC) {
CERROR("invalid record in catalog\n");
if (rec->lrh_type != LLOG_LOGID_MAGIC) {
CERROR("invalid record in catalog\n");
/* test catalog wrap around */
static int llog_test_10(const struct lu_env *env, struct obd_device *obd)
{
/* test catalog wrap around */
static int llog_test_10(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_handle *cath;
- char name[10];
- int rc, rc2, i, enospc, eok;
- struct llog_mini_rec lmr;
- struct llog_ctxt *ctxt;
- struct lu_attr la;
- __u64 cat_max_size;
- struct dt_device *dt;
+ struct llog_handle *cath;
+ char name[10];
+ int rc, rc2, i, enospc, eok;
+ struct llog_mini_rec lmr;
+ struct llog_ctxt *ctxt;
+ struct lu_attr la;
+ __u64 cat_max_size;
+ struct dt_device *dt;
cat_logid = cath->lgh_id;
dt = lu2dt_dev(cath->lgh_obj->do_lu.lo_dev);
cat_logid = cath->lgh_id;
dt = lu2dt_dev(cath->lgh_obj->do_lu.lo_dev);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10c: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10c: sync failed: %d\n", rc);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10b: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10b: sync failed: %d\n", rc);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10c: sync failed: %d\n", rc);
GOTO(out, rc);
}
rc = dt_sync(env, dt);
if (rc) {
CERROR("10c: sync failed: %d\n", rc);
GOTO(out, rc);
}
- /* fill last allocated plain LLOG and reach -ENOSPC condition
- * because no slot available in Catalog */
+ /*
+ * fill last allocated plain LLOG and reach -ENOSPC condition
+ * because no slot available in Catalog
+ */
enospc = 0;
eok = 0;
CWARN("10c: write %d more log records\n", LLOG_TEST_RECNUM);
enospc = 0;
eok = 0;
CWARN("10c: write %d more log records\n", LLOG_TEST_RECNUM);
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
- /* after last added plain LLOG has filled up, all new
- * records add should fail with -ENOSPC */
+ /*
+ * after last added plain LLOG has filled up, all new
+ * records add should fail with -ENOSPC
+ */
if (rc == -ENOSPC) {
enospc++;
} else {
if (rc == -ENOSPC) {
enospc++;
} else {
}
cat_max_size = la.la_size;
}
cat_max_size = la.la_size;
- /* cancel all 1st plain llog records to empty it, this will also cause
- * its catalog entry to be freed for next forced wrap in 10e */
+ /*
+ * cancel all 1st plain llog records to empty it, this will also cause
+ * its catalog entry to be freed for next forced wrap in 10e
+ */
CWARN("10d: Cancel %d records, see one log zapped\n", LLOG_TEST_RECNUM);
cancel_count = 0;
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10d: process with llog_cancel_rec_cb failed: %d\n", rc);
CWARN("10d: Cancel %d records, see one log zapped\n", LLOG_TEST_RECNUM);
cancel_count = 0;
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10d: process with llog_cancel_rec_cb failed: %d\n", rc);
- /* need to indicate error if for any reason LLOG_TEST_RECNUM is
- * not reached */
+ /*
+ * need to indicate error if for any reason LLOG_TEST_RECNUM is
+ * not reached
+ */
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10d: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10d: sync failed: %d\n", rc);
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
- /* after last added plain LLOG has filled up, all new
- * records add should fail with -ENOSPC */
+ /*
+ * after last added plain LLOG has filled up, all new
+ * records add should fail with -ENOSPC
+ */
if (rc == -ENOSPC) {
enospc++;
} else {
if (rc == -ENOSPC) {
enospc++;
} else {
CWARN("10e: catalog successfully wrap around, last_idx %d, first %d\n",
cath->lgh_last_idx, cath->lgh_hdr->llh_cat_idx);
CWARN("10e: catalog successfully wrap around, last_idx %d, first %d\n",
cath->lgh_last_idx, cath->lgh_hdr->llh_cat_idx);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10e: sync failed: %d\n", rc);
GOTO(out, rc);
}
rc = dt_sync(env, dt);
if (rc) {
CERROR("10e: sync failed: %d\n", rc);
GOTO(out, rc);
}
- /* cancel more records to free one more slot in Catalog
- * see if it is re-allocated when adding more records */
+ /*
+ * cancel more records to free one more slot in Catalog
+ * see if it is re-allocated when adding more records
+ */
CWARN("10f: Cancel %d records, see one log zapped\n", LLOG_TEST_RECNUM);
cancel_count = 0;
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10f: process with llog_cancel_rec_cb failed: %d\n", rc);
CWARN("10f: Cancel %d records, see one log zapped\n", LLOG_TEST_RECNUM);
cancel_count = 0;
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10f: process with llog_cancel_rec_cb failed: %d\n", rc);
- /* need to indicate error if for any reason LLOG_TEST_RECNUM is
- * not reached */
+ /*
+ * need to indicate error if for any reason LLOG_TEST_RECNUM is
+ * not reached
+ */
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10f: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10f: sync failed: %d\n", rc);
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
- /* after last added plain LLOG has filled up, all new
- * records add should fail with -ENOSPC */
+ /*
+ * after last added plain LLOG has filled up, all new
+ * records add should fail with -ENOSPC
+ */
if (rc == -ENOSPC) {
enospc++;
} else {
if (rc == -ENOSPC) {
enospc++;
} else {
GOTO(out, rc = -EINVAL);
}
GOTO(out, rc = -EINVAL);
}
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10f: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10f: sync failed: %d\n", rc);
/* will llh_cat_idx also successfully wrap ? */
/* will llh_cat_idx also successfully wrap ? */
- /* cancel all records in the plain LLOGs referenced by 2 last indexes in
- * Catalog */
+ /*
+ * cancel all records in the plain LLOGs referenced by 2 last indexes in
+ * Catalog
+ */
/* cancel more records to free one more slot in Catalog */
CWARN("10g: Cancel %d records, see one log zapped\n", LLOG_TEST_RECNUM);
/* cancel more records to free one more slot in Catalog */
CWARN("10g: Cancel %d records, see one log zapped\n", LLOG_TEST_RECNUM);
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10g: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10g: sync failed: %d\n", rc);
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10g: process with llog_cancel_rec_cb failed: %d\n", rc);
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10g: process with llog_cancel_rec_cb failed: %d\n", rc);
- /* need to indicate error if for any reason LLOG_TEST_RECNUM is
- * not reached */
+ /*
+ * need to indicate error if for any reason LLOG_TEST_RECNUM is
+ * not reached
+ */
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
GOTO(out, rc = -EINVAL);
}
GOTO(out, rc = -EINVAL);
}
- /* sync device to commit all recent LLOG changes to disk and avoid
+ /*
+ * sync device to commit all recent LLOG changes to disk and avoid
* to consume a huge space with delayed journal commit callbacks
* to consume a huge space with delayed journal commit callbacks
- * particularly on low memory nodes or VMs */
+ * particularly on low memory nodes or VMs
+ */
rc = dt_sync(env, dt);
if (rc) {
CERROR("10g: sync failed: %d\n", rc);
rc = dt_sync(env, dt);
if (rc) {
CERROR("10g: sync failed: %d\n", rc);
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10g: process with llog_cancel_rec_cb failed: %d\n", rc);
rc = llog_cat_process(env, cath, llog_cancel_rec_cb, "foobar", 0, 0);
if (rc != -LLOG_EEMPTY) {
CERROR("10g: process with llog_cancel_rec_cb failed: %d\n", rc);
- /* need to indicate error if for any reason LLOG_TEST_RECNUM is
- * not reached */
+ /*
+ * need to indicate error if for any reason LLOG_TEST_RECNUM is
+ * not reached
+ */
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
if (rc == 0)
rc = -ERANGE;
GOTO(out, rc);
CWARN("10g: llh_cat_idx has also successfully wrapped!\n");
CWARN("10g: llh_cat_idx has also successfully wrapped!\n");
- /* catalog has only one valid entry other slots has outdated
+ /*
+ * catalog has only one valid entry other slots has outdated
* records. Trying to race the llog_thread_process with llog_add
* llog_thread_process read buffer and loop record on it.
* llog_add adds a record and mark a record in bitmap.
* records. Trying to race the llog_thread_process with llog_add
* llog_thread_process read buffer and loop record on it.
* llog_add adds a record and mark a record in bitmap.
- * llog_thread_process process record with old data. */
+ * llog_thread_process process record with old data.
+ */
{
struct llog_process_info lpi;
struct lu_fid test_fid = {0};
{
struct llog_process_info lpi;
struct lu_fid test_fid = {0};
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
LLOG_TEST_RECNUM, i + 1, rc);
GOTO(out, rc);
}
- /* after last added plain LLOG has filled up, all new
- * records add should fail with -ENOSPC */
+ /*
+ * after last added plain LLOG has filled up, all new
+ * records add should fail with -ENOSPC
+ */
if (rc == -ENOSPC) {
enospc++;
} else {
if (rc == -ENOSPC) {
enospc++;
} else {
-/* -------------------------------------------------------------------------
+/*
+ * -------------------------------------------------------------------------
* Tests above, boring obd functions below
* Tests above, boring obd functions below
- * ------------------------------------------------------------------------- */
+ * -------------------------------------------------------------------------
+ */
static int llog_run_tests(const struct lu_env *env, struct obd_device *obd)
{
static int llog_run_tests(const struct lu_env *env, struct obd_device *obd)
{
- struct llog_handle *llh = NULL;
- struct llog_ctxt *ctxt;
- int rc, err;
- char name[10];
+ struct llog_handle *llh = NULL;
+ struct llog_ctxt *ctxt;
+ int rc, err;
+ char name[10];
ENTRY;
ctxt = llog_get_context(obd, LLOG_TEST_ORIG_CTXT);
ENTRY;
ctxt = llog_get_context(obd, LLOG_TEST_ORIG_CTXT);
static int llog_test_cleanup(struct obd_device *obd)
{
static int llog_test_cleanup(struct obd_device *obd)
{
- struct obd_device *tgt;
- struct lu_env env;
- int rc;
+ struct obd_device *tgt;
+ struct lu_env env;
+ int rc;
static int llog_test_setup(struct obd_device *obd, struct lustre_cfg *lcfg)
{
static int llog_test_setup(struct obd_device *obd, struct lustre_cfg *lcfg)
{
- struct obd_device *tgt;
- struct llog_ctxt *ctxt;
- struct dt_object *o;
- struct lu_env env;
- struct lu_context test_session;
- int rc;
-
- ENTRY;
-
- if (lcfg->lcfg_bufcount < 2) {
- CERROR("requires a TARGET OBD name\n");
- RETURN(-EINVAL);
- }
+ struct obd_device *tgt;
+ struct llog_ctxt *ctxt;
+ struct dt_object *o;
+ struct lu_env env;
+ struct lu_context test_session;
+ int rc;
- if (lcfg->lcfg_buflens[1] < 1) {
- CERROR("requires a TARGET OBD name\n");
- RETURN(-EINVAL);
- }
- /* disk obd */
- tgt = class_name2obd(lustre_cfg_string(lcfg, 1));
- if (!tgt || !tgt->obd_attached || !tgt->obd_set_up) {
- CERROR("target device not attached or not set up (%s)\n",
- lustre_cfg_string(lcfg, 1));
- RETURN(-EINVAL);
- }
+ if (lcfg->lcfg_bufcount < 2) {
+ CERROR("requires a TARGET OBD name\n");
+ RETURN(-EINVAL);
+ }
+
+ if (lcfg->lcfg_buflens[1] < 1) {
+ CERROR("requires a TARGET OBD name\n");
+ RETURN(-EINVAL);
+ }
+
+ /* disk obd */
+ tgt = class_name2obd(lustre_cfg_string(lcfg, 1));
+ if (!tgt || !tgt->obd_attached || !tgt->obd_set_up) {
+ CERROR("target device not attached or not set up (%s)\n",
+ lustre_cfg_string(lcfg, 1));
+ RETURN(-EINVAL);
+ }
rc = lu_env_init(&env, LCT_LOCAL | LCT_MG_THREAD);
if (rc)
rc = lu_env_init(&env, LCT_LOCAL | LCT_MG_THREAD);
if (rc)
}
static struct obd_ops llog_obd_ops = {
}
static struct obd_ops llog_obd_ops = {
- .o_owner = THIS_MODULE,
- .o_setup = llog_test_setup,
- .o_cleanup = llog_test_cleanup,
+ .o_owner = THIS_MODULE,
+ .o_setup = llog_test_setup,
+ .o_cleanup = llog_test_cleanup,
};
static int __init llog_test_init(void)
};
static int __init llog_test_init(void)