Added guide to add table entries in P4 dpdk using native control plan… by Vineet1101 · Pull Request #116 · p4lang/p4-dpdk-target

Vineet1101 · 2025-03-02T05:56:40Z

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:

Setting up the environment for P4 DPDK.
Writing and compiling a P4 program for DPDK.
Using the native control plane API to insert table entries dynamically.
Verifying table entries using appropriate commands or logging outputs.

Related Issues:

Fixes #27

Vineet1101 · 2025-03-02T05:59:07Z

@jafingerhut please review this

swaroopsarma

initial placement comments

doc/P4-DPDK-starting-guide/mac_modifier.p4

doc/P4-DPDK-starting-guide/p4-dpdk-table-entries-guide.md

doc/P4-DPDK-starting-guide/table_manager.c

doc/P4-DPDK-starting-guide/test_packets.py

Vineet1101 · 2025-03-03T10:44:00Z

@swaroopsarma I have made the changes can you review them

swaroopsarma · 2025-03-03T11:42:25Z

Makefile in e2e-test p4rt missing

Vineet1101 · 2025-03-03T12:41:45Z

The gitignore file has mentioned Makefile so should i remove it from there or I add that part in the guide??

swaroopsarma · 2025-03-03T13:20:20Z

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

Vineet1101 · 2025-03-03T13:58:24Z

@swaroopsarma I have added the makefile

jafingerhut · 2025-03-03T17:00:47Z

@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.

Vineet1101 · 2025-03-03T19:39:01Z

@jafingerhut I have updated the code of makefile

jafingerhut · 2025-03-04T16:29:29Z

@jafingerhut please review this

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?

jafingerhut · 2025-07-07T20:51:19Z

@jafingerhut please review this

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?

Davide Scano and I discussed this PR today. My apologies for not commenting on this much sooner.

My preference would be that before I review this, I would really want someone else to confirm the following:

I (person X, not Andy) started from version Y of Linux distribution Z (e.g. Ubuntu 22.04 or Ubuntu 24.04 are ideal, but others are OK, too), and followed exactly the sequence of steps S, using the changes in this PR, to install all of the software required. As a result, I could run this "hello world" style example, i.e. compiling some simple P4 program and adding table entries and sending test packets through. And, it all works.

I am not personally interested in spending hours of experimentation to learn what those steps might be. I'd rather be told what they are, and have someone else confirm that they work.

Vineet1101 · 2025-10-15T01:32:40Z

@jafingerhut I am working on this. Some steps are not working , I will update those steps and will update the pr soon

jafingerhut · 2025-10-15T04:19:11Z

@jafingerhut I am working on this. Some steps are not working , I will update those steps and will update the pr soon

Thanks for doing that.

I would also recommend that for ease of following the instructions by others in the future, that all of the installation steps before the step "Verify P4RT Installation" would be good to put into a Bash script as a separate file. Then the install instructions can be reduced to simply "Run this command:" with a command something like ./install.sh. Following the current instructions involves over 50 separate commands, which is extremely tedious and error-prone for others to follow.

Vineet1101 · 2025-10-15T07:40:11Z

sure will do that

@jafingerhut I am working on this. Some steps are not working , I will update those steps and will update the pr soon

Thanks for doing that.

I would also recommend that for ease of following the instructions by others in the future, that all of the installation steps before the step "Verify P4RT Installation" would be good to put into a Bash script as a separate file. Then the install instructions can be reduced to simply "Run this command:" with a command something like ./install.sh. Following the current instructions involves over 50 separate commands, which is extremely tedious and error-prone for others to follow.

Vineet1101 · 2025-12-19T15:22:36Z

@Dscano @jafingerhut can you please review it, I have tested these instructions on a Ubuntu22.04 VM and my switch started and it was running fine

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp

examples/psa/simple_l2l3_lpm/ports.json

examples/psa/simple_l2l3_lpm/switchd.config

install_dpdk.sh

examples/psa/simple_l2l3_lpm/build.sh

Dscano · 2025-12-20T08:31:10Z

@Vineet1101

Can you add riyazahm22 and jafingerhut to the reviewers?

