On Thu, Aug 4, 2016 at 7:46 PM, Russell Bryant <russ...@ovn.org> wrote: > > > On Wed, Aug 3, 2016 at 6:31 PM, Ben Pfaff <b...@ovn.org> wrote: >> >> On Wed, Aug 03, 2016 at 11:14:20PM +0800, Zong Kai LI wrote: >> > This patch aims to extend Address_Set to Macros, make it more common to >> > accept variable set, not only address. And by that, we can skinny down >> > ACLs, >> > if we use Macros to defines port name sets for ACL to use, since lots of >> > ACL >> > entries are similar but only "inport" and "outport" fields are >> > different. >> > >> > Address_Set improves the performance of OVN control plane. But what more >> > important is, its implementation introduced a "$macros -- element_set" >> > mechanism, it's good way to do bunch jobs for ACLs, and even lflows. >> >> I'd prefer to defer this feature past the Open vSwitch 2.6 release. > > > On an earlier patch, I suggested the idea that we might want to at least > rename the table if we see that coming, even if we don't apply the rest of > the changes. That way, we get the non-backwards compatible part out of the > way before 2.6. What do you think? > > -- > Russell Bryant
Hi, Russell and Ben. I submitted another patch @ http://patchwork.ozlabs.org/patch/657589/ , it's a only rename table Address_Set version. Since we only have address set supported for now, so I didn't renames and changes variables and methods used for address set. I hope this can get merged recently, and I don't get misunderstand on Russell's suggestion. Thanks. Zongkai, LI _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev