All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v4 net-next 0/4] Add ethtool interface for SyncE
@ 2021-12-01 18:02 ` Maciej Machnikowski
  0 siblings, 0 replies; 56+ messages in thread
From: Maciej Machnikowski @ 2021-12-01 18:02 UTC (permalink / raw)
  To: maciej.machnikowski, netdev, intel-wired-lan, arkadiusz.kubalewski
  Cc: richardcochran, abyagowi, anthony.l.nguyen, davem, kuba,
	linux-kselftest, idosch, mkubecek, saeed, michael.chan, petrm

Synchronous Ethernet networks use a physical layer clock to syntonize
the frequency across different network elements.

Basic SyncE node defined in the ITU-T G.8264 consist of an Ethernet
Equipment Clock (EEC) and have the ability to recover synchronization
from the synchronization inputs - either traffic interfaces or external
frequency sources.
The EEC can synchronize its frequency (syntonize) to any of those sources.
It is also able to select synchronization source through priority tables
and synchronization status messaging. It also provides neccessary
filtering and holdover capabilities

This patch series introduces basic interface for reading and configuring
recover clocks on a SyncE capable device

v4:
- Dropped EEC_STATE reporting (TBD: DPLL subsystem)
- moved recovered clock configuration to ethtool netlink

v3:
- remove RTM_GETRCLKRANGE
- return state of all possible pins in the RTM_GETRCLKSTATE
- clarify documentation

v2:
- improved documentation
- fixed kdoc warning

RFC history:
v2:
- removed whitespace changes
- fix issues reported by test robot
v3:
- Changed naming from SyncE to EEC
- Clarify cover letter and commit message for patch 1
v4:
- Removed sync_source and pin_idx info
- Changed one structure to attributes
- Added EEC_SRC_PORT flag to indicate that the EEC is synchronized
  to the recovered clock of a port that returns the state
v5:
- add EEC source as an optiona attribute
- implement support for recovered clocks
- align states returned by EEC to ITU-T G.781
v6:
- fix EEC clock state reporting
- add documentation
- fix descriptions in code comments

Maciej Machnikowski (4):
  ice: add support detecting features based on netlist
  ethtool: Add ability to configure recovered clock for SyncE feature
  ice: add support for monitoring SyncE DPLL state
  ice: add support for SyncE recovered clocks

 Documentation/networking/ethtool-netlink.rst  |  67 +++++
 drivers/net/ethernet/intel/ice/ice.h          |   7 +
 .../net/ethernet/intel/ice/ice_adminq_cmd.h   |  70 ++++-
 drivers/net/ethernet/intel/ice/ice_common.c   | 224 +++++++++++++++
 drivers/net/ethernet/intel/ice/ice_common.h   |  20 +-
 drivers/net/ethernet/intel/ice/ice_devids.h   |   3 +
 drivers/net/ethernet/intel/ice/ice_ethtool.c  |  97 +++++++
 drivers/net/ethernet/intel/ice/ice_lib.c      |   6 +-
 drivers/net/ethernet/intel/ice/ice_ptp.c      |  35 +++
 drivers/net/ethernet/intel/ice/ice_ptp_hw.c   |  49 ++++
 drivers/net/ethernet/intel/ice/ice_ptp_hw.h   |  36 +++
 drivers/net/ethernet/intel/ice/ice_type.h     |   1 +
 include/linux/ethtool.h                       |   9 +
 include/uapi/linux/ethtool_netlink.h          |  21 ++
 net/ethtool/Makefile                          |   3 +-
 net/ethtool/netlink.c                         |  20 ++
 net/ethtool/netlink.h                         |   4 +
 net/ethtool/synce.c                           | 267 ++++++++++++++++++
 18 files changed, 935 insertions(+), 4 deletions(-)
 create mode 100644 net/ethtool/synce.c

-- 
2.26.3


^ permalink raw reply	[flat|nested] 56+ messages in thread
* Re: [PATCH v4 net-next 2/4] ethtool: Add ability to configure recovered clock for SyncE feature
@ 2021-12-02 12:29 kernel test robot
  0 siblings, 0 replies; 56+ messages in thread
From: kernel test robot @ 2021-12-02 12:29 UTC (permalink / raw)
  To: kbuild

[-- Attachment #1: Type: text/plain, Size: 2844 bytes --]

CC: kbuild-all(a)lists.01.org
In-Reply-To: <20211201180208.640179-3-maciej.machnikowski@intel.com>
References: <20211201180208.640179-3-maciej.machnikowski@intel.com>
TO: Maciej Machnikowski <maciej.machnikowski@intel.com>
TO: maciej.machnikowski(a)intel.com
TO: netdev(a)vger.kernel.org
TO: intel-wired-lan(a)lists.osuosl.org
TO: arkadiusz.kubalewski(a)intel.com
CC: richardcochran(a)gmail.com
CC: abyagowi(a)fb.com
CC: anthony.l.nguyen(a)intel.com
CC: davem(a)davemloft.net
CC: kuba(a)kernel.org
CC: linux-kselftest(a)vger.kernel.org

Hi Maciej,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Maciej-Machnikowski/Add-ethtool-interface-for-SyncE/20211202-021915
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 23ea630f86c70cbe6691f9f839e7b6742f0e9ad3
:::::: branch date: 18 hours ago
:::::: commit date: 18 hours ago
config: i386-randconfig-c001-20211202 (https://download.01.org/0day-ci/archive/20211202/202112022021.bjtxq8wO-lkp(a)intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Julia Lawall <julia.lawall@lip6.fr>


cocci warnings: (new ones prefixed by >>)
>> net/ethtool/synce.c:156:1-3: WARNING: possible condition with no effect (if == else)

vim +156 net/ethtool/synce.c

1890b45913d75d Maciej Machnikowski 2021-12-01  150  
1890b45913d75d Maciej Machnikowski 2021-12-01  151  static int rclk_reply_size(const struct ethnl_req_info *req_base,
1890b45913d75d Maciej Machnikowski 2021-12-01  152  			   const struct ethnl_reply_data *reply_base)
1890b45913d75d Maciej Machnikowski 2021-12-01  153  {
1890b45913d75d Maciej Machnikowski 2021-12-01  154  	const struct rclk_request_data *request = RCLK_REQDATA(req_base);
1890b45913d75d Maciej Machnikowski 2021-12-01  155  
1890b45913d75d Maciej Machnikowski 2021-12-01 @156  	if (request->out_pin.valid)
1890b45913d75d Maciej Machnikowski 2021-12-01  157  		return nla_total_size(sizeof(u32)) +	/* ETHTOOL_A_RCLK_PIN_FLAGS */
1890b45913d75d Maciej Machnikowski 2021-12-01  158  		       nla_total_size(sizeof(u32));	/* ETHTOOL_A_RCLK_OUT_PIN_IDX */
1890b45913d75d Maciej Machnikowski 2021-12-01  159  	else
1890b45913d75d Maciej Machnikowski 2021-12-01  160  		return nla_total_size(sizeof(u32)) +	/* ETHTOOL_A_RCLK_PIN_MIN */
1890b45913d75d Maciej Machnikowski 2021-12-01  161  		       nla_total_size(sizeof(u32));	/* ETHTOOL_A_RCLK_PIN_MAX */
1890b45913d75d Maciej Machnikowski 2021-12-01  162  }
1890b45913d75d Maciej Machnikowski 2021-12-01  163  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

^ permalink raw reply	[flat|nested] 56+ messages in thread

end of thread, other threads:[~2021-12-06 18:49 UTC | newest]

Thread overview: 56+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-01 18:02 [PATCH v4 net-next 0/4] Add ethtool interface for SyncE Maciej Machnikowski
2021-12-01 18:02 ` [Intel-wired-lan] " Maciej Machnikowski
2021-12-01 18:02 ` [PATCH v4 net-next 1/4] ice: add support detecting features based on netlist Maciej Machnikowski
2021-12-01 18:02   ` [Intel-wired-lan] " Maciej Machnikowski
2021-12-01 18:02 ` [PATCH v4 net-next 2/4] ethtool: Add ability to configure recovered clock for SyncE feature Maciej Machnikowski
2021-12-01 18:02   ` [Intel-wired-lan] " Maciej Machnikowski
2021-12-02 11:48   ` kernel test robot
2021-12-02 11:48     ` kernel test robot
2021-12-02 11:48     ` [Intel-wired-lan] " kernel test robot
2021-12-02 12:43   ` Ido Schimmel
2021-12-02 12:43     ` [Intel-wired-lan] " Ido Schimmel
2021-12-02 15:17     ` Machnikowski, Maciej
2021-12-02 15:17       ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-02 16:35       ` Ido Schimmel
2021-12-02 16:35         ` [Intel-wired-lan] " Ido Schimmel
2021-12-02 17:20         ` Machnikowski, Maciej
2021-12-02 17:20           ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-03 14:26           ` Petr Machata
2021-12-03 14:26             ` [Intel-wired-lan] " Petr Machata
2021-12-03 14:55             ` Machnikowski, Maciej
2021-12-03 14:55               ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-03 15:58               ` Ido Schimmel
2021-12-03 15:58                 ` [Intel-wired-lan] " Ido Schimmel
2021-12-03 16:26               ` Petr Machata
2021-12-03 16:26                 ` [Intel-wired-lan] " Petr Machata
2021-12-03 16:50                 ` Machnikowski, Maciej
2021-12-03 16:50                   ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-03 18:44                   ` Petr Machata
2021-12-03 18:44                     ` [Intel-wired-lan] " Petr Machata
2021-12-03 19:07                     ` Machnikowski, Maciej
2021-12-03 19:07                       ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-06 14:40                       ` Petr Machata
2021-12-06 14:40                         ` [Intel-wired-lan] " Petr Machata
2021-12-06 15:09                         ` Machnikowski, Maciej
2021-12-06 15:09                           ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-06 16:00                           ` Petr Machata
2021-12-06 16:00                             ` [Intel-wired-lan] " Petr Machata
2021-12-06 18:49                             ` Machnikowski, Maciej
2021-12-06 18:49                               ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-03 15:45           ` Ido Schimmel
2021-12-03 15:45             ` [Intel-wired-lan] " Ido Schimmel
2021-12-03 16:18             ` Machnikowski, Maciej
2021-12-03 16:18               ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-03 18:21               ` Petr Machata
2021-12-03 18:21                 ` [Intel-wired-lan] " Petr Machata
2021-12-05 12:24               ` Ido Schimmel
2021-12-05 12:24                 ` [Intel-wired-lan] " Ido Schimmel
2021-12-06  9:15                 ` Machnikowski, Maciej
2021-12-06  9:15                   ` [Intel-wired-lan] " Machnikowski, Maciej
2021-12-01 18:02 ` [PATCH v4 net-next 3/4] ice: add support for monitoring SyncE DPLL state Maciej Machnikowski
2021-12-01 18:02   ` [Intel-wired-lan] " Maciej Machnikowski
2021-12-01 18:02 ` [PATCH v4 net-next 4/4] ice: add support for SyncE recovered clocks Maciej Machnikowski
2021-12-01 18:02   ` [Intel-wired-lan] " Maciej Machnikowski
2021-12-02  1:56   ` Jakub Kicinski
2021-12-02  1:56     ` [Intel-wired-lan] " Jakub Kicinski
2021-12-02 12:29 [PATCH v4 net-next 2/4] ethtool: Add ability to configure recovered clock for SyncE feature kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.