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

DOCA TX/RX statistics and leftovers #261

Open
wants to merge 7 commits into
base: doca_xlio_vNext
Choose a base branch
from

Conversation

AlexanderGrissik
Copy link
Collaborator

Description

  1. Removal of tap ring and related code
  2. Fixing TX ring migration
  3. Adding hw_queue_tx stats
  4. Adding hw_queue_rx stats
What

DOCA leftovers and statistics fix

Why ?

DOCA integration

Change type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Tests
  • Other

Check list

  • Code follows the style de facto guidelines of this project
  • Comments have been inserted in hard to understand places
  • Documentation has been updated (if necessary)
  • Test has been added (if possible)

Removing TC and flow-message functionality, from Daemon, which was used by TAP logic.

Signed-off-by: Alexander Grissik <[email protected]>
Attempt TX migration as part of DOCA data path

Signed-off-by: Alexander Grissik <[email protected]>
Removing the 'simple' union substructure used to differentitate ring_tap.

Signed-off-by: Alexander Grissik <[email protected]>
1. Moving hw_queue_tx related stats from the ring to a seperate object inside hw_queue_tx in order to avoid unnecessary indirections.
2. Fixing TX offload statistics

Signed-off-by: Alexander Grissik <[email protected]>
1. Moving hw_queue_rx related stats from the ring to a seperate object inside hw_queue_rx in order to avoid unnecessary indirections.
2. Fixing RX offload statistics
3. Moving DOCA RX related stats from CQ to hw_queue_rx. CQ will be deleted in the future.

Signed-off-by: Alexander Grissik <[email protected]>
It is crucial to revert expected seq num in case of TX drop for TLS offload.
Adding this handling now to avoid serious debugging in the future.

Signed-off-by: Alexander Grissik <[email protected]>
Adding statistics for TX interrupt request and receive.

Signed-off-by: Alexander Grissik <[email protected]>
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.

1 participant