Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added octep-cp plugin, updated virtio header length and egress chaining support #4

Merged

Conversation

kmonendra
Copy link
Contributor

octep-cp: added OCTEON endpoint control plugin
vppinfra: export utils for non vppinfra clib users
session: add api to shrink fifos
svm: add api to shrink allocated
crypto-native: fix AES-CBC encrypt loop
linux-cp: fix esn and anti-replay issue
octeon: add readme for octeon-virtio plugin
octeon: fix library link issue
octeon: add egress chaining support for virtio
octeon: update virtio header length
octeon: use proper refs for roc item spec and mask

Bheemappa Agasimundin and others added 15 commits September 3, 2024 22:59
This patch adds a new PCIe octeon end point control path
plugin which reads mbox messages using octep library. Once
messages are read, corresponding actions are applied to
datapath.

Type: feature
JIRA: https://essjira.marvell.com/browse/IPBUSW-37643
      https://essjira.marvell.com/browse/IPBUSW-39413

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Signed-off-by: Aakash <[email protected]>
Change-Id: Iadd5971185d02ce558c26058cc72577e545d367e
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/115172
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 854c7fb98973a667411f123365ae32a13922748e)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/121115
Reviewed-by: Shiva Shankar Kommula <[email protected]>
Tested-by: Shiva Shankar Kommula <[email protected]>
This patch contains changes to provide OCTEON offload capabilities to
HOST.

Type: feature
JIRA: https://essjira.marvell.com/browse/IPBUSW-37647

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: I3461a9a50897b768750b519eb6c85b6b3a3af3ac
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/119477
Reviewed-by: Abed Mohammad Kamaluddin <[email protected]>
Tested-by: sa_ip-sw-jenkins <[email protected]>
(cherry picked from commit 27d2aab102f0c6fdd40f9c8c39d7d21c821214d3)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/121129
Reviewed-by: Shiva Shankar Kommula <[email protected]>
Tested-by: Shiva Shankar Kommula <[email protected]>
This patch adds generic configuration file for OCTEON end point
instead of having specific configuration like cn106xx or cn103xx.

Type: feature
JIRA: https://essjira.marvell.com/browse/IPBUSW-37647

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: Ieec6bac5f5017523cef09df37ac12d389fb48e93
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/119565
Reviewed-by: Abed Mohammad Kamaluddin <[email protected]>
Tested-by: sa_ip-sw-jenkins <[email protected]>
(cherry picked from commit 2060c1a1da3ec24037bda90a22d5c4271bfa75e4)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/121130
Reviewed-by: Shiva Shankar Kommula <[email protected]>
Tested-by: Shiva Shankar Kommula <[email protected]>
This patch adds support for runtime checksum offload ON/OFF from
HOST.

Type: feature
JIRA:https://essjira.marvell.com/browse/IPBUSW-41157

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: Icc1fc7023f363fe61e5e96cab3c6e92b0d6ef143
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/122465
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Shiva Shankar Kommula <[email protected]>
Type: feature

Signed-off-by: Kommula Shiva Shankar <[email protected]>
Change-Id: I012c9755bb614fc9b15431d6c9fda7c51fa939ea
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/121457
Tested-by: sa_ip-toolkits-Jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
vnet flow enable is failing due to bogus bytes pointed by spec, mask
variables of roc_npc_flow_item structure. Using reference to local
variables defined in block scope is causing this. Moving the variable
declarations to function block scope fixes this issue.

Fixes: 064762e20

Type: fix

Signed-off-by: Sriram Vatala <[email protected]>
Change-Id: I3904199b5b2bd88cd02ada5604059ab6fd12eef7
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/133108
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 0670542d8a6eda382d7b82de3df2d2cea1046a4f)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134661
Reviewed-by: Nithinsen Kaithakadan
The length of the Virtio header fluctuates depending on the features
that have been negotiated with the host. This patch integrates this
modification.

Type: fix
JIRA: https://essjira.marvell.com/browse/IPBUSW-49355

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: Icd448b14a4a4a1bb3590ba99b4e33260c6d5c320
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/132773
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 16d63c8047974e03b2856508bf858feb663c5d1b)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134662
Reviewed-by: Nithinsen Kaithakadan
This patch adds octeon virtio egress chaining support.

Type: feature
JIRA: https://essjira.marvell.com/browse/IPBUSW-50557

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: Iae2328560584f8b4da21e10e9771ad557a2f0b78
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/133543
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134664
Reviewed-by: Nithinsen Kaithakadan
This patch fixes OCTEON DAO library link issue.

