diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-14 01:29:41 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:02:03 -0500 |
commit | f16f3026db6fa63cbb0f4a37833562aa999c93e5 (patch) | |
tree | 94cf1a39795d04315a1dd2eadbe2d77a453b3844 /net/ax25/af_ax25.c | |
parent | 6bf1574ee33270e7c0b9d43103e8cedffd9f74db (diff) |
[AX25]: sparse cleanups
net/ax25/ax25_route.c:251:13: warning: context imbalance in
'ax25_rt_seq_start' - wrong count at exit
net/ax25/ax25_route.c:276:13: warning: context imbalance in 'ax25_rt_seq_stop'
- unexpected unlock
net/ax25/ax25_std_timer.c:65:25: warning: expensive signed divide
net/ax25/ax25_uid.c:46:1: warning: symbol 'ax25_uid_list' was not declared.
Should it be static?
net/ax25/ax25_uid.c:146:13: warning: context imbalance in 'ax25_uid_seq_start'
- wrong count at exit
net/ax25/ax25_uid.c:169:13: warning: context imbalance in 'ax25_uid_seq_stop'
- unexpected unlock
net/ax25/af_ax25.c:573:28: warning: expensive signed divide
net/ax25/af_ax25.c:1865:13: warning: context imbalance in 'ax25_info_start' -
wrong count at exit
net/ax25/af_ax25.c:1888:13: warning: context imbalance in 'ax25_info_stop' -
unexpected unlock
net/ax25/ax25_ds_timer.c:133:25: warning: expensive signed divide
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ax25/af_ax25.c')
-rw-r--r-- | net/ax25/af_ax25.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c index a028d37ba2e0..1bc0e85f04a5 100644 --- a/net/ax25/af_ax25.c +++ b/net/ax25/af_ax25.c | |||
@@ -570,7 +570,7 @@ static int ax25_setsockopt(struct socket *sock, int level, int optname, | |||
570 | res = -EINVAL; | 570 | res = -EINVAL; |
571 | break; | 571 | break; |
572 | } | 572 | } |
573 | ax25->rtt = (opt * HZ) / 2; | 573 | ax25->rtt = (opt * HZ) >> 1; |
574 | ax25->t1 = opt * HZ; | 574 | ax25->t1 = opt * HZ; |
575 | break; | 575 | break; |
576 | 576 | ||
@@ -1863,6 +1863,7 @@ static int ax25_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg) | |||
1863 | #ifdef CONFIG_PROC_FS | 1863 | #ifdef CONFIG_PROC_FS |
1864 | 1864 | ||
1865 | static void *ax25_info_start(struct seq_file *seq, loff_t *pos) | 1865 | static void *ax25_info_start(struct seq_file *seq, loff_t *pos) |
1866 | __acquires(ax25_list_lock) | ||
1866 | { | 1867 | { |
1867 | struct ax25_cb *ax25; | 1868 | struct ax25_cb *ax25; |
1868 | struct hlist_node *node; | 1869 | struct hlist_node *node; |
@@ -1886,6 +1887,7 @@ static void *ax25_info_next(struct seq_file *seq, void *v, loff_t *pos) | |||
1886 | } | 1887 | } |
1887 | 1888 | ||
1888 | static void ax25_info_stop(struct seq_file *seq, void *v) | 1889 | static void ax25_info_stop(struct seq_file *seq, void *v) |
1890 | __releases(ax25_list_lock) | ||
1889 | { | 1891 | { |
1890 | spin_unlock_bh(&ax25_list_lock); | 1892 | spin_unlock_bh(&ax25_list_lock); |
1891 | } | 1893 | } |