diff options
author | Michal Schmidt <mschmidt@redhat.com> | 2015-12-04 17:22:34 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-12-06 01:00:02 +0100 |
commit | 5c9ffde4a02144c4f7362152853d69138c437f17 (patch) | |
tree | ff7919ebbdd448217691f6151a1f174b84f11edd | |
parent | net: constify netif_is_* helpers net_device param (diff) | |
download | linux-5c9ffde4a02144c4f7362152853d69138c437f17.tar.xz linux-5c9ffde4a02144c4f7362152853d69138c437f17.zip |
bnx2x: drop redundant error message about allocation failure
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>
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c index d9add7c02e42..5b6c17c53d37 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c @@ -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; } |