diff mbox series

[ovs-dev] learn: Support learn output with OFPP_TABLE port

Message ID 20240712015355.4172140-1-i@liuyulong.me
State Superseded
Headers show
Series [ovs-dev] learn: Support learn output with OFPP_TABLE port | expand

Checks

Context Check Description
ovsrobot/apply-robot warning apply and check: warning
ovsrobot/github-robot-_Build_and_Test success github build: passed

Commit Message

LIU Yulong July 12, 2024, 1:53 a.m. UTC 
The ofport OFPP_TABLE(0xfff9) is defined as perform
actions in flow table. It will send the packet back
to table=0 to do reprocess. Add this port to learn
output action then users can leverage it to do some
flow table re-entry.
---
 lib/learn.c | 1 +
 1 file changed, 1 insertion(+)

Comments

0-day Robot July 12, 2024, 2:21 a.m. UTC | #1 
Bleep bloop.  Greetings LIU Yulong, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: The subject summary should end with a dot.
Subject: learn: Support learn output with OFPP_TABLE port
ERROR: Author LIU Yulong <i@liuyulong.me> needs to sign off.
Lines checked: 31, Warnings: 1, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@redhat.com

Thanks,
0-day Robot
diff mbox series

Patch

diff --git a/lib/learn.c b/lib/learn.c
index b465a9954..63b06a7c8 100644
--- a/lib/learn.c
+++ b/lib/learn.c
@@ -171,6 +171,7 @@  learn_execute(const struct ofpact_learn *learn, const struct flow *flow,
                 if (ofp_to_u16(port) < ofp_to_u16(OFPP_MAX)
                     || port == OFPP_IN_PORT
                     || port == OFPP_FLOOD
+                    || port == OFPP_TABLE
                     || port == OFPP_LOCAL
                     || port == OFPP_ALL) {
                     ofpact_put_OUTPUT(ofpacts)->port = port;