netdev - [PATCH net-next v1 3/3] amd-xgbe: Enhance auto-negotiation parallel detection

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 29 Aug 2014 10:22:07 -0500
From:	Tom Lendacky <thomas.lendacky@....com>
To:	<netdev@...r.kernel.org>
CC:	<davem@...emloft.net>
Subject: [PATCH net-next v1 3/3] amd-xgbe: Enhance auto-negotiation parallel
 detection

Add support for parallel detection during auto negotiation while in
KR speed mode.  Additionally, don't use auto negotiation registers
to determine state if parallel detection was used.

Signed-off-by: Tom Lendacky <thomas.lendacky@....com>
---
 drivers/net/ethernet/amd/xgbe/xgbe-phy.c |   19 ++++++++++++++++++-
 drivers/net/ethernet/amd/xgbe/xgbe.h     |    1 +
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-phy.c b/drivers/net/ethernet/amd/xgbe/xgbe-phy.c
index d072201..d95c2a8 100644
--- a/drivers/net/ethernet/amd/xgbe/xgbe-phy.c
+++ b/drivers/net/ethernet/amd/xgbe/xgbe-phy.c
@@ -108,6 +108,10 @@
 #define MDIO_AN_INT			0x8002
 #endif
 
+#ifndef MDIO_AN_KR_CTRL
+#define MDIO_AN_KR_CTRL			0x8003
+#endif
+
 #ifndef MDIO_CTRL1_SPEED1G
 #define MDIO_CTRL1_SPEED1G		(MDIO_CTRL1_SPEED10G & ~BMCR_SPEED100)
 #endif
@@ -120,6 +124,10 @@
 #define MDIO_PMA_10GBR_KR_TRAIN_ENABLE	0x02
 #endif
 
+#ifndef MDIO_KR_CTRL_PDETECT
+#define MDIO_KR_CTRL_PDETECT		1
+#endif
+
 /* SerDes integration register offsets */
 #define SIR0_KR_RT_1			0x002c
 #define SIR0_STATUS			0x0040
@@ -642,6 +650,9 @@ static enum xgbe_an xgbe_phy_an_start(struct phy_device *phydev)
 	/* Enable and start auto-negotiation */
 	XMDIO_WRITE(pdata, MDIO_MMD_AN, MDIO_AN_INT, 0);
 
+	XMDIO_WRITE_BITS(pdata, MDIO_MMD_AN, MDIO_AN_KR_CTRL,
+			 MDIO_KR_CTRL_PDETECT, MDIO_KR_CTRL_PDETECT);
+
 	val = XMDIO_READ(pdata, MDIO_MMD_AN, MDIO_CTRL1);
 	val |= MDIO_AN_CTRL1_ENABLE;
 	val |= MDIO_AN_CTRL1_RESTART;
@@ -714,6 +725,9 @@ static void xgbe_phy_an_state_machine(struct work_struct *work)
 	unsigned int sleep;
 	unsigned int an_supported = 0;
 
+	/* Start in KX mode */
+	xgbe_phy_set_mode(phydev, XGBE_SPEED_MODE_KX);
+
 	while (1) {
 		mutex_lock(&pdata->phy.an_mutex);
 
@@ -722,6 +736,7 @@ static void xgbe_phy_an_state_machine(struct work_struct *work)
 		switch (pdata->phy.an_state) {
 		case XGBE_AN_START:
 			an_supported = 0;
+			pdata->phy.parallel_detect = 0;
 			pdata->phy.an_state = xgbe_phy_an_start(phydev);
 			break;
 
@@ -739,6 +754,7 @@ static void xgbe_phy_an_state_machine(struct work_struct *work)
 			break;
 
 		case XGBE_AN_COMPLETE:
+			pdata->phy.parallel_detect = an_supported ? 0 : 1;
 			netdev_info(phydev->attached_dev, "%s successful\n",
 				    an_supported ? "Auto negotiation"
 						 : "Parallel detection");
@@ -956,7 +972,8 @@ static int xgbe_phy_read_status(struct phy_device *phydev)
 	if (ret)
 		return ret;
 
-	if (phydev->autoneg == AUTONEG_ENABLE) {
+	if ((phydev->autoneg == AUTONEG_ENABLE) &&
+	    !pdata->phy.parallel_detect) {
 		if (!(mmd_mask & MDIO_DEVS_AN))
 			return -EINVAL;
 
diff --git a/drivers/net/ethernet/amd/xgbe/xgbe.h b/drivers/net/ethernet/amd/xgbe/xgbe.h
index 2d057cf..a964266 100644
--- a/drivers/net/ethernet/amd/xgbe/xgbe.h
+++ b/drivers/net/ethernet/amd/xgbe/xgbe.h
@@ -464,6 +464,7 @@ struct xgbe_phy {
 	enum xgbe_an_rx kx_state;
 	struct work_struct an_work;
 	struct workqueue_struct *an_workqueue;
+	unsigned int parallel_detect;
 };
 
 struct xgbe_hw_if {

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html