From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay6-d.mail.gandi.net (relay6-d.mail.gandi.net [217.70.183.198]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 245DF21578D; Tue, 10 Jun 2025 08:18:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.198 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749543520; cv=none; b=Iav9A7NZ4s2k2QRsmQWO52ew1/OU9acb0VpDd0DpAlKt2JPFb5GbL78VjFCtqqbx4SHLKqaHnGlyl5lsZgk4AB49pGLbDQf5yq0nHJNFalFqyLD8mtAoUMrhEneprRz8S1z4tA6LUlIJSMuGxPPTzlPBCd/efGMB4gS2Ciba/Ac= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749543520; c=relaxed/simple; bh=0O8yujLEJLMJ3hUgzFfiYx8X6y8n0uKkiblzYtmGvtw=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=Skv6e4+unATqlVJKoypkA8GMT6wbvGHV9k33N2tXQKUr5h15xiNSdZWJvsvXgDCfpLs/Rwzt5vQxoJdNtk76gQFkTHhpxp+ixnlbREEjT7fWi+SNoV3AGjn1vpt2sMH9nPWiwX3XLBtAUxNNvqc4FCSQymi5E3gZ+NKRCFIEYkA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=jO4QuuS+; arc=none smtp.client-ip=217.70.183.198 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="jO4QuuS+" Received: by mail.gandi.net (Postfix) with ESMTPSA id 57B464426E; Tue, 10 Jun 2025 08:18:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1749543515; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=0LzaDuer9xNWjAddXjLU6iPOUE3PiHIk5+pn0I9K6wQ=; b=jO4QuuS+a5O24WThoIYuFwA9x+6PkLFXHYythzHObGmD7fdH3sIFh5+8NRdT62+P6fqdsQ DPKBQk7aozNSwYh0LQnOMckCSYj7rPMno9cIHtCfjgn6hbF/8sAtuxGZagdMuiDghiNw96 jqYmvbTqn8eDw+Cf4wn4Rb4/kNNG6d1PhHGm1bi2+zaWJ91f83hfavM5AvD9qadnfe8H87 WSM4pb7UxwNDsnVlGzOs42vahY3EH7870uDH2WCcRrHeLSdvfDXLwXguRHFfdIEJp+CPEw /J5Q4twYyfZ7E73jT9y4SVko2umiptJGV727zr5lgmsMHotTJv7a/XiImPc1pg== From: Kory Maincent Subject: [PATCH net-next v13 00/13] Add support for PSE budget evaluation strategy Date: Tue, 10 Jun 2025 10:11:34 +0200 Message-Id: <20250610-feature_poe_port_prio-v13-0-c5edc16b9ee2@bootlin.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-B4-Tracking: v=1; b=H4sIALfoR2gC/3XSzW7DIAwA4FepOC8T5s+w095jmioCZkXakirJo k5V330k0tRoKQcOYPFhY1/ZSEOmkb0crmygOY+578oG5NOBhZPvPqjJsRwwwYXiDmSTyE/fAx3 P/bKG6Xgect94DZ4iSikosHL3PFDKlxV+Yx1NTUeXib2XyCmPUz/8rC/OsMYXGzgXFXuGhjdok WuVUFHUr23fT5+5ew7912rOYuNIXnNEcZzWzghBpNDvHXl3QEDNkcWxUjiXvHEGw95Rf47mJaG ao4oTgwMvAw9gce/ouyPA1hxdHBm9soZ0ikbtHbw7ilcdXP4n6RQclepC3Dt244CpObY4SpkgX QwSpd07buOIat/d4gCmwDlqFA/yAX6HNK8mBLxIWht05ZOcTu6BBBtJVEcIlllsW5+UQoJWPKg NxFZSVWmZxmiWhkFM6P91/3a7/QJC9sZYnQMAAA== To: Andrew Lunn , Oleksij Rempel , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Jonathan Corbet , Donald Hunter , Rob Herring , Andrew Lunn , Simon Horman , Heiner Kallweit , Russell King , Krzysztof Kozlowski , Conor Dooley Cc: Liam Girdwood , Mark Brown , Thomas Petazzoni , netdev@vger.kernel.org, linux-doc@vger.kernel.org, Kyle Swenson , Dent Project , kernel@pengutronix.de, Maxime Chevallier , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, "Kory Maincent (Dent Project)" , Krzysztof Kozlowski X-Mailer: b4 0.15-dev-8cb71 X-GND-State: clean X-GND-Score: -100 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeeffedrtddugddutdeglecutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfitefpfffkpdcuggftfghnshhusghstghrihgsvgenuceurghilhhouhhtmecufedtudenucesvcftvggtihhpihgvnhhtshculddquddttddmnecujfgurhephffufffkgggtgffvvefosehtkeertdertdejnecuhfhrohhmpefmohhrhicuofgrihhntggvnhhtuceokhhorhihrdhmrghinhgtvghnthessghoohhtlhhinhdrtghomheqnecuggftrfgrthhtvghrnhepudfhveduteffgfekvdfhveehgeehtdelgefhffduiefffedvheefgeeiiedvkeetnecuffhomhgrihhnpehkvghrnhgvlhdrohhrghdpsghoohhtlhhinhdrtghomhenucfkphepledtrdekledrudeifedruddvjeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepihhnvghtpeeltddrkeelrdduieefrdduvdejpdhhvghloheplgduvdejrddtrddurddungdpmhgrihhlfhhrohhmpehkohhrhidrmhgrihhntggvnhhtsegsohhothhlihhnrdgtohhmpdhnsggprhgtphhtthhopedvkedprhgtphhtthhopegrnhgurhgvfieslhhunhhnrdgthhdprhgtphhtthhopegvughumhgriigvthesghhoohhglhgvrdgtohhmpdhrtghpthhtohepuggvnhhtphhrohhjvggttheslhhinhhugihfohhunhgurghtihhonhdrohhrghdprhgtphhtthhopehmrgigihhmvgdrtghhvghvrghllhhivghrsegsohhothhli hhnrdgtohhmpdhrtghpthhtohepnhgvthguvghvsehvghgvrhdrkhgvrhhnvghlrdhorhhgpdhrtghpthhtohephhhkrghllhifvghithdusehgmhgrihhlrdgtohhmpdhrtghpthhtohepohdrrhgvmhhpvghlsehpvghnghhuthhrohhnihigrdguvgdprhgtphhtthhopehkvghrnhgvlhesphgvnhhguhhtrhhonhhigidruggv X-GND-Sasl: kory.maincent@bootlin.com From: Kory Maincent (Dent Project) This series brings support for budget evaluation strategy in the PSE subsystem. PSE controllers can set priorities to decide which ports should be turned off in case of special events like over-current. This patch series adds support for two budget evaluation strategy. 1. Static Method: This method involves distributing power based on PD classification. It’s straightforward and stable, the PSE core keeping track of the budget and subtracting the power requested by each PD’s class. Advantages: Every PD gets its promised power at any time, which guarantees reliability. Disadvantages: PD classification steps are large, meaning devices request much more power than they actually need. As a result, the power supply may only operate at, say, 50% capacity, which is inefficient and wastes money. 2. Dynamic Method: To address the inefficiencies of the static method, vendors like Microchip have introduced dynamic power budgeting, as seen in the PD692x0 firmware. This method monitors the current consumption per port and subtracts it from the available power budget. When the budget is exceeded, lower-priority ports are shut down. Advantages: This method optimizes resource utilization, saving costs. Disadvantages: Low-priority devices may experience instability. The UAPI allows adding support for software port priority mode managed from userspace later if needed. Patches 1-2: Add support for interrupt event report in PSE core, ethtool and ethtool specs. Patch 3: Adds support for interrupt and event report in TPS23881 driver. Patches 4,5: Add support for PSE power domain in PSE core and ethtool. Patches 6-8: Add support for budget evaluation strategy in PSE core, ethtool and ethtool specs. Patches 9-11: Add support for port priority and power supplies in PD692x0 drivers. Patches 12,13: Add support for port priority in TPS23881 drivers. Signed-off-by: Kory Maincent (Dent Project) --- Changes in v13: - Small change, no need for >0 condition check for unsigned variables. - Link to v12: https://lore.kernel.org/r/20250524-feature_poe_port_prio-v12-0-d65fd61df7a7@bootlin.com Changes in v12: - Rebase on net-next. - Link to v11: https://lore.kernel.org/r/20250520-feature_poe_port_prio-v11-0-bbaf447e1b28@bootlin.com Changes in v11: - Move the PSE events enum description fully in the ethtool spec. - Remove the first patch which was useless as not used. - Split the second patch to separate the attached_phydev introduction to the PSE interrupt support. - Link to v10: https://lore.kernel.org/r/20250506-feature_poe_port_prio-v10-0-55679a4895f9@bootlin.com Changes in v10: - Change patch 2 and 7 due to possible used after free scenario or deadlock scenario. Move the PSE notification send management to a workqueue to protect it from the deadlock scenario. - Link to v9: https://lore.kernel.org/r/20250422-feature_poe_port_prio-v9-0-417fc007572d@bootlin.com Changes in v9: - Add a missing check after skb creation. - Link to v8: https://lore.kernel.org/r/20250416-feature_poe_port_prio-v8-0-446c39dc3738@bootlin.com Changes in v8: - Rename a few functions for better clarity. - Add missing kref_init in PSE power domain support and a wrong error check condition. - Link to v7: https://lore.kernel.org/r/20250408-feature_poe_port_prio-v7-0-9f5fc9e329cd@bootlin.com Changes in v7: - Add reference count and mutex lock for PSE power domain. - Add support to retry enabling port that failed to be powered in case of port disconnection or priority change. - Use flags definition for pse events in ethtool specs. - Small changes in the TPS23881 driver. - Link to v6: https://lore.kernel.org/r/20250304-feature_poe_port_prio-v6-0-3dc0c5ebaf32@bootlin.com Changes in v6: - Few typos. - Use uint instead of bitset for PSE_EVENT. - Remove report of budget evaluation strategy in the uAPI. - Link to v5: https://lore.kernel.org/r/20250218-feature_poe_port_prio-v5-0-3da486e5fd64@bootlin.com Changes in v5: - Remove the first part of the patch series which tackled PSE improvement and already gets merged: https://lore.kernel.org/netdev/20250110-b4-feature_poe_arrange-v3-0-142279aedb94@bootlin.com/ - Remove the PSE index support which is useless for now. The PSE power domain ID is sufficient. - Add support for PD692x0 power supplies other than Vmain which was already in the patch series. - Few other small fixes. - Link to v4: https://lore.kernel.org/r/20250103-feature_poe_port_prio-v4-0-dc91a3c0c187@bootlin.com Changes in v4: - Remove disconnection policy. - Rename port priority mode to budget evaluation strategy. - Add cosmetic changes in PSE core. - Add support for port priority in PD692x0 driver. - Link to v3: https://lore.kernel.org/r/20241121-feature_poe_port_prio-v3-0-83299fa6967c@bootlin.com Changes in v3: - Move power budget to regulator core. - Add disconnection policies with PIs using the same priority. - Several fixes on the TPS23881 drivers. - Several new cosmetic patches. - Link to v2: https://lore.kernel.org/r/20241030-feature_poe_port_prio-v2-0-9559622ee47a@bootlin.com Changes in v2: - Rethink the port priority management. - Add PSE id. - Add support for PSE power domains. - Add get power budget regulator constraint. - Link to v1: https://lore.kernel.org/r/20241002-feature_poe_port_prio-v1-0-787054f74ed5@bootlin.com --- Kory Maincent (13): net: pse-pd: Introduce attached_phydev to pse control net: pse-pd: Add support for reporting events net: pse-pd: tps23881: Add support for PSE events and interrupts net: pse-pd: Add support for PSE power domains net: ethtool: Add support for new power domains index description net: pse-pd: Add helper to report hardware enable status of the PI net: pse-pd: Add support for budget evaluation strategies net: ethtool: Add PSE port priority support feature net: pse-pd: pd692x0: Add support for PSE PI priority feature net: pse-pd: pd692x0: Add support for controller and manager power supplies dt-bindings: net: pse-pd: microchip,pd692x0: Add manager regulator supply net: pse-pd: tps23881: Add support for static port priority feature dt-bindings: net: pse-pd: ti,tps23881: Add interrupt description .../bindings/net/pse-pd/microchip,pd692x0.yaml | 22 +- .../bindings/net/pse-pd/ti,tps23881.yaml | 8 + Documentation/netlink/specs/ethtool.yaml | 76 ++ Documentation/networking/ethtool-netlink.rst | 49 + drivers/net/mdio/fwnode_mdio.c | 26 +- drivers/net/pse-pd/pd692x0.c | 225 +++++ drivers/net/pse-pd/pse_core.c | 1068 +++++++++++++++++++- drivers/net/pse-pd/tps23881.c | 403 +++++++- include/linux/ethtool_netlink.h | 9 + include/linux/pse-pd/pse.h | 108 +- include/uapi/linux/ethtool_netlink_generated.h | 40 + net/ethtool/pse-pd.c | 63 ++ 12 files changed, 2043 insertions(+), 54 deletions(-) --- base-commit: 62937b656898ae9039071a41f6e06e8bea9cb67c change-id: 20240913-feature_poe_port_prio-a51aed7332ec Best regards, -- Köry Maincent, Bootlin Embedded Linux and kernel engineering https://bootlin.com