From: Luis R. Rodriguez Date: Wed, 2 Apr 2014 00:54:30 +0000 (+0000) Subject: backports: backport inet_frag_maybe_warn_overflow() X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=9cffb9dd036e0e3e0ee6d91c715bea4d82365c28;p=openwrt%2Fstaging%2Fblogic.git backports: backport inet_frag_maybe_warn_overflow() This is used by the ieee802154 reassembly code. This is a straight forward backport. mcgrof@ergon ~/linux-next (git::master)$ git describe --contains 5a3da1fe95 v3.9-rc4~27^2 commit 5a3da1fe9561828d0ca7eca664b16ec2b9bf0055 Author: Hannes Frederic Sowa Date: Fri Mar 15 11:32:30 2013 +0000 inet: limit length of fragment queue hash table bucket lists This patch introduces a constant limit of the fragment queue hash table bucket list lengths. Currently the limit 128 is choosen somewhat arbitrary and just ensures that we can fill up the fragment cache with empty packets up to the default ip_frag_high_thresh limits. It should just protect from list iteration eating considerable amounts of cpu. If we reach the maximum length in one hash bucket a warning is printed. This is implemented on the caller side of inet_frag_find to distinguish between the different users of inet_fragment.c. I dropped the out of memory warning in the ipv4 fragment lookup path, because we already get a warning by the slab allocator. Cc: Eric Dumazet Cc: Jesper Dangaard Brouer Signed-off-by: Hannes Frederic Sowa Acked-by: Eric Dumazet Signed-off-by: David S. Miller Cc: Hannes Frederic Sowa Signed-off-by: Luis R. Rodriguez --- diff --git a/backport/backport-include/net/inet_frag.h b/backport/backport-include/net/inet_frag.h index 167792bfdd59..3c9e11a47e69 100644 --- a/backport/backport-include/net/inet_frag.h +++ b/backport/backport-include/net/inet_frag.h @@ -34,6 +34,11 @@ static inline int sum_frag_mem_limit(struct netns_frags *nf) { return atomic_read(&nf->mem); } + +#define inet_frag_maybe_warn_overflow LINUX_BACKPORT(inet_frag_maybe_warn_overflow) +void inet_frag_maybe_warn_overflow(struct inet_frag_queue *q, + const char *prefix); #endif /* LINUX_VERSION_CODE < KERNEL_VERSION(3,9,0) */ + #endif /* __BACKPORT__NET_FRAG_H__ */ diff --git a/backport/compat/compat-3.9.c b/backport/compat/compat-3.9.c index df10d7d64e1e..5a4f52390cbe 100644 --- a/backport/compat/compat-3.9.c +++ b/backport/compat/compat-3.9.c @@ -15,6 +15,8 @@ #include #include #include +#include +#include void __iomem *devm_ioremap_resource(struct device *dev, struct resource *res) { @@ -56,3 +58,15 @@ void eth_commit_mac_addr_change(struct net_device *dev, void *p) memcpy(dev->dev_addr, addr->sa_data, ETH_ALEN); } EXPORT_SYMBOL_GPL(eth_commit_mac_addr_change); + +void inet_frag_maybe_warn_overflow(struct inet_frag_queue *q, + const char *prefix) +{ + static const char msg[] = "inet_frag_find: Fragment hash bucket" + " list length grew over limit " __stringify(INETFRAGS_MAXDEPTH) + ". Dropping fragment.\n"; + + if (PTR_ERR(q) == -ENOBUFS) + LIMIT_NETDEBUG(KERN_WARNING "%s%s", prefix, msg); +} +EXPORT_SYMBOL_GPL(inet_frag_maybe_warn_overflow);