-
Notifications
You must be signed in to change notification settings - Fork 20
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 guide to add table entries in P4 dpdk using native control plan… #116
base: main
Are you sure you want to change the base?
Conversation
@jafingerhut please review this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
initial placement comments
@@ -0,0 +1,190 @@ | |||
/* -*- P4_16 -*- */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move the P4 to examples directory
|
||
CC = gcc | ||
CFLAGS = -Wall -Werror | ||
LDFLAGS = -lrte_eal -lrte_mempool -lrte_mbuf -lrte_ethdev -lp4rt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add instructions on how to download p4rt
@@ -0,0 +1,226 @@ | |||
#include <stdio.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
create and move this to e2e-test/p4rt. Provide make file also as part of test being added rather than providing in guide
@@ -0,0 +1,100 @@ | |||
#!/usr/bin/env python3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to e2e-test/p4rt
Signed-off-by: Vineet1101 <Vineetgoel692@gmail.com>
@swaroopsarma I have made the changes can you review them |
Makefile in e2e-test p4rt missing |
The gitignore file has mentioned Makefile so should i remove it from there or I add that part in the guide?? |
you can still add it with git add -f though its in gitignore |
@swaroopsarma I have added the makefile |
The makefile I see added in the PR is an empty file. That was not the intent of the comment, I don't think. |
@jafingerhut I have updated the code of makefile |
I would be happy to do so when I have some time to try out the steps and see if they work. I would like to ask: have you tried out the steps in your instructions and successfully installed the necessary software on a aystem of your own, and successfully run the test P4 program and control plane code interacting with it? |
PR Title: Add Guide for Adding Table Entries to P4 DPDK Using Native Control Plane API
Description:
This PR adds a comprehensive guide on how to add table entries to a P4 DPDK pipeline using its native control plane API. The guide walks through the necessary steps to interact with the P4 DPDK control plane, providing an in-depth explanation of:
Related Issues:
Fixes #27