Skip to content

Commit d072531

Browse files
jacob-kelleranguy11
authored andcommitted
ixgbevf: stop attempting IPSEC offload on Mailbox API 1.5
Commit 339f289 ("ixgbevf: Add support for new mailbox communication between PF and VF") added support for v1.5 of the PF to VF mailbox communication API. This commit mistakenly enabled IPSEC offload for API v1.5. No implementation of the v1.5 API has support for IPSEC offload. This offload is only supported by the Linux PF as mailbox API v1.4. In fact, the v1.5 API is not implemented in any Linux PF. Attempting to enable IPSEC offload on a PF which supports v1.5 API will not work. Only the Linux upstream ixgbe and ixgbevf support IPSEC offload, and only as part of the v1.4 API. Fix the ixgbevf Linux driver to stop attempting IPSEC offload when the mailbox API does not support it. The existing API design choice makes it difficult to support future API versions, as other non-Linux hosts do not implement IPSEC offload. If we add support for v1.5 to the Linux PF, then we lose support for IPSEC offload. A full solution likely requires a new mailbox API with a proper negotiation to check that IPSEC is actually supported by the host. Fixes: 339f289 ("ixgbevf: Add support for new mailbox communication between PF and VF") Signed-off-by: Jacob Keller <[email protected]> Reviewed-by: Przemek Kitszel <[email protected]> Tested-by: Rafal Romanowski <[email protected]> Signed-off-by: Tony Nguyen <[email protected]>
1 parent 4c69c77 commit d072531

File tree

1 file changed

+0
-1
lines changed
  • drivers/net/ethernet/intel/ixgbevf

1 file changed

+0
-1
lines changed

drivers/net/ethernet/intel/ixgbevf/ipsec.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -629,7 +629,6 @@ void ixgbevf_init_ipsec_offload(struct ixgbevf_adapter *adapter)
629629

630630
switch (adapter->hw.api_version) {
631631
case ixgbe_mbox_api_14:
632-
case ixgbe_mbox_api_15:
633632
break;
634633
default:
635634
return;

0 commit comments

Comments
 (0)