Skip to content

Convert drivers to return XFRM configuration errors through extack

Leon Romanovsky intel-wired-lan linux-doc lkml llvm netdev oe-kbuild-all 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest10 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10xfrm: extend add policy callback to set failure reasonDocumentation/networking/xfrm_device.rst drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c include/linux/netdevice.h +1 more+5 -50 replies
2/10net/mlx5e: Fill IPsec policy validation failure reasondrivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c+12 -100 replies
3/10xfrm: extend add state callback to set failure reasonDocumentation/networking/xfrm_device.rst drivers/net/bonding/bond_main.c drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c +9 more+30 -200 replies
4/10net/mlx5e: Fill IPsec state validation failure reasondrivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c+32 -431 reply
5/10netdevsim: Fill IPsec state validation failure reasondrivers/net/netdevsim/ipsec.c+5 -60 replies
6/10nfp: fill IPsec state validation failure reasondrivers/net/ethernet/netronome/nfp/crypto/ipsec.c+19 -192 replies
7/10ixgbevf: fill IPsec state validation failure reasondrivers/net/ethernet/intel/ixgbevf/ipsec.c+8 -90 replies
8/10ixgbe: fill IPsec state validation failure reasondrivers/net/ethernet/intel/ixgbe/ixgbe_ipsec.c+10 -110 replies
9/10bonding: fill IPsec state validation failure reasondrivers/net/bonding/bond_main.c+1 -12 replies
10/10cxgb4: fill IPsec state validation failure reasondrivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c drivers/net/ethernet/chelsio/inline_crypto/ch_ipsec/chcr_ipsec.c+15 -160 replies

Reviews trailers per patch

1 of 10 patches has a Reviewed-by or Acked-by.

PatchSimon Horman
1/10
2/10
3/10
4/10
5/10
6/10R-b
7/10
8/10
9/10
10/10

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH net-next 00/10] Convert drivers to return XFRM configuration errors through extack

Hi,

This series continues effort started by Sabrina to return XFRM configuration errors through extack. It allows for user space software stack easily present driver failure reasons to users.

As a note, Intel drivers have a path where extack is equal to NULL, and error prints won't be available in current patchset. If it is needed, it can be changed by adding special to Intel macro to print to dmesg in case of extack == NULL.

Thanks

Leon Romanovsky (10):
xfrm: extend add policy callback to set failure reason net/mlx5e: Fill IPsec policy validation failure reason
xfrm: extend add state callback to set failure reason net/mlx5e: Fill IPsec state validation failure reason
netdevsim: Fill IPsec state validation failure reason
nfp: fill IPsec state validation failure reason
ixgbevf: fill IPsec state validation failure reason
ixgbe: fill IPsec state validation failure reason
bonding: fill IPsec state validation failure reason
cxgb4: fill IPsec state validation failure reason

 Documentation/networking/xfrm_device.rst      |   4 +-
 drivers/net/bonding/bond_main.c               |  10 +-
 .../net/ethernet/chelsio/cxgb4/cxgb4_main.c   |   8 +-
 .../inline_crypto/ch_ipsec/chcr_ipsec.c       |  34 +++---
 .../net/ethernet/intel/ixgbe/ixgbe_ipsec.c    |  27 ++---
 drivers/net/ethernet/intel/ixgbevf/ipsec.c    |  21 ++--
 .../mellanox/mlx5/core/en_accel/ipsec.c       | 103 ++++++++----------
 .../net/ethernet/netronome/nfp/crypto/ipsec.c |  41 +++----
 drivers/net/netdevsim/ipsec.c                 |  14 +--
 include/linux/netdevice.h                     |   4 +-
 net/xfrm/xfrm_device.c                        |   9 +-
 net/xfrm/xfrm_state.c                         |   2 +-
 12 files changed, 137 insertions(+), 140 deletions(-)
Signature · 1 line
2.39.1