Ah, I see, I will continue reading now that I understand that, thanks. On Nov 3, 2013 12:42 PM, "Jarno Rajahalme" <jrajaha...@nicira.com> wrote:
> > > 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
