[ovs-dev] [PATCH v6 12/18] lib/rstp: Inline trivial predicate functions.
Jarno Rajahalme
jrajahalme at nicira.com
Wed Aug 20 23:57:24 UTC 2014
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
---
lib/rstp.c | 34 ----------------------------------
lib/rstp.h | 42 +++++++++++++++++++++++++++++++++++++++---
2 files changed, 39 insertions(+), 37 deletions(-)
diff --git a/lib/rstp.c b/lib/rstp.c
index c674105..80479fb 100644
--- a/lib/rstp.c
+++ b/lib/rstp.c
@@ -1140,40 +1140,6 @@ rstp_port_get_aux(struct rstp_port *p)
return aux;
}
-/* Returns true if 'state' is one in which BPDU packets should be received
- * and transmitted on a port, false otherwise.
- */
- bool
- rstp_should_manage_bpdu(enum rstp_state state)
- {
- return (state == RSTP_DISCARDING || state == RSTP_LEARNING ||
- state == RSTP_FORWARDING);
- }
-
-/* Returns true if 'state' is one in which packets received on a port should
- * be forwarded, false otherwise.
- *
- * Returns true if 'state' is RSTP_DISABLED, since presumably in that case the
- * port should still work, just not have RSTP applied to it.
- */
-bool
-rstp_forward_in_state(enum rstp_state state)
-{
- return (state == RSTP_DISABLED || state == RSTP_FORWARDING);
-}
-
-/* Returns true if 'state' is one in which MAC learning should be done on
- * packets received on a port, false otherwise.
- *
- * Returns true if 'state' is RSTP_DISABLED, since presumably in that case the
- * port should still work, just not have RSTP applied to it. */
-bool
-rstp_learn_in_state(enum rstp_state state)
-{
- return (state == RSTP_DISABLED || state == RSTP_LEARNING ||
- state == RSTP_FORWARDING);
-}
-
/* Unixctl. */
static struct rstp *
rstp_find(const char *name)
diff --git a/lib/rstp.h b/lib/rstp.h
index 5d7c7f4..0e8d101 100644
--- a/lib/rstp.h
+++ b/lib/rstp.h
@@ -119,9 +119,9 @@ struct rstp_port;
struct ofproto_rstp_settings;
const char *rstp_state_name(enum rstp_state);
-bool rstp_forward_in_state(enum rstp_state);
-bool rstp_learn_in_state(enum rstp_state);
-bool rstp_should_manage_bpdu(enum rstp_state state);
+static inline bool rstp_forward_in_state(enum rstp_state);
+static inline bool rstp_learn_in_state(enum rstp_state);
+static inline bool rstp_should_manage_bpdu(enum rstp_state state);
const char *rstp_port_role_name(enum rstp_port_role);
void rstp_init(void);
@@ -198,4 +198,40 @@ enum rstp_port_role rstp_port_get_role(const struct rstp_port *);
void rstp_port_get_counts(const struct rstp_port *, int *tx_count,
int *rx_count, int *error_count, int *uptime);
void * rstp_port_get_aux(struct rstp_port *);
+
+/* Inline functions. */
+/* Returns true if 'state' is one in which BPDU packets should be received
+ * and transmitted on a port, false otherwise.
+ */
+static inline bool
+rstp_should_manage_bpdu(enum rstp_state state)
+{
+ return (state == RSTP_DISCARDING || state == RSTP_LEARNING ||
+ state == RSTP_FORWARDING);
+}
+
+/* Returns true if 'state' is one in which packets received on a port should
+ * be forwarded, false otherwise.
+ *
+ * Returns true if 'state' is RSTP_DISABLED, since presumably in that case the
+ * port should still work, just not have RSTP applied to it.
+ */
+static inline bool
+rstp_forward_in_state(enum rstp_state state)
+{
+ return (state == RSTP_DISABLED || state == RSTP_FORWARDING);
+}
+
+/* Returns true if 'state' is one in which MAC learning should be done on
+ * packets received on a port, false otherwise.
+ *
+ * Returns true if 'state' is RSTP_DISABLED, since presumably in that case the
+ * port should still work, just not have RSTP applied to it. */
+static inline bool
+rstp_learn_in_state(enum rstp_state state)
+{
+ return (state == RSTP_DISABLED || state == RSTP_LEARNING ||
+ state == RSTP_FORWARDING);
+}
+
#endif /* rstp.h */
--
1.7.10.4
More information about the dev
mailing list