On Fri, Feb 23, 2018 at 5:20 PM, Vinicius Costa Gomes
<vinicius.go...@intel.com> wrote:
> This will allow functionality depending on the hardware being traffic
> class aware to work. In particular the tc-flower offloading checks
> verifies that this bit is set.
>
> Signed-off-by: Vinicius Costa Gomes <vinicius.go...@intel.com>
> ---
> drivers/net/ethernet/intel/igb/igb_main.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/igb/igb_main.c
> b/drivers/net/ethernet/intel/igb/igb_main.c
> index 0ea32be07d71..543aa99892eb 100644
> --- a/drivers/net/ethernet/intel/igb/igb_main.c
> +++ b/drivers/net/ethernet/intel/igb/igb_main.c
> @@ -2820,8 +2820,10 @@ static int igb_probe(struct pci_dev *pdev, const
> struct pci_device_id *ent)
> NETIF_F_HW_VLAN_CTAG_TX |
> NETIF_F_RXALL;
>
> - if (hw->mac.type >= e1000_i350)
> - netdev->hw_features |= NETIF_F_NTUPLE;
> + if (hw->mac.type >= e1000_i350) {
> + netdev->hw_features |= (NETIF_F_NTUPLE | NETIF_F_HW_TC);
> + netdev->features |= NETIF_F_HW_TC;
The parens aren't needed.
Also you might consider moving this block up to where we have a
similar one for 82576. Then you wouldn't need to set both features and
hw_features in the case of the HW_TC flag.
> + }
>
> if (pci_using_dac)
> netdev->features |= NETIF_F_HIGHDMA;
> --
> 2.16.2
>
> _______________________________________________
> Intel-wired-lan mailing list
> intel-wired-...@osuosl.org
> https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
