diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-15 06:30:35 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:02:44 -0500 |
commit | 95b7d924a589dbefc7ae2ea6c7144b86b75d6a47 (patch) | |
tree | 887766c4c98864db4fa1ac0ec5b97bcc90bc2ab5 /net/rose/af_rose.c | |
parent | 5c17d5f11212af5f12b91991b1132cf301dd1f28 (diff) |
[ROSE]: Supress sparse warnings
CHECK net/rose/af_rose.c
net/rose/af_rose.c:125:11: warning: expensive signed divide
net/rose/af_rose.c:976:46: warning: expensive signed divide
net/rose/af_rose.c:1379:13: warning: context imbalance in 'rose_info_start' - wrong count at exit
net/rose/af_rose.c:1406:13: warning: context imbalance in 'rose_info_stop' - unexpected unlock
CHECK net/rose/rose_in.c
net/rose/rose_in.c:185:25: warning: expensive signed divide
CHECK net/rose/rose_route.c
net/rose/rose_route.c:997:46: warning: expensive signed divide
net/rose/rose_route.c:1070:13: warning: context imbalance in 'rose_node_start' - wrong count at exit
net/rose/rose_route.c:1093:13: warning: context imbalance in 'rose_node_stop' - unexpected unlock
net/rose/rose_route.c:1146:13: warning: context imbalance in 'rose_neigh_start' - wrong count at exit
net/rose/rose_route.c:1169:13: warning: context imbalance in 'rose_neigh_stop' - unexpected unlock
net/rose/rose_route.c:1229:13: warning: context imbalance in 'rose_route_start' - wrong count at exit
net/rose/rose_route.c:1252:13: warning: context imbalance in 'rose_route_stop' - unexpected unlock
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rose/af_rose.c')
-rw-r--r-- | net/rose/af_rose.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/net/rose/af_rose.c b/net/rose/af_rose.c index 323d42a7b360..4a31a81059ab 100644 --- a/net/rose/af_rose.c +++ b/net/rose/af_rose.c | |||
@@ -116,7 +116,7 @@ int rosecmp(rose_address *addr1, rose_address *addr2) | |||
116 | */ | 116 | */ |
117 | int rosecmpm(rose_address *addr1, rose_address *addr2, unsigned short mask) | 117 | int rosecmpm(rose_address *addr1, rose_address *addr2, unsigned short mask) |
118 | { | 118 | { |
119 | int i, j; | 119 | unsigned int i, j; |
120 | 120 | ||
121 | if (mask > 10) | 121 | if (mask > 10) |
122 | return 1; | 122 | return 1; |
@@ -973,8 +973,8 @@ int rose_rx_call_request(struct sk_buff *skb, struct net_device *dev, struct ros | |||
973 | */ | 973 | */ |
974 | memset(&facilities, 0x00, sizeof(struct rose_facilities_struct)); | 974 | memset(&facilities, 0x00, sizeof(struct rose_facilities_struct)); |
975 | 975 | ||
976 | len = (((skb->data[3] >> 4) & 0x0F) + 1) / 2; | 976 | len = (((skb->data[3] >> 4) & 0x0F) + 1) >> 1; |
977 | len += (((skb->data[3] >> 0) & 0x0F) + 1) / 2; | 977 | len += (((skb->data[3] >> 0) & 0x0F) + 1) >> 1; |
978 | if (!rose_parse_facilities(skb->data + len + 4, &facilities)) { | 978 | if (!rose_parse_facilities(skb->data + len + 4, &facilities)) { |
979 | rose_transmit_clear_request(neigh, lci, ROSE_INVALID_FACILITY, 76); | 979 | rose_transmit_clear_request(neigh, lci, ROSE_INVALID_FACILITY, 76); |
980 | return 0; | 980 | return 0; |
@@ -1377,6 +1377,7 @@ static int rose_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg) | |||
1377 | 1377 | ||
1378 | #ifdef CONFIG_PROC_FS | 1378 | #ifdef CONFIG_PROC_FS |
1379 | static void *rose_info_start(struct seq_file *seq, loff_t *pos) | 1379 | static void *rose_info_start(struct seq_file *seq, loff_t *pos) |
1380 | __acquires(rose_list_lock) | ||
1380 | { | 1381 | { |
1381 | int i; | 1382 | int i; |
1382 | struct sock *s; | 1383 | struct sock *s; |
@@ -1404,6 +1405,7 @@ static void *rose_info_next(struct seq_file *seq, void *v, loff_t *pos) | |||
1404 | } | 1405 | } |
1405 | 1406 | ||
1406 | static void rose_info_stop(struct seq_file *seq, void *v) | 1407 | static void rose_info_stop(struct seq_file *seq, void *v) |
1408 | __releases(rose_list_lock) | ||
1407 | { | 1409 | { |
1408 | spin_unlock_bh(&rose_list_lock); | 1410 | spin_unlock_bh(&rose_list_lock); |
1409 | } | 1411 | } |