struct nrs_tbf_rule *rule = cli->tc_rule;
cli->tc_rpc_rate = rule->tr_rpc_rate;
- cli->tc_nsecs = rule->tr_nsecs;
+ cli->tc_nsecs = rule->tr_nsecs_per_rpc;
cli->tc_depth = rule->tr_depth;
cli->tc_ntoken = rule->tr_depth;
cli->tc_check_time = ktime_to_ns(ktime_get());
memcpy(rule->tr_name, start->tc_name, strlen(start->tc_name));
rule->tr_rpc_rate = start->u.tc_start.ts_rpc_rate;
rule->tr_flags = start->u.tc_start.ts_rule_flags;
- rule->tr_nsecs = NSEC_PER_SEC;
- do_div(rule->tr_nsecs, rule->tr_rpc_rate);
+ rule->tr_nsecs_per_rpc = NSEC_PER_SEC / rule->tr_rpc_rate;
rule->tr_depth = tbf_depth;
atomic_set(&rule->tr_ref, 1);
INIT_LIST_HEAD(&rule->tr_cli_list);
head->th_rule = rule;
}
- CDEBUG(D_RPCTRACE, "TBF starts rule@%p rate %llu gen %llu\n",
+ CDEBUG(D_RPCTRACE, "TBF starts rule@%p rate %u gen %llu\n",
rule, rule->tr_rpc_rate, rule->tr_generation);
return 0;
return -ENOENT;
rule->tr_rpc_rate = rate;
- rule->tr_nsecs = NSEC_PER_SEC;
- do_div(rule->tr_nsecs, rule->tr_rpc_rate);
+ rule->tr_nsecs_per_rpc = NSEC_PER_SEC / rule->tr_rpc_rate;
rule->tr_generation++;
nrs_tbf_rule_put(rule);
if (hnode == NULL)
return NULL;
- cli = container_of0(hnode, struct nrs_tbf_client, tc_hnode);
+ cli = container_of(hnode, struct nrs_tbf_client, tc_hnode);
if (!list_empty(&cli->tc_lru))
list_del_init(&cli->tc_lru);
return cli;
struct cfs_hash *hs = head->th_cli_hash;
struct nrs_tbf_bucket *bkt;
int hw;
- struct list_head zombies;
+ LIST_HEAD(zombies);
- INIT_LIST_HEAD(&zombies);
cfs_hash_bd_get(hs, &cli->tc_jobid, &bd);
bkt = cfs_hash_bd_extra_get(hs, &bd);
if (!cfs_hash_bd_dec_and_lock(hs, &bd, &cli->tc_ref))
cfs_hash_bd_unlock(head->th_cli_hash, &bd, 1);
while (!list_empty(&zombies)) {
- cli = container_of0(zombies.next,
- struct nrs_tbf_client, tc_lru);
+ cli = container_of(zombies.next,
+ struct nrs_tbf_client, tc_lru);
list_del_init(&cli->tc_lru);
nrs_tbf_cli_fini(cli);
}
list_for_each_entry_safe(jobid, n, jobid_list, tj_linkage) {
OBD_FREE(jobid->tj_id, strlen(jobid->tj_id) + 1);
list_del(&jobid->tj_linkage);
- OBD_FREE(jobid, sizeof(struct nrs_tbf_jobid));
+ OBD_FREE_PTR(jobid);
}
}
struct nrs_tbf_jobid *jobid;
char *ptr;
- OBD_ALLOC(jobid, sizeof(struct nrs_tbf_jobid));
+ OBD_ALLOC_PTR(jobid);
if (jobid == NULL)
return -ENOMEM;
OBD_ALLOC(jobid->tj_id, id->ls_len + 1);
if (jobid->tj_id == NULL) {
- OBD_FREE(jobid, sizeof(struct nrs_tbf_jobid));
+ OBD_FREE_PTR(jobid);
return -ENOMEM;
}
static int
nrs_tbf_jobid_rule_dump(struct nrs_tbf_rule *rule, struct seq_file *m)
{
- seq_printf(m, "%s {%s} %llu, ref %d\n", rule->tr_name,
+ seq_printf(m, "%s {%s} %u, ref %d\n", rule->tr_name,
rule->tr_jobids_str, rule->tr_rpc_rate,
atomic_read(&rule->tr_ref) - 1);
return 0;
static int
nrs_tbf_nid_rule_dump(struct nrs_tbf_rule *rule, struct seq_file *m)
{
- seq_printf(m, "%s {%s} %llu, ref %d\n", rule->tr_name,
+ seq_printf(m, "%s {%s} %u, ref %d\n", rule->tr_name,
rule->tr_nids_str, rule->tr_rpc_rate,
atomic_read(&rule->tr_ref) - 1);
return 0;
if (hnode == NULL)
return NULL;
- cli = container_of0(hnode, struct nrs_tbf_client, tc_hnode);
+ cli = container_of(hnode, struct nrs_tbf_client, tc_hnode);
if (!list_empty(&cli->tc_lru))
list_del_init(&cli->tc_lru);
return cli;
struct cfs_hash *hs = head->th_cli_hash;
struct nrs_tbf_bucket *bkt;
int hw;
- struct list_head zombies;
+ LIST_HEAD(zombies);
- INIT_LIST_HEAD(&zombies);
cfs_hash_bd_get(hs, &cli->tc_key, &bd);
bkt = cfs_hash_bd_extra_get(hs, &bd);
if (!cfs_hash_bd_dec_and_lock(hs, &bd, &cli->tc_ref))
cfs_hash_bd_unlock(head->th_cli_hash, &bd, 1);
while (!list_empty(&zombies)) {
- cli = container_of0(zombies.next,
- struct nrs_tbf_client, tc_lru);
+ cli = container_of(zombies.next,
+ struct nrs_tbf_client, tc_lru);
list_del_init(&cli->tc_lru);
nrs_tbf_cli_fini(cli);
}
struct cfs_lstr field;
int rc = 0;
- OBD_ALLOC(expr, sizeof(struct nrs_tbf_expression));
+ OBD_ALLOC_PTR(expr);
if (expr == NULL)
return -ENOMEM;
struct cfs_lstr expr;
int rc = 0;
- OBD_ALLOC(conjunction, sizeof(struct nrs_tbf_conjunction));
+ OBD_ALLOC_PTR(conjunction);
if (conjunction == NULL)
return -ENOMEM;
static int
nrs_tbf_generic_rule_dump(struct nrs_tbf_rule *rule, struct seq_file *m)
{
- seq_printf(m, "%s %s %llu, ref %d\n", rule->tr_name,
+ seq_printf(m, "%s %s %u, ref %d\n", rule->tr_name,
rule->tr_conds_str, rule->tr_rpc_rate,
atomic_read(&rule->tr_ref) - 1);
return 0;
if (head->th_cli_hash == NULL)
return -ENOMEM;
- start.u.tc_start.ts_opcodes = NULL;
start.u.tc_start.ts_opcodes_str = "*";
start.u.tc_start.ts_rpc_rate = tbf_rate;
break;
}
- if (rc == 0)
+ if (rc == 0 && bitmaptr)
*bitmaptr = opcodes;
else
CFS_FREE_BITMAP(opcodes);
static void nrs_tbf_opcode_cmd_fini(struct nrs_tbf_cmd *cmd)
{
- if (cmd->u.tc_start.ts_opcodes)
- CFS_FREE_BITMAP(cmd->u.tc_start.ts_opcodes);
-
if (cmd->u.tc_start.ts_opcodes_str)
OBD_FREE(cmd->u.tc_start.ts_opcodes_str,
strlen(cmd->u.tc_start.ts_opcodes_str) + 1);
/* parse opcode list */
rc = nrs_tbf_opcode_list_parse(cmd->u.tc_start.ts_opcodes_str,
strlen(cmd->u.tc_start.ts_opcodes_str),
- &cmd->u.tc_start.ts_opcodes);
+ NULL);
if (rc)
nrs_tbf_opcode_cmd_fini(cmd);
strlen(start->u.tc_start.ts_opcodes_str) + 1);
/* Default rule '*' */
- if (start->u.tc_start.ts_opcodes == NULL)
+ if (strcmp(start->u.tc_start.ts_opcodes_str, "*") == 0)
return 0;
rc = nrs_tbf_opcode_list_parse(rule->tr_opcodes_str,
static int
nrs_tbf_opcode_rule_dump(struct nrs_tbf_rule *rule, struct seq_file *m)
{
- seq_printf(m, "%s {%s} %llu, ref %d\n", rule->tr_name,
+ seq_printf(m, "%s {%s} %u, ref %d\n", rule->tr_name,
rule->tr_opcodes_str, rule->tr_rpc_rate,
atomic_read(&rule->tr_ref) - 1);
return 0;
static int
nrs_tbf_id_rule_dump(struct nrs_tbf_rule *rule, struct seq_file *m)
{
- seq_printf(m, "%s {%s} %llu, ref %d\n", rule->tr_name,
+ seq_printf(m, "%s {%s} %u, ref %d\n", rule->tr_name,
rule->tr_ids_str, rule->tr_rpc_rate,
atomic_read(&rule->tr_ref) - 1);
return 0;
*/
case NRS_CTL_TBF_RD_RULE: {
struct nrs_tbf_head *head = policy->pol_private;
- struct seq_file *m = (struct seq_file *) arg;
+ struct seq_file *m = arg;
struct ptlrpc_service_part *svcpt;
svcpt = policy->pol_nrs->nrs_svcpt;
struct nrs_tbf_rule *rule;
CDEBUG(D_RPCTRACE,
- "TBF class@%p rate %llu sequence %d, "
+ "TBF class@%p rate %u sequence %d, "
"rule flags %d, head sequence %d\n",
cli, cli->tc_rpc_rate,
cli->tc_rule_sequence,
&cli->tc_node);
}
CDEBUG(D_RPCTRACE,
- "TBF dequeues: class@%p rate %llu gen %llu "
- "token %llu, rule@%p rate %llu gen %llu\n",
+ "TBF dequeues: class@%p rate %u gen %llu "
+ "token %llu, rule@%p rate %u gen %llu\n",
cli, cli->tc_rpc_rate,
cli->tc_rule_generation, cli->tc_ntoken,
cli->tc_rule, cli->tc_rule->tr_rpc_rate,
if (rc == 0)
CDEBUG(D_RPCTRACE,
- "TBF enqueues: class@%p rate %llu gen %llu "
- "token %llu, rule@%p rate %llu gen %llu\n",
+ "TBF enqueues: class@%p rate %u gen %llu "
+ "token %llu, rule@%p rate %u gen %llu\n",
cli, cli->tc_rpc_rate,
cli->tc_rule_generation, cli->tc_ntoken,
cli->tc_rule, cli->tc_rule->tr_rpc_rate,
if (!svc->srv_debugfs_entry)
return 0;
- return ldebugfs_add_vars(svc->srv_debugfs_entry, nrs_tbf_lprocfs_vars,
- NULL);
+ ldebugfs_add_vars(svc->srv_debugfs_entry, nrs_tbf_lprocfs_vars, NULL);
+
+ return 0;
}
/**