Rev 333: - get rid of ctdb_ctrl_get_config in http://samba.org/~tridge/ctdb

[tridge]

------------------------------------------------------------
revno: 333
revision-id: [EMAIL PROTECTED]
parent: [EMAIL PROTECTED]
committer: Andrew Tridgell <[EMAIL PROTECTED]>
branch nick: tridge
timestamp: Wed 2007-05-23 15:18:30 +1000
message:
  - get rid of ctdb_ctrl_get_config
  - cope with zero timeout in ctdb_control
modified:
  common/cmdline.c               cmdline.c-20070416041216-w1zvz91bkdsgjckw-1
  common/ctdb_client.c           ctdb_client.c-20070411010216-3kd8v37k61steeya-1
  common/ctdb_daemon.c           ctdb_daemon.c-20070409200331-3el1kqgdb9m4ib0g-1
=== modified file 'common/cmdline.c'
--- a/common/cmdline.c  2007-05-15 05:13:36 +0000
+++ b/common/cmdline.c  2007-05-23 05:18:30 +0000
@@ -183,10 +183,10 @@
                return NULL;
        }
 
-       /* get our config */
-       ret = ctdb_ctrl_get_config(ctdb);
-       if (ret != 0) {
-               DEBUG(0,(__location__ " Failed to get ctdb config\n"));
+       /* get our vnn */
+       ctdb->vnn = ctdb_ctrl_getvnn(ctdb, timeval_zero(), CTDB_CURRENT_NODE);
+       if (ctdb->vnn == (uint32_t)-1) {
+               DEBUG(0,(__location__ " Failed to get ctdb vnn\n"));
                talloc_free(ctdb);
                return NULL;
        }

=== modified file 'common/ctdb_client.c'
--- a/common/ctdb_client.c      2007-05-23 04:50:41 +0000
+++ b/common/ctdb_client.c      2007-05-23 05:18:30 +0000
@@ -469,9 +469,6 @@
        /* now we can go into the normal wait routine, as the reply packet
           will update the ctdb->num_connected variable */
        ctdb_daemon_connect_wait(ctdb);
-
-       /* get other config variables */
-       ctdb_ctrl_get_config(ctdb);
 }
 
 /*
@@ -745,7 +742,7 @@
 
        /* semi-async operation */
        timed_out = 0;
-       if (timeout) {
+       if (timeout && !timeval_is_zero(timeout)) {
                event_add_timed(ctdb->ev, state, *timeout, timeout_func, 
&timed_out);
        }
        while ((state->state == CTDB_CALL_WAIT)
@@ -1236,38 +1233,6 @@
 }
 
 /*
-  get ctdb config
- */
-int ctdb_ctrl_get_config(struct ctdb_context *ctdb)
-{
-       int ret;
-       int32_t res;
-       TDB_DATA data;
-       struct ctdb_context c;
-
-       ZERO_STRUCT(data);
-       ret = ctdb_control(ctdb, CTDB_CURRENT_NODE, 0, CTDB_CONTROL_CONFIG, 0,
-                          data, ctdb, &data, &res, NULL, NULL);
-       if (ret != 0 || res != 0) {
-               return -1;
-       }
-       if (data.dsize != sizeof(c)) {
-               DEBUG(0,("Bad config size %u - expected %u\n", data.dsize, 
sizeof(c)));
-               return -1;
-       }
-
-       c = *(struct ctdb_context *)data.dptr;
-       talloc_free(data.dptr);
-
-       ctdb->num_nodes = c.num_nodes;
-       ctdb->num_connected = c.num_connected;
-       ctdb->vnn = c.vnn;
-       ctdb->max_lacount = c.max_lacount;
-       
-       return 0;
-}
-
-/*
   find the real path to a ltdb 
  */
 int ctdb_ctrl_getdbpath(struct ctdb_context *ctdb, struct timeval timeout, 
uint32_t destnode, uint32_t dbid, TALLOC_CTX *mem_ctx, 

=== modified file 'common/ctdb_daemon.c'
--- a/common/ctdb_daemon.c      2007-05-23 02:23:07 +0000
+++ b/common/ctdb_daemon.c      2007-05-23 05:18:30 +0000
@@ -682,7 +682,11 @@
                close(fd[0]);
                close(ctdb->daemon.sd);
                ctdb->daemon.sd = -1;
-               ctdb_ctrl_get_config(ctdb);
+               ctdb->vnn = ctdb_ctrl_getvnn(ctdb, timeval_zero(), 
CTDB_CURRENT_NODE);
+               if (ctdb->vnn == (uint32_t)-1) {
+                       DEBUG(0,(__location__ " Failed to get ctdb vnn\n"));
+                       return -1;
+               }
                return 0;
        }