> 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

Reply via email to