svn commit: r292008 - head/sys/dev/sfxge/common

Andrew Rybchenko arybchik at FreeBSD.org
Wed Dec 9 06:24:24 UTC 2015


Author: arybchik
Date: Wed Dec  9 06:24:22 2015
New Revision: 292008
URL: https://svnweb.freebsd.org/changeset/base/292008

Log:
  sfxge: use MAC spoofing TX and MAC change privileges
  
  Update of common code to provide a query on the MAC_SPOOFING_TX and
  CHANGE_MAC privileges instead of the deprecated MAC_SPOOFING privilege.
  
  Submitted by:   Richard Houldsworth <rhouldsworth at solarflare.com>
  Reviewed by:    gnn
  Sponsored by:   Solarflare Communications, Inc.
  MFC after:      2 days
  Differential Revision: https://reviews.freebsd.org/D4436

Modified:
  head/sys/dev/sfxge/common/efx_impl.h
  head/sys/dev/sfxge/common/efx_mcdi.c
  head/sys/dev/sfxge/common/efx_mcdi.h
  head/sys/dev/sfxge/common/hunt_impl.h
  head/sys/dev/sfxge/common/hunt_mcdi.c

Modified: head/sys/dev/sfxge/common/efx_impl.h
==============================================================================
--- head/sys/dev/sfxge/common/efx_impl.h	Wed Dec  9 06:14:47 2015	(r292007)
+++ head/sys/dev/sfxge/common/efx_impl.h	Wed Dec  9 06:24:22 2015	(r292008)
@@ -463,6 +463,7 @@ typedef struct efx_mcdi_ops_s {
 	efx_rc_t	(*emco_fw_update_supported)(efx_nic_t *, boolean_t *);
 	efx_rc_t	(*emco_macaddr_change_supported)(efx_nic_t *, boolean_t *);
 	efx_rc_t	(*emco_link_control_supported)(efx_nic_t *, boolean_t *);
+	efx_rc_t	(*emco_mac_spoofing_supported)(efx_nic_t *, boolean_t *);
 	void		(*emco_read_response)(efx_nic_t *, void *, size_t, size_t);
 } efx_mcdi_ops_t;
 

Modified: head/sys/dev/sfxge/common/efx_mcdi.c
==============================================================================
--- head/sys/dev/sfxge/common/efx_mcdi.c	Wed Dec  9 06:14:47 2015	(r292007)
+++ head/sys/dev/sfxge/common/efx_mcdi.c	Wed Dec  9 06:24:22 2015	(r292008)
@@ -55,6 +55,7 @@ static efx_mcdi_ops_t	__efx_mcdi_siena_o
 					/* emco_macaddr_change_supported */
 	siena_mcdi_link_control_supported,
 					/* emco_link_control_supported */
+	NULL,				/* emco_mac_spoofing_supported */
 	siena_mcdi_read_response,	/* emco_read_response */
 };
 
@@ -74,6 +75,8 @@ static efx_mcdi_ops_t	__efx_mcdi_hunt_op
 					/* emco_macaddr_change_supported */
 	hunt_mcdi_link_control_supported,
 					/* emco_link_control_supported */
+	hunt_mcdi_mac_spoofing_supported,
+					/* emco_mac_spoofing_supported */
 	hunt_mcdi_read_response,	/* emco_read_response */
 };
 
@@ -1307,6 +1310,31 @@ fail1:
 	return (rc);
 }
 
+	__checkReturn		efx_rc_t
+efx_mcdi_mac_spoofing_supported(
+	__in			efx_nic_t *enp,
+	__out			boolean_t *supportedp)
+{
+	efx_mcdi_ops_t *emcop = enp->en_mcdi.em_emcop;
+	efx_rc_t rc;
+
+	if (emcop != NULL && emcop->emco_mac_spoofing_supported != NULL) {
+		if ((rc = emcop->emco_mac_spoofing_supported(enp, supportedp))
+		    != 0)
+			goto fail1;
+	} else {
+		/* Earlier devices always supported MAC spoofing */
+		*supportedp = B_TRUE;
+	}
+
+	return (0);
+
+fail1:
+	EFSYS_PROBE1(fail1, efx_rc_t, rc);
+
+	return (rc);
+}
+
 #if EFSYS_OPT_BIST
 
 #if EFSYS_OPT_HUNTINGTON

