>Thanks for the review.
>The intention of this example is not to replace the existing l3fwd ACL mode, 
>but to provide a minimal and easier starting point for >users who want to 
>understand how to use the DPDK ACL library in isolation.

For that thing, I think  we do have an ACL UT: app/test/*acl*

>The l3fwd application is quite large and includes full L3 forwarding logic, 
>routing tables, multiple lookup paths, and various >configuration options. 
>Because of that, it can be difficult for new users to identify the specific 
>steps required to
>define ACL fields, create and build an ACL context, insert rules, and perform 
>classification on packets.
>This example focuses only on demonstrating ACL usage in a small, 
>self-contained program, without routing or complex forwarding >logic. The goal 
>is to make it easier for beginners to learn how ACL classification works 
>before integrating it into larger applications.
>If the maintainers feel that this use case is already sufficiently covered by 
>existing examples, I’m happy to adjust or improve the patch >accordingly.

I don't think there is a need to introduce a new sample app just for that.
Feel free to submit a patch that improves l3fwd docs and/or comments to make
It easier for newcomers to understand it.

On a slightly different subject:
when you replying to patch's comments - don't forget to hit 'reply all' button,
and please use plain-text format.

Thanks
Konstantin

 

Reply via email to