[ovs-dev] [classifier-opt 25/28] classifier: Break cls_rule 'flow' and 'wc' members into new "struct match".

Ethan Jackson ethan at nicira.com
Tue Aug 7 21:49:19 UTC 2012


> I dropped the parameter and return value from
> ofputil_match_from_ofp10_match().  After some thought, it seemed that
> I only needed the priority calculation in one place (in
> ofputil_decode_flow_mod()) so I inlined it there.

The other caller didn't need to calculate it?

Ethan


>
>> The indentation is incorrect in ofputil_match_from_ofp11_match().
>
> OK, fixed.
>
>> Everything else looks good, thanks.
>
> Thank you.



More information about the dev mailing list