diff options
| author | Greg Rose <gregory.v.rose@intel.com> | 2012-09-24 22:25:30 -0400 |
|---|---|---|
| committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2012-10-30 01:32:07 -0400 |
| commit | 3970c3234bbaac962a6c9213536dfcabc421d0d0 (patch) | |
| tree | 1f7ef755b91291612e5c2d3867a88cae1fd6506c /drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | |
| parent | ec74a4715549ed4823df7ad4ecc81a8416b7f6bd (diff) | |
ixgbe: Return success or failure on VF MAC filter set
When setting a MAC filter for the VF the function should return a success
or failure code, not the index of the new filter. It causes spurious NACK
returns to the VF driver.
Signed-off-by: Greg Rose <gregory.v.rose@intel.com>
Tested-by: Phil Schmitt <phillip.j.schmitt@intel.com>
Tested-by: Sibai Li <sibai.li@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c')
| -rw-r--r-- | drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c index 8bdb341c14bf..bfeb7ec2abd9 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | |||
| @@ -675,7 +675,7 @@ static int ixgbe_set_vf_mac_addr(struct ixgbe_adapter *adapter, | |||
| 675 | return -1; | 675 | return -1; |
| 676 | } | 676 | } |
| 677 | 677 | ||
| 678 | return ixgbe_set_vf_mac(adapter, vf, new_mac); | 678 | return ixgbe_set_vf_mac(adapter, vf, new_mac) < 0; |
| 679 | } | 679 | } |
| 680 | 680 | ||
| 681 | static int ixgbe_set_vf_vlan_msg(struct ixgbe_adapter *adapter, | 681 | static int ixgbe_set_vf_vlan_msg(struct ixgbe_adapter *adapter, |