tcp: metrics: Avoid duplicate entries with the same destination-IP
authorChristoph Paasch <christoph.paasch@uclouvain.be>
Thu, 16 Jan 2014 19:01:21 +0000 (20:01 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sat, 18 Jan 2014 02:05:34 +0000 (18:05 -0800)
commit77f99ad16a07aa062c2d30fae57b1fee456f6ef6
treed95bb59a1fdcab37cd1eef6d909a4e75fc75d4c5
parentc196403b79aa241c3fefb3ee5bb328aa7c5cc860
tcp: metrics: Avoid duplicate entries with the same destination-IP

Because the tcp-metrics is an RCU-list, it may be that two
soft-interrupts are inside __tcp_get_metrics() for the same
destination-IP at the same time. If this destination-IP is not yet part of
the tcp-metrics, both soft-interrupts will end up in tcpm_new and create
a new entry for this IP.
So, we will have two tcp-metrics with the same destination-IP in the list.

This patch checks twice __tcp_get_metrics(). First without holding the
lock, then while holding the lock. The second one is there to confirm
that the entry has not been added by another soft-irq while waiting for
the spin-lock.

Fixes: 51c5d0c4b169b (tcp: Maintain dynamic metrics in local cache.)
Signed-off-by: Christoph Paasch <christoph.paasch@uclouvain.be>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_metrics.c