Merge branch 'macsec-offload-fixes'
authorDavid S. Miller <davem@davemloft.net>
Fri, 4 Nov 2022 10:43:57 +0000 (10:43 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 4 Nov 2022 10:43:57 +0000 (10:43 +0000)
commitf4e405f566ff57db0c59e68a1b5ff4ff09989a91
tree1056d292135103ec50d2fb60c614e4c4a2d450fa
parentcdb525ca92b196f8916102b62431aa0d9a644ff2
parentaaab73f8fba4fd38f4d2617440d541a1c334e819
Merge branch 'macsec-offload-fixes'

Sabrina Dubroca says:

====================
macsec: offload-related fixes

I'm working on a dummy offload for macsec on netdevsim. It just has a
small SecY and RXSC table so I can trigger failures easily on the
ndo_* side. It has exposed a couple of issues.

The first patch is a revert of commit c850240b6c41 ("net: macsec:
report real_dev features when HW offloading is enabled"). That commit
tried to improve the performance of macsec offload by taking advantage
of some of the NIC's features, but in doing so, broke macsec offload
when the lower device supports both macsec and ipsec offload, as the
ipsec offload feature flags were copied from the real device. Since
the macsec device doesn't provide xdo_* ops, the XFRM core rejects the
registration of the new macsec device in xfrm_api_check.

I'm working on re-adding those feature flags when offload is
available, but I haven't fully solved that yet. I think it would be
safer to do that second part in net-next considering how complex
feature interactions tend to be.

v2:
 - better describe the issue introduced by commit c850240b6c41 (Leon
   Romanovsky)
 - patch #3: drop unnecessary !! (Leon Romanovsky)

v3:
 - patch #3: drop extra newline (Jakub Kicinski)
====================

Signed-off-by: David S. Miller <davem@davemloft.net>