[GNUnet-SVN] r37308 - gnunet/src/nse

[gnunet]

Author: grothoff
Date: 2016-06-20 21:56:20 +0200 (Mon, 20 Jun 2016)
New Revision: 37308

Modified:
   gnunet/src/nse/nse_api.c
Log:
convering nse_api.c to new MQ API

Modified: gnunet/src/nse/nse_api.c
===================================================================
--- gnunet/src/nse/nse_api.c    2016-06-20 19:55:38 UTC (rev 37307)
+++ gnunet/src/nse/nse_api.c    2016-06-20 19:56:20 UTC (rev 37308)
@@ -1,6 +1,6 @@
 /*
      This file is part of GNUnet.
-     Copyright (C) 2009, 2010, 2011 GNUnet e.V.
+     Copyright (C) 2009, 2010, 2011, 2016 GNUnet e.V.
 
      GNUnet is free software; you can redistribute it and/or modify
      it under the terms of the GNU General Public License as published
@@ -45,19 +45,14 @@
   const struct GNUNET_CONFIGURATION_Handle *cfg;
 
   /**
-   * Socket (if available).
+   * Message queue (if available).
    */
-  struct GNUNET_CLIENT_Connection *client;
+  struct GNUNET_MQ_Handle *mq;
 
   /**
-   * Currently pending transmission request.
-   */
-  struct GNUNET_CLIENT_TransmitHandle *th;
-
-  /**
    * Task doing exponential back-off trying to reconnect.
    */
-  struct GNUNET_SCHEDULER_Task * reconnect_task;
+  struct GNUNET_SCHEDULER_Task *reconnect_task;
 
   /**
    * Time for next connect retry.
@@ -80,7 +75,7 @@
 /**
  * Try again to connect to network size estimation service.
  *
- * @param cls the handle to the transport service
+ * @param cls closure with the `struct GNUNET_NSE_Handle *`
  */
 static void
 reconnect (void *cls);
@@ -87,111 +82,48 @@
 
 
 /**
- * Type of a function to call when we receive a message
- * from the service.
+ * Generic error handler, called with the appropriate
+ * error code and the same closure specified at the creation of
+ * the message queue.
+ * Not every message queue implementation supports an error handler.
  *
- * @param cls closure
- * @param msg message received, NULL on timeout or fatal error
+ * @param cls closure with the `struct GNUNET_NSE_Handle *`
+ * @param error error code
  */
 static void
-message_handler (void *cls,
-                const struct GNUNET_MessageHeader *msg)
+mq_error_handler (void *cls,
+                  enum GNUNET_MQ_Error error)
 {
   struct GNUNET_NSE_Handle *h = cls;
-  const struct GNUNET_NSE_ClientMessage *client_msg;
 
-  if (NULL == msg)
-  {
-    /* Error, timeout, death */
-    GNUNET_CLIENT_disconnect (h->client);
-    h->client = NULL;
-    h->reconnect_task =
-        GNUNET_SCHEDULER_add_delayed (h->reconnect_delay,
-                                     &reconnect, h);
-    return;
-  }
-  if ((ntohs (msg->size) != sizeof (struct GNUNET_NSE_ClientMessage)) ||
-      (ntohs (msg->type) != GNUNET_MESSAGE_TYPE_NSE_ESTIMATE))
-  {
-    GNUNET_break (0);
-    return;
-  }
-  client_msg = (const struct GNUNET_NSE_ClientMessage *) msg;
-  h->recv_cb (h->recv_cb_cls, GNUNET_TIME_absolute_ntoh 
(client_msg->timestamp),
-              GNUNET_ntoh_double (client_msg->size_estimate),
-             GNUNET_ntoh_double (client_msg->std_deviation));
-  GNUNET_CLIENT_receive (h->client, &message_handler, h,
-                         GNUNET_TIME_UNIT_FOREVER_REL);
+  GNUNET_MQ_destroy (h->mq);
+  h->mq = NULL;
+  h->reconnect_task
+    = GNUNET_SCHEDULER_add_delayed (h->reconnect_delay,
+                                    &reconnect,
+                                    h);
+  h->reconnect_delay = GNUNET_TIME_STD_BACKOFF (h->reconnect_delay);
 }
 
 
 /**
- * Reschedule a connect attempt to the service.
+ * Type of a function to call when we receive a message
+ * from the service.
  *
- * @param h transport service to reconnect
+ * @param cls closure
+ * @param cklient_msg message received
  */
 static void
