net/mlx4_en: remove fallback after kzalloc_node()
authorEric Dumazet <edumazet@google.com>
Thu, 13 Dec 2018 11:03:37 +0000 (03:03 -0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Dec 2018 19:18:02 +0000 (11:18 -0800)
kzalloc_node(..., GFP_KERNEL, node) will attempt to allocate
memory as close as possible to the node.

There is no need to fallback to kzalloc() if this has failed.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Tariq Toukan <tariqt@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlx4/en_cq.c
drivers/net/ethernet/mellanox/mlx4/en_rx.c
drivers/net/ethernet/mellanox/mlx4/en_tx.c

index 062a88f..74d4667 100644 (file)
@@ -54,11 +54,8 @@ int mlx4_en_create_cq(struct mlx4_en_priv *priv,
 
        cq = kzalloc_node(sizeof(*cq), GFP_KERNEL, node);
        if (!cq) {
-               cq = kzalloc(sizeof(*cq), GFP_KERNEL);
-               if (!cq) {
-                       en_err(priv, "Failed to allocate CQ structure\n");
-                       return -ENOMEM;
-               }
+               en_err(priv, "Failed to allocate CQ structure\n");
+               return -ENOMEM;
        }
 
        cq->size = entries;
index fd09ba9..9a0881c 100644 (file)
@@ -271,11 +271,8 @@ int mlx4_en_create_rx_ring(struct mlx4_en_priv *priv,
 
        ring = kzalloc_node(sizeof(*ring), GFP_KERNEL, node);
        if (!ring) {
-               ring = kzalloc(sizeof(*ring), GFP_KERNEL);
-               if (!ring) {
-                       en_err(priv, "Failed to allocate RX ring structure\n");
-                       return -ENOMEM;
-               }
+               en_err(priv, "Failed to allocate RX ring structure\n");
+               return -ENOMEM;
        }
 
        ring->prod = 0;
index 6f5153a..2cbd2bd 100644 (file)
@@ -57,11 +57,8 @@ int mlx4_en_create_tx_ring(struct mlx4_en_priv *priv,
 
        ring = kzalloc_node(sizeof(*ring), GFP_KERNEL, node);
        if (!ring) {
-               ring = kzalloc(sizeof(*ring), GFP_KERNEL);
-               if (!ring) {
-                       en_err(priv, "Failed allocating TX ring\n");
-                       return -ENOMEM;
-               }
+               en_err(priv, "Failed allocating TX ring\n");
+               return -ENOMEM;
        }
 
        ring->size = size;