Vineet1101 · 2025-12-20T08:47:02Z

@Vineet1101

Can you add riyazahm22 and jafingerhut to the reviewers?

@riyazahm22 can you please review this pr

Vineet1101 · 2025-12-25T13:26:13Z

@Dscano @swaroopsarma @jafingerhut @riyazahm22 can you please review this pr

build_dpdk.sh

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp

README.md

Vineet1101 · 2025-12-26T04:23:42Z

@swaroopsarma this was the error

In file included from /home/p4/sde/install/include/tdi/common/tdi_attributes.hpp:35,
                 from /home/p4/sde/install/include/tdi/common/tdi_table.hpp:32,
                 from /home/p4/sde/install/include/tdi/common/tdi_utils.hpp:28,
                 from tdi_rt/tdi_port/dpdk/tdi_port_table_impl.cpp:21:
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
In file included from ./tdi_rt/tdi_port/dpdk/tdi_port_table_impl.hpp:22,
                 from tdi_rt/tdi_port/dpdk/tdi_port_table_impl.cpp:23:
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:101:16: note:   by 'tdi::PortHdlInfoTableKey::getValue'
  101 |   tdi_status_t getValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:95:16: note:   by 'tdi::PortHdlInfoTableKey::setValue'
   95 |   tdi_status_t setValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:133:16: note:   by 'tdi::PortFpIdxInfoTableKey::getValue'
  133 |   tdi_status_t getValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:127:16: note:   by 'tdi::PortFpIdxInfoTableKey::setValue'
  127 |   tdi_status_t setValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:155:16: note:   by 'tdi_status_t tdi::PortStrInfoTableKey::getValue(const tdi_id_t&, std::string*) const'
  155 |   tdi_status_t getValue(const tdi_id_t &field_id, std::string *value) const;
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:153:16: note:   by 'tdi_status_t tdi::PortStrInfoTableKey::setValue(const tdi_id_t&, const std::string&)'
  153 |   tdi_status_t setValue(const tdi_id_t &field_id, const std::string &value);
      |                ^~~~~~~~
cc1plus: all warnings being treated as errors
make[4]: *** [Makefile:2651: tdi_rt/tdi_port/dpdk/libbfrt_la-tdi_port_table_impl.lo] Error 1
make[4]: *** Waiting for unfinished jobs....
make[4]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[3]: *** [Makefile:3083: all-recursive] Error 1
make[3]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[2]: *** [Makefile:1181: all] Error 2
make[2]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[1]: *** [Makefile:529: all-recursive] Error 1
make[1]: Leaving directory '/home/p4/sde/p4-dpdk-target'
make: *** [Makefile:416: all] Error 2
(.venv) p4@p4dev:~/sde/p4-dpdk-target$

examples/psa/simple_l2l3_lpm/build.sh

README.md

build_dpdk.sh

install_dpdk.sh

Signed-off-by: Vineet1101 <[email protected]>

Vineet1101 · 2025-12-29T08:41:46Z

Two sections need to be removed; see inline comments.

removed them

Dscano

@Vineet1101 I suggest taking a quick pass over the changes before pushing. A few times the updates were only partially correct.

README.md

Vineet1101 · 2025-12-29T09:06:57Z

@Vineet1101 I suggest taking a quick pass over the changes before pushing. A few times the updates were only partially correct.

Sorry for that I thought both the sections should be removed

Dscano · 2025-12-29T09:43:03Z

@Vineet1101 I suggest taking a quick pass over the changes before pushing. A few times the updates were only partially correct.

Sorry for that I thought both the sections should be removed

To summarize, keep the section titled Installing P4-DPDK directly from installation script.

Remove the following sections:

Install dependent packages
Building from install.sh file

Signed-off-by: Vineet1101 <[email protected]>

Vineet1101 · 2025-12-29T10:25:22Z

@Vineet1101 I suggest taking a quick pass over the changes before pushing. A few times the updates were only partially correct.

Sorry for that I thought both the sections should be removed

To summarize, keep the section titled Installing P4-DPDK directly from installation script.

Remove the following sections:

Install dependent packages

Building from install.sh file

There are two files README and README.md so should I make these changes in both the files or only in README.md

