> On Jun 10, 2015, at 5:53 PM, Ben Pfaff <b...@nicira.com> wrote: > > On Wed, Jun 10, 2015 at 05:52:21PM -0700, Jarno Rajahalme wrote: >> >>> On Jun 10, 2015, at 5:46 PM, Ben Pfaff <b...@nicira.com> wrote: >>> >>> On Tue, Jun 09, 2015 at 05:24:18PM -0700, Jarno Rajahalme wrote: >>>> struct match in ofputil_flow_mod uses a lot of space, when used for a >>>> stored bundle message. This patch adds a new struct >>>> ofputil_miniflow_mod, that uses a minimatch instead of match in hopes >>>> of using less memory when handling large bundles. >>>> >>>> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> >>> >>> Hmm. It's uglier than I expected. Might still be worthwhile. >>> >>> I wonder whether it would be a big deal to change ofputil_flow_mod to >>> always use a minimatch? >> >> I tried that briefly and decided at that time that it is too much >> work. But I might try again and take a better look. > > This whole idea might be "premature optimization"; not sure.
I’ll drop this patch for now, then. Jarno _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev