On Fri, Feb 15, 2019 at 7:35 AM Vlad Buslov <vla...@mellanox.com> wrote: > > Another problem that I found in cls_fw and cls_route is that they set > arg->stop when empty. Both of them have code unchanged since it was > committed initially in 2005 so I assume this convention is no longer > relevant because all other classifiers don't do that (they only set > arg->stop when arg->fn returns negative value). >
The question is why do you want to use arg->stop==0 as an indication for emptiness? Isn't what arg->count==0 supposed to be?