Dscano · 2025-12-29T11:43:20Z

@Vineet1101 I suggest taking a quick pass over the changes before pushing. A few times the updates were only partially correct.

Sorry for that I thought both the sections should be removed

To summarize, keep the section titled Installing P4-DPDK directly from installation script.
Remove the following sections:

Install dependent packages

Building from install.sh file

There are two files README and README.md so should I make these changes in both the files or only in README.md

In my opinion, we can delete the README file and keep only README.md. What do you think @swaroopsarma ?

Signed-off-by: Vineet1101 <[email protected]>

README.md

Dscano · 2025-12-30T07:56:47Z

@swaroopsarma this was the error

In file included from /home/p4/sde/install/include/tdi/common/tdi_attributes.hpp:35,
                 from /home/p4/sde/install/include/tdi/common/tdi_table.hpp:32,
                 from /home/p4/sde/install/include/tdi/common/tdi_utils.hpp:28,
                 from tdi_rt/tdi_port/dpdk/tdi_port_table_impl.cpp:21:
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
In file included from ./tdi_rt/tdi_port/dpdk/tdi_port_table_impl.hpp:22,
                 from tdi_rt/tdi_port/dpdk/tdi_port_table_impl.cpp:23:
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:101:16: note:   by 'tdi::PortHdlInfoTableKey::getValue'
  101 |   tdi_status_t getValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:95:16: note:   by 'tdi::PortHdlInfoTableKey::setValue'
   95 |   tdi_status_t setValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:133:16: note:   by 'tdi::PortFpIdxInfoTableKey::getValue'
  133 |   tdi_status_t getValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:127:16: note:   by 'tdi::PortFpIdxInfoTableKey::setValue'
  127 |   tdi_status_t setValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:155:16: note:   by 'tdi_status_t tdi::PortStrInfoTableKey::getValue(const tdi_id_t&, std::string*) const'
  155 |   tdi_status_t getValue(const tdi_id_t &field_id, std::string *value) const;
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:153:16: note:   by 'tdi_status_t tdi::PortStrInfoTableKey::setValue(const tdi_id_t&, const std::string&)'
  153 |   tdi_status_t setValue(const tdi_id_t &field_id, const std::string &value);
      |                ^~~~~~~~
cc1plus: all warnings being treated as errors
make[4]: *** [Makefile:2651: tdi_rt/tdi_port/dpdk/libbfrt_la-tdi_port_table_impl.lo] Error 1
make[4]: *** Waiting for unfinished jobs....
make[4]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[3]: *** [Makefile:3083: all-recursive] Error 1
make[3]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[2]: *** [Makefile:1181: all] Error 2
make[2]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[1]: *** [Makefile:529: all-recursive] Error 1
make[1]: Leaving directory '/home/p4/sde/p4-dpdk-target'
make: *** [Makefile:416: all] Error 2
(.venv) p4@p4dev:~/sde/p4-dpdk-target$

@swaroopsarma, do you have any thoughts on this?

swaroopsarma · 2025-12-30T08:46:34Z

@swaroopsarma this was the error

