Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread David Marchand
On Thu, Nov 16, 2023 at 12:32 PM Ilya Maximets wrote: > > +AT_CHECK([ > > +ovs-vsctl set bridge br0 \ > > +datapath_type=dummy \ > > +mcast_snooping_enable=true \ > > +other-config:mcast-snooping-disable-flood-unregistered=false > > Nit: > Not a full review, but in case you're send

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread Ilya Maximets
On 11/10/23 18:52, David Marchand wrote: > Various options affect how the mcast snooping module work. > > When multicast snooping is enabled and a reporter is known, it is still > possible to flood associated packets to some other port via the > mcast-snooping-flood option. > > If flooding unregi

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread Eelco Chaudron
On 16 Nov 2023, at 12:13, David Marchand wrote: > On Thu, Nov 16, 2023 at 11:57 AM Eelco Chaudron wrote: >> On 10 Nov 2023, at 18:52, David Marchand wrote: >>> +Final flow: unchanged >>> +Megaflow: >>> recirc_id=0,eth,udp,in_port=3,dl_src=aa:55:aa:55:00:ff,dl_dst=01:00:5e:01:01:01,nw_dst=224.1

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread Eelco Chaudron
On 16 Nov 2023, at 12:10, David Marchand wrote: > Hello Eelco, > > On Thu, Nov 16, 2023 at 11:57 AM Eelco Chaudron wrote: > > [snip] > >>> +bridge("br0") >>> +- >>> + 0. priority 32768 >>> +NORMAL >>> + -> forwarding to mcast group port >>> + -> forwarding to mcast flood

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread David Marchand
On Thu, Nov 16, 2023 at 11:57 AM Eelco Chaudron wrote: > On 10 Nov 2023, at 18:52, David Marchand wrote: > > +Final flow: unchanged > > +Megaflow: > > recirc_id=0,eth,udp,in_port=3,dl_src=aa:55:aa:55:00:ff,dl_dst=01:00:5e:01:01:01,nw_dst=224.1.1.1,nw_frag=no > > +]) > > +AT_CHECK([sed -ne 's/^Dat

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread David Marchand
Hello Eelco, On Thu, Nov 16, 2023 at 11:57 AM Eelco Chaudron wrote: [snip] > > +bridge("br0") > > +- > > + 0. priority 32768 > > +NORMAL > > + -> forwarding to mcast group port > > + -> forwarding to mcast flood port > > + > > +Final flow: unchanged > > +Megaflow: > > r

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-16 Thread Eelco Chaudron
On 10 Nov 2023, at 18:52, David Marchand wrote: > Various options affect how the mcast snooping module work. > > When multicast snooping is enabled and a reporter is known, it is still > possible to flood associated packets to some other port via the > mcast-snooping-flood option. > > If floodi

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-15 Thread Paolo Valerio
David Marchand writes: > Various options affect how the mcast snooping module work. > > When multicast snooping is enabled and a reporter is known, it is still > possible to flood associated packets to some other port via the > mcast-snooping-flood option. > > If flooding unregistered traffic is

Re: [ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-15 Thread Simon Horman
On Fri, Nov 10, 2023 at 06:52:03PM +0100, David Marchand wrote: > Various options affect how the mcast snooping module work. > > When multicast snooping is enabled and a reporter is known, it is still > possible to flood associated packets to some other port via the > mcast-snooping-flood option.

[ovs-dev] [PATCH v2 1/2] mcast-snooping: Test per port explicit flooding.

2023-11-10 Thread David Marchand
Various options affect how the mcast snooping module work. When multicast snooping is enabled and a reporter is known, it is still possible to flood associated packets to some other port via the mcast-snooping-flood option. If flooding unregistered traffic is disabled, it is still possible to flo