notifiers: Rename run and wait functions.
authorEthan Jackson <ethan@nicira.com>
Thu, 15 Sep 2011 18:23:08 +0000 (11:23 -0700)
committerEthan Jackson <ethan@nicira.com>
Fri, 16 Sep 2011 18:22:30 +0000 (11:22 -0700)
It makes more sense to call nln_notifier_run() and
nln_notifier_wait() simply nln_run() and nln_wait() since they
don't operate on notifiers but the entire nln object.  This patch
changes the nln and the rtnetlink-link modules to the new
convention.

lib/dpif-linux.c
lib/netdev-linux.c
lib/netlink-notifier.c
lib/netlink-notifier.h
lib/route-table.c
lib/rtnetlink-link.c
lib/rtnetlink-link.h
vswitchd/ovs-brcompatd.c

index 3d081aab50a67aee5ef1899dd98bdbf3d7a955d1..ee4879514e8c5eed7c2896a7032a7b30faedbd17 100644 (file)
@@ -320,7 +320,7 @@ static void
 dpif_linux_run(struct dpif *dpif OVS_UNUSED)
 {
     if (nln) {
-        nln_notifier_run(nln);
+        nln_run(nln);
     }
 }
 
@@ -328,7 +328,7 @@ static void
 dpif_linux_wait(struct dpif *dpif OVS_UNUSED)
 {
     if (nln) {
-        nln_notifier_wait(nln);
+        nln_wait(nln);
     }
 }
 
index ee3c5f52a5cc931e59caf8444586728768606b4c..cc930e1d2b3fd0d3d973684eaa75c9332f974068 100644 (file)
@@ -469,14 +469,14 @@ netdev_linux_init(void)
 static void
 netdev_linux_run(void)
 {
-    rtnetlink_link_notifier_run();
+    rtnetlink_link_run();
     netdev_linux_miimon_run();
 }
 
 static void
 netdev_linux_wait(void)
 {
-    rtnetlink_link_notifier_wait();
+    rtnetlink_link_wait();
     netdev_linux_miimon_wait();
 }
 
index 1b7529dd7726ce9f2c358d780f536c994aa80dd4..d0a031b2f8efaa9a5f2d217629cc4f0300d73d70 100644 (file)
@@ -109,7 +109,7 @@ nln_notifier_register(struct nln *nln, struct nln_notifier *notifier,
     } else {
         /* Catch up on notification work so that the new notifier won't
          * receive any stale notifications. */
-        nln_notifier_run(nln);
+        nln_run(nln);
     }
 
     list_push_back(&nln->all_notifiers, &notifier->node);
@@ -133,7 +133,7 @@ nln_notifier_unregister(struct nln *nln, struct nln_notifier *notifier)
 /* Calls all of the registered notifiers, passing along any as-yet-unreported
  * change events. */
 void
-nln_notifier_run(struct nln *nln)
+nln_run(struct nln *nln)
 {
     static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
 
@@ -171,7 +171,7 @@ nln_notifier_run(struct nln *nln)
 
 /* Causes poll_block() to wake up when change notifications are ready. */
 void
-nln_notifier_wait(struct nln *nln)
+nln_wait(struct nln *nln)
 {
     nln->has_run = false;
     if (nln->notify_sock) {
index 60b5991b1655c898146aaf86f72f9a8e0fe265d2..e3dacef0ad126e56edb7c0d84e502fb4af1d62d4 100644 (file)
@@ -49,6 +49,6 @@ void nln_destroy(struct nln *);
 int nln_notifier_register(struct nln *, struct nln_notifier *,
                           nln_notify_func *, void *aux);
 void nln_notifier_unregister(struct nln *, struct nln_notifier *);
-void nln_notifier_run(struct nln *);
-void nln_notifier_wait(struct nln *);
+void nln_run(struct nln *);
+void nln_wait(struct nln *);
 #endif /* netlink-notifier.h */
index de335f57b3efef3edefeb4b235dfd65ed8f6e8e0..c8245dfd015cbe5c87aa8ad3c579f2b3061adbad 100644 (file)
@@ -199,8 +199,8 @@ void
 route_table_run(void)
 {
     if (nln) {
-        rtnetlink_link_notifier_run();
-        nln_notifier_run(nln);
+        rtnetlink_link_run();
+        nln_run(nln);
     }
 }
 
@@ -209,8 +209,8 @@ void
 route_table_wait(void)
 {
     if (nln) {
-        rtnetlink_link_notifier_wait();
-        nln_notifier_wait(nln);
+        rtnetlink_link_wait();
+        nln_wait(nln);
     }
 }
 
index dbdf724ba30ad1abd4a7abcaa269699ecc2aef23..eef07d695c122a4e4f26d3e0c98786738f64cae8 100644 (file)
@@ -109,19 +109,19 @@ rtnetlink_link_notifier_unregister(struct nln_notifier *notifier)
 /* Calls all of the registered notifiers, passing along any as-yet-unreported
  * netdev change events. */
 void
-rtnetlink_link_notifier_run(void)
+rtnetlink_link_run(void)
 {
     if (nln) {
-        nln_notifier_run(nln);
+        nln_run(nln);
     }
 }
 
 /* Causes poll_block() to wake up when network device change notifications are
  * ready. */
 void
-rtnetlink_link_notifier_wait(void)
+rtnetlink_link_wait(void)
 {
     if (nln) {
-        nln_notifier_wait(nln);
+        nln_wait(nln);
     }
 }
index 1e52a7dc3822c68598b6ed620ba3076965c5d9d2..80fbd5cf401b380dfd7f49d9ba6a4ee91cdc308a 100644 (file)
@@ -53,6 +53,6 @@ bool rtnetlink_link_parse(struct ofpbuf *buf,
 int rtnetlink_link_notifier_register(struct nln_notifier*,
                                      rtnetlink_link_notify_func *, void *aux);
 void rtnetlink_link_notifier_unregister(struct nln_notifier *);
-void rtnetlink_link_notifier_run(void);
-void rtnetlink_link_notifier_wait(void);
+void rtnetlink_link_run(void);
+void rtnetlink_link_wait(void);
 #endif /* rtnetlink-link.h */
index bfa13ca26b7476c56ea8e3fa49c3bed641d58837..df2b75dad467105cb0ffdd943ce9030ec5984218 100644 (file)
@@ -724,7 +724,7 @@ brc_recv_update(void)
      * (XenServer Tools 5.5.0 does not exhibit this behavior, and neither does
      * a VM without Tools installed at all.)
      */
-    rtnetlink_link_notifier_run();
+    rtnetlink_link_run();
 
     switch (genlmsghdr->cmd) {
     case BRC_GENL_C_DP_ADD:
@@ -830,14 +830,14 @@ main(int argc, char *argv[])
 
     for (;;) {
         unixctl_server_run(unixctl);
-        rtnetlink_link_notifier_run();
+        rtnetlink_link_run();
         brc_recv_update();
 
         netdev_run();
 
         nl_sock_wait(brc_sock, POLLIN);
         unixctl_server_wait(unixctl);
-        rtnetlink_link_notifier_wait();
+        rtnetlink_link_wait();
         netdev_wait();
         poll_block();
     }