[ovs-dev] [nxm 05/42] Make the ovs_be<N> types generally available.

Ben Pfaff blp at nicira.com
Thu Oct 28 17:27:36 UTC 2010


Using these types for data in network byte order makes code clearer, and
allows the "sparse" checker to give warnings for misuse.
---
 build-aux/check-structs                 |    3 ++
 include/openvswitch/datapath-protocol.h |    5 +---
 include/openvswitch/types.h             |   37 +++++++++++++++++++++++++++++++
 3 files changed, 41 insertions(+), 4 deletions(-)
 create mode 100644 include/openvswitch/types.h

diff --git a/build-aux/check-structs b/build-aux/check-structs
index 545c80a..f582e2b 100755
--- a/build-aux/check-structs
+++ b/build-aux/check-structs
@@ -13,6 +13,9 @@ types['uint8_t'] = {"size": 1, "alignment": 1}
 types['uint16_t'] = {"size": 2, "alignment": 2}
 types['uint32_t'] = {"size": 4, "alignment": 4}
 types['uint64_t'] = {"size": 8, "alignment": 8}
+types['ovs_be16'] = {"size": 2, "alignment": 2}
+types['ovs_be32'] = {"size": 4, "alignment": 4}
+types['ovs_be64'] = {"size": 8, "alignment": 8}
 
 token = None
 line = ""
diff --git a/include/openvswitch/datapath-protocol.h b/include/openvswitch/datapath-protocol.h
index b0e9dfb..15c05e0 100644
--- a/include/openvswitch/datapath-protocol.h
+++ b/include/openvswitch/datapath-protocol.h
@@ -56,10 +56,7 @@
 #define ovs_be32 __be32
 #define ovs_be64 __be64
 #else
-#include <stdint.h>
-#define ovs_be16 uint16_t
-#define ovs_be32 uint32_t
-#define ovs_be64 uint64_t
+#include "openvswitch/types.h"
 #endif
 
 #define ODP_MAX 256             /* Maximum number of datapaths. */
diff --git a/include/openvswitch/types.h b/include/openvswitch/types.h
new file mode 100644
index 0000000..2f670c0
--- /dev/null
+++ b/include/openvswitch/types.h
@@ -0,0 +1,37 @@
+/*
+ * Copyright (c) 2010 Nicira Networks.
+ *
+ * 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 OPENVSWITCH_TYPES_H
+#define OPENVSWITCH_TYPES_H 1
+
+#include <stdint.h>
+
+#ifdef __CHECKER__
+#define OVS_BITWISE __attribute__((bitwise))
+#else
+#define OVS_BITWISE
+#endif
+
+/* The ovs_be<N> types indicate that an object is in big-endian, not
+ * native-endian, byte order.  They are otherwise equivalent to uint<N>_t.
+ *
+ * The OVS_BITWISE annotation allows the sparse checker to issue warnings
+ * for incorrect use of values in network byte order. */
+typedef uint16_t OVS_BITWISE ovs_be16;
+typedef uint32_t OVS_BITWISE ovs_be32;
+typedef uint64_t OVS_BITWISE ovs_be64;
+
+#endif /* openvswitch/types.h */
-- 
1.7.1





More information about the dev mailing list