"dev" <[email protected]> wrote on 07/02/2016 10:30:29 PM:

> From: Ben Pfaff <[email protected]>
> To: [email protected]
> Cc: [email protected]
> Date: 07/02/2016 10:30 PM
> Subject: Re: [ovs-dev] [PATCH v6 1/2] Add address set support.
> Sent by: "dev" <[email protected]>
>
> On Tue, Jun 28, 2016 at 02:20:40PM +0530, [email protected] wrote:
> > From: Russel Bryant <[email protected]>
>
> This misspells Russell's name (I fixed it).
>
> > Update the OVN expression parser to support address sets.  Previously,
> > you could have a set of IP or MAC addresses in this form:
> >
> >     {addr1, addr2, ..., addrN}
> >
> > This patch adds support for a bit of indirection where we can define a
> > set of addresses and refer to them by name.
> >
> >     $name
> >
> > This '$name' can be used in the expresssions like
> >
> >     {addr1, addr2, $name, ... }
> >     {$name}
> >     $name
> >
> > A future patch will expose the ability to define address sets for use.
> >
> > Signed-off-by: Russell Bryant <[email protected]>
> > Co-authored-by: Babu Shanmugam <[email protected]>
> > Signed-off-by: Babu Shanmugam <[email protected]>
>
> Thanks a lot!
>
> The results that Kyle reported are really impressive, so I want to start
> getting this in.  Unfortunately I think that patch 2 probably conflicts
> with Ryan's incremental series, which I'm trying to prioritize because
> it's been in progress so long.  So I'm just looking at patch 1 in this
> review round.

Because of our interest in this set, I've already taken a pass
at merging part 2 with the incremental patch series and can
share the needed delta (if it would be of interested here...)

Ryan
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to