diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-14 01:27:52 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:02:03 -0500 |
commit | 6bf1574ee33270e7c0b9d43103e8cedffd9f74db (patch) | |
tree | 926e7008acacd0e49c1de0d615073a0d916b5b89 /net/x25/af_x25.c | |
parent | 4dde4610c4ab54e9d36a4afaa98c23b017f7f9e3 (diff) |
[X25]: Avoid divides and sparse warnings
CHECK net/x25/af_x25.c
net/x25/af_x25.c:117:46: warning: expensive signed divide
CHECK net/x25/x25_facilities.c
net/x25/x25_facilities.c:209:30: warning: expensive signed divide
CHECK net/x25/x25_in.c
net/x25/x25_in.c:250:26: warning: expensive signed divide
CHECK net/x25/x25_proc.c
net/x25/x25_proc.c:48:11: warning: context imbalance in 'x25_seq_route_start'
- wrong count at exit
net/x25/x25_proc.c:72:13: warning: context imbalance in 'x25_seq_route_stop' -
unexpected unlock
net/x25/x25_proc.c:112:11: warning: context imbalance in
'x25_seq_socket_start' - wrong count at exit
net/x25/x25_proc.c:129:13: warning: context imbalance in 'x25_seq_socket_stop'
- unexpected unlock
net/x25/x25_proc.c:190:11: warning: context imbalance in
'x25_seq_forward_start' - wrong count at exit
net/x25/x25_proc.c:215:13: warning: context imbalance in
'x25_seq_forward_stop' - unexpected unlock
CHECK net/x25/x25_subr.c
net/x25/x25_subr.c:362:57: 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/x25/af_x25.c')
-rw-r--r-- | net/x25/af_x25.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/x25/af_x25.c b/net/x25/af_x25.c index 92cfe8e3e0b8..07fad7ccf832 100644 --- a/net/x25/af_x25.c +++ b/net/x25/af_x25.c | |||
@@ -83,9 +83,9 @@ struct compat_x25_subscrip_struct { | |||
83 | int x25_addr_ntoa(unsigned char *p, struct x25_address *called_addr, | 83 | int x25_addr_ntoa(unsigned char *p, struct x25_address *called_addr, |
84 | struct x25_address *calling_addr) | 84 | struct x25_address *calling_addr) |
85 | { | 85 | { |
86 | int called_len, calling_len; | 86 | unsigned int called_len, calling_len; |
87 | char *called, *calling; | 87 | char *called, *calling; |
88 | int i; | 88 | unsigned int i; |
89 | 89 | ||
90 | called_len = (*p >> 0) & 0x0F; | 90 | called_len = (*p >> 0) & 0x0F; |
91 | calling_len = (*p >> 4) & 0x0F; | 91 | calling_len = (*p >> 4) & 0x0F; |