Skip to content

Commit

Permalink
IPv6: Delete redundant counter of IPSTATS_MIB_REASMFAILS
Browse files Browse the repository at this point in the history
When no more memory can be allocated, fq_find() will return NULL and
increase the value of IPSTATS_MIB_REASMFAILS. In this case,
ipv6_frag_rcv() also increase the value of IPSTATS_MIB_REASMFAILS.

So, the patch deletes redundant counter of IPSTATS_MIB_REASMFAILS in fq_find().
and deletes the unused parameter of idev.

Signed-off-by: Shan Wei <shanwei@cn.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Shan Wei authored and David S. Miller committed Feb 16, 2010
1 parent 564517e commit 9546377
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions net/ipv6/reassembly.c
Original file line number Diff line number Diff line change
Expand Up @@ -237,8 +237,7 @@ static void ip6_frag_expire(unsigned long data)
}

static __inline__ struct frag_queue *
fq_find(struct net *net, __be32 id, struct in6_addr *src, struct in6_addr *dst,
struct inet6_dev *idev)
fq_find(struct net *net, __be32 id, struct in6_addr *src, struct in6_addr *dst)
{
struct inet_frag_queue *q;
struct ip6_create_arg arg;
Expand All @@ -254,13 +253,9 @@ fq_find(struct net *net, __be32 id, struct in6_addr *src, struct in6_addr *dst,

q = inet_frag_find(&net->ipv6.frags, &ip6_frags, &arg, hash);
if (q == NULL)
goto oom;
return NULL;

return container_of(q, struct frag_queue, q);

oom:
IP6_INC_STATS_BH(net, idev, IPSTATS_MIB_REASMFAILS);
return NULL;
}

static int ip6_frag_queue(struct frag_queue *fq, struct sk_buff *skb,
Expand Down Expand Up @@ -606,8 +601,8 @@ static int ipv6_frag_rcv(struct sk_buff *skb)
if (atomic_read(&net->ipv6.frags.mem) > net->ipv6.frags.high_thresh)
ip6_evictor(net, ip6_dst_idev(skb_dst(skb)));

if ((fq = fq_find(net, fhdr->identification, &hdr->saddr, &hdr->daddr,
ip6_dst_idev(skb_dst(skb)))) != NULL) {
fq = fq_find(net, fhdr->identification, &hdr->saddr, &hdr->daddr);
if (fq != NULL) {
int ret;

spin_lock(&fq->q.lock);
Expand Down

0 comments on commit 9546377

Please sign in to comment.