Introduce a mac_address_t on the API and in VPP 50/14550/4
authorNeale Ranns <neale.ranns@cisco.com>
Wed, 29 Aug 2018 13:37:18 +0000 (06:37 -0700)
committerDave Barach <openvpp@barachs.net>
Fri, 31 Aug 2018 11:58:46 +0000 (11:58 +0000)
Change-Id: I05d6c2cb5d34de469eb050e4ee10dc6b954c986d
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
14 files changed:
extras/vom/vom/Makefile.am
extras/vom/vom/api_types.cpp [new file with mode: 0644]
extras/vom/vom/api_types.hpp [new file with mode: 0644]
extras/vom/vom/types.cpp
extras/vom/vom/types.hpp
src/vnet.am
src/vnet/CMakeLists.txt
src/vnet/ethernet/ethernet.h
src/vnet/ethernet/ethernet_types.api [new file with mode: 0644]
src/vnet/ethernet/ethernet_types_api.c [new file with mode: 0644]
src/vnet/ethernet/ethernet_types_api.h [new file with mode: 0644]
src/vnet/ethernet/mac_address.c [new file with mode: 0644]
src/vnet/ethernet/mac_address.h [new file with mode: 0644]
src/vnet/ip/ip.api

index f976b6f..797bef0 100644 (file)
@@ -80,6 +80,7 @@ endif
 
 libvom_la_SOURCES =                    \
        types.cpp                       \
+       api_types.cpp                   \
        arp_proxy_binding_cmds.cpp      \
        arp_proxy_binding.cpp           \
        arp_proxy_config_cmds.cpp       \
diff --git a/extras/vom/vom/api_types.cpp b/extras/vom/vom/api_types.cpp
new file mode 100644 (file)
index 0000000..1b9ff29
--- /dev/null
@@ -0,0 +1,123 @@
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <vom/api_types.hpp>
+
+namespace VOM {
+
+static vapi_type_ip4_address
+to_api(const boost::asio::ip::address_v4& a)
+{
+  vapi_type_ip4_address v;
+
+  std::copy_n(a.to_bytes().data(), 4, v.address);
+
+  return v;
+}
+
+static vapi_type_ip6_address
+to_api(const boost::asio::ip::address_v6& a)
+{
+  vapi_type_ip6_address v;
+
+  std::copy_n(a.to_bytes().data(), 16, v.address);
+
+  return v;
+}
+
+vapi_type_address
+to_api(const ip_address_t& a)
+{
+  if (a.is_v4()) {
+    vapi_type_address v = {
+      .af = ADDRESS_IP4,
+      .un =
+        {
+          .ip4 = to_api(a.to_v4()),
+        },
+    };
+    return (v);
+  } else {
+    vapi_type_address v = {
+      .af = ADDRESS_IP6,
+      .un =
+        {
+          .ip6 = to_api(a.to_v6()),
+        },
+    };
+    return (v);
+  }
+}
+
+ip_address_t
+from_api(const vapi_type_address& v)
+{
+  boost::asio::ip::address addr;
+
+  if (ADDRESS_IP6 == v.af) {
+    std::array<uint8_t, 16> a;
+    std::copy(v.un.ip6.address, v.un.ip6.address + 16, std::begin(a));
+    boost::asio::ip::address_v6 v6(a);
+    addr = v6;
+  } else {
+    std::array<uint8_t, 4> a;
+    std::copy(v.un.ip6.address, v.un.ip6.address + 4, std::begin(a));
+    boost::asio::ip::address_v4 v4(a);
+    addr = v4;
+  }
+
+  return addr;
+}
+
+vapi_type_mac_address
+to_api(const mac_address_t& a)
+{
+  vapi_type_mac_address v;
+
+  std::copy(std::begin(a.bytes), std::end(a.bytes), v.bytes);
+
+  return (v);
+}
+
+mac_address_t
+from_api(const vapi_type_mac_address& v)
+{
+  return mac_address_t(v.bytes);
+}
+
+route::prefix_t
+from_api(const vapi_type_prefix& v)
+{
+  return route::prefix_t(from_api(v.address), v.address_length);
+}
+
+vapi_type_prefix
+to_api(const route::prefix_t& p)
+{
+  vapi_type_prefix v = {
+    .address = to_api(p.address()), .address_length = p.mask_width(),
+  };
+
+  return v;
+}
+};
+
+/*
+ * fd.io coding-style-patch-verification: ON
+ *
+ * Local Variables:
+ * eval: (c-set-style "mozilla")
+ * End:
+ */
diff --git a/extras/vom/vom/api_types.hpp b/extras/vom/vom/api_types.hpp
new file mode 100644 (file)
index 0000000..b555fc3
--- /dev/null
@@ -0,0 +1,45 @@
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <boost/asio/ip/address.hpp>
+#include <vom/prefix.hpp>
+#include <vom/types.hpp>
+
+#include <vapi/ip.api.vapi.hpp>
+
+namespace VOM {
+
+typedef boost::asio::ip::address ip_address_t;
+
+vapi_type_address to_api(const ip_address_t& a);
+
+ip_address_t from_api(const vapi_type_address& v);
+
+vapi_type_mac_address to_api(const mac_address_t& a);
+
+mac_address_t from_api(const vapi_type_mac_address& v);
+
+route::prefix_t from_api(const vapi_type_prefix&);
+
+vapi_type_prefix to_api(const route::prefix_t&);
+};
+
+/*
+ * fd.io coding-style-patch-verification: ON
+ *
+ * Local Variables:
+ * eval: (c-set-style "mozilla")
+ * End:
+ */
index c6093eb..a67a7f3 100644 (file)
@@ -109,7 +109,7 @@ operator<<(std::ostream& os, const handle_t& h)
   return (os);
 }
 