Type: fix
JIRA: https://essjira.marvell.com/browse/IPBUSW-52142

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: Ic8f85d3ba5cbd0ddb3946fcb60b36bd530f8c9a1
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/133895
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 82f6e4fbd8f6d5f3611f4aad103dc4e238634924)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134663
Reviewed-by: Nithinsen Kaithakadan
This document provides an overview of the OCTEON virtio plugin and
includes configuration details for using the plugin.

Type: feature
JIRA: https://essjira.marvell.com/browse/IPBUSW-50902
Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: Ie151d8d84fe5c19b3862e93013a04acdef9c8810
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/132586
Reviewed-by: Abed Mohammad Kamaluddin <[email protected]>
Tested-by: sa_ip-sw-jenkins <[email protected]>
(cherry picked from commit 432105b7e391c21b065a0610569b0352967e1ced)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134673
Reviewed-by: Nithinsen Kaithakadan
This patch enables anti-replay when ESN is enabled on a Security
Association (SA) configured via strongSwan.

Type: fix
JIRA: https://essjira.marvell.com/browse/IPBUSW-52363

Signed-off-by: Bheemappa Agasimundin <[email protected]>
Change-Id: I0efd5ba77088214245eb3b19b35309eed2ab66ca
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134340
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 88d83f4567f7441184143a4e400af554d1819853)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134672
Reviewed-by: Nithinsen Kaithakadan
Type: fix
JIRA: https://essjira.marvell.com/browse/IPBUSW-47134

Change-Id: I11cc52ff3867277e6591efb061f96cadfcc70c88
Signed-off-by: Damjan Marion <[email protected]>
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134381
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 9a1fb77f89207cd1cc865a899a9568219b6dc0f2)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134713
Reviewed-by: Nithinsen Kaithakadan
This patch adds a new API to shrink fifos allocated in fifo segments.
In svm_fifo_clear_deq_ntf, calls svm_fifo_del_want_deq_ntf when fifo
is empty.

Type: improvement

JIRA: https://essjira.marvell.com/browse/IPBUSW-50810
Signed-off-by: Varun Rapelly <[email protected]>
Change-Id: Ic2f66730d949f2ea6d59b8c1c016ec261a818a1a
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/133077
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
Reviewed-by: Venkata Ravichandra Mynidi <[email protected]>
Tested-by: sa_ip-sw-jenkins <[email protected]>
(cherry picked from commit d0c2bb324731f60ab136afed14cf283556f2cc64)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134723
Reviewed-by: Nithinsen Kaithakadan
This patch adds new API to shrink fifos in session layer.
Shrinks svm_fifo chunks when the TCP connection is in TCP_STATE_FIN_WAIT_2
and TCP_STATE_CLOSING states.
The current problem with the TIME_WAIT TCP connection state in VPP is that
it keeps all session resources (including RX/TX svm_fifos and their chunks)
for the duration of the 2MSL timeout as required by the RFC. Such behavior
might introduce the out-of-memory scenario when running many TCP sessions.
The session in the TIME_STATE_CLOSING and TCP_STATE_FIN_WAIT_2 states does
not need svm_fifo chunks anymore because it doesn't expect any data transfer.
Deallocating svm_fifo chunks allows memory saving while keeping svm_fifo
control structures used for signaling and notifications.

Type: improvement

JIRA: https://essjira.marvell.com/browse/IPBUSW-50810
Signed-off-by: Varun Rapelly <[email protected]>
Change-Id: Ibc92c021e8b47928b351059c3ed46ae77458a72e
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/133676
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
Reviewed-by: Venkata Ravichandra Mynidi <[email protected]>
(cherry picked from commit 0e746baddfcd19f59291384bc3b2b3607f63177b)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134724
Reviewed-by: Nithinsen Kaithakadan
Exports below functions for non src/vppinfra clib users:
1. clib_file_n_bytes
2. sprintf_clib_timebase_time

Type: improvement

JIRA: https://essjira.marvell.com/browse/IPBUSW-50810
Signed-off-by: Varun Rapelly <[email protected]>
Change-Id: Ia8356e15559290625b4ef0e22507d112abf9b900
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/133076
Tested-by: sa_ip-sw-jenkins <[email protected]>
Reviewed-by: Venkata Ravichandra Mynidi <[email protected]>
Reviewed-by: Monendra Singh Kushwaha <[email protected]>
(cherry picked from commit 342e3cb50e02a02a6476e275ecc33c39fc4c331f)
Reviewed-on: https://sj1git1.cavium.com/c/IP/SW/toolkits/vpp/+/134725
Reviewed-by: Nithinsen Kaithakadan
@kmonendra kmonendra merged commit 4b9c810 into MarvellEmbeddedProcessors:stable/2402 Sep 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants