Genric API types format/unformat support for VAT and custom dump

Change-Id: I8bc3a991f0ede0605d78b51ba609fbe5889513f2
Signed-off-by: Neale Ranns <nranns@cisco.com>

6 files changed, 21 insertions, 57 deletions

diff --git a/src/vnet/ethernet/ethernet_types_api.c b/src/vnet/ethernet/ethernet_types_api.c
index 9b2e99a6371..67e6231cfc8 100644
--- a/src/vnet/ethernet/ethernet_types_api.c
+++ b/src/vnet/ethernet/ethernet_types_api.c
@@ -41,14 +41,6 @@ mac_address_encode (const mac_address_t * in, vl_api_mac_address_t * out)
   clib_memcpy (out->bytes, in->bytes, 6);
 }
 
-extern u8 *
-format_vl_api_mac_address (u8 * s, va_list * args)
-{
-  vl_api_mac_address_t *mac = va_arg (*args, vl_api_mac_address_t *);
-
-  return (format (s, "%U", format_mac_address, mac->bytes));
-}
-
 /*
  * fd.io coding-style-patch-verification: ON
  *
diff --git a/src/vnet/ethernet/ethernet_types_api.h b/src/vnet/ethernet/ethernet_types_api.h
index f326f8afa56..b65d9d46c86 100644
--- a/src/vnet/ethernet/ethernet_types_api.h
+++ b/src/vnet/ethernet/ethernet_types_api.h
@@ -32,8 +32,6 @@ extern void mac_address_decode (const struct _vl_api_mac_address *in,
 extern void mac_address_encode (const mac_address_t * in,
 				struct _vl_api_mac_address *out);
 
-extern u8 *format_vl_api_mac_address (u8 * s, va_list * args);
-
 #endif
 
 /*
diff --git a/src/vnet/ethernet/mac_address.c b/src/vnet/ethernet/mac_address.c
index 72725c8e593..419a5b0879f 100644
--- a/src/vnet/ethernet/mac_address.c
+++ b/src/vnet/ethernet/mac_address.c
@@ -15,20 +15,36 @@
 
 #include <vnet/ethernet/mac_address.h>
 
+/* *INDENT-OFF* */
 const mac_address_t ZERO_MAC_ADDRESS = {
   .bytes = {
-	    0, 0, 0, 0, 0, 0,
-	    },
+    0, 0, 0, 0, 0, 0,
+  },
 };
+/* *INDENT-ON* */
 
 u8 *
 format_mac_address_t (u8 * s, va_list * args)
 {
   const mac_address_t *mac = va_arg (*args, mac_address_t *);
 
-  return (format (s, "%U", format_mac_address, mac->bytes));
+  return format (s, "%02x:%02x:%02x:%02x:%02x:%02x",
+		 mac->bytes[0], mac->bytes[1], mac->bytes[2],
+		 mac->bytes[3], mac->bytes[4], mac->bytes[5]);
 }
 
+uword
+unformat_mac_address_t (unformat_input_t * input, va_list * args)
+{
+  mac_address_t *mac = va_arg (*args, mac_address_t *);
+
+  if (!unformat (input, "%_%x:%x:%x:%x:%x:%x%_",
+		 &mac->bytes[0], &mac->bytes[1], &mac->bytes[2],
+		 &mac->bytes[3], &mac->bytes[4], &mac->bytes[5]))
+    return 0;
+
+  return 1;
+}
 
 /*
  * fd.io coding-style-patch-verification: ON
diff --git a/src/vnet/ethernet/mac_address.h b/src/vnet/ethernet/mac_address.h
index e97eab6c3d7..8e1559b8654 100644
--- a/src/vnet/ethernet/mac_address.h
+++ b/src/vnet/ethernet/mac_address.h
@@ -49,6 +49,8 @@ mac_address_from_u64 (u64 u, mac_address_t * mac)
   ethernet_mac_address_from_u64 (u, mac->bytes);
 }
 
+extern uword unformat_mac_address_t (unformat_input_t * input,
+				     va_list * args);
 extern u8 *format_mac_address_t (u8 * s, va_list * args);
 
 #endif
diff --git a/src/vnet/ip/ip_types_api.c b/src/vnet/ip/ip_types_api.c
index d5a56fa5547..11b52760fad 100644
--- a/src/vnet/ip/ip_types_api.c
+++ b/src/vnet/ip/ip_types_api.c
@@ -140,45 +140,6 @@ ip_mprefix_decode (const vl_api_mprefix_t * in, mfib_prefix_t * out)
   ip_address_union_decode (&in->src_address, in->af, &out->fp_src_addr);
 }
 
-u8 *
-format_vl_api_address (u8 * s, va_list * args)
-{
-  const vl_api_address_t *addr = va_arg (*args, vl_api_address_t *);
-
-  if (ADDRESS_IP6 == clib_net_to_host_u32 (addr->af))
-    s = format (s, "ip6:%U", format_ip6_address, addr->un.ip6.address);
-  else
-    s = format (s, "ip4:%U", format_ip4_address, addr->un.ip4.address);
-
-  return s;
-}
-
-u8 *
-format_vl_api_address_union (u8 * s, va_list * args)
-{
-  const vl_api_address_union_t *addr =
-    va_arg (*args, vl_api_address_union_t *);
-  vl_api_address_family_t af = va_arg (*args, vl_api_address_family_t);
-
-  if (ADDRESS_IP6 == af)
-    s = format (s, "ip6:%U", format_ip6_address, addr->ip6.address);
-  else
-    s = format (s, "ip4:%U", format_ip4_address, addr->ip4.address);
-
-  return s;
-}
-
-u8 *
-format_vl_api_prefix (u8 * s, va_list * args)
-{
-  const vl_api_prefix_t *pfx = va_arg (*args, vl_api_prefix_t *);
-
-  s = format (s, "%U/%d", format_vl_api_address,
-	      &pfx->address, pfx->address_length);
-
-  return s;
-}
-
 /*
  * fd.io coding-style-patch-verification: ON
  *
diff --git a/src/vnet/ip/ip_types_api.h b/src/vnet/ip/ip_types_api.h
index 1bc899a4588..be41bf59ec3 100644
--- a/src/vnet/ip/ip_types_api.h
+++ b/src/vnet/ip/ip_types_api.h
@@ -46,11 +46,6 @@ extern void ip_mprefix_decode (const struct _vl_api_mprefix *in,
 extern void ip_mprefix_encode (const mfib_prefix_t * in,
 			       struct _vl_api_mprefix *out);
 
-extern u8 *format_vl_api_address (u8 * s, va_list * args);
-extern u8 *format_vl_api_address_union (u8 * s, va_list * args);
-extern u8 *format_vl_api_prefix (u8 * s, va_list * args);
-extern u8 *format_vl_api_mprefix (u8 * s, va_list * args);
-
 #endif
 
 /*