-mac_address_t::mac_address_t(uint8_t b[6])
+mac_address_t::mac_address_t(const uint8_t b[6])
 {
   std::copy(b, b + 6, std::begin(bytes));
 }
index 7b08d83..ab21408 100644 (file)
@@ -289,7 +289,7 @@ std::ostream& operator<<(std::ostream& os, const handle_t& h);
  */
 struct mac_address_t
 {
-  mac_address_t(uint8_t bytes[6]);
+  mac_address_t(const uint8_t bytes[6]);
   mac_address_t(const std::string& str);
   mac_address_t(std::initializer_list<uint8_t> bytes);
   /**
index b9402b1..8235d9b 100644 (file)
@@ -120,6 +120,7 @@ libvnet_la_SOURCES +=                               \
  vnet/ethernet/format.c                                \
  vnet/ethernet/init.c                          \
  vnet/ethernet/interface.c                     \
+ vnet/ethernet/mac_address.c                   \
  vnet/ethernet/node.c                          \
  vnet/ethernet/pg.c                            \
  vnet/ethernet/sfp.c                           \
@@ -139,7 +140,9 @@ nobase_include_HEADERS +=                   \
  vnet/ethernet/p2p_ethernet.api.h              \
  vnet/ethernet/p2p_ethernet.h
 
-API_FILES += vnet/ethernet/p2p_ethernet.api
+API_FILES += \
+ vnet/ethernet/p2p_ethernet.api                        \
+ vnet/ethernet/ethernet_types.api
 
 ########################################
 # Layer 2 protocol: Ethernet bridging
index a10ec82..61945a4 100644 (file)
@@ -109,9 +109,11 @@ list(APPEND VNET_API_FILES cop/cop.api)
 # Layer 2 protocol: Ethernet
 ##############################################################################
 list(APPEND VNET_SOURCES
+  ethernet/ethernet_types_api.c
   ethernet/format.c
   ethernet/init.c
   ethernet/interface.c
+  ethernet/mac_address.c
   ethernet/node.c
   ethernet/pg.c
   ethernet/sfp.c
index 0bfd487..9a23448 100644 (file)
@@ -46,7 +46,7 @@
 #include <vnet/feature/feature.h>
 
 always_inline u64
-ethernet_mac_address_u64 (u8 * a)
+ethernet_mac_address_u64 (const u8 * a)
 {
   return (((u64) a[0] << (u64) (5 * 8))
          | ((u64) a[1] << (u64) (4 * 8))
@@ -55,6 +55,18 @@ ethernet_mac_address_u64 (u8 * a)
          | ((u64) a[4] << (u64) (1 * 8)) | ((u64) a[5] << (u64) (0 * 8)));
 }
 
+always_inline void
+ethernet_mac_address_from_u64 (u64 u, u8 * a)
+{
+  i8 ii;
+
+  for (ii = 5; ii >= 0; ii--)
+    {
+      a[ii] = u & 0xFF;
+      u = u >> 8;
+    }
+}
+
 static inline int
 ethernet_mac_address_is_multicast_u64 (u64 a)
 {
@@ -62,7 +74,7 @@ ethernet_mac_address_is_multicast_u64 (u64 a)
 }
 
 static inline int
-ethernet_mac_address_is_zero (u8 * mac)
+ethernet_mac_address_is_zero (const u8 * mac)
 {
   return ((*((u32 *) mac) == 0) && (*((u16 *) (mac + 4)) == 0));
 }
diff --git a/src/vnet/ethernet/ethernet_types.api b/src/vnet/ethernet/ethernet_types.api
new file mode 100644 (file)
index 0000000..c33a02c
--- /dev/null
@@ -0,0 +1,19 @@
+/* Hey Emacs use -*- mode: C -*- */
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+typedef mac_address {
+  u8 bytes[6];
+};
diff --git a/src/vnet/ethernet/ethernet_types_api.c b/src/vnet/ethernet/ethernet_types_api.c
new file mode 100644 (file)
index 0000000..d9e0601
--- /dev/null
@@ -0,0 +1,57 @@
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <vnet/ethernet/ethernet_types_api.h>
+
+#define vl_typedefs            /* define message structures */
+#include <vnet/vnet_all_api_h.h>
+#undef vl_typedefs
+
+#define vl_endianfun           /* define message structures */
+#include <vnet/vnet_all_api_h.h>
+#undef vl_endianfun
+
+/* instantiate all the print functions we know about */
+#define vl_print(handle, ...) vlib_cli_output (handle, __VA_ARGS__)
+#define vl_printfun
+#include <vnet/vnet_all_api_h.h>
+#undef vl_printfun
+
+void
+mac_address_decode (const vl_api_mac_address_t * in, mac_address_t * out)
+{
+  mac_address_from_bytes (out, in->bytes);
+}
+
+void
+mac_address_encode (const mac_address_t * in, vl_api_mac_address_t * out)
+{
+}
+
+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
+ *
+ * Local Variables:
+ * eval: (c-set-style "gnu")
+ * End:
+ */
diff --git a/src/vnet/ethernet/ethernet_types_api.h b/src/vnet/ethernet/ethernet_types_api.h
new file mode 100644 (file)
index 0000000..f326f8a
--- /dev/null
@@ -0,0 +1,45 @@
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#ifndef __ETHERNET_TYPES_API_H__
+#define __ETHERNET_TYPES_API_H__
+
+/**
+ * Conversion functions to/from (decode/encode) API types to VPP internal types
+ */
+
+#include <vnet/ethernet/mac_address.h>
+
+/**
+ * Forward declarations so we need not #include the API definitions here
+ */
+struct _vl_api_mac_address;
+
+extern void mac_address_decode (const struct _vl_api_mac_address *in,
+                               mac_address_t * out);
+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
+
+/*
+ * fd.io coding-style-patch-verification: ON
+ *
+ * Local Variables:
+ * eval: (c-set-style "gnu")
+ * End:
+ */
diff --git a/src/vnet/ethernet/mac_address.c b/src/vnet/ethernet/mac_address.c
new file mode 100644 (file)
index 0000000..72725c8
--- /dev/null
@@ -0,0 +1,39 @@
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <vnet/ethernet/mac_address.h>
+
+const mac_address_t ZERO_MAC_ADDRESS = {
+  .bytes = {
+           0, 0, 0, 0, 0, 0,
+           },
+};
+
+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));
+}
+
+
+/*
+ * fd.io coding-style-patch-verification: ON
+ *
+ * Local Variables:
+ * eval: (c-set-style "gnu")
+ * End:
+ */
diff --git a/src/vnet/ethernet/mac_address.h b/src/vnet/ethernet/mac_address.h
new file mode 100644 (file)
index 0000000..e97eab6
--- /dev/null
@@ -0,0 +1,62 @@
+/*
+ * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#ifndef __MAC_ADDRESS_H__
+#define __MAC_ADDRESS_H__
+
+#include <vnet/ethernet/ethernet.h>
+
+typedef struct mac_address_t_
+{
+  u8 bytes[6];
+} mac_address_t;
+
+extern const mac_address_t ZERO_MAC_ADDRESS;
+
+static_always_inline void
+mac_address_from_bytes (mac_address_t * mac, const u8 * bytes)
+{
+  clib_memcpy (mac->bytes, bytes, sizeof (*mac));
+}
+
+static_always_inline int
+mac_address_is_zero (const mac_address_t * mac)
+{
+  return (ethernet_mac_address_is_zero (mac->bytes));
+}
+
+static_always_inline u64
+mac_address_as_u64 (const mac_address_t * mac)
+{
+  return (ethernet_mac_address_u64 (mac->bytes));
+}
+
+static_always_inline void
+mac_address_from_u64 (u64 u, mac_address_t * mac)
+{
+  ethernet_mac_address_from_u64 (u, mac->bytes);
+}
+
+extern u8 *format_mac_address_t (u8 * s, va_list * args);
+
+#endif
+
+/*
+ * fd.io coding-style-patch-verification: ON
+ *
+ * Local Variables:
+ * eval: (c-set-style "gnu")
+ * End:
+ */
index 5f664f3..616d621 100644 (file)
@@ -22,6 +22,7 @@
 option version = "1.3.0";
 import "vnet/ip/ip_types.api";
 import "vnet/fib/fib_types.api";
+import "vnet/ethernet/ethernet_types.api";
 
 /** \brief Add / del table request
            A table can be added multiple times, but need be deleted only once.