[ovs-dev] [PATCH v5 3/3] dpif-netdev: split out generic lookup function

Van Haaren, Harry harry.van.haaren at intel.com
Mon Mar 25 16:30:21 UTC 2019


> -----Original Message-----
> From: Ilya Maximets [mailto:i.maximets at samsung.com]
> Sent: Friday, March 22, 2019 1:19 PM
> To: Van Haaren, Harry <harry.van.haaren at intel.com>; Eelco Chaudron
> <echaudro at redhat.com>
> Cc: ovs-dev at openvswitch.org
> Subject: Re: [ovs-dev] [PATCH v5 3/3] dpif-netdev: split out generic lookup
> function
> 
> On 22.03.2019 16:08, Van Haaren, Harry wrote:
> > Hey Eelco,
> >
> >> -----Original Message-----

<snip>

> > I'll work on addressing your comments early next week - thanks again! -H
> 
> Hi Harry,
> 
> I didn't look at this series closely yet. But if you're going to prepare new
> version, it'll be good to fix checkpatch issues reported by 0-day robot too:
> 
>   https://mail.openvswitch.org/pipermail/ovs-dev/2019-February/356615.html
>   https://mail.openvswitch.org/pipermail/ovs-dev/2019-February/356616.html
> 
> BTW, commented out counters looks suspicious.
> 
> Best regards, Ilya Maximets.

Thanks Ilya, fixing checkpatch issues now, no problems.

Correct the lookups_match counter isn't implemented yet - will add it in v6 implementation.

The name of the variable is a bit misleading - it really counts the number of subtables searched for a match, per packet that it found a hit for.

Eg: a packet that matched the 3rd subtable searched, will += counter by 3.

So really, it counts "subtable-effort per packet match". Does this align with you understanding of it?

Thanks, -Harry


More information about the dev mailing list