Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-17 Thread Simon Horman
On Tue, Jan 16, 2024 at 03:20:08PM +0100, Ilya Maximets wrote: > On 12/27/23 12:15, Mohammad Heib wrote: > > Store igmp/mld protocol version into the > > mcast_group internally, the multicast snooping feature > > is used by many OVS consumers and those consumers heavily rely > > on the OVS

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-17 Thread Ilya Maximets
On 1/17/24 13:14, Eelco Chaudron wrote: > > > On 17 Jan 2024, at 12:16, Ilya Maximets wrote: > >> On 1/16/24 15:26, Eelco Chaudron wrote: >>> >>> >>> On 16 Jan 2024, at 15:20, Ilya Maximets wrote: >>> On 12/27/23 12:15, Mohammad Heib wrote: > Store igmp/mld protocol version into the

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-17 Thread Eelco Chaudron
On 17 Jan 2024, at 12:16, Ilya Maximets wrote: > On 1/16/24 15:26, Eelco Chaudron wrote: >> >> >> On 16 Jan 2024, at 15:20, Ilya Maximets wrote: >> >>> On 12/27/23 12:15, Mohammad Heib wrote: Store igmp/mld protocol version into the mcast_group internally, the multicast snooping

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-17 Thread Ilya Maximets
On 1/16/24 15:26, Eelco Chaudron wrote: > > > On 16 Jan 2024, at 15:20, Ilya Maximets wrote: > >> On 12/27/23 12:15, Mohammad Heib wrote: >>> Store igmp/mld protocol version into the >>> mcast_group internally, the multicast snooping feature >>> is used by many OVS consumers and those consumers

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-16 Thread Eelco Chaudron
On 27 Dec 2023, at 12:15, Mohammad Heib wrote: > Store igmp/mld protocol version into the > mcast_group internally, the multicast snooping feature > is used by many OVS consumers and those consumers heavily rely > on the OVS implementation to manage/deal with mcast groups, > some of those

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-16 Thread Eelco Chaudron
On 16 Jan 2024, at 15:20, Ilya Maximets wrote: > On 12/27/23 12:15, Mohammad Heib wrote: >> Store igmp/mld protocol version into the >> mcast_group internally, the multicast snooping feature >> is used by many OVS consumers and those consumers heavily rely >> on the OVS implementation to

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-16 Thread Ilya Maximets
On 12/27/23 12:15, Mohammad Heib wrote: > Store igmp/mld protocol version into the > mcast_group internally, the multicast snooping feature > is used by many OVS consumers and those consumers heavily rely > on the OVS implementation to manage/deal with mcast groups, > some of those consumers also

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-04 Thread Mohammad Heib
Hi Simon, Thank you for the review :) On Thu, Jan 4, 2024 at 1:27 PM Simon Horman wrote: > On Wed, Dec 27, 2023 at 01:15:22PM +0200, Mohammad Heib wrote: > > Store igmp/mld protocol version into the > > mcast_group internally, the multicast snooping feature > > is used by many OVS consumers and

Re: [ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2024-01-04 Thread Simon Horman
On Wed, Dec 27, 2023 at 01:15:22PM +0200, Mohammad Heib wrote: > Store igmp/mld protocol version into the > mcast_group internally, the multicast snooping feature > is used by many OVS consumers and those consumers heavily rely > on the OVS implementation to manage/deal with mcast groups, > some

[ovs-dev] [PATCH v6 1/2] mcast-snooping: Store IGMP/MLD protocol version.

2023-12-27 Thread Mohammad Heib
Store igmp/mld protocol version into the mcast_group internally, the multicast snooping feature is used by many OVS consumers and those consumers heavily rely on the OVS implementation to manage/deal with mcast groups, some of those consumers also need to deal/expose the mcast protocol to the end