2018-01-01 19:57:10

by SF Markus Elfring

[permalink] [raw]
Subject: [PATCH] igb: Delete an error message for a failed memory allocation in igb_enable_sriov()

From: Markus Elfring <[email protected]>
Date: Mon, 1 Jan 2018 20:53:10 +0100

Omit an extra message for a memory allocation failure in this function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <[email protected]>
---
drivers/net/ethernet/intel/igb/igb_main.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/net/ethernet/intel/igb/igb_main.c b/drivers/net/ethernet/intel/igb/igb_main.c
index e94d3c256667..1621aaa3eb28 100644
--- a/drivers/net/ethernet/intel/igb/igb_main.c
+++ b/drivers/net/ethernet/intel/igb/igb_main.c
@@ -3200,8 +3200,6 @@ static int igb_enable_sriov(struct pci_dev *pdev, int num_vfs)
/* if allocation failed then we do not support SR-IOV */
if (!adapter->vf_data) {
adapter->vfs_allocated_count = 0;
- dev_err(&pdev->dev,
- "Unable to allocate memory for VF Data Storage\n");
err = -ENOMEM;
goto out;
}
--
2.15.1


2018-01-04 03:41:47

by Brown, Aaron F

[permalink] [raw]
Subject: RE: [Intel-wired-lan] [PATCH] igb: Delete an error message for a failed memory allocation in igb_enable_sriov()

> From: Intel-wired-lan [mailto:[email protected]] On
> Behalf Of SF Markus Elfring
> Sent: Monday, January 1, 2018 11:57 AM
> To: [email protected]; [email protected]; Kirsher,
> Jeffrey T <[email protected]>
> Cc: [email protected]; LKML <[email protected]>
> Subject: [Intel-wired-lan] [PATCH] igb: Delete an error message for a failed
> memory allocation in igb_enable_sriov()
>
> From: Markus Elfring <[email protected]>
> Date: Mon, 1 Jan 2018 20:53:10 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfring <[email protected]>
> ---
> drivers/net/ethernet/intel/igb/igb_main.c | 2 --
> 1 file changed, 2 deletions(-)
>

Tested-by: Aaron Brown <[email protected]>