On Tue, Jun 19, 2018 at 03:34:29PM +0530, Anoob Joseph wrote: > Hi Bruce, Pablo, > > Any comments on this series? > > Thanks, > Anoob > > On 14/06/18 17:18, Anoob Joseph wrote: > > This patchset modularizes l2fwd application to prepare it for eventmode > > additions. This patchset doesn't change the code flow or logic, except > > for few minor improvements. Some of the newly added functions are used > > in just one place, but is added for efficient usage with eventmode. > > > > v1: > > * Fix all checkpatch reported issues > > My main concern here is how much this eventmode addition is going to complicate the l2fwd example. l2fwd has always been a pretty basic example app to get users started on the basics of DPDK use, and I'm not sure how much we want to move away from that. Is this eventmode-l2fwd better being a separate app, to allow l2fwd to be kept as simple as it can be?
Looking for more thoughts from others here, since it's a community decision as to the scope of the examples. /Bruce
