diff options
author | Stephen Hemminger <shemminger@vyatta.com> | 2008-01-21 20:28:31 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:10:18 -0500 |
commit | dd329bfa96dd9dabfc3b5154317bf5fbe9440455 (patch) | |
tree | ad68995cd1d5484a1141f39726ff9afc5967de97 /net/ipv4 | |
parent | 1402c8519acba215f5a6101b9e5ada07ab371273 (diff) |
[IPV4]: igmp sparse warnings
Partial sparse warning fix. The other conditional locking
is too much for sparse to handle.
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r-- | net/ipv4/igmp.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/ipv4/igmp.c b/net/ipv4/igmp.c index 285d26218a51..016cfdb184f3 100644 --- a/net/ipv4/igmp.c +++ b/net/ipv4/igmp.c | |||
@@ -2329,6 +2329,7 @@ static struct ip_mc_list *igmp_mc_get_idx(struct seq_file *seq, loff_t pos) | |||
2329 | } | 2329 | } |
2330 | 2330 | ||
2331 | static void *igmp_mc_seq_start(struct seq_file *seq, loff_t *pos) | 2331 | static void *igmp_mc_seq_start(struct seq_file *seq, loff_t *pos) |
2332 | __acquires(dev_base_lock) | ||
2332 | { | 2333 | { |
2333 | read_lock(&dev_base_lock); | 2334 | read_lock(&dev_base_lock); |
2334 | return *pos ? igmp_mc_get_idx(seq, *pos - 1) : SEQ_START_TOKEN; | 2335 | return *pos ? igmp_mc_get_idx(seq, *pos - 1) : SEQ_START_TOKEN; |
@@ -2346,6 +2347,7 @@ static void *igmp_mc_seq_next(struct seq_file *seq, void *v, loff_t *pos) | |||
2346 | } | 2347 | } |
2347 | 2348 | ||
2348 | static void igmp_mc_seq_stop(struct seq_file *seq, void *v) | 2349 | static void igmp_mc_seq_stop(struct seq_file *seq, void *v) |
2350 | __releases(dev_base_lock) | ||
2349 | { | 2351 | { |
2350 | struct igmp_mc_iter_state *state = igmp_mc_seq_private(seq); | 2352 | struct igmp_mc_iter_state *state = igmp_mc_seq_private(seq); |
2351 | if (likely(state->in_dev != NULL)) { | 2353 | if (likely(state->in_dev != NULL)) { |