Skip to content

Commit

Permalink
tracepoints: use TABLE_SIZE macro
Browse files Browse the repository at this point in the history
Steven Rostedt suggested:

| Wouldn't it look nicer to have: (TRACEPOINT_TABLE_SIZE - 1) ?

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
Cc: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
  • Loading branch information
Mathieu Desnoyers authored and Ingo Molnar committed Oct 14, 2008
1 parent 611b159 commit 9795302
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions kernel/tracepoint.c
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ static struct tracepoint_entry *get_tracepoint(const char *name)
struct tracepoint_entry *e;
u32 hash = jhash(name, strlen(name), 0);

head = &tracepoint_table[hash & ((1 << TRACEPOINT_HASH_BITS)-1)];
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
hlist_for_each_entry(e, node, head, hlist) {
if (!strcmp(name, e->name))
return e;
Expand All @@ -197,7 +197,7 @@ static struct tracepoint_entry *add_tracepoint(const char *name)
size_t name_len = strlen(name) + 1;
u32 hash = jhash(name, name_len-1, 0);

head = &tracepoint_table[hash & ((1 << TRACEPOINT_HASH_BITS)-1)];
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
hlist_for_each_entry(e, node, head, hlist) {
if (!strcmp(name, e->name)) {
printk(KERN_NOTICE
Expand Down Expand Up @@ -233,7 +233,7 @@ static int remove_tracepoint(const char *name)
size_t len = strlen(name) + 1;
u32 hash = jhash(name, len-1, 0);

head = &tracepoint_table[hash & ((1 << TRACEPOINT_HASH_BITS)-1)];
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
hlist_for_each_entry(e, node, head, hlist) {
if (!strcmp(name, e->name)) {
found = 1;
Expand Down

0 comments on commit 9795302

Please sign in to comment.