diff options
| author | Hannes Eder <hannes@hanneseder.net> | 2009-02-25 11:32:25 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-02-27 08:13:34 +0100 |
| commit | 8521c27ee79533db444dcac291992b6bf8d61fc8 (patch) | |
| tree | 2002369a6b2fbc370b7b4075b23ec67e61dde669 /net/decnet | |
| parent | sysctl: fix sparse warning: Should it be static? (diff) | |
| download | linux-8521c27ee79533db444dcac291992b6bf8d61fc8.tar.xz linux-8521c27ee79533db444dcac291992b6bf8d61fc8.zip | |
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>
Diffstat (limited to 'net/decnet')
| -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) } static void *dn_dev_seq_start(struct seq_file *seq, loff_t *pos) + __acquires(&dev_base_lock) { int i; struct net_device *dev; @@ -1365,6 +1366,7 @@ static void *dn_dev_seq_next(struct seq_file *seq, void *v, loff_t *pos) } static void dn_dev_seq_stop(struct seq_file *seq, void *v) + __releases(&dev_base_lock) { read_unlock(&dev_base_lock); } |