Commit 5c17d5f1 authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller

[ATM]: Suppress some sparse warnings

  CHECK   net/atm/br2684.c
net/atm/br2684.c:665:13: warning: context imbalance in 'br2684_seq_start' - wrong count at exit
net/atm/br2684.c:676:13: warning: context imbalance in 'br2684_seq_stop' - unexpected unlock
  CHECK   net/atm/lec.c
net/atm/lec.c:196:23: warning: expensive signed divide
  CHECK   net/atm/proc.c
net/atm/proc.c:151:14: warning: context imbalance in 'vcc_seq_start' - wrong count at exit
net/atm/proc.c:154:13: warning: context imbalance in 'vcc_seq_stop' - unexpected unlock
Signed-off-by: default avatarEric Dumazet <dada1@cosmosbay.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ca629f24
...@@ -663,6 +663,7 @@ static struct atm_ioctl br2684_ioctl_ops = { ...@@ -663,6 +663,7 @@ static struct atm_ioctl br2684_ioctl_ops = {
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
static void *br2684_seq_start(struct seq_file *seq, loff_t * pos) static void *br2684_seq_start(struct seq_file *seq, loff_t * pos)
__acquires(devs_lock)
{ {
read_lock(&devs_lock); read_lock(&devs_lock);
return seq_list_start(&br2684_devs, *pos); return seq_list_start(&br2684_devs, *pos);
...@@ -674,6 +675,7 @@ static void *br2684_seq_next(struct seq_file *seq, void *v, loff_t * pos) ...@@ -674,6 +675,7 @@ static void *br2684_seq_next(struct seq_file *seq, void *v, loff_t * pos)
} }
static void br2684_seq_stop(struct seq_file *seq, void *v) static void br2684_seq_stop(struct seq_file *seq, void *v)
__releases(devs_lock)
{ {
read_unlock(&devs_lock); read_unlock(&devs_lock);
} }
......
...@@ -176,7 +176,7 @@ static void lec_handle_bridge(struct sk_buff *skb, struct net_device *dev) ...@@ -176,7 +176,7 @@ static void lec_handle_bridge(struct sk_buff *skb, struct net_device *dev)
static unsigned char *get_tr_dst(unsigned char *packet, unsigned char *rdesc) static unsigned char *get_tr_dst(unsigned char *packet, unsigned char *rdesc)
{ {
struct trh_hdr *trh; struct trh_hdr *trh;
int riflen, num_rdsc; unsigned int riflen, num_rdsc;
trh = (struct trh_hdr *)packet; trh = (struct trh_hdr *)packet;
if (trh->daddr[0] & (uint8_t) 0x80) if (trh->daddr[0] & (uint8_t) 0x80)
......
...@@ -142,6 +142,7 @@ static int vcc_seq_release(struct inode *inode, struct file *file) ...@@ -142,6 +142,7 @@ static int vcc_seq_release(struct inode *inode, struct file *file)
} }
static void *vcc_seq_start(struct seq_file *seq, loff_t *pos) static void *vcc_seq_start(struct seq_file *seq, loff_t *pos)
__acquires(vcc_sklist_lock)
{ {
struct vcc_state *state = seq->private; struct vcc_state *state = seq->private;
loff_t left = *pos; loff_t left = *pos;
...@@ -152,6 +153,7 @@ static void *vcc_seq_start(struct seq_file *seq, loff_t *pos) ...@@ -152,6 +153,7 @@ static void *vcc_seq_start(struct seq_file *seq, loff_t *pos)
} }
static void vcc_seq_stop(struct seq_file *seq, void *v) static void vcc_seq_stop(struct seq_file *seq, void *v)
__releases(vcc_sklist_lock)
{ {
read_unlock(&vcc_sklist_lock); read_unlock(&vcc_sklist_lock);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment