diff options
| author | Hannes Eder <hannes@hanneseder.net> | 2009-02-14 11:48:44 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-02-17 19:34:49 -0800 |
| commit | b1ae1e99bd9f611f16a09c1bca2a0a38e67b88c3 (patch) | |
| tree | df9b742f01d081f4767041327d121cf1b2a1a32f | |
| parent | 5ed1e981bd243d78f95738a355bd117cf1f6daa1 (diff) | |
| download | linux-b1ae1e99bd9f611f16a09c1bca2a0a38e67b88c3.tar.bz2 | |
drivers/net/wireless: fix sparse warning: context imbalance
Impact: Attribute function with __acquires(...) resp. __releases(...).
Fix this sparse warnings:
drivers/net/wireless/strip.c:955:21: warning: context imbalance in 'strip_seq_start' - wrong count at exit
drivers/net/wireless/strip.c:975:13: warning: context imbalance in 'strip_seq_stop' - unexpected unlock
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/wireless/strip.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wireless/strip.c b/drivers/net/wireless/strip.c index d6bf8d2ef8ea..39f525efdc8d 100644 --- a/drivers/net/wireless/strip.c +++ b/drivers/net/wireless/strip.c @@ -950,6 +950,7 @@ static struct strip *strip_get_idx(loff_t pos) } static void *strip_seq_start(struct seq_file *seq, loff_t *pos) + __acquires(RCU) { rcu_read_lock(); return *pos ? strip_get_idx(*pos - 1) : SEQ_START_TOKEN; @@ -973,6 +974,7 @@ static void *strip_seq_next(struct seq_file *seq, void *v, loff_t *pos) } static void strip_seq_stop(struct seq_file *seq, void *v) + __releases(RCU) { rcu_read_unlock(); } |