-reschedule_connect (struct GNUNET_NSE_Handle *h)
+handle_estimate (void *cls,
+                const struct GNUNET_NSE_ClientMessage *client_msg)
 {
-  GNUNET_assert (h->reconnect_task == NULL);
-
-  if (NULL != h->th)
-  {
-    GNUNET_CLIENT_notify_transmit_ready_cancel (h->th);
-    h->th = NULL;
-  }
-  if (NULL != h->client)
-  {
-    GNUNET_CLIENT_disconnect (h->client);
-    h->client = NULL;
-  }
-
-  LOG (GNUNET_ERROR_TYPE_DEBUG,
-       "Scheduling task to reconnect to nse service in %s.\n",
-       GNUNET_STRINGS_relative_time_to_string (h->reconnect_delay,
-                                              GNUNET_YES));
-  h->reconnect_task =
-      GNUNET_SCHEDULER_add_delayed (h->reconnect_delay,
-                                   &reconnect, h);
-  h->reconnect_delay = GNUNET_TIME_STD_BACKOFF (h->reconnect_delay);
-}
-
-
-/**
- * Transmit START message to service.
- *
- * @param cls the `struct GNUNET_NSE_Handle *`
- * @param size number of bytes available in @a buf
- * @param buf where to copy the message
- * @return number of bytes copied to @a buf
- */
-static size_t
-send_start (void *cls, size_t size, void *buf)
-{
   struct GNUNET_NSE_Handle *h = cls;
-  struct GNUNET_MessageHeader *msg;
 
-  h->th = NULL;
-  if (NULL == buf)
-  {
-    /* Connect error... */
-    LOG (GNUNET_ERROR_TYPE_DEBUG,
-         "Error while trying to transmit `%s' request.\n",
-        "START");
-    reschedule_connect (h);
-    return 0;
-  }
-  LOG (GNUNET_ERROR_TYPE_DEBUG,
-       "Transmitting `%s' request.\n",
-       "START");
-  GNUNET_assert (size >= sizeof (struct GNUNET_MessageHeader));
-
-  msg = (struct GNUNET_MessageHeader *) buf;
-  msg->size = htons (sizeof (struct GNUNET_MessageHeader));
-  msg->type = htons (GNUNET_MESSAGE_TYPE_NSE_START);
-  GNUNET_CLIENT_receive (h->client, &message_handler, h,
-                         GNUNET_TIME_UNIT_FOREVER_REL);
-  return sizeof (struct GNUNET_MessageHeader);
+  h->reconnect_delay = GNUNET_TIME_UNIT_ZERO;
+  h->recv_cb (h->recv_cb_cls,
+              GNUNET_TIME_absolute_ntoh (client_msg->timestamp),
+              GNUNET_ntoh_double (client_msg->size_estimate),
+             GNUNET_ntoh_double (client_msg->std_deviation));
 }
 
 
@@ -203,21 +135,32 @@
 static void
 reconnect (void *cls)
 {
+  GNUNET_MQ_hd_fixed_size (estimate,
+                           GNUNET_MESSAGE_TYPE_NSE_ESTIMATE,
+                           struct GNUNET_NSE_ClientMessage);
   struct GNUNET_NSE_Handle *h = cls;
+  struct GNUNET_MQ_MessageHandler handlers[] = {
+    make_estimate_handler (h),
+    GNUNET_MQ_handler_end ()
+  };
+  struct GNUNET_MessageHeader *msg;
+  struct GNUNET_MQ_Envelope *env;
 
   h->reconnect_task = NULL;
   LOG (GNUNET_ERROR_TYPE_DEBUG,
        "Connecting to network size estimation service.\n");
-  GNUNET_assert (NULL == h->client);
-  h->client = GNUNET_CLIENT_connect ("nse", h->cfg);
-  GNUNET_assert (NULL != h->client);
-  GNUNET_assert (NULL == h->th);
-  h->th =
-      GNUNET_CLIENT_notify_transmit_ready (h->client,
-                                           sizeof (struct 
GNUNET_MessageHeader),
-                                           GNUNET_TIME_UNIT_FOREVER_REL,
-                                           GNUNET_NO, &send_start, h);
-  GNUNET_assert (NULL != h->th);
+  GNUNET_assert (NULL == h->mq);
+  h->mq = GNUNET_CLIENT_connecT (h->cfg,
+                                 "nse",
+                                 handlers,
+                                 &mq_error_handler,
+                                 h);
+  if (NULL == h->mq)
+    return;
+  env = GNUNET_MQ_msg (msg,
+                       GNUNET_MESSAGE_TYPE_NSE_START);
+  GNUNET_MQ_send (h->mq,
+                  env);
 }
 
 
@@ -231,18 +174,24 @@
  */
 struct GNUNET_NSE_Handle *
 GNUNET_NSE_connect (const struct GNUNET_CONFIGURATION_Handle *cfg,
-                    GNUNET_NSE_Callback func, void *func_cls)
+                    GNUNET_NSE_Callback func,
+                    void *func_cls)
 {
-  struct GNUNET_NSE_Handle *ret;
+  struct GNUNET_NSE_Handle *h;
 
-  GNUNET_assert (func != NULL);
-  ret = GNUNET_new (struct GNUNET_NSE_Handle);
-  ret->cfg = cfg;
-  ret->recv_cb = func;
-  ret->recv_cb_cls = func_cls;
-  ret->reconnect_delay = GNUNET_TIME_UNIT_ZERO;
-  ret->reconnect_task = GNUNET_SCHEDULER_add_now (&reconnect, ret);
-  return ret;
+  GNUNET_assert (NULL != func);
+  h = GNUNET_new (struct GNUNET_NSE_Handle);
+  h->cfg = cfg;
+  h->recv_cb = func;
+  h->recv_cb_cls = func_cls;
+  h->reconnect_delay = GNUNET_TIME_UNIT_ZERO;
+  reconnect (h);
+  if (NULL == h->mq)
+  {
+    GNUNET_free (h);
+    return NULL;
+  }
+  return h;
 }
 
 
@@ -254,22 +203,16 @@
 void
 GNUNET_NSE_disconnect (struct GNUNET_NSE_Handle *h)
 {
-  GNUNET_assert (NULL != h);
-  if (h->reconnect_task != NULL)
+  if (NULL != h->reconnect_task)
   {
     GNUNET_SCHEDULER_cancel (h->reconnect_task);
     h->reconnect_task = NULL;
   }
-  if (NULL != h->th)
+  if (NULL != h->mq)
   {
-    GNUNET_CLIENT_notify_transmit_ready_cancel (h->th);
-    h->th = NULL;
+    GNUNET_MQ_destroy (h->mq);
+    h->mq = NULL;
   }
-  if (NULL != h->client)
-  {
-    GNUNET_CLIENT_disconnect (h->client);
-    h->client = NULL;
-  }
   GNUNET_free (h);
 }