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.

Sign up for GitHub

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

Jump to bottom

[DO NOT MERGE] Add TNA multipipe #148

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[DO NOT MERGE] Add TNA multipipe #148

wants to merge 1 commit into from

Conversation

pudelkoM
Copy link
Member

@pudelkoM pudelkoM commented Jan 8, 2021

Supersedes #11

@pudelkoM
Convert to multipipe
31874da 
Basic setup

Convert to multipipe

Use ALPM to scale ipv4 to 750k

Fix multicast pruning not working

By carrying the original ingress port all the way to the
last pipe via bridged metadata.

Also fix the build script which was looking for missing files
from older runs.

Cleanup

Commit p4c output
@pudelkoM pudelkoM added the 🍩 merge Do not merge label Jan 8, 2021
@onf-bot
Copy link
Collaborator

onf-bot commented Oct 28, 2021

Can one of the admins verify this patch? For help please reach out to support@opennetworking.org. The list of admins can be updated in the Jenkins configuration.

@super4hanlin
Copy link

Any plan about this patch?
I have tried to build this patch, but get following errors:
error: Inferred incompatible container alignments for field ingress::hdr.bridge_md.inner_vlan_id:
Source unknown: alignment = 3 (little endian)
Previously inferred alignments:
/home/P4/fabric-tna-multi-pipe-new/p4src/include/parser.p4(61): alignment = 0 (little endian)
fabric_md.inner_vlan_id = hdr.inner_vlan_tag.vlan_id;
^^^^^^^^^^^^^^^^^^^^^^^

Number of errors exceeded set maximum of 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
🍩 merge Do not merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pudelkoM @onf-bot @super4hanlin