bnx2x: drop redundant error message about allocation failure
authorMichal Schmidt <mschmidt@redhat.com>
Fri, 4 Dec 2015 16:22:34 +0000 (17:22 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sun, 6 Dec 2015 00:00:02 +0000 (19:00 -0500)
alloc_pages() already prints a warning when it fails. No need to emit
another message. Certainly not at KERN_ERR level, because it is no big
deal if this GFP_ATOMIC allocation fails occasionally.

Signed-off-by: Michal Schmidt <mschmidt@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c

index d9add7c02e42ca89f4be202b390a24d704a746e4..5b6c17c53d37c707f4b59a4890498ad39d6f6da6 100644 (file)
@@ -558,10 +558,8 @@ static int bnx2x_alloc_rx_sge(struct bnx2x *bp, struct bnx2x_fastpath *fp,
                        put_page(pool->page);
 
                pool->page = alloc_pages(gfp_mask, PAGES_PER_SGE_SHIFT);
-               if (unlikely(!pool->page)) {
-                       BNX2X_ERR("Can't alloc sge\n");
+               if (unlikely(!pool->page))
                        return -ENOMEM;
-               }
 
                pool->offset = 0;
        }