[PATCH 0/7] ice: add support for devlink health events

Przemek Kitszel posted 7 patches 1 month, 4 weeks ago
drivers/net/ethernet/intel/ice/Makefile       |   3 +-
scripts/checkpatch.pl                         |   2 +
.../net/ethernet/intel/ice/devlink/health.h   |  59 ++++
.../ice/devlink/{devlink_port.h => port.h}    |   0
drivers/net/ethernet/intel/ice/ice.h          |   2 +
drivers/net/ethernet/intel/ice/ice_eswitch.h  |   2 +-
drivers/net/ethernet/intel/ice/ice_ethtool.h  |   2 +
.../ethernet/intel/ice/ice_ethtool_common.h   |  19 ++
include/net/devlink.h                         |  13 +
.../net/ethernet/intel/ice/devlink/devlink.c  |   2 +-
.../net/ethernet/intel/ice/devlink/health.c   | 302 ++++++++++++++++++
.../ice/devlink/{devlink_port.c => port.c}    |   2 +-
drivers/net/ethernet/intel/ice/ice_ethtool.c  |  10 +-
drivers/net/ethernet/intel/ice/ice_main.c     |  26 +-
drivers/net/ethernet/intel/ice/ice_repr.c     |   2 +-
drivers/net/ethernet/intel/ice/ice_sf_eth.c   |   2 +-
net/devlink/health.c                          |  67 ++++
17 files changed, 498 insertions(+), 17 deletions(-)
create mode 100644 drivers/net/ethernet/intel/ice/devlink/health.h
rename drivers/net/ethernet/intel/ice/devlink/{devlink_port.h => port.h} (100%)
create mode 100644 drivers/net/ethernet/intel/ice/ice_ethtool_common.h
create mode 100644 drivers/net/ethernet/intel/ice/devlink/health.c
rename drivers/net/ethernet/intel/ice/devlink/{devlink_port.c => port.c} (99%)
[PATCH 0/7] ice: add support for devlink health events
Posted by Przemek Kitszel 1 month, 4 weeks ago
Reports for two kinds of events are implemented, Malicious Driver
Detection (MDD) and Tx hang.

Patches 1, 2, 3: core improvements (checkpatch.pl, devlink extension)
Patch 4: rename current ice devlink/ files
Patches 5, 6, 7: ice devlink health infra + reporters

Mateusz did good job caring for this series, and hardening the code.
---
v4:
    - rebase, added patch 4 that renames curent devlink_port files

v3: - extracted devlink_fmsg_dump_skb(), and thus removed ugly copy-pasta
      present in v2 (Jakub);
    - tx hang reported is now called from service_task, to resolve calling
      it from atomic (watchog) context - patch 4
https://lore.kernel.org/netdev/20240821133714.61417-5-przemyslaw.kitszel@intel.com

v2: patch 3 (patch 4 in v3)
    - added additional cast to long in ice_tx_hang_reporter_dump()
    - removed size_mul() in devlink_fmsg_binary_pair_put() call
https://lore.kernel.org/netdev/20240712093251.18683-1-mateusz.polchlopek@intel.com

v1:
https://lore.kernel.org/netdev/20240703125922.5625-1-mateusz.polchlopek@intel.com
---

Ben Shelton (1):
  ice: Add MDD logging via devlink health

Mateusz Polchlopek (1):
  devlink: add devlink_fmsg_dump_skb() function

Przemek Kitszel (5):
  checkpatch: don't complain on _Generic() use
  devlink: add devlink_fmsg_put() macro
  ice: rename devlink_port.[ch] to port.[ch]
  ice: add Tx hang devlink health reporter
  ice: dump ethtool stats and skb by Tx hang devlink health reporter

 drivers/net/ethernet/intel/ice/Makefile       |   3 +-
 scripts/checkpatch.pl                         |   2 +
 .../net/ethernet/intel/ice/devlink/health.h   |  59 ++++
 .../ice/devlink/{devlink_port.h => port.h}    |   0
 drivers/net/ethernet/intel/ice/ice.h          |   2 +
 drivers/net/ethernet/intel/ice/ice_eswitch.h  |   2 +-
 drivers/net/ethernet/intel/ice/ice_ethtool.h  |   2 +
 .../ethernet/intel/ice/ice_ethtool_common.h   |  19 ++
 include/net/devlink.h                         |  13 +
 .../net/ethernet/intel/ice/devlink/devlink.c  |   2 +-
 .../net/ethernet/intel/ice/devlink/health.c   | 302 ++++++++++++++++++
 .../ice/devlink/{devlink_port.c => port.c}    |   2 +-
 drivers/net/ethernet/intel/ice/ice_ethtool.c  |  10 +-
 drivers/net/ethernet/intel/ice/ice_main.c     |  26 +-
 drivers/net/ethernet/intel/ice/ice_repr.c     |   2 +-
 drivers/net/ethernet/intel/ice/ice_sf_eth.c   |   2 +-
 net/devlink/health.c                          |  67 ++++
 17 files changed, 498 insertions(+), 17 deletions(-)
 create mode 100644 drivers/net/ethernet/intel/ice/devlink/health.h
 rename drivers/net/ethernet/intel/ice/devlink/{devlink_port.h => port.h} (100%)
 create mode 100644 drivers/net/ethernet/intel/ice/ice_ethtool_common.h
 create mode 100644 drivers/net/ethernet/intel/ice/devlink/health.c
 rename drivers/net/ethernet/intel/ice/devlink/{devlink_port.c => port.c} (99%)

-- 
2.39.3