In file included from /home/p4/sde/install/include/tdi/common/tdi_attributes.hpp:35,
                 from /home/p4/sde/install/include/tdi/common/tdi_table.hpp:32,
                 from /home/p4/sde/install/include/tdi/common/tdi_utils.hpp:28,
                 from tdi_rt/tdi_port/dpdk/tdi_port_table_impl.cpp:21:
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
In file included from ./tdi_rt/tdi_port/dpdk/tdi_port_table_impl.hpp:22,
                 from tdi_rt/tdi_port/dpdk/tdi_port_table_impl.cpp:23:
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:101:16: note:   by 'tdi::PortHdlInfoTableKey::getValue'
  101 |   tdi_status_t getValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:95:16: note:   by 'tdi::PortHdlInfoTableKey::setValue'
   95 |   tdi_status_t setValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:133:16: note:   by 'tdi::PortFpIdxInfoTableKey::getValue'
  133 |   tdi_status_t getValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:127:16: note:   by 'tdi::PortFpIdxInfoTableKey::setValue'
  127 |   tdi_status_t setValue(const tdi_id_t &field_id,
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:198:24: error: 'virtual tdi_status_t tdi::TableKey::getValue(const tdi_id_t&, tdi::KeyFieldValue*) const' was hidden [-Werror=overloaded-virtual=]
  198 |   virtual tdi_status_t getValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:155:16: note:   by 'tdi_status_t tdi::PortStrInfoTableKey::getValue(const tdi_id_t&, std::string*) const'
  155 |   tdi_status_t getValue(const tdi_id_t &field_id, std::string *value) const;
      |                ^~~~~~~~
/home/p4/sde/install/include/tdi/common/tdi_table_key.hpp:187:24: error: 'virtual tdi_status_t tdi::TableKey::setValue(const tdi_id_t&, const tdi::KeyFieldValue&)' was hidden [-Werror=overloaded-virtual=]
  187 |   virtual tdi_status_t setValue(const tdi_id_t &field_id,
      |                        ^~~~~~~~
./tdi_rt/tdi_port/tdi_port_table_key_impl.hpp:153:16: note:   by 'tdi_status_t tdi::PortStrInfoTableKey::setValue(const tdi_id_t&, const std::string&)'
  153 |   tdi_status_t setValue(const tdi_id_t &field_id, const std::string &value);
      |                ^~~~~~~~
cc1plus: all warnings being treated as errors
make[4]: *** [Makefile:2651: tdi_rt/tdi_port/dpdk/libbfrt_la-tdi_port_table_impl.lo] Error 1
make[4]: *** Waiting for unfinished jobs....
make[4]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[3]: *** [Makefile:3083: all-recursive] Error 1
make[3]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[2]: *** [Makefile:1181: all] Error 2
make[2]: Leaving directory '/home/p4/sde/p4-dpdk-target/src'
make[1]: *** [Makefile:529: all-recursive] Error 1
make[1]: Leaving directory '/home/p4/sde/p4-dpdk-target'
make: *** [Makefile:416: all] Error 2
(.venv) p4@p4dev:~/sde/p4-dpdk-target$

@swaroopsarma, do you have any thoughts on this?

Fix should be otherway around. I believe this is warning with latest compilers. The reporting routines are missing override final to tell compiler that we are overriding these routines from base class without which those base class will be hidden and compiler is indicating same. SO isuggest to add override final which is our intention

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp

Signed-off-by: Vineet1101 <[email protected]>

Dscano

LGTM

Dscano

The section text looks good, but the build_p4dpdk_ubuntu "job" needs to pass.

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp

Signed-off-by: Vineet1101 <[email protected]>

swaroopsarma · 2026-01-01T07:01:49Z

@Vineet1101 if you dont find a matching function to override , your initial version of using base class is right thing for those. I missed those details when proposing to add override.

Vineet1101 · 2026-01-01T07:02:49Z

@Vineet1101 if you dont find a matching function to override , your initial version of using base class is right thing for those. I missed those details when proposing to add override.

Ok even when I tried to build using override it was failing

Signed-off-by: Vineet1101 <[email protected]>

Vineet1101 · 2026-01-01T07:30:38Z

@Dscano @swaroopsarma the build was completed so can we merge this??

swaroopsarma

looks good

Dscano

LGTM

Vineet1101 · 2026-01-01T11:14:43Z

So can we merge this??

swaroopsarma reviewed Mar 3, 2025

View reviewed changes

Vineet1101 requested a review from swaroopsarma March 3, 2025 10:44

Dscano mentioned this pull request Dec 10, 2025

bf_switchd crashes with "Assertion mempool_obj_arr != NULL" when loading custom P4 program #119

Open

Vineet1101 force-pushed the vineet-work branch from f99ec0d to 764c376 Compare December 19, 2025 15:19

Vineet1101 force-pushed the vineet-work branch from 764c376 to 83aaf84 Compare December 19, 2025 15:28

Dscano suggested changes Dec 19, 2025

View reviewed changes

Vineet1101 requested a review from Dscano December 25, 2025 13:26

Vineet1101 force-pushed the vineet-work branch from ef6c02a to 49eb2f5 Compare December 25, 2025 13:28

swaroopsarma reviewed Dec 25, 2025

View reviewed changes

build_dpdk.sh Outdated Show resolved Hide resolved

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

Dscano suggested changes Dec 26, 2025

View reviewed changes

examples/psa/simple_l2l3_lpm/build.sh Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

build_dpdk.sh Outdated Show resolved Hide resolved

Vineet1101 commented Dec 26, 2025

View reviewed changes

install_dpdk.sh Show resolved Hide resolved

added build commands for p4-dpdk

9a5704a

Signed-off-by: Vineet1101 <[email protected]>

changes in top level readme file

49f50ca

Signed-off-by: Vineet1101 <[email protected]>

Vineet1101 requested a review from Dscano December 29, 2025 08:44

Dscano suggested changes Dec 29, 2025

View reviewed changes

README.md Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

removed unecessary section from top level readme

37b8d97

Signed-off-by: Vineet1101 <[email protected]>

removed readme file

9d8669c

Signed-off-by: Vineet1101 <[email protected]>

Vineet1101 requested a review from Dscano December 30, 2025 04:43

Dscano suggested changes Dec 30, 2025

View reviewed changes

README.md Show resolved Hide resolved

swaroopsarma requested changes Dec 30, 2025

View reviewed changes

fixed compiler error

05a65f4

Signed-off-by: Vineet1101 <[email protected]>

Dscano approved these changes Dec 30, 2025

View reviewed changes

Dscano suggested changes Dec 30, 2025

View reviewed changes

swaroopsarma reviewed Dec 30, 2025

View reviewed changes

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp Outdated Show resolved Hide resolved

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp Outdated Show resolved Hide resolved

src/tdi_rt/tdi_port/tdi_port_table_key_impl.hpp Outdated Show resolved Hide resolved

added override final in required functions

a23c2be

Signed-off-by: Vineet1101 <[email protected]>

fixed override issues

0754562

Signed-off-by: Vineet1101 <[email protected]>

swaroopsarma approved these changes Jan 1, 2026

View reviewed changes

Dscano approved these changes Jan 1, 2026

View reviewed changes

swaroopsarma merged commit 43f59db into p4lang:main Jan 1, 2026
2 checks passed

Dscano mentioned this pull request Jan 1, 2026

Document how to add table entries to P4 DPDK using its native control plane API p4lang/project-ideas#27

Open

Conversation

Vineet1101 commented Mar 2, 2025

PR Title: Add Guide for Adding Table Entries to P4 DPDK Using Native Control Plane API

Description:

Related Issues:

Uh oh!

Vineet1101 commented Mar 2, 2025

Uh oh!

swaroopsarma left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Vineet1101 commented Mar 3, 2025

Uh oh!

swaroopsarma commented Mar 3, 2025

Uh oh!

Vineet1101 commented Mar 3, 2025

Uh oh!

swaroopsarma commented Mar 3, 2025

Uh oh!

Vineet1101 commented Mar 3, 2025

Uh oh!

jafingerhut commented Mar 3, 2025

Uh oh!

Vineet1101 commented Mar 3, 2025

Uh oh!

jafingerhut commented Mar 4, 2025

Uh oh!

jafingerhut commented Jul 7, 2025

Uh oh!

Vineet1101 commented Oct 15, 2025

Uh oh!

jafingerhut commented Oct 15, 2025

Uh oh!

Vineet1101 commented Oct 15, 2025

Uh oh!

Vineet1101 commented Dec 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Dscano commented Dec 20, 2025

Uh oh!

Vineet1101 commented Dec 20, 2025

Uh oh!

Vineet1101 commented Dec 25, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Vineet1101 commented Dec 26, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Vineet1101 commented Dec 29, 2025

Uh oh!

Dscano left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Vineet1101 commented Dec 29, 2025

Uh oh!

Dscano commented Dec 29, 2025

Uh oh!

Vineet1101 commented Dec 29, 2025

Uh oh!

Dscano commented Dec 29, 2025

Uh oh!

Uh oh!

Dscano commented Dec 30, 2025

Uh oh!

Vineet1101 commented Dec 19, 2025 •

edited

Loading