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]
Message-ID: <20260127134202.8208-12-maxime.chevallier@bootlin.com>
Date: Tue, 27 Jan 2026 14:41:59 +0100
From: Maxime Chevallier <maxime.chevallier@...tlin.com>
To: davem@...emloft.net,
	Andrew Lunn <andrew@...n.ch>,
	Jakub Kicinski <kuba@...nel.org>,
	Eric Dumazet <edumazet@...gle.com>,
	Paolo Abeni <pabeni@...hat.com>,
	Russell King <linux@...linux.org.uk>,
	Heiner Kallweit <hkallweit1@...il.com>
Cc: Maxime Chevallier <maxime.chevallier@...tlin.com>,
	netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	thomas.petazzoni@...tlin.com,
	Christophe Leroy <christophe.leroy@...roup.eu>,
	Herve Codina <herve.codina@...tlin.com>,
	Florian Fainelli <f.fainelli@...il.com>,
	Vladimir Oltean <vladimir.oltean@....com>,
	Köry Maincent <kory.maincent@...tlin.com>,
	Marek Behún <kabel@...nel.org>,
	Oleksij Rempel <o.rempel@...gutronix.de>,
	Nicolò Veronese <nicveronese@...il.com>,
	Simon Horman <horms@...nel.org>,
	mwojtas@...omium.org,
	Romain Gantois <romain.gantois@...tlin.com>,
	Daniel Golle <daniel@...rotopia.org>,
	Dimitri Fedrau <dimitri.fedrau@...bherr.com>
Subject: [PATCH net-next 11/13] net: phy: store phy_modes in a static array

The phy_modes() helper provided by include/linux/phy.h returns the name of
a phy_interface_t value.

As it may be used by callers that don't depend on PHYLIB, it's a static
inline function.

We want to report using netlink the phy_interface_t types that may be
accessible on a phy_port, especially for SFP cages. We need access to
that array of phy_interface_t names, let's therefore store it into a
dedicated array in net/ethtool/common.c. This is part of CONFIG_NET, so
it's accessible even without dependency on PHYLIB.

Signed-off-by: Maxime Chevallier <maxime.chevallier@...tlin.com>
---
 include/linux/phy.h  | 84 +++-----------------------------------------
 net/ethtool/common.c | 44 +++++++++++++++++++++++
 2 files changed, 49 insertions(+), 79 deletions(-)

diff --git a/include/linux/phy.h b/include/linux/phy.h
index dc788ae4da64..cf607c1a5127 100644
--- a/include/linux/phy.h
+++ b/include/linux/phy.h
@@ -199,6 +199,8 @@ static inline void phy_interface_set_rgmii(unsigned long *intf)
 	__set_bit(PHY_INTERFACE_MODE_RGMII_TXID, intf);
 }
 
