* [B.A.T.M.A.N.] [PATCH] batman-adv: TT_CLIENT_PENDING is never set in struct tt_global_entry
@ 2012-01-12 18:07 Antonio Quartulli
2012-01-15 19:57 ` Marek Lindner
0 siblings, 1 reply; 2+ messages in thread
From: Antonio Quartulli @ 2012-01-12 18:07 UTC (permalink / raw)
To: b.a.t.m.a.n
Actually the TT_CLIENT_PENDING flag is never set in the tt_global_entry
structure, therefore this code is useless and can be removed.
Reported-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
---
translation-table.c | 9 +--------
1 files changed, 1 insertions(+), 8 deletions(-)
diff --git a/translation-table.c b/translation-table.c
index d105cbd..27753a0 100644
--- a/translation-table.c
+++ b/translation-table.c
@@ -603,7 +603,7 @@ int tt_global_seq_print_text(struct seq_file *seq, void *offset)
struct tt_global_entry,
common);
seq_printf(seq, " * %pM (%3u) via %pM (%3u) "
- "[%c%c%c]\n",
+ "[%c%c]\n",
tt_global_entry->common.addr,
tt_global_entry->ttvn,
tt_global_entry->orig_node->orig,
@@ -613,8 +613,6 @@ int tt_global_seq_print_text(struct seq_file *seq, void *offset)
(tt_global_entry->common.flags &
TT_CLIENT_ROAM ? 'R' : '.'),
(tt_global_entry->common.flags &
- TT_CLIENT_PENDING ? 'X' : '.'),
- (tt_global_entry->common.flags &
TT_CLIENT_WIFI ? 'W' : '.'));
}
rcu_read_unlock();
@@ -838,11 +836,6 @@ struct orig_node *transtable_search(struct bat_priv *bat_priv,
if (!atomic_inc_not_zero(&tt_global_entry->orig_node->refcount))
goto out;
- /* A global client marked as PENDING has already moved from that
- * originator */
- if (tt_global_entry->common.flags & TT_CLIENT_PENDING)
- goto out;
-
orig_node = tt_global_entry->orig_node;
out:
--
1.7.3.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [B.A.T.M.A.N.] [PATCH] batman-adv: TT_CLIENT_PENDING is never set in struct tt_global_entry
2012-01-12 18:07 [B.A.T.M.A.N.] [PATCH] batman-adv: TT_CLIENT_PENDING is never set in struct tt_global_entry Antonio Quartulli
@ 2012-01-15 19:57 ` Marek Lindner
0 siblings, 0 replies; 2+ messages in thread
From: Marek Lindner @ 2012-01-15 19:57 UTC (permalink / raw)
To: The list for a Better Approach To Mobile Ad-hoc Networking
On Friday, January 13, 2012 02:07:00 Antonio Quartulli wrote:
> Actually the TT_CLIENT_PENDING flag is never set in the tt_global_entry
> structure, therefore this code is useless and can be removed.
Applied in revision e40ea15.
Thanks,
Marek
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-01-15 19:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-12 18:07 [B.A.T.M.A.N.] [PATCH] batman-adv: TT_CLIENT_PENDING is never set in struct tt_global_entry Antonio Quartulli
2012-01-15 19:57 ` Marek Lindner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).