Commit dae0587e authored by James Simmons's avatar James Simmons Committed by Greg Kroah-Hartman

staging: lustre: cleanup comment style for lnet selftest

Apply a consistent style for comments in the lnet selftest
code. Realign some of the comments to make it easier to read.
This also fixes a few checkpatch issues as well.
Signed-off-by: default avatarJames Simmons <jsimmons@infradead.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0b4427de
...@@ -562,10 +562,10 @@ lstcon_rpc_trans_destroy(lstcon_rpc_trans_t *trans) ...@@ -562,10 +562,10 @@ lstcon_rpc_trans_destroy(lstcon_rpc_trans_t *trans)
} }
/* /*
* rpcs can be still not callbacked (even LNetMDUnlink is called) * rpcs can be still not callbacked (even LNetMDUnlink is
* because huge timeout for inaccessible network, don't make * called) because huge timeout for inaccessible network,
* user wait for them, just abandon them, they will be recycled * don't make user wait for them, just abandon them, they
* in callback * will be recycled in callback
*/ */
LASSERT(crpc->crp_status); LASSERT(crpc->crp_status);
...@@ -1178,7 +1178,8 @@ lstcon_rpc_pinger(void *arg) ...@@ -1178,7 +1178,8 @@ lstcon_rpc_pinger(void *arg)
int count = 0; int count = 0;
int rc; int rc;
/* RPC pinger is a special case of transaction, /*
* RPC pinger is a special case of transaction,
* it's called by timer at 8 seconds interval. * it's called by timer at 8 seconds interval.
*/ */
mutex_lock(&console_session.ses_mutex); mutex_lock(&console_session.ses_mutex);
......
...@@ -1446,7 +1446,8 @@ lstcon_test_batch_query(char *name, int testidx, int client, ...@@ -1446,7 +1446,8 @@ lstcon_test_batch_query(char *name, int testidx, int client,
lstcon_rpc_trans_postwait(trans, timeout); lstcon_rpc_trans_postwait(trans, timeout);
if (!testidx && /* query a batch, not a test */ /* query a batch, not a test */
if (!testidx &&
!lstcon_rpc_stat_failure(lstcon_trans_stat(), 0) && !lstcon_rpc_stat_failure(lstcon_trans_stat(), 0) &&
!lstcon_tsbqry_stat_run(lstcon_trans_stat(), 0)) { !lstcon_tsbqry_stat_run(lstcon_trans_stat(), 0)) {
/* all RPCs finished, and no active test */ /* all RPCs finished, and no active test */
......
...@@ -844,7 +844,7 @@ sfw_test_unit_done(sfw_test_unit_t *tsu) ...@@ -844,7 +844,7 @@ sfw_test_unit_done(sfw_test_unit_t *tsu)
spin_lock(&sfw_data.fw_lock); spin_lock(&sfw_data.fw_lock);
if (!atomic_dec_and_test(&tsb->bat_nactive) ||/* tsb still active */ if (!atomic_dec_and_test(&tsb->bat_nactive) || /* tsb still active */
sn == sfw_data.fw_session) { /* sn also active */ sn == sfw_data.fw_session) { /* sn also active */
spin_unlock(&sfw_data.fw_lock); spin_unlock(&sfw_data.fw_lock);
return; return;
...@@ -1273,7 +1273,7 @@ sfw_handle_server_rpc(struct srpc_server_rpc *rpc) ...@@ -1273,7 +1273,7 @@ sfw_handle_server_rpc(struct srpc_server_rpc *rpc)
} }
} else if (request->msg_ses_feats & ~LST_FEATS_MASK) { } else if (request->msg_ses_feats & ~LST_FEATS_MASK) {
/** /*
* NB: at this point, old version will ignore features and * NB: at this point, old version will ignore features and
* create new session anyway, so console should be able * create new session anyway, so console should be able
* to handle this * to handle this
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment