> On Nov 3, 2013, at 9:22 AM, Ben Pfaff <b...@nicira.com> wrote: > >> On Fri, Nov 01, 2013 at 03:20:10PM -0700, Jarno Rajahalme wrote: >> v2: Properly finish hashes, more flexible configuration. >> >> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > I'm just starting to look at this. I have an initial questions: it > looks like there's the intent to have some flexibility, since > cls->flow_segments[] exists instead of being fixed, but the ovs_assert() > in classifier_init() looks like it prevents any of that flexibility from > being used. What's the story there? >
No story, it is a leftover from a bug hunt and should not be there. There are similar issues with later patches and there is certainly space for design discussion, how to configure things etc. Jarno _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
