Re: [ovs-dev] [PATCH] ovn-controller: handle multicast group for local datapath only

2018-05-08 Thread Han Zhou
On Tue, May 8, 2018 at 11:21 AM, Mark Michelson wrote: > > On 05/08/2018 01:26 PM, Han Zhou wrote: >> >> >> >> On Tue, May 8, 2018 at 8:53 AM, Ben Pfaff wrote: >> > >> > On Thu, May 03, 2018 at 07:30:56PM -0700, Han Zhou wrote: >> > > Most mc related flows

Re: [ovs-dev] [PATCH] ovn-controller: handle multicast group for local datapath only

2018-05-08 Thread Mark Michelson
On 05/08/2018 01:26 PM, Han Zhou wrote: On Tue, May 8, 2018 at 8:53 AM, Ben Pfaff > wrote: > > On Thu, May 03, 2018 at 07:30:56PM -0700, Han Zhou wrote: > > Most mc related flows are added only if the mc is related to local > > datapath, but there is a

Re: [ovs-dev] [PATCH] ovn-controller: handle multicast group for local datapath only

2018-05-08 Thread Han Zhou
On Tue, May 8, 2018 at 8:53 AM, Ben Pfaff wrote: > > On Thu, May 03, 2018 at 07:30:56PM -0700, Han Zhou wrote: > > Most mc related flows are added only if the mc is related to local > > datapath, but there is a flow added before the local datapath > > check, out of the function

Re: [ovs-dev] [PATCH] ovn-controller: handle multicast group for local datapath only

2018-05-08 Thread Ben Pfaff
On Thu, May 03, 2018 at 07:30:56PM -0700, Han Zhou wrote: > Most mc related flows are added only if the mc is related to local > datapath, but there is a flow added before the local datapath > check, out of the function consider_mc_group(). This patch fixes it. > > Signed-off-by: Han Zhou

Re: [ovs-dev] [PATCH] ovn-controller: handle multicast group for local datapath only

2018-05-07 Thread Mark Michelson
Looks good to me. Acked-by: Mark Michelson On 05/03/2018 10:30 PM, Han Zhou wrote: Most mc related flows are added only if the mc is related to local datapath, but there is a flow added before the local datapath check, out of the function consider_mc_group(). This patch

[ovs-dev] [PATCH] ovn-controller: handle multicast group for local datapath only

2018-05-03 Thread Han Zhou
Most mc related flows are added only if the mc is related to local datapath, but there is a flow added before the local datapath check, out of the function consider_mc_group(). This patch fixes it. Signed-off-by: Han Zhou --- ovn/controller/physical.c | 31