Re: [ovs-dev] [PATCH ovn 1/4] multicast: Document reserved multicast groups.

2022-07-19 Thread Numan Siddique
On Tue, Jul 5, 2022 at 7:09 AM Dumitru Ceara  wrote:
>
> Signed-off-by: Dumitru Ceara 

Thanks.  I applied this patch to the main branch.

Numan

> ---
>  lib/mcast-group-index.h |   26 +-
>  1 file changed, 21 insertions(+), 5 deletions(-)
>
> diff --git a/lib/mcast-group-index.h b/lib/mcast-group-index.h
> index 5bc725451..e8d82b126 100644
> --- a/lib/mcast-group-index.h
> +++ b/lib/mcast-group-index.h
> @@ -28,11 +28,27 @@ struct sbrec_datapath_binding;
>  enum ovn_mcast_tunnel_keys {
>
>  OVN_MCAST_FLOOD_TUNNEL_KEY = OVN_MIN_MULTICAST,
> -OVN_MCAST_UNKNOWN_TUNNEL_KEY,
> -OVN_MCAST_MROUTER_FLOOD_TUNNEL_KEY,
> -OVN_MCAST_MROUTER_STATIC_TUNNEL_KEY,
> -OVN_MCAST_STATIC_TUNNEL_KEY,
> -OVN_MCAST_FLOOD_L2_TUNNEL_KEY,
> +OVN_MCAST_UNKNOWN_TUNNEL_KEY,/* For L2 unknown dest traffic. */
> +OVN_MCAST_MROUTER_FLOOD_TUNNEL_KEY,  /* For L3 multicast traffic that 
> must
> +  * be relayed (multicast routed).
> +  */
> +OVN_MCAST_MROUTER_STATIC_TUNNEL_KEY, /* For multicast reports that need 
> to
> +  * be forwarded statically towards
> +  * mrouters.
> +  */
> +OVN_MCAST_STATIC_TUNNEL_KEY, /* For switches:
> +  * - for L3 multicast traffic that
> +  *   needs to be forwarded
> +  *   statically.
> +  * For routers:
> +  * - for L3 multicast traffic AND
> +  *   reports that need to be
> +  *   forwarded statically.
> +  */
> +OVN_MCAST_FLOOD_L2_TUNNEL_KEY,   /* Logical switch broadcast domain
> +  * excluding ports towards logical
> +  * routers.
> +  */
>  OVN_MIN_IP_MULTICAST,
>  OVN_MAX_IP_MULTICAST = OVN_MAX_MULTICAST,
>  };
>
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH ovn 1/4] multicast: Document reserved multicast groups.

2022-07-05 Thread Dumitru Ceara
Signed-off-by: Dumitru Ceara 
---
 lib/mcast-group-index.h |   26 +-
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/lib/mcast-group-index.h b/lib/mcast-group-index.h
index 5bc725451..e8d82b126 100644
--- a/lib/mcast-group-index.h
+++ b/lib/mcast-group-index.h
@@ -28,11 +28,27 @@ struct sbrec_datapath_binding;
 enum ovn_mcast_tunnel_keys {
 
 OVN_MCAST_FLOOD_TUNNEL_KEY = OVN_MIN_MULTICAST,
-OVN_MCAST_UNKNOWN_TUNNEL_KEY,
-OVN_MCAST_MROUTER_FLOOD_TUNNEL_KEY,
-OVN_MCAST_MROUTER_STATIC_TUNNEL_KEY,
-OVN_MCAST_STATIC_TUNNEL_KEY,
-OVN_MCAST_FLOOD_L2_TUNNEL_KEY,
+OVN_MCAST_UNKNOWN_TUNNEL_KEY,/* For L2 unknown dest traffic. */
+OVN_MCAST_MROUTER_FLOOD_TUNNEL_KEY,  /* For L3 multicast traffic that must
+  * be relayed (multicast routed).
+  */
+OVN_MCAST_MROUTER_STATIC_TUNNEL_KEY, /* For multicast reports that need to
+  * be forwarded statically towards
+  * mrouters.
+  */
+OVN_MCAST_STATIC_TUNNEL_KEY, /* For switches:
+  * - for L3 multicast traffic that
+  *   needs to be forwarded
+  *   statically.
+  * For routers:
+  * - for L3 multicast traffic AND
+  *   reports that need to be
+  *   forwarded statically.
+  */
+OVN_MCAST_FLOOD_L2_TUNNEL_KEY,   /* Logical switch broadcast domain
+  * excluding ports towards logical
+  * routers.
+  */
 OVN_MIN_IP_MULTICAST,
 OVN_MAX_IP_MULTICAST = OVN_MAX_MULTICAST,
 };

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev