On Thu, Jul 28, 2016 at 05:56:01PM -0700, Jarno Rajahalme wrote: > This changes ofproto providers modify_group() to never fail. > > Separating major refactoring to a separate patch should make following > patches easier to review. > > Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
Needs the "sparse" warning fix from RULE_COLLECTION(S)_FOR_EACH for GROUP_COLLECTION(S)_FOR_EACH as well. I recommend being really careful about self-review for modify_group_start(). That code has been super-buggy in the past, for the cases of inserting and deleting buckets. Acked-by: Ben Pfaff <b...@ovn.org> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev