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

Ben Pfaff blp at nicira.com
Tue Aug 7 21:53:51 UTC 2012


On Tue, Aug 07, 2012 at 02:49:19PM -0700, Ethan Jackson wrote:
> > 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?

It was in decoding a flow stats reply, IIRC.  I think that there it's
actually deceptive to change something else to 65535 because it
misrepresents what the switch actually reported.



More information about the dev mailing list