mm: add a low limit to alloc_large_system_hash

UDP stack needs a minimum hash size value for proper operation and also
uses alloc_large_system_hash() for proper NUMA distribution of its hash
tables and automatic sizing depending on available system memory.

On some low memory situations, udp_table_init() must ignore the
alloc_large_system_hash() result and reallocs a bigger memory area.

As we cannot easily free old hash table, we leak it and kmemleak can
issue a warning.

This patch adds a low limit parameter to alloc_large_system_hash() to
solve this problem.

We then specify UDP_HTABLE_SIZE_MIN for UDP/UDPLite hash table
allocation.

Reported-by: Mark Asselstine <mark.asselstine@windriver.com>
Reported-by: Tim Bird <tim.bird@am.sony.com>
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 10 insertions, 20 deletions

diff --git a/net/ipv4/udp.c b/net/ipv4/udp.c
index 609397ee78fb..eaca73644e79 100644
--- a/net/ipv4/udp.c
+++ b/net/ipv4/udp.c
@@ -2192,26 +2192,16 @@ void __init udp_table_init(struct udp_table *table, const char *name)
 {
         unsigned int i;
 
-        if (!CONFIG_BASE_SMALL)
-                table->hash = alloc_large_system_hash(name,
-                        2 * sizeof(struct udp_hslot),
-                        uhash_entries,
-                        21, /* one slot per 2 MB */
-                        0,
-                        &table->log,
-                        &table->mask,
-                        64 * 1024);
-        /*
-         * Make sure hash table has the minimum size
-         */
-        if (CONFIG_BASE_SMALL || table->mask < UDP_HTABLE_SIZE_MIN - 1) {
-                table->hash = kmalloc(UDP_HTABLE_SIZE_MIN *
-                                      2 * sizeof(struct udp_hslot), GFP_KERNEL);
-                if (!table->hash)
-                        panic(name);
-                table->log = ilog2(UDP_HTABLE_SIZE_MIN);
-                table->mask = UDP_HTABLE_SIZE_MIN - 1;
-        }
+        table->hash = alloc_large_system_hash(name,
+                                              2 * sizeof(struct udp_hslot),
+                                              uhash_entries,
+                                              21, /* one slot per 2 MB */
+                                              0,
+                                              &table->log,
+                                              &table->mask,
+                                              UDP_HTABLE_SIZE_MIN,
+                                              64 * 1024);
+
         table->hash2 = table->hash + (table->mask + 1);
         for (i = 0; i <= table->mask; i++) {
                 INIT_HLIST_NULLS_HEAD(&table->hash[i].head, i);