Author: marius
Date: Sun Jul 15 18:02:50 2018
New Revision: 336309
URL: https://svnweb.freebsd.org/changeset/base/336309

Log:
  As suggested by a comment in ixl_initialize_vsi(), use if_getcapenable(9)
  instead of directly interrogating ifp->if_capenable.
  
  Reviewed by:  erj (ixl_initialize_vsi())
  Differential Revision:        https://reviews.freebsd.org/D15720 (part of)

Modified:
  head/sys/dev/ixl/ixl_pf_main.c
  head/sys/dev/ixl/ixl_txrx.c

Modified: head/sys/dev/ixl/ixl_pf_main.c
==============================================================================
--- head/sys/dev/ixl/ixl_pf_main.c      Sun Jul 15 17:31:50 2018        
(r336308)
+++ head/sys/dev/ixl/ixl_pf_main.c      Sun Jul 15 18:02:50 2018        
(r336309)
@@ -1219,8 +1219,7 @@ ixl_initialize_vsi(struct ixl_vsi *vsi)
        /* Set VLAN receive stripping mode */
        ctxt.info.valid_sections |= I40E_AQ_VSI_PROP_VLAN_VALID;
        ctxt.info.port_vlan_flags = I40E_AQ_VSI_PVLAN_MODE_ALL;
-       // TODO: Call function to get this cap bit, instead
-       if (vsi->ifp->if_capenable & IFCAP_VLAN_HWTAGGING)
+       if (if_getcapenable(vsi->ifp) & IFCAP_VLAN_HWTAGGING)
                ctxt.info.port_vlan_flags |= I40E_AQ_VSI_PVLAN_EMOD_STR_BOTH;
        else
                ctxt.info.port_vlan_flags |= I40E_AQ_VSI_PVLAN_EMOD_NOTHING;

Modified: head/sys/dev/ixl/ixl_txrx.c
==============================================================================
--- head/sys/dev/ixl/ixl_txrx.c Sun Jul 15 17:31:50 2018        (r336308)
+++ head/sys/dev/ixl/ixl_txrx.c Sun Jul 15 18:02:50 2018        (r336309)
@@ -687,7 +687,7 @@ ixl_isc_rxd_pkt_get(void *arg, if_rxd_info_t ri)
        rxr->packets++;
        rxr->rx_packets++;
 
-       if ((vsi->ifp->if_capenable & IFCAP_RXCSUM) != 0)
+       if ((if_getcapenable(vsi->ifp) & IFCAP_RXCSUM) != 0)
                ixl_rx_checksum(ri, status, error, ptype);
        ri->iri_flowid = le32toh(cur->wb.qword0.hi_dword.rss);
        ri->iri_rsstype = ixl_ptype_to_hash(ptype);
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to