Modified: head/sys/dev/sfxge/common/efx_mcdi.h
==============================================================================
--- head/sys/dev/sfxge/common/efx_mcdi.h	Wed Dec  9 06:14:47 2015	(r292007)
+++ head/sys/dev/sfxge/common/efx_mcdi.h	Wed Dec  9 06:24:22 2015	(r292008)
@@ -164,6 +164,12 @@ efx_mcdi_link_control_supported(
 	__in			efx_nic_t *enp,
 	__out			boolean_t *supportedp);
 
+extern	__checkReturn		efx_rc_t
+efx_mcdi_mac_spoofing_supported(
+	__in			efx_nic_t *enp,
+	__out			boolean_t *supportedp);
+
+
 #if EFSYS_OPT_BIST
 #if EFSYS_OPT_HUNTINGTON
 extern	__checkReturn		efx_rc_t

Modified: head/sys/dev/sfxge/common/hunt_impl.h
==============================================================================
--- head/sys/dev/sfxge/common/hunt_impl.h	Wed Dec  9 06:14:47 2015	(r292007)
+++ head/sys/dev/sfxge/common/hunt_impl.h	Wed Dec  9 06:24:22 2015	(r292008)
@@ -298,6 +298,12 @@ hunt_mcdi_link_control_supported(
 	__in		efx_nic_t *enp,
 	__out		boolean_t *supportedp);
 
+extern	__checkReturn	efx_rc_t
+hunt_mcdi_mac_spoofing_supported(
+	__in		efx_nic_t *enp,
+	__out		boolean_t *supportedp);
+
+
 #endif /* EFSYS_OPT_MCDI */
 
 /* NVRAM */

Modified: head/sys/dev/sfxge/common/hunt_mcdi.c
==============================================================================
--- head/sys/dev/sfxge/common/hunt_mcdi.c	Wed Dec  9 06:14:47 2015	(r292007)
+++ head/sys/dev/sfxge/common/hunt_mcdi.c	Wed Dec  9 06:24:22 2015	(r292008)
@@ -435,9 +435,12 @@ hunt_mcdi_macaddr_change_supported(
 	/*
 	 * Use privilege mask state at MCDI attach.
 	 * Admin privilege must be used prior to introduction of
-	 * specific flag (at v4.6).
+	 * mac spoofing privilege (at v4.6), which is used up to
+	 * introduction of change mac spoofing privilege (at v4.7)
 	 */
 	*supportedp =
+	    ((privilege_mask & MC_CMD_PRIVILEGE_MASK_IN_GRP_CHANGE_MAC) ==
+	    MC_CMD_PRIVILEGE_MASK_IN_GRP_CHANGE_MAC) ||
 	    ((privilege_mask & MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING) ==
 	    MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING) ||
 	    ((privilege_mask & MC_CMD_PRIVILEGE_MASK_IN_GRP_ADMIN) ==
@@ -447,6 +450,34 @@ hunt_mcdi_macaddr_change_supported(
 }
 
 	__checkReturn	efx_rc_t
+hunt_mcdi_mac_spoofing_supported(
+	__in		efx_nic_t *enp,
+	__out		boolean_t *supportedp)
+{
+	efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
+	uint32_t privilege_mask = encp->enc_privilege_mask;
+
+	EFSYS_ASSERT3U(enp->en_family, ==, EFX_FAMILY_HUNTINGTON);
+
+	/*
+	 * Use privilege mask state at MCDI attach.
+	 * Admin privilege must be used prior to introduction of
+	 * mac spoofing privilege (at v4.6), which is used up to
+	 * introduction of mac spoofing TX privilege (at v4.7)
+	 */
+	*supportedp =
+	    ((privilege_mask & MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING_TX) ==
+	    MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING_TX) ||
+	    ((privilege_mask & MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING) ==
+	    MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING) ||
+	    ((privilege_mask & MC_CMD_PRIVILEGE_MASK_IN_GRP_ADMIN) ==
+	    MC_CMD_PRIVILEGE_MASK_IN_GRP_ADMIN);
+
+	return (0);
+}
+
+
+	__checkReturn	efx_rc_t
 hunt_mcdi_link_control_supported(
 	__in		efx_nic_t *enp,
 	__out		boolean_t *supportedp)


More information about the svn-src-all mailing list