zebra: SRv6 manager multi chunks support

bgpd/bgp_vty.c

@@ -9243,13 +9243,10 @@ DEFPY (no_bgp_srv6_locator,
 	return CMD_SUCCESS;
 }
 
-DEFPY (show_bgp_srv6,
-       show_bgp_srv6_cmd,
-       "show bgp segment-routing srv6",
-       SHOW_STR
-       BGP_STR
-       "BGP Segment Routing\n"
-       "BGP Segment Routing SRv6\n")
+DEFPY(show_bgp_srv6, show_bgp_srv6_cmd, "show bgp segment-routing srv6 [json]",
+      SHOW_STR BGP_STR
+      "BGP Segment Routing\n"
+      "BGP Segment Routing SRv6\n" JSON_STR)
 {
 	struct bgp *bgp;
 	struct listnode *node;
@@ -9260,43 +9257,123 @@ DEFPY (show_bgp_srv6,
 	char buf[256];
 	char buf_tovpn4_sid[256];
 	char buf_tovpn6_sid[256];
+	json_object *json = NULL;
+	json_object *json_chunks = NULL;
+	json_object *json_function = NULL;
+	json_object *json_functions = NULL;
+	json_object *json_bgp = NULL;
+	json_object *json_bgps = NULL;
+	json_object *json_vpn_policy = NULL;
+	json_object *json_vpn_policy_ip = NULL;
+	json_object *json_vpn_policy_ip6 = NULL;
+	bool uj = use_json(argc, argv);
 
 	bgp = bgp_get_default();
 	if (!bgp)
 		return CMD_SUCCESS;
 
-	vty_out(vty, "locator_name: %s\n", bgp->srv6_locator_name);
-	vty_out(vty, "locator_chunks:\n");
-	for (ALL_LIST_ELEMENTS_RO(bgp->srv6_locator_chunks, node, chunk)) {
-		prefix2str(chunk, buf, sizeof(buf));
-		vty_out(vty, "- %s\n", buf);
-	}
-
-	vty_out(vty, "functions:\n");
-	for (ALL_LIST_ELEMENTS_RO(bgp->srv6_functions, node, func)) {
-		inet_ntop(AF_INET6, &func->sid, buf, sizeof(buf));
-		vty_out(vty, "- sid: %s\n", buf);
-		vty_out(vty, "  locator: %s\n", func->locator_name);
-	}
-
-	vty_out(vty, "bgps:\n");
-	for (ALL_LIST_ELEMENTS_RO(bm->bgp, node, bgp)) {
-		vty_out(vty, "- name: %s\n",
-			bgp->name ? bgp->name : "default");
-
-		tovpn4_sid = bgp->vpn_policy[AFI_IP].tovpn_sid;
-		tovpn6_sid = bgp->vpn_policy[AFI_IP6].tovpn_sid;
-		if (tovpn4_sid)
-			inet_ntop(AF_INET6, tovpn4_sid, buf_tovpn4_sid,
-				  sizeof(buf_tovpn4_sid));
-		if (tovpn6_sid)
-			inet_ntop(AF_INET6, tovpn6_sid, buf_tovpn6_sid,
-				  sizeof(buf_tovpn6_sid));
-
-		vty_out(vty, "  vpn_policy[AFI_IP].tovpn_sid: %s\n",
-			tovpn4_sid ? buf_tovpn4_sid : "none");
-		vty_out(vty, "  vpn_policy[AFI_IP6].tovpn_sid: %s\n",
-			tovpn6_sid ? buf_tovpn6_sid : "none");
+	if (uj) {
+		json = json_object_new_object();
+		json_chunks = json_object_new_array();
+		json_functions = json_object_new_array();
+		json_bgps = json_object_new_array();
+		json_object_string_add(json, "locatorName",
+				       bgp->srv6_locator_name);
+		json_object_object_add(json, "locatorChunks", json_chunks);
+		json_object_object_add(json, "functions", json_functions);
+		json_object_object_add(json, "bgps", json_bgps);
+
+		/* collect all chunk to json array*/
+		for (ALL_LIST_ELEMENTS_RO(bgp->srv6_locator_chunks, node,
+					  chunk)) {
+			prefix2str(chunk, buf, sizeof(buf));
+			json_array_string_add(json_chunks, buf);
+		}
+
+		/* collect all function to json array*/
+		for (ALL_LIST_ELEMENTS_RO(bgp->srv6_functions, node, func)) {
+			json_function = json_object_new_object();
+			json_object_array_add(json_functions, json_function);
+			inet_ntop(AF_INET6, &func->sid, buf, sizeof(buf));
+			json_object_string_add(json_function, "Sid", buf);
+			json_object_string_add(json_function, "locator",
+					       func->locator_name);
+		}
+
+		/* collect all bgp instance to json array*/
+		for (ALL_LIST_ELEMENTS_RO(bm->bgp, node, bgp)) {
+			json_bgp = json_object_new_object();
+			json_object_array_add(json_bgps, json_bgp);
+			json_object_string_add(json_bgp, "name",
+					       bgp->name ? bgp->name
+							 : "default");
+
+			json_vpn_policy = json_object_new_object();
+			json_vpn_policy_ip = json_object_new_object();
+			json_vpn_policy_ip6 = json_object_new_object();
+			json_object_object_add(json_bgp, "vpnPolicy",
+					       json_vpn_policy);
+			json_object_object_add(json_vpn_policy, "ip",
+					       json_vpn_policy_ip);
+			json_object_object_add(json_vpn_policy, "ip6",
+					       json_vpn_policy_ip6);
+
+			tovpn4_sid = bgp->vpn_policy[AFI_IP].tovpn_sid;
+			tovpn6_sid = bgp->vpn_policy[AFI_IP6].tovpn_sid;
+			if (tovpn4_sid)
+				inet_ntop(AF_INET6, tovpn4_sid, buf_tovpn4_sid,
+					  sizeof(buf_tovpn4_sid));
+			if (tovpn6_sid)
+				inet_ntop(AF_INET6, tovpn6_sid, buf_tovpn6_sid,
+					  sizeof(buf_tovpn6_sid));
+
+			json_object_string_add(json_vpn_policy_ip, "toVpnSid",
+					       tovpn4_sid ? buf_tovpn4_sid
+							  : "none");
+			json_object_string_add(json_vpn_policy_ip6, "toVpnSid",
+					       tovpn6_sid ? buf_tovpn6_sid
+							  : "none");
+		}
+
+		vty_out(vty, "%s\n",
+			json_object_to_json_string_ext(
+				json, JSON_C_TO_STRING_PRETTY));
+		json_object_free(json);
+	} else {
+		vty_out(vty, "locator_name: %s\n", bgp->srv6_locator_name);
+		vty_out(vty, "locator_chunks:\n");
+		for (ALL_LIST_ELEMENTS_RO(bgp->srv6_locator_chunks, node,
+					  chunk)) {
+			prefix2str(chunk, buf, sizeof(buf));
+			vty_out(vty, "- %s\n", buf);
+		}
+
+		vty_out(vty, "functions:\n");
+		for (ALL_LIST_ELEMENTS_RO(bgp->srv6_functions, node, func)) {
+			inet_ntop(AF_INET6, &func->sid, buf, sizeof(buf));
+			vty_out(vty, "- sid: %s\n", buf);
+			vty_out(vty, "  locator: %s\n", func->locator_name);
+		}
+
+		vty_out(vty, "bgps:\n");
+		for (ALL_LIST_ELEMENTS_RO(bm->bgp, node, bgp)) {
+			vty_out(vty, "- name: %s\n",
+				bgp->name ? bgp->name : "default");
+
+			tovpn4_sid = bgp->vpn_policy[AFI_IP].tovpn_sid;
+			tovpn6_sid = bgp->vpn_policy[AFI_IP6].tovpn_sid;
+			if (tovpn4_sid)
+				inet_ntop(AF_INET6, tovpn4_sid, buf_tovpn4_sid,
+					  sizeof(buf_tovpn4_sid));
+			if (tovpn6_sid)
+				inet_ntop(AF_INET6, tovpn6_sid, buf_tovpn6_sid,
+					  sizeof(buf_tovpn6_sid));
+
+			vty_out(vty, "  vpn_policy[AFI_IP].tovpn_sid: %s\n",
+				tovpn4_sid ? buf_tovpn4_sid : "none");
+			vty_out(vty, "  vpn_policy[AFI_IP6].tovpn_sid: %s\n",
+				tovpn6_sid ? buf_tovpn6_sid : "none");
+		}
 	}
 
 	return CMD_SUCCESS;

lib/srv6.c

@@ -159,6 +159,25 @@ void srv6_locator_chunk_free(struct srv6_locator_chunk *chunk)
 	XFREE(MTYPE_SRV6_LOCATOR_CHUNK, chunk);
 }
 
+/**
+ * check all chunk in the locator are used by clients
+ *
+ * @param loc SRv6 locator
+ * @return true if all chunks are used.
+ */
+bool srv6_locator_chunks_exhausted(const struct srv6_locator *loc)
+{
+	struct listnode *node;
+	struct srv6_locator_chunk *chunk;
+
+	for (ALL_LIST_ELEMENTS_RO(loc->chunks, node, chunk))
+		if (chunk->proto == 0 && chunk->instance == 0
+		    && chunk->session_id == 0 && chunk->keep == 0)
+			return false;
+
+	return true;
+}
+
 json_object *srv6_locator_chunk_json(const struct srv6_locator_chunk *chunk)
 {
 	char str[256];

lib/srv6.h

@@ -90,6 +90,7 @@ struct srv6_locator {
 	uint64_t current;
 	bool status_up;
 	struct list *chunks;
+	uint8_t chunk_bits_length;
 
 	QOBJ_FIELDS;
 };
@@ -187,6 +188,7 @@ extern struct srv6_locator *srv6_locator_alloc(const char *name);
 extern struct srv6_locator_chunk *srv6_locator_chunk_alloc(void);
 extern void srv6_locator_free(struct srv6_locator *locator);
 extern void srv6_locator_chunk_free(struct srv6_locator_chunk *chunk);
+extern bool srv6_locator_chunks_exhausted(const struct srv6_locator *loc);
 json_object *srv6_locator_chunk_json(const struct srv6_locator_chunk *chunk);
 json_object *srv6_locator_json(const struct srv6_locator *loc);
 

sharpd/sharp_zebra.c

@@ -942,23 +942,17 @@ static int sharp_zebra_process_srv6_locator_chunk(ZAPI_CALLBACK_ARGS)
 	zapi_srv6_locator_chunk_decode(s, &s6c);
 
 	for (ALL_LIST_ELEMENTS(sg.srv6_locators, node, nnode, loc)) {
-		struct prefix_ipv6 *chunk = NULL;
-		struct listnode *chunk_node;
-		struct prefix_ipv6 *c;
+		struct prefix_ipv6 *chunk_prefix = NULL;
 
 		if (strcmp(loc->name, s6c.locator_name) != 0) {
 			zlog_err("%s: Locator name unmatch %s:%s", __func__,
 				 loc->name, s6c.locator_name);
 			continue;
 		}
 
-		for (ALL_LIST_ELEMENTS_RO(loc->chunks, chunk_node, c))
-			if (!prefix_cmp(c, &s6c.prefix))
-				return 0;
-
-		chunk = prefix_ipv6_new();
-		*chunk = s6c.prefix;
-		listnode_add(loc->chunks, chunk);
+		chunk_prefix = prefix_ipv6_new();
+		*chunk_prefix = s6c.prefix;
+		listnode_add(loc->chunks, chunk_prefix);
 		return 0;
 	}
 

tests/topotests/srv6_locator/r1/bgpd.conf

@@ -0,0 +1,3 @@
+router bgp 100
+ bgp router-id 1.1.1.1
+!

tests/topotests/srv6_locator/step1/expected_bgpd_chunks.json

@@ -0,0 +1,18 @@
+{
+  "locatorName": "",
+  "locatorChunks": [],
+  "functions": [],
+  "bgps": [
+    {
+      "name": "default",
+      "vpnPolicy": {
+        "ip": {
+          "toVpnSid": "none"
+        },
+        "ip6": {
+          "toVpnSid": "none"
+        }
+      }
+    }
+  ]
+}

tests/topotests/srv6_locator/step1/expected_locators.json

@@ -0,0 +1,16 @@
+{
+  "locators": [
+    {
+      "name": "loc1",
+      "prefix": "2001:db8:1:1::/64",
+      "statusUp": true,
+      "chunks": []
+    },
+    {
+      "name": "loc2",
+      "prefix": "2001:db8:2:2::/64",
+      "statusUp": true,
+      "chunks": []
+    }
+  ]
+}

tests/topotests/srv6_locator/step10/expected_bgpd_chunks.json

@@ -0,0 +1,18 @@
+{
+  "locatorName": "",
+  "locatorChunks": [],
+  "functions": [],
+  "bgps": [
+    {
+      "name": "default",
+      "vpnPolicy": {
+        "ip": {
+          "toVpnSid": "none"
+        },
+        "ip6": {
+          "toVpnSid": "none"
+        }
+      }
+    }
+  ]
+}

tests/topotests/srv6_locator/step10/expected_locators.json

@@ -0,0 +1,3 @@
+{
+  "locators": []
+}

tests/topotests/srv6_locator/step2/expected_bgpd_chunks.json

@@ -0,0 +1,18 @@
+{
+  "locatorName": "",
+  "locatorChunks": [],
+  "functions": [],
+  "bgps": [
+    {
+      "name": "default",
+      "vpnPolicy": {
+        "ip": {
+          "toVpnSid": "none"
+        },
+        "ip6": {
+          "toVpnSid": "none"
+        }
+      }
+    }
+  ]
+}

tests/topotests/srv6_locator/step2/expected_locators.json

@@ -0,0 +1,21 @@
+{
+  "locators": [
+    {
+      "name": "loc1",
+      "prefix": "2001:db8:1:1::/64",
+      "statusUp": true,
+      "chunks": [
+        {
+          "prefix": "2001:db8:1:1::/68",
+          "proto": "sharp"
+        }
+      ]
+    },
+    {
+      "name": "loc2",
+      "prefix": "2001:db8:2:2::/64",
+      "statusUp": true,
+      "chunks": []
+    }
+  ]
+}