Skip to content

Commit

Permalink
ipv6 flowlabel: add __rcu annotations
Browse files Browse the repository at this point in the history
Commit 1836768 (ipv6 flowlabel: Convert np->ipv6_fl_list to RCU.)
omitted proper __rcu annotations.

Signed-off-by: Eric Dumazet <[email protected]>
Cc: YOSHIFUJI Hideaki <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
Eric Dumazet authored and davem330 committed Mar 7, 2013
1 parent bf5e4dd commit 7f0e44a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 7 deletions.
8 changes: 4 additions & 4 deletions include/net/ipv6.h
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ struct ipv6_txoptions {
};

struct ip6_flowlabel {
struct ip6_flowlabel *next;
struct ip6_flowlabel __rcu *next;
__be32 label;
atomic_t users;
struct in6_addr dst;
Expand All @@ -238,9 +238,9 @@ struct ip6_flowlabel {
#define IPV6_FLOWLABEL_MASK cpu_to_be32(0x000FFFFF)

struct ipv6_fl_socklist {
struct ipv6_fl_socklist *next;
struct ip6_flowlabel *fl;
struct rcu_head rcu;
struct ipv6_fl_socklist __rcu *next;
struct ip6_flowlabel *fl;
struct rcu_head rcu;
};

extern struct ip6_flowlabel *fl6_sock_lookup(struct sock *sk, __be32 label);
Expand Down
11 changes: 8 additions & 3 deletions net/ipv6/ip6_flowlabel.c
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,9 @@ static void ip6_fl_gc(unsigned long dummy)
spin_lock(&ip6_fl_lock);

for (i=0; i<=FL_HASH_MASK; i++) {
struct ip6_flowlabel *fl, **flp;
struct ip6_flowlabel *fl;
struct ip6_flowlabel __rcu **flp;

flp = &fl_ht[i];
while ((fl = rcu_dereference_protected(*flp,
lockdep_is_held(&ip6_fl_lock))) != NULL) {
Expand Down Expand Up @@ -179,7 +181,9 @@ static void __net_exit ip6_fl_purge(struct net *net)

spin_lock(&ip6_fl_lock);
for (i = 0; i <= FL_HASH_MASK; i++) {
struct ip6_flowlabel *fl, **flp;
struct ip6_flowlabel *fl;
struct ip6_flowlabel __rcu **flp;

flp = &fl_ht[i];
while ((fl = rcu_dereference_protected(*flp,
lockdep_is_held(&ip6_fl_lock))) != NULL) {
Expand Down Expand Up @@ -506,7 +510,8 @@ int ipv6_flowlabel_opt(struct sock *sk, char __user *optval, int optlen)
struct ipv6_pinfo *np = inet6_sk(sk);
struct in6_flowlabel_req freq;
struct ipv6_fl_socklist *sfl1=NULL;
struct ipv6_fl_socklist *sfl, **sflp;
struct ipv6_fl_socklist *sfl;
struct ipv6_fl_socklist __rcu **sflp;
struct ip6_flowlabel *fl, *fl1 = NULL;


Expand Down

0 comments on commit 7f0e44a

Please sign in to comment.