diff options
author | Hannes Eder <hannes@hanneseder.net> | 2009-02-25 05:32:25 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-02-27 02:13:34 -0500 |
commit | 8521c27ee79533db444dcac291992b6bf8d61fc8 (patch) | |
tree | 2002369a6b2fbc370b7b4075b23ec67e61dde669 | |
parent | 63d819caebcc3ca260ca0214d29044293969aa2f (diff) |
decnet: fix sparse warnings: context imbalance
Impact: Attribute functions with __acquires(...) resp. __releases(...).
Fix this sparse warnings:
net/decnet/dn_dev.c:1324:13: warning: context imbalance in 'dn_dev_seq_start' - wrong count at exit
net/decnet/dn_dev.c:1366:13: warning: context imbalance in 'dn_dev_seq_stop' - unexpected unlock
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/decnet/dn_dev.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/decnet/dn_dev.c b/net/decnet/dn_dev.c index e457769bf7a7..b6c80f3b8185 100644 --- a/net/decnet/dn_dev.c +++ b/net/decnet/dn_dev.c | |||
@@ -1323,6 +1323,7 @@ static inline int is_dn_dev(struct net_device *dev) | |||
1323 | } | 1323 | } |
1324 | 1324 | ||
1325 | static void *dn_dev_seq_start(struct seq_file *seq, loff_t *pos) | 1325 | static void *dn_dev_seq_start(struct seq_file *seq, loff_t *pos) |
1326 | __acquires(&dev_base_lock) | ||
1326 | { | 1327 | { |
1327 | int i; | 1328 | int i; |
1328 | struct net_device *dev; | 1329 | struct net_device *dev; |
@@ -1365,6 +1366,7 @@ static void *dn_dev_seq_next(struct seq_file *seq, void *v, loff_t *pos) | |||
1365 | } | 1366 | } |
1366 | 1367 | ||
1367 | static void dn_dev_seq_stop(struct seq_file *seq, void *v) | 1368 | static void dn_dev_seq_stop(struct seq_file *seq, void *v) |
1369 | __releases(&dev_base_lock) | ||
1368 | { | 1370 | { |
1369 | read_unlock(&dev_base_lock); | 1371 | read_unlock(&dev_base_lock); |
1370 | } | 1372 | } |