+extern const char phy_interface_names[][ETH_GSTRING_LEN];
+
 /**
  * phy_modes - map phy_interface_t enum to device tree binding of phy-mode
  * @interface: enum phy_interface_t value
@@ -209,86 +211,10 @@ static inline void phy_interface_set_rgmii(unsigned long *intf)
  */
 static inline const char *phy_modes(phy_interface_t interface)
 {
-	switch (interface) {
-	case PHY_INTERFACE_MODE_NA:
-		return "";
-	case PHY_INTERFACE_MODE_INTERNAL:
-		return "internal";
-	case PHY_INTERFACE_MODE_MII:
-		return "mii";
-	case PHY_INTERFACE_MODE_GMII:
-		return "gmii";
-	case PHY_INTERFACE_MODE_SGMII:
-		return "sgmii";
-	case PHY_INTERFACE_MODE_TBI:
-		return "tbi";
-	case PHY_INTERFACE_MODE_REVMII:
-		return "rev-mii";
-	case PHY_INTERFACE_MODE_RMII:
-		return "rmii";
-	case PHY_INTERFACE_MODE_REVRMII:
-		return "rev-rmii";
-	case PHY_INTERFACE_MODE_RGMII:
-		return "rgmii";
-	case PHY_INTERFACE_MODE_RGMII_ID:
-		return "rgmii-id";
-	case PHY_INTERFACE_MODE_RGMII_RXID:
-		return "rgmii-rxid";
-	case PHY_INTERFACE_MODE_RGMII_TXID:
-		return "rgmii-txid";
-	case PHY_INTERFACE_MODE_RTBI:
-		return "rtbi";
-	case PHY_INTERFACE_MODE_SMII:
-		return "smii";
-	case PHY_INTERFACE_MODE_XGMII:
-		return "xgmii";
-	case PHY_INTERFACE_MODE_XLGMII:
-		return "xlgmii";
-	case PHY_INTERFACE_MODE_MOCA:
-		return "moca";
-	case PHY_INTERFACE_MODE_PSGMII:
-		return "psgmii";
-	case PHY_INTERFACE_MODE_QSGMII:
-		return "qsgmii";
-	case PHY_INTERFACE_MODE_TRGMII:
-		return "trgmii";
-	case PHY_INTERFACE_MODE_1000BASEX:
-		return "1000base-x";
-	case PHY_INTERFACE_MODE_1000BASEKX:
-		return "1000base-kx";
-	case PHY_INTERFACE_MODE_2500BASEX:
-		return "2500base-x";
-	case PHY_INTERFACE_MODE_5GBASER:
-		return "5gbase-r";
-	case PHY_INTERFACE_MODE_RXAUI:
-		return "rxaui";
-	case PHY_INTERFACE_MODE_XAUI:
-		return "xaui";
-	case PHY_INTERFACE_MODE_10GBASER:
-		return "10gbase-r";
-	case PHY_INTERFACE_MODE_25GBASER:
-		return "25gbase-r";
-	case PHY_INTERFACE_MODE_USXGMII:
-		return "usxgmii";
-	case PHY_INTERFACE_MODE_10GKR:
-		return "10gbase-kr";
-	case PHY_INTERFACE_MODE_100BASEX:
-		return "100base-x";
-	case PHY_INTERFACE_MODE_QUSGMII:
-		return "qusgmii";
-	case PHY_INTERFACE_MODE_10G_QXGMII:
-		return "10g-qxgmii";
-	case PHY_INTERFACE_MODE_50GBASER:
-		return "50gbase-r";
-	case PHY_INTERFACE_MODE_LAUI:
-		return "laui";
-	case PHY_INTERFACE_MODE_100GBASEP:
-		return "100gbase-p";
-	case PHY_INTERFACE_MODE_MIILITE:
-		return "mii-lite";
-	default:
+	if (interface > PHY_INTERFACE_MODE_MAX)
 		return "unknown";
-	}
+
+	return phy_interface_names[interface];
 }
 
 /**
diff --git a/net/ethtool/common.c b/net/ethtool/common.c
index 4036561b078b..d795d9a8b0cd 100644
--- a/net/ethtool/common.c
+++ b/net/ethtool/common.c
@@ -483,6 +483,50 @@ static const char ethtool_link_medium_names[][ETH_GSTRING_LEN] = {
 };
 static_assert(ARRAY_SIZE(ethtool_link_medium_names) == __ETHTOOL_LINK_MEDIUM_LAST);
 
+const char phy_interface_names[][ETH_GSTRING_LEN] = {
+	[PHY_INTERFACE_MODE_NA] =	"",
+	[PHY_INTERFACE_MODE_INTERNAL] =	"internal",
+	[PHY_INTERFACE_MODE_MII] =	"mii",
+	[PHY_INTERFACE_MODE_GMII] =	"gmii",
+	[PHY_INTERFACE_MODE_SGMII] =	"sgmii",
+	[PHY_INTERFACE_MODE_TBI] =	"tbi",
+	[PHY_INTERFACE_MODE_REVMII] =	"rev-mii",
+	[PHY_INTERFACE_MODE_RMII] =	"rmii",
+	[PHY_INTERFACE_MODE_REVRMII] =	"rev-rmii",
+	[PHY_INTERFACE_MODE_RGMII] =	"rgmii",
+	[PHY_INTERFACE_MODE_RGMII_ID] = "rgmii-id",
+	[PHY_INTERFACE_MODE_RGMII_RXID] = "rgmii-rxid",
+	[PHY_INTERFACE_MODE_RGMII_TXID] = "rgmii-txid",
+	[PHY_INTERFACE_MODE_RTBI] =	"rtbi",
+	[PHY_INTERFACE_MODE_SMII] =	"smii",
+	[PHY_INTERFACE_MODE_XGMII] =	"xgmii",
+	[PHY_INTERFACE_MODE_XLGMII] =	"xlgmii",
+	[PHY_INTERFACE_MODE_MOCA] =	"moca",
+	[PHY_INTERFACE_MODE_PSGMII] =	"psgmii",
+	[PHY_INTERFACE_MODE_QSGMII] =	"qsgmii",
+	[PHY_INTERFACE_MODE_TRGMII] =	"trgmii",
+	[PHY_INTERFACE_MODE_1000BASEX] = "1000base-x",
+	[PHY_INTERFACE_MODE_1000BASEKX] = "1000base-kx",
+	[PHY_INTERFACE_MODE_2500BASEX] = "2500base-x",
+	[PHY_INTERFACE_MODE_5GBASER] =	"5gbase-r",
+	[PHY_INTERFACE_MODE_RXAUI] =	"rxaui",
+	[PHY_INTERFACE_MODE_XAUI] =	"xaui",
+	[PHY_INTERFACE_MODE_10GBASER] =	"10gbase-r",
+	[PHY_INTERFACE_MODE_25GBASER] =	"25gbase-r",
+	[PHY_INTERFACE_MODE_USXGMII] =	"usxgmii",
+	[PHY_INTERFACE_MODE_10GKR] =	"10gbase-kr",
+	[PHY_INTERFACE_MODE_100BASEX] =	"100base-x",
+	[PHY_INTERFACE_MODE_QUSGMII] =	"qusgmii",
+	[PHY_INTERFACE_MODE_10G_QXGMII] = "10g-qxgmii",
+	[PHY_INTERFACE_MODE_50GBASER] =	"50gbase-r",
+	[PHY_INTERFACE_MODE_LAUI] =	"laui",
+	[PHY_INTERFACE_MODE_100GBASEP] = "100gbase-p",
+	[PHY_INTERFACE_MODE_MIILITE] =	"mii-lite",
+
+};
+static_assert(ARRAY_SIZE(phy_interface_names) == PHY_INTERFACE_MODE_MAX);
+EXPORT_SYMBOL_GPL(phy_interface_names);
+
 const char netif_msg_class_names[][ETH_GSTRING_LEN] = {
 	[NETIF_MSG_DRV_BIT]		= "drv",
 	[NETIF_MSG_PROBE_BIT]		= "probe",
-- 
2.49.0


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