-improve UDP logging
[oweals/gnunet.git] / src / ats / gnunet-service-ats_addresses.c
index c2010e8d2ab56801c62f19ea36775fed4f2261fb..c4d7f2da9737239cd898c2af3aada1c7db943347 100644 (file)
@@ -1,6 +1,6 @@
 /*
  This file is part of GNUnet.
(C) 2011 Christian Grothoff (and other contributing authors)
Copyright (C) 2011-2015 Christian Grothoff (and other contributing authors)
 
  GNUnet is free software; you can redistribute it and/or modify
  it under the terms of the GNU General Public License as published
  * @author Christian Grothoff
  */
 #include "platform.h"
-#include "gnunet_ats_service.h"
-#include "gnunet_ats_plugin.h"
-#include "gnunet-service-ats.h"
 #include "gnunet-service-ats_addresses.h"
-#include "gnunet-service-ats_normalization.h"
 #include "gnunet-service-ats_performance.h"
-#include "gnunet-service-ats_scheduling.h"
-#include "gnunet-service-ats_reservations.h"
+#include "gnunet-service-ats_normalization.h"
+#include "gnunet-service-ats_plugins.h"
 
 
 /**
  * NOTE: Do not change this documentation. This documentation is based on
- * gnunet.org:/vcs/fsnsg/ats-paper.git/tech-doku/ats-tech-guide.tex
+ * gnunet.org:/vcs/fsnsg/2014-p2p-ats.git/tech-doku/ats-tech-guide.tex
  * use build_txt.sh to generate plaintext output
  *
  *   1 ATS addresses : ATS address management
  *    available it will not respond at all If the client is not interested
  *    anymore, it has to cancel the address suggestion request.
  *
- *       1.7.6 Suggestions blocks and reset
- *
- *    After suggesting an address it is blocked for ATS_BLOCKING_DELTA sec. to
- *    prevent the client from being thrashed. If the client requires immediately
- *    it can reset this block using GAS_addresses_handle_backoff_reset.
- *
- *       1.7.7 Marking address in use
- *
- *    The client can notify addresses that it successfully uses an address and
- *    wants this address to be kept by calling GSA_address_in_use. Adresses will
- *    mark the address as used an notify the solver about the use.
- *
- *       1.7.8 Address lifecycle
+ *       1.7.6 Address lifecycle
  *
  *      * (add address)
- *      * (updated address) || (address in use)
+ *      * (updated address)
  *      * (delete address)
  *
  *     1.8 Bandwidth assignment
  *    The bandwidth assigned to a peer can be influenced by setting a preference
  *    for a peer. The prefernce will be given to to the solver with s_pref which
  *    has to take care of the preference value
-
- */
-
-
-/**
- * Pending Address suggestion requests
- */
-struct GAS_Addresses_Suggestion_Requests
-{
-  /**
-   * Next in DLL
-   */
-  struct GAS_Addresses_Suggestion_Requests *next;
-
-  /**
-   * Previous in DLL
-   */
-  struct GAS_Addresses_Suggestion_Requests *prev;
-
-  /**
-   * Peer ID
-   */
-  struct GNUNET_PeerIdentity id;
-};
-
-/**
- * Pending Address suggestion requests
  */
-struct GAS_Addresses_Preference_Clients
-{
-  /**
-   * Next in DLL
-   */
-  struct GAS_Addresses_Preference_Clients *next;
-
-  /**
-   * Previous in DLL
-   */
-  struct GAS_Addresses_Preference_Clients *prev;
-
-  /**
-   * Peer ID
-   */
-  void *client;
-};
 
 
-/**
- *
- */
-static struct GNUNET_STATISTICS_Handle *stats;
-
 /**
  * A multihashmap to store all addresses
  */
-static struct GNUNET_CONTAINER_MultiPeerMap *addresses;
-
-/**
- * Is ATS addresses running
- */
-static int running;
-
-/**
- * Preferences clients
- */
-static int pref_clients;
-
-/**
- * Configured ATS solver
- */
-static int ats_mode;
-
-/**
- * Solver handle
- */
-static void *solver;
-
-/**
- * Address suggestion requests DLL head
- */
-static struct GAS_Addresses_Suggestion_Requests *pending_requests_head;
-
-/**
- * Address suggestion requests DLL tail
- */
-static struct GAS_Addresses_Suggestion_Requests *pending_requests_tail;
+struct GNUNET_CONTAINER_MultiPeerMap *GSA_addresses;
 
-/**
- * Preference requests DLL head
- */
-static struct GAS_Addresses_Preference_Clients *preference_clients_head;
-
-/**
- * Preference requests DLL head
- */
-static struct GAS_Addresses_Preference_Clients *preference_clients_tail;
-
-/**
- * Solver functions
- */
-static struct GNUNET_ATS_PluginEnvironment env;
 
 /**
- * Solver plugin name as string
+ * Update statistic on number of addresses.
  */
-static char *plugin;
-
-/**
- * Value we pass for zero bandwidth.
- */
-static const struct GNUNET_BANDWIDTH_Value32NBO zero_bw;
+static void
+update_addresses_stat ()
+{
+  GNUNET_STATISTICS_set (GSA_stats,
+                         "# addresses",
+                         GNUNET_CONTAINER_multipeermap_size (GSA_addresses),
+                         GNUNET_NO);
+}
 
 
 /**
@@ -380,9 +273,10 @@ disassemble_ats_information (struct ATS_Address *dest,
     dest->atsi =
         GNUNET_malloc (update_count * sizeof (struct GNUNET_ATS_Information));
     dest->atsi_count = update_count;
-    memcpy (dest->atsi, update,
-        update_count * sizeof(struct GNUNET_ATS_Information));
-    (*delta_dest) =
+    memcpy (dest->atsi,
+            update,
+            update_count * sizeof(struct GNUNET_ATS_Information));
+    *delta_dest =
         GNUNET_malloc (update_count * sizeof (struct GNUNET_ATS_Information));
     for (c1 = 0; c1 < update_count; c1++)
     {
@@ -434,7 +328,7 @@ disassemble_ats_information (struct ATS_Address *dest,
         dest->atsi_count * sizeof(struct GNUNET_ATS_Information));
     memcpy (&tmp_atsi[dest->atsi_count], add_atsi,
         add_atsi_count * sizeof(struct GNUNET_ATS_Information));
-    GNUNET_free(dest->atsi);
+    GNUNET_free (dest->atsi);
     dest->atsi = tmp_atsi;
     dest->atsi_count = dest->atsi_count + add_atsi_count;
     change = GNUNET_YES;
@@ -462,6 +356,19 @@ disassemble_ats_information (struct ATS_Address *dest,
 static void
 free_address (struct ATS_Address *addr)
 {
+  GNUNET_CONTAINER_multipeermap_remove (GSA_addresses,
+                                        &addr->peer,
+                                        addr);
+  update_addresses_stat ();
+  GAS_plugin_delete_address (addr);
+  GAS_performance_notify_all_clients (&addr->peer,
+                                      addr->plugin,
+                                      addr->addr,
+                                      addr->addr_len,
+                                      GNUNET_NO,
+                                      NULL, 0,
+                                      GNUNET_BANDWIDTH_ZERO,
+                                      GNUNET_BANDWIDTH_ZERO);
   GNUNET_free (addr->plugin);
   GNUNET_free_non_null (addr->atsi);
   GNUNET_free (addr);
@@ -495,7 +402,9 @@ create_address (const struct GNUNET_PeerIdentity *peer,
   aa->peer = *peer;
   aa->addr_len = plugin_addr_len;
   aa->addr = &aa[1];
-  memcpy (&aa[1], plugin_addr, plugin_addr_len);
+  memcpy (&aa[1],
+          plugin_addr,
+          plugin_addr_len);
   aa->plugin = GNUNET_strdup (plugin_name);
   aa->session_id = session_id;
   aa->local_address_info = local_address_info;
@@ -503,6 +412,7 @@ create_address (const struct GNUNET_PeerIdentity *peer,
   for (c1 = 0; c1 < GNUNET_ATS_QualityPropertiesCount; c1++)
   {
     aa->atsin[c1].avg_queue_index = 0;
+    aa->atsin[c1].norm = DEFAULT_REL_QUALITY;
     for (c2 = 0; c2 < GAS_normalization_queue_length; c2++)
       aa->atsin[c1].atsi_abs[c2] = GNUNET_ATS_VALUE_UNDEFINED;
   }
@@ -510,129 +420,6 @@ create_address (const struct GNUNET_PeerIdentity *peer,
 }
 
 
-/**
- * Closure for #compare_address_it()
- */
-struct CompareAddressContext
-{
-  const struct ATS_Address *search;
-
-  /* exact_address != NULL if address and session is equal */
-  struct ATS_Address *exact_address;
-  /* exact_address != NULL if address and session is 0 */
-  struct ATS_Address *base_address;
-};
-
-
-/**
- * Comapre addresses.
- *
- * @param cls a CompareAddressContext containin the source address
- * @param key peer id
- * @param value the address to compare with
- * @return #GNUNET_YES to continue, #GNUNET_NO if address is founce
- */
-static int
-compare_address_it (void *cls,
-                   const struct GNUNET_PeerIdentity *key,
-                   void *value)
-{
-  struct CompareAddressContext *cac = cls;
-  struct ATS_Address *aa = value;
-
-  /* Find an matching exact address:
-   *
-   * Compare by:
-   * aa->addr_len == cac->search->addr_len
-   * aa->plugin == cac->search->plugin
-   * aa->addr == cac->search->addr
-   * aa->session == cac->search->session
-   *
-   * return as exact address
-   */
-  if ((aa->addr_len == cac->search->addr_len)
-      && (0 == strcmp (aa->plugin, cac->search->plugin)))
-  {
-    if ((0 == memcmp (aa->addr, cac->search->addr, aa->addr_len))
-        && (aa->session_id == cac->search->session_id))
-      cac->exact_address = aa;
-  }
-
-  /* Find an matching base address:
-   *
-   * Properties:
-   *
-   * aa->session_id == 0
-   *
-   * Compare by:
-   * aa->addr_len == cac->search->addr_len
-   * aa->plugin == cac->search->plugin
-   * aa->addr == cac->search->addr
-   *
-   * return as base address
-   */
-  if ((aa->addr_len == cac->search->addr_len)
-      && (0 == strcmp (aa->plugin, cac->search->plugin)))
-  {
-    if ((0 == memcmp (aa->addr, cac->search->addr, aa->addr_len))
-        && (aa->session_id == 0))
-      cac->base_address = aa;
-  }
-
-  /* Find an matching exact address based on session:
-   *
-   * Properties:
-   *
-   * cac->search->addr_len == 0
-   *
-   * Compare by:
-   * aa->plugin == cac->search->plugin
-   * aa->session_id == cac->search->session_id
-   *
-   * return as exact address
-   */
-  if (0 == cac->search->addr_len)
-  {
-    if ((0 == strcmp (aa->plugin, cac->search->plugin))
-        && (aa->session_id == cac->search->session_id))
-      cac->exact_address = aa;
-  }
-
-  if (cac->exact_address == NULL )
-    return GNUNET_YES; /* Continue iteration to find exact address */
-  else
-    return GNUNET_NO; /* Stop iteration since we have an exact address */
-}
-
-
-/**
- * Find an existing equivalent address record.
- * Compares by peer identity and network address OR by session ID
- * (one of the two must match).
- *
- * @param peer peer to lookup addresses for
- * @param addr existing address record
- * @return existing address record, NULL for none
- */
-struct ATS_Address *
-find_equivalent_address (const struct GNUNET_PeerIdentity *peer,
-                         const struct ATS_Address *addr)
-{
-  struct CompareAddressContext cac;
-
-  cac.exact_address = NULL;
-  cac.base_address = NULL;
-  cac.search = addr;
-  GNUNET_CONTAINER_multipeermap_get_multiple (addresses,
-                                             peer,
-                                             &compare_address_it, &cac);
-
-  if (NULL == cac.exact_address)
-    return cac.base_address;
-  return cac.exact_address;
-}
-
-
 /**
  * Closure for #find_address_cb()
  */
@@ -691,45 +478,13 @@ find_exact_address (const struct GNUNET_PeerIdentity *peer,
 
   fac.exact_address = NULL;
   fac.session_id = session_id;
-  GNUNET_CONTAINER_multipeermap_get_multiple (addresses,
+  GNUNET_CONTAINER_multipeermap_get_multiple (GSA_addresses,
                                              peer,
                                              &find_address_cb, &fac);
   return fac.exact_address;
 }
 
 
-/**
- * Function allowing the solver to obtain normalized preference
- * values from solver
- *
- * @param cls unused
- * @param id the peer to return the normalized properties for
- * @return array of double values with |GNUNET_ATS_PreferenceCount| elements
- */
-const double *
-get_preferences_cb (void *cls,
-                    const struct GNUNET_PeerIdentity *id)
-{
-  return GAS_normalization_get_preferences_by_peer (id);
-}
-
-
-/**
- * Function allowing the solver to obtain normalized property
- * values for an address from solver
- *
- * @param cls unused
- * @param address the address
- * @return array of double values with |GNUNET_ATS_QualityPropertiesCount| elements
- */
-const double *
-get_property_cb (void *cls,
-                 const struct ATS_Address *address)
-{
-  return GAS_normalization_get_properties (address);
-}
-
-
 /**
  * Extract an ATS performance info from an address
  *
@@ -738,10 +493,10 @@ get_property_cb (void *cls,
  * @return the value in HBO or #GNUNET_ATS_VALUE_UNDEFINED in HBO if value does not exist
  */
 static int
-get_performance_info (struct ATS_Address *address, uint32_t type)
+get_performance_info (struct ATS_Address *address,
+                      uint32_t type)
 {
-  int c1;
-  GNUNET_assert(NULL != address);
+  uint32_t c1;
 
   if ((NULL == address->atsi) || (0 == address->atsi_count))
     return GNUNET_ATS_VALUE_UNDEFINED;
@@ -778,18 +533,11 @@ GAS_addresses_add (const struct GNUNET_PeerIdentity *peer,
                    uint32_t atsi_count)
 {
   struct ATS_Address *new_address;
-  struct ATS_Address *existing_address;
   struct GNUNET_ATS_Information *atsi_delta;
   uint32_t atsi_delta_count;
   uint32_t addr_net;
-  uint32_t previous_session;
-  int c1;
 
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Received `%s' for peer `%s'\n",
-              "ADDRESS ADD",
-              GNUNET_i2s (peer));
-  if (GNUNET_NO == running)
+  if (NULL != find_exact_address (peer, session_id))
   {
     GNUNET_break (0);
     return;
@@ -810,129 +558,37 @@ GAS_addresses_add (const struct GNUNET_PeerIdentity *peer,
   if (GNUNET_ATS_VALUE_UNDEFINED == addr_net)
     addr_net = GNUNET_ATS_NET_UNSPECIFIED;
 
-  /* Get existing address or address with session == 0 */
-  existing_address = find_equivalent_address (peer, new_address);
-  if (NULL == existing_address)
-  {
-    /* Add a new address */
-    new_address->t_added = GNUNET_TIME_absolute_get();
-    new_address->t_last_activity = GNUNET_TIME_absolute_get();
-    GNUNET_assert(GNUNET_OK ==
-                  GNUNET_CONTAINER_multipeermap_put (addresses,
-                                                     peer,
-                                                     new_address,
-                                                     GNUNET_CONTAINER_MULTIHASHMAPOPTION_MULTIPLE));
-
-    GNUNET_STATISTICS_set (stats,
-                           "# addresses",
-                           GNUNET_CONTAINER_multipeermap_size (addresses),
-                           GNUNET_NO);
-
-    GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                "Adding new address %p for peer `%s', length %u, session id %u, %s\n",
-                new_address,
-                GNUNET_i2s (peer),
-                plugin_addr_len,
-                session_id,
-                GNUNET_ATS_print_network_type (addr_net));
-
-    /* Tell solver about new address */
-    env.sf.s_add (solver, new_address, addr_net);
-
-    env.sf.s_bulk_start (solver);
-    GAS_normalization_normalize_property (addresses,
-                                          new_address,
-                                          atsi,
-                                          atsi_count);
-    env.sf.s_bulk_stop (solver);
-
-    /* Notify performance clients about new address */
-    GAS_performance_notify_all_clients (&new_address->peer, new_address->plugin,
-        new_address->addr, new_address->addr_len, new_address->active,
-        new_address->atsi, new_address->atsi_count,
-        GNUNET_BANDWIDTH_value_init (new_address->assigned_bw_out),
-        GNUNET_BANDWIDTH_value_init (new_address->assigned_bw_in));
-    return;
-  }
-
-  /* FIXME: this case should probably not be allowed... */
-  /* We have an existing address we can use, clean up new */
-  GNUNET_free(new_address->plugin);
-  GNUNET_free_non_null(new_address->atsi);
-  GNUNET_free(new_address);
-  new_address = NULL;
-
-  if (0 != existing_address->session_id)
-  {
-    /* Should not happen */
-    GNUNET_break(0);
-    return;
-  }
-
-  addr_net = get_performance_info (existing_address, GNUNET_ATS_NETWORK_TYPE);
-  if (GNUNET_ATS_VALUE_UNDEFINED == addr_net)
-    addr_net = GNUNET_ATS_NET_UNSPECIFIED;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-      "Found existing address for peer `%s' %p with new session %u in network %s\n",
-      GNUNET_i2s (peer), existing_address, session_id,
-      GNUNET_ATS_print_network_type (addr_net));
-  /* We have an address without an session, update this address */
-  existing_address->t_added = GNUNET_TIME_absolute_get();
-  existing_address->t_last_activity = GNUNET_TIME_absolute_get();
-  atsi_delta = NULL;
-  atsi_delta_count = 0;
-  if (GNUNET_YES
-      == disassemble_ats_information (existing_address, atsi, atsi_count,
-          &atsi_delta, &atsi_delta_count))
-  {
-    /* Notify performance clients about properties */
-    GAS_performance_notify_all_clients (&existing_address->peer,
-        existing_address->plugin, existing_address->addr,
-        existing_address->addr_len, existing_address->active,
-        existing_address->atsi, existing_address->atsi_count,
-        GNUNET_BANDWIDTH_value_init (existing_address->assigned_bw_out),
-        GNUNET_BANDWIDTH_value_init (existing_address->assigned_bw_in));
-
-    for (c1 = 0; c1 < atsi_delta_count; c1++)
-    {
-      if ((GNUNET_ATS_NETWORK_TYPE == ntohl (atsi_delta[c1].type))
-          && (addr_net != ntohl (atsi_delta[c1].value)))
-      {
-        /* Network type changed */
-        GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-            "Address for peer `%s' %p changed from network %s to %s\n",
-            GNUNET_i2s (peer), existing_address,
-            GNUNET_ATS_print_network_type (addr_net),
-            GNUNET_ATS_print_network_type (ntohl (atsi_delta[c1].value)));
-        env.sf.s_address_update_network (solver, existing_address,
-            ntohl (atsi_delta[c1].value),
-            get_performance_info (existing_address, GNUNET_ATS_NETWORK_TYPE));
-        addr_net = get_performance_info (existing_address,
-            GNUNET_ATS_NETWORK_TYPE);
-      }
-    }
-    /* Notify solver about update with atsi information and session */
-    env.sf.s_bulk_start (solver);
-    GAS_normalization_normalize_property (addresses, existing_address,
-                                          atsi, atsi_count);
-    env.sf.s_bulk_stop (solver);
-  }
-  GNUNET_free_non_null(atsi_delta);
-
-  /* Notify solver about new session */
-  if (existing_address->session_id == session_id)
-    return; /* possible, can both be 0 since address is revalidated */
-
-  previous_session = existing_address->session_id;
-  existing_address->session_id = session_id;
-  env.sf.s_address_update_session (solver, existing_address,
-      previous_session, session_id);
-
-  GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-      "Updated existing address for peer `%s' %p length %u with new session %u in network %s\n",
-      GNUNET_i2s (peer), existing_address, existing_address->addr_len,
-      session_id, GNUNET_ATS_print_network_type (addr_net));
+  /* Add a new address */
+  new_address->t_added = GNUNET_TIME_absolute_get();
+  new_address->t_last_activity = GNUNET_TIME_absolute_get();
+  GNUNET_assert(GNUNET_OK ==
+               GNUNET_CONTAINER_multipeermap_put (GSA_addresses,
+                                                  peer,
+                                                  new_address,
+                                                  GNUNET_CONTAINER_MULTIHASHMAPOPTION_MULTIPLE));
+  update_addresses_stat ();
+  GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+             "Adding new address for peer `%s' slot %u\n",
+             GNUNET_i2s (peer),
+             session_id);
+  /* Tell solver about new address */
+  GAS_plugin_solver_lock ();
+  GAS_plugin_new_address (new_address,
+                         addr_net);
+  GAS_normalization_update_property (new_address,
+                                     atsi,
+                                     atsi_count);
+  GAS_plugin_solver_unlock ();
+  /* Notify performance clients about new address */
+  GAS_performance_notify_all_clients (&new_address->peer,
+                                     new_address->plugin,
+                                     new_address->addr,
+                                     new_address->addr_len,
+                                     new_address->active,
+                                     new_address->atsi,
+                                     new_address->atsi_count,
+                                     GNUNET_BANDWIDTH_value_init (new_address->assigned_bw_out),
+                                     GNUNET_BANDWIDTH_value_init (new_address->assigned_bw_in));
 }
 
 
@@ -953,12 +609,6 @@ GAS_addresses_update (const struct GNUNET_PeerIdentity *peer,
   struct ATS_Address *aa;
   struct GNUNET_ATS_Information *atsi_delta;
   uint32_t atsi_delta_count;
-  uint32_t prev_session;
-  int c1;
-
-  if (GNUNET_NO == running)
-    return;
-  GNUNET_assert (NULL != addresses);
 
   /* Get existing address */
   aa = find_exact_address (peer,
@@ -973,26 +623,13 @@ GAS_addresses_update (const struct GNUNET_PeerIdentity *peer,
     GNUNET_break (0);
     return;
   }
-
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Received `%s' for peer `%s' address \n",
-              "ADDRESS UPDATE",
+              "Received ADDRESS_UPDATE for peer `%s' slot %u\n",
               GNUNET_i2s (peer),
-              aa);
+              (unsigned int) session_id);
 
   /* Update address */
   aa->t_last_activity = GNUNET_TIME_absolute_get();
-  if (session_id != aa->session_id)
-  {
-    /* Session changed */
-    prev_session = aa->session_id;
-    aa->session_id = session_id;
-    env.sf.s_address_update_session (solver,
-                                     aa,
-                                     prev_session,
-                                     aa->session_id);
-  }
-
   atsi_delta = NULL;
   atsi_delta_count = 0;
   if (GNUNET_YES ==
@@ -1001,37 +638,27 @@ GAS_addresses_update (const struct GNUNET_PeerIdentity *peer,
                                    &atsi_delta,
                                    &atsi_delta_count))
   {
-    /* ATS properties changed */
-    for (c1 = 0; c1 < atsi_delta_count; c1++)
-    {
-      if (GNUNET_ATS_NETWORK_TYPE == ntohl (atsi_delta[c1].type))
-      {
-        /* Network type changed */
-        env.sf.s_address_update_network (solver, aa,
-            ntohl (atsi_delta[c1].value),
-            get_performance_info (aa, GNUNET_ATS_NETWORK_TYPE));
-      }
-    }
-
     /* Notify performance clients about updated address */
-    GAS_performance_notify_all_clients (&aa->peer, aa->plugin, aa->addr,
-        aa->addr_len, aa->active, aa->atsi, aa->atsi_count,
-        GNUNET_BANDWIDTH_value_init (aa->assigned_bw_out),
-        GNUNET_BANDWIDTH_value_init (aa->assigned_bw_in));
-
-    env.sf.s_bulk_start (solver);
-    GAS_normalization_normalize_property (addresses,
-                                          aa,
+    GAS_performance_notify_all_clients (&aa->peer,
+                                       aa->plugin,
+                                       aa->addr,
+                                       aa->addr_len,
+                                       aa->active,
+                                       aa->atsi,
+                                       aa->atsi_count,
+                                       GNUNET_BANDWIDTH_value_init (aa->assigned_bw_out),
+                                       GNUNET_BANDWIDTH_value_init (aa->assigned_bw_in));
+
+    GAS_normalization_update_property (aa,
                                           atsi,
                                           atsi_count);
-    env.sf.s_bulk_stop (solver);
   }
   GNUNET_free_non_null (atsi_delta);
 }
 
 
 /**
- * Remove an address or just a session for a peer.
+ * Remove an address for a peer.
  *
  * @param peer peer
  * @param session_id session id, can never be 0
@@ -1042,750 +669,19 @@ GAS_addresses_destroy (const struct GNUNET_PeerIdentity *peer,
 {
   struct ATS_Address *ea;
 
-  if (GNUNET_NO == running)
-    return;
-
   /* Get existing address */
   ea = find_exact_address (peer,
                            session_id);
   if (NULL == ea)
   {
     GNUNET_break (0);
-    GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-                "Tried to destroy unknown address for peer `%s' session id %u\n",
-                GNUNET_i2s (peer),
-                session_id);
     return;
   }
-
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Received `%s' for peer `%s' address %p session %u\n",
-              "ADDRESS DESTROYED",
+              "Received ADDRESS_DESTROYED for peer `%s' session %u\n",
               GNUNET_i2s (peer),
-              ea,
               session_id);
-  GNUNET_CONTAINER_multipeermap_remove (addresses,
-                                        peer,
-                                        ea);
-
-  env.sf.s_del (solver, ea, GNUNET_NO);
-  GAS_performance_notify_all_clients (peer,
-                                      ea->plugin,
-                                      ea->addr,
-                                      ea->addr_len,
-                                      GNUNET_SYSERR,
-                                      NULL, 0,
-                                      zero_bw,
-                                      zero_bw);
   free_address (ea);
-  GNUNET_STATISTICS_set (stats,
-                         "# addresses",
-                         GNUNET_CONTAINER_multipeermap_size (addresses),
-                         GNUNET_NO);
-}
-
-
-/**
- * Notification about active use of an address.
- * in_use == #GNUNET_YES:
- *     This address is used to maintain an active connection with a peer.
- * in_use == #GNUNET_NO:
- *     This address is no longer used to maintain an active connection with a peer.
- *
- * Note: can only be called with in_use == #GNUNET_NO if called with #GNUNET_YES
- * before
- *
- * @param peer peer
- * @param session_id session id, can be 0
- * @param in_use #GNUNET_YES if #GNUNET_NO FIXME
- * @return #GNUNET_SYSERR on failure (address unknown ...)
- */
-int
-GAS_addresses_in_use (const struct GNUNET_PeerIdentity *peer,
-                      uint32_t session_id,
-                      int in_use)
-{
-  struct ATS_Address *ea;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Received `%s' for peer `%s'\n",
-             "ADDRESS IN USE",
-             GNUNET_i2s (peer));
-  if (GNUNET_NO == running)
-    return GNUNET_SYSERR;
-  ea = find_exact_address (peer,
-                           session_id);
-  if (NULL == ea)
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
-                "Trying to set unknown address `%s' `%u' to %s \n",
-                GNUNET_i2s (peer),
-                session_id,
-                (GNUNET_NO == in_use) ? "NO" : "YES");
-    GNUNET_break (0);
-    return GNUNET_SYSERR;
-  }
-  if (ea->used == in_use)
-  {
-    GNUNET_break (0);
-    GNUNET_log(GNUNET_ERROR_TYPE_WARNING,
-               "Address in use called multiple times for peer `%s': %s -> %s \n",
-               GNUNET_i2s (peer),
-               (GNUNET_NO == ea->used) ? "NO" : "YES",
-               (GNUNET_NO == in_use) ? "NO" : "YES");
-    return GNUNET_SYSERR;
-  }
-  /* Tell solver about update */
-  ea->used = in_use;
-  ea->t_last_activity = GNUNET_TIME_absolute_get();
-  env.sf.s_address_update_inuse (solver,
-                                 ea,
-                                 ea->used);
-  return GNUNET_OK;
-}
-
-
-/**
- * Cancel address suggestions for a peer
- *
- * @param peer the peer id
- */
-void
-GAS_addresses_request_address_cancel (const struct GNUNET_PeerIdentity *peer)
-{
-  struct GAS_Addresses_Suggestion_Requests *cur = pending_requests_head;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Received request: `%s' for peer %s\n",
-             "request_address_cancel",
-             GNUNET_i2s (peer));
-
-  while (NULL != cur)
-  {
-    if (0 == memcmp (peer, &cur->id, sizeof(cur->id)))
-      break; /* found */
-    cur = cur->next;
-  }
-
-  if (NULL == cur)
-  {
-    GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-        "No address requests pending for peer `%s', cannot remove!\n",
-        GNUNET_i2s (peer));
-    return;
-  }
-  env.sf.s_get_stop (solver, peer);
-  GAS_addresses_handle_backoff_reset (peer);
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Removed request pending for peer `%s\n",
-             GNUNET_i2s (peer));
-  GNUNET_CONTAINER_DLL_remove (pending_requests_head,
-                               pending_requests_tail,
-                               cur);
-  GNUNET_free(cur);
-}
-
-
-/**
- * Request address suggestions for a peer
- *
- * @param peer the peer id
- */
-void
-GAS_addresses_request_address (const struct GNUNET_PeerIdentity *peer)
-{
-  struct GAS_Addresses_Suggestion_Requests *cur = pending_requests_head;
-  struct ATS_Address *aa;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Received `%s' for peer `%s'\n",
-             "REQUEST ADDRESS",
-             GNUNET_i2s (peer));
-
-  if (GNUNET_NO == running)
-    return;
-  while (NULL != cur)
-  {
-    if (0 == memcmp (peer, &cur->id, sizeof(cur->id)))
-      break; /* already suggesting */
-    cur = cur->next;
-  }
-  if (NULL == cur)
-  {
-    cur = GNUNET_new (struct GAS_Addresses_Suggestion_Requests);
-    cur->id = (*peer);
-    GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-        "Adding new address suggestion request for `%s'\n",
-         GNUNET_i2s (peer));
-    GNUNET_CONTAINER_DLL_insert (pending_requests_head,
-                                 pending_requests_tail,
-                                 cur);
-  }
-
-  /* Get prefered address from solver */
-  aa = (struct ATS_Address *) env.sf.s_get (solver, peer);
-  if (NULL == aa)
-  {
-    GNUNET_log(GNUNET_ERROR_TYPE_INFO, "Cannot suggest address for peer `%s'\n",
-        GNUNET_i2s (peer));
-    return;
-  }
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG, "Suggesting address %p for peer `%s'\n",
-      aa, GNUNET_i2s (peer));
-
-  GAS_scheduling_transmit_address_suggestion (peer,
-                                              aa->session_id,
-                                              GNUNET_BANDWIDTH_value_init (aa->assigned_bw_out),
-                                              GNUNET_BANDWIDTH_value_init (aa->assigned_bw_in));
-
-  aa->block_interval = GNUNET_TIME_relative_add (aa->block_interval,
-      ATS_BLOCKING_DELTA);
-  aa->blocked_until = GNUNET_TIME_absolute_add (GNUNET_TIME_absolute_get (),
-      aa->block_interval);
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-      "Address %p ready for suggestion, block interval now %llu \n", aa,
-      aa->block_interval);
-}
-
-/**
- * Iterator to reset address blocking
- *
- * @param cls not used
- * @param key the peer
- * @param value the address to reset
- * @return #GNUNET_OK to continue
- */
-static int
-reset_address_it (void *cls,
-                 const struct GNUNET_PeerIdentity *key,
-                 void *value)
-{
-  struct ATS_Address *aa = value;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-            "Resetting interval for peer `%s' address %p from %llu to 0\n",
-            GNUNET_i2s (&aa->peer),
-            aa,
-            aa->block_interval);
-  aa->blocked_until = GNUNET_TIME_UNIT_ZERO_ABS;
-  aa->block_interval = GNUNET_TIME_UNIT_ZERO;
-  return GNUNET_OK;
-}
-
-
-/**
- * Reset suggestion backoff for a peer
- *
- * Suggesting addresses is blocked for ATS_BLOCKING_DELTA. Blocking can be
- * reset using this function
- *
- * @param peer the peer id
- */
-void
-GAS_addresses_handle_backoff_reset (const struct GNUNET_PeerIdentity *peer)
-{
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Received `%s' for peer `%s'\n",
-             "RESET BACKOFF",
-             GNUNET_i2s (peer));
-
-  GNUNET_break(GNUNET_SYSERR !=
-               GNUNET_CONTAINER_multipeermap_get_multiple (addresses,
-                                                           peer,
-                                                           &reset_address_it,
-                                                           NULL));
-}
-
-
-/**
- * Solver information callback
- *
- * @param cls the closure
- * @param op the operation
- * @param status operation status
- * @param add additional information
- */
-static void
-solver_info_cb (void *cls,
-    enum GAS_Solver_Operation op,
-    enum GAS_Solver_Status status,
-    enum GAS_Solver_Additional_Information add)
-{
-  char *add_info;
-
-  switch (add) {
-    case GAS_INFO_NONE:
-      add_info = "GAS_INFO_NONE";
-      break;
-    case GAS_INFO_FULL:
-      add_info = "GAS_INFO_MLP_FULL";
-      break;
-    case GAS_INFO_UPDATED:
-      add_info = "GAS_INFO_MLP_UPDATED";
-      break;
-    case GAS_INFO_PROP_ALL:
-      add_info = "GAS_INFO_PROP_ALL";
-      break;
-    case GAS_INFO_PROP_SINGLE:
-      add_info = "GAS_INFO_PROP_SINGLE";
-      break;
-    default:
-      add_info = "INVALID";
-      break;
-  }
-  switch (op)
-  {
-    case GAS_OP_SOLVE_START:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s' `%s'\n", "GAS_OP_SOLVE_START",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL", add_info);
-      return;
-    case GAS_OP_SOLVE_STOP:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_STOP",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL", add_info);
-      return;
-
-    case GAS_OP_SOLVE_SETUP_START:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_SETUP_START",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-
-    case GAS_OP_SOLVE_SETUP_STOP:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_SETUP_STOP",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-
-    case GAS_OP_SOLVE_MLP_LP_START:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_LP_START",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-    case GAS_OP_SOLVE_MLP_LP_STOP:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_LP_STOP",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-
-    case GAS_OP_SOLVE_MLP_MLP_START:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_MLP_START",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-    case GAS_OP_SOLVE_MLP_MLP_STOP:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_MLP_STOP",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-    case GAS_OP_SOLVE_UPDATE_NOTIFICATION_START:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_UPDATE_NOTIFICATION_START",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-    case GAS_OP_SOLVE_UPDATE_NOTIFICATION_STOP:
-      GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-          "Solver notifies `%s' with result `%s'\n", "GAS_OP_SOLVE_UPDATE_NOTIFICATION_STOP",
-          (GAS_STAT_SUCCESS == status) ? "SUCCESS" : "FAIL");
-      return;
-    default:
-      break;
-    }
-}
-
-
-/**
- * The preference changed for a peer
- *
- * @param cls NULL
- * @param peer the peer
- * @param kind the ATS kind
- * @param pref_rel the new relative preference value
- */
-static void
-normalized_preference_changed_cb (void *cls,
-                                  const struct GNUNET_PeerIdentity *peer,
-                                  enum GNUNET_ATS_PreferenceKind kind,
-                                  double pref_rel)
-{
-  /* Tell solver about update */
-  env.sf.s_pref (solver, peer, kind, pref_rel);
-}
-
-
-/**
- * The relative value for a property changed
- *
- * @param cls NULL
- * @param address the peer
- * @param type the ATS type
- * @param prop_rel the new relative preference value
- */
-static void
-normalized_property_changed_cb (void *cls,
-                                struct ATS_Address *address,
-                                uint32_t type,
-                                double prop_rel)
-{
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Normalized property %s for peer `%s' changed to %.3f \n",
-             GNUNET_ATS_print_property_type (type),
-             GNUNET_i2s (&address->peer),
-             prop_rel);
-  env.sf.s_address_update_property (solver,
-                                    address,
-                                    type,
-                                    0,
-                                    prop_rel);
-}
-
-
-static struct GAS_Addresses_Preference_Clients *
-find_preference_client (void *client)
-{
-  struct GAS_Addresses_Preference_Clients *cur;
-
-  for (cur = preference_clients_head; NULL != cur; cur = cur->next)
-    if (cur->client == client)
-      return cur;
-  return NULL;
-}
-
-
-/**
- * A performance client disconnected
- *
- * @param client the client
- */
-void
-GAS_addresses_preference_client_disconnect (void *client)
-{
-  struct GAS_Addresses_Preference_Clients *pc;
-
-  if (NULL != (pc = find_preference_client (client)))
-  {
-    GNUNET_CONTAINER_DLL_remove (preference_clients_head,
-                                 preference_clients_tail,
-                                 pc);
-    GNUNET_free (pc);
-    GNUNET_assert (pref_clients > 0);
-    pref_clients --;
-    GNUNET_STATISTICS_set (stats,
-                           "# active performance clients",
-                           pref_clients,
-                           GNUNET_NO);
-  }
-  GAS_normalization_preference_client_disconnect (client);
-}
-
-
-/**
- * Change the preference for a peer
- *
- * @param client the client sending this request
- * @param peer the peer id
- * @param kind the preference kind to change
- * @param score_abs the new preference score
- */
-void
-GAS_addresses_preference_change (void *client,
-                                 const struct GNUNET_PeerIdentity *peer,
-                                 enum GNUNET_ATS_PreferenceKind kind,
-                                 float score_abs)
-{
-  struct GAS_Addresses_Preference_Clients *pc;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-      "Received `%s' for peer `%s' for client %p\n", "CHANGE PREFERENCE",
-      GNUNET_i2s (peer), client);
-
-  if (GNUNET_NO == running)
-    return;
-
-  if (GNUNET_NO ==
-      GNUNET_CONTAINER_multipeermap_contains (addresses,
-                                             peer))
-  {
-    GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-        "Received `%s' for unknown peer `%s' from client %p\n",
-        "CHANGE PREFERENCE", GNUNET_i2s (peer), client);
-    return;
-  }
-
-  if (NULL == find_preference_client (client))
-  {
-    pc = GNUNET_new (struct GAS_Addresses_Preference_Clients);
-    pc->client = client;
-    GNUNET_CONTAINER_DLL_insert (preference_clients_head,
-                                 preference_clients_tail,
-                                 pc);
-    pref_clients ++;
-    GNUNET_STATISTICS_set (stats,
-                           "# active performance clients",
-                           pref_clients,
-                           GNUNET_NO);
-  }
-
-  env.sf.s_bulk_start (solver);
-  /* Tell normalization about change, normalization will call callback if preference changed */
-  GAS_normalization_normalize_preference (client, peer, kind, score_abs);
-  env.sf.s_bulk_stop (solver);
-}
-
-
-/**
- * Change the preference for a peer
- *
- * @param application the client sending this request
- * @param peer the peer id
- * @param scope the time interval for this feedback: [now - scope .. now]
- * @param kind the preference kind to change
- * @param score_abs the new preference score
- */
-void
-GAS_addresses_preference_feedback (void *application,
-                                   const struct GNUNET_PeerIdentity *peer,
-                                   const struct GNUNET_TIME_Relative scope,
-                                   enum GNUNET_ATS_PreferenceKind kind,
-                                   float score_abs)
-{
-  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-             "Received `%s' for peer `%s' for client %p\n",
-             "PREFERENCE FEEDBACK",
-             GNUNET_i2s (peer),
-             application);
-
-  if (GNUNET_NO == running)
-    return;
-
-  if (GNUNET_NO ==
-      GNUNET_CONTAINER_multipeermap_contains (addresses,
-                                             peer))
-  {
-    GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-        "Received `%s' for unknown peer `%s' from client %p\n",
-        "PREFERENCE FEEDBACK", GNUNET_i2s (peer), application);
-    return;
-  }
-
-  env.sf.s_feedback (solver, application, peer, scope, kind,
-                     score_abs);
-}
-
-
-/**
- * Load quotas for networks from configuration
- *
- * @param cfg configuration handle
- * @param out_dest where to write outbound quotas
- * @param in_dest where to write inbound quotas
- * @param dest_length length of inbound and outbound arrays
- * @return number of networks loaded
- */
-static unsigned int
-load_quotas (const struct GNUNET_CONFIGURATION_Handle *cfg,
-    unsigned long long *out_dest, unsigned long long *in_dest, int dest_length)
-{
-  char * entry_in = NULL;
-  char * entry_out = NULL;
-  char * quota_out_str;
-  char * quota_in_str;
-  int c;
-  int res;
-
-  for (c = 0; (c < GNUNET_ATS_NetworkTypeCount) && (c < dest_length); c++)
-  {
-    in_dest[c] = 0;
-    out_dest[c] = 0;
-    GNUNET_asprintf (&entry_out,
-                     "%s_QUOTA_OUT",
-                     GNUNET_ATS_print_network_type (c));
-    GNUNET_asprintf (&entry_in,
-                     "%s_QUOTA_IN",
-                     GNUNET_ATS_print_network_type (c));
-
-    /* quota out */
-    if (GNUNET_OK
-        == GNUNET_CONFIGURATION_get_value_string (cfg, "ats", entry_out,
-            &quota_out_str))
-    {
-      res = GNUNET_NO;
-      if (0 == strcmp (quota_out_str, GNUNET_ATS_MaxBandwidthString))
-      {
-        out_dest[c] = GNUNET_ATS_MaxBandwidth;
-        res = GNUNET_YES;
-      }
-      if ((GNUNET_NO == res)
-          && (GNUNET_OK
-              == GNUNET_STRINGS_fancy_size_to_bytes (quota_out_str,
-                  &out_dest[c])))
-        res = GNUNET_YES;
-      if ((GNUNET_NO == res)
-          && (GNUNET_OK
-              == GNUNET_CONFIGURATION_get_value_number (cfg, "ats", entry_out,
-                  &out_dest[c])))
-        res = GNUNET_YES;
-
-      if (GNUNET_NO == res)
-      {
-        GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
-                   _("Could not load quota for network `%s':  `%s', assigning default bandwidth %llu\n"),
-                   GNUNET_ATS_print_network_type (c),
-                   quota_out_str,
-                   GNUNET_ATS_DefaultBandwidth);
-        out_dest[c] = GNUNET_ATS_DefaultBandwidth;
-      }
-      else
-      {
-        GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-                   _("Outbound quota configure for network `%s' is %llu\n"),
-                   GNUNET_ATS_print_network_type (c),
-                   out_dest[c]);
-      }
-      GNUNET_free(quota_out_str);
-    }
-    else
-    {
-      GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
-                 _("No outbound quota configured for network `%s', assigning default bandwidth %llu\n"),
-                 GNUNET_ATS_print_network_type (c),
-                 GNUNET_ATS_DefaultBandwidth);
-      out_dest[c] = GNUNET_ATS_DefaultBandwidth;
-    }
-
-    /* quota in */
-    if (GNUNET_OK
-        == GNUNET_CONFIGURATION_get_value_string (cfg, "ats", entry_in,
-            &quota_in_str))
-    {
-      res = GNUNET_NO;
-      if (0 == strcmp (quota_in_str, GNUNET_ATS_MaxBandwidthString))
-      {
-        in_dest[c] = GNUNET_ATS_MaxBandwidth;
-        res = GNUNET_YES;
-      }
-      if ((GNUNET_NO == res)
-          && (GNUNET_OK
-              == GNUNET_STRINGS_fancy_size_to_bytes (quota_in_str, &in_dest[c])))
-        res = GNUNET_YES;
-      if ((GNUNET_NO == res)
-          && (GNUNET_OK
-              == GNUNET_CONFIGURATION_get_value_number (cfg, "ats", entry_in,
-                  &in_dest[c])))
-        res = GNUNET_YES;
-
-      if (GNUNET_NO == res)
-      {
-        GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
-                   _("Could not load quota for network `%s':  `%s', assigning default bandwidth %llu\n"),
-                   GNUNET_ATS_print_network_type (c),
-                   quota_in_str,
-                   GNUNET_ATS_DefaultBandwidth);
-        in_dest[c] = GNUNET_ATS_DefaultBandwidth;
-      }
-      else
-      {
-        GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-                   _("Inbound quota configured for network `%s' is %llu\n"),
-                   GNUNET_ATS_print_network_type (c),
-                   in_dest[c]);
-      }
-      GNUNET_free(quota_in_str);
-    }
-    else
-    {
-      GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
-                 _("No outbound quota configure for network `%s', assigning default bandwidth %llu\n"),
-                 GNUNET_ATS_print_network_type (c),
-                 GNUNET_ATS_DefaultBandwidth);
-      in_dest[c] = GNUNET_ATS_DefaultBandwidth;
-    }
-    GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
-               "Loaded quota for network `%s' (in/out): %llu %llu\n",
-               GNUNET_ATS_print_network_type (c),
-               in_dest[c],
-               out_dest[c]);
-    GNUNET_free(entry_out);
-    GNUNET_free(entry_in);
-  }
-  return GNUNET_ATS_NetworkTypeCount;
-}
-
-
-/**
- * Callback for solver to notify about assignment changes
- *
- * @param cls NULL
- * @param address the address with changes
- */
-static void
-bandwidth_changed_cb (void *cls, struct ATS_Address *address)
-{
-  struct GAS_Addresses_Suggestion_Requests *cur;
-  uint32_t diff_out;
-  uint32_t diff_in;
-
-  GNUNET_assert(address != NULL);
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Bandwidth assignment changed for peer %s \n",
-              GNUNET_i2s (&address->peer));
-
-  /* Notify performance clients about changes to address */
-  GAS_performance_notify_all_clients (&address->peer, address->plugin,
-      address->addr, address->addr_len, address->active, address->atsi,
-      address->atsi_count,
-      GNUNET_BANDWIDTH_value_init (address->assigned_bw_out),
-      GNUNET_BANDWIDTH_value_init (address->assigned_bw_in));
-
-  for (cur = pending_requests_head;NULL != cur; cur = cur->next)
-    if (0 == memcmp (&address->peer, &cur->id, sizeof(cur->id)))
-      break; /* we have an address request pending*/
-  if (NULL == cur)
-  {
-    GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-               "Nobody is interested in peer `%s' :(\n",
-               GNUNET_i2s (&address->peer));
-    return;
-  }
-
-  if ((0 == address->assigned_bw_in) && (0 == address->assigned_bw_out))
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-               "Telling transport to disconnect peer `%s'\n",
-                GNUNET_i2s (&address->peer));
-
-    /* Notify scheduling clients about suggestion */
-    GAS_scheduling_transmit_address_suggestion (&address->peer,
-                                                address->session_id,
-                                                GNUNET_BANDWIDTH_value_init (0),
-                                                GNUNET_BANDWIDTH_value_init (0));
-    return;
-  }
-
-  /* Do bandwidth stability check */
-  diff_out = abs (address->assigned_bw_out - address->last_notified_bw_out);
-  diff_in = abs (address->assigned_bw_in - address->last_notified_bw_in);
-
-  if ( (diff_out < htonl(GNUNET_CONSTANTS_DEFAULT_BW_IN_OUT.value__)) &&
-       (diff_in < htonl(GNUNET_CONSTANTS_DEFAULT_BW_IN_OUT.value__)) )
-    return;
-
-  GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-      "Sending bandwidth update for peer `%s': %u %u\n",
-      GNUNET_i2s (&address->peer), address->assigned_bw_out,
-      address->assigned_bw_out);
-
-  /* *Notify scheduling clients about suggestion */
-  GAS_scheduling_transmit_address_suggestion (&address->peer,
-                                              address->session_id,
-                                              GNUNET_BANDWIDTH_value_init (address->assigned_bw_out),
-                                              GNUNET_BANDWIDTH_value_init (address->assigned_bw_in));
-
-  address->last_notified_bw_out = address->assigned_bw_out;
-  address->last_notified_bw_in = address->assigned_bw_in;
 }
 
 
@@ -1795,146 +691,13 @@ bandwidth_changed_cb (void *cls, struct ATS_Address *address)
  * responsible for the resource allocation. It tells the solver about changes
  * and receives updates when the solver changes the resource allocation.
  *
- * @param cfg configuration to use
- * @param stats_ the statistics handle to use
- * @return #GNUNET_OK on success, #GNUNET_SYSERR on error (failed to load
- *         solver plugin)
+ * @param server handle to our server
  */
-int
-GAS_addresses_init (const struct GNUNET_CONFIGURATION_Handle *cfg,
-                    struct GNUNET_STATISTICS_Handle *stats_)
+void
+GAS_addresses_init (struct GNUNET_SERVER_Handle *server)
 {
-  unsigned long long quotas_in[GNUNET_ATS_NetworkTypeCount];
-  unsigned long long quotas_out[GNUNET_ATS_NetworkTypeCount];
-  char *mode_str;
-  char *plugin_short;
-  int c;
-
-  running = GNUNET_NO;
-  stats = stats_;
-  /* Initialize the addresses database */
-  addresses = GNUNET_CONTAINER_multipeermap_create (128, GNUNET_NO);
-  pref_clients = 0;
-
-  /* Figure out configured solution method */
-  if (GNUNET_SYSERR ==
-      GNUNET_CONFIGURATION_get_value_string (cfg, "ats", "MODE", &mode_str))
-  {
-    GNUNET_log(GNUNET_ERROR_TYPE_WARNING,
-               "No resource assignment method configured, using proportional approach\n");
-    ats_mode = MODE_PROPORTIONAL;
-  }
-  else
-  {
-    for (c = 0; c < strlen (mode_str); c++)
-      mode_str[c] = toupper (mode_str[c]);
-    if (0 == strcmp (mode_str, "PROPORTIONAL"))
-      ats_mode = MODE_PROPORTIONAL;
-    else if (0 == strcmp (mode_str, "MLP"))
-    {
-      ats_mode = MODE_MLP;
-#if !HAVE_LIBGLPK
-      GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
-                 "Assignment method `%s' configured, but GLPK is not available, please install \n",
-                 mode_str);
-      ats_mode = MODE_PROPORTIONAL;
-#endif
-    }
-    else if (0 == strcmp (mode_str, "RIL"))
-      ats_mode = MODE_RIL;
-    else
-    {
-      GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
-                 "Invalid resource assignment method `%s' configured, using proportional approach\n",
-                 mode_str);
-      ats_mode = MODE_PROPORTIONAL;
-    }
-    GNUNET_free(mode_str);
-  }
-
-  load_quotas (cfg, quotas_out, quotas_in, GNUNET_ATS_NetworkTypeCount);
-  env.info_cb = &solver_info_cb;
-  env.info_cb_cls = NULL;
-  env.bandwidth_changed_cb = &bandwidth_changed_cb;
-  env.bw_changed_cb_cls = NULL;
-  env.get_preferences = &get_preferences_cb;
-  env.get_preference_cls = NULL;
-  env.get_property = &get_property_cb;
-  env.get_property_cls = NULL;
-  env.cfg = cfg;
-  env.stats = stats;
-  env.addresses = addresses;
-
-  env.network_count = GNUNET_ATS_NetworkTypeCount;
-  int networks[GNUNET_ATS_NetworkTypeCount] = GNUNET_ATS_NetworkType;
-  for (c = 0; c < GNUNET_ATS_NetworkTypeCount; c++)
-  {
-    env.networks[c] = networks[c];
-    env.out_quota[c] = quotas_out[c];
-    env.in_quota[c] = quotas_in[c];
-  }
-
-  switch (ats_mode) {
-    case MODE_PROPORTIONAL:
-      plugin_short = "proportional";
-      break;
-    case MODE_MLP:
-      plugin_short = "mlp";
-      break;
-    case MODE_RIL:
-      plugin_short = "ril";
-      break;
-    default:
-      plugin_short = NULL;
-      break;
-  }
-  GNUNET_asprintf (&plugin,
-                   "libgnunet_plugin_ats_%s",
-                   plugin_short);
-  GNUNET_log(GNUNET_ERROR_TYPE_INFO,
-             _("Initializing solver `%s '`%s'\n"),
-             plugin_short,
-             plugin);
-  if (NULL == (solver = GNUNET_PLUGIN_load (plugin, &env)))
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                _("Failed to initialize solver `%s'!\n"),
-                plugin);
-    return GNUNET_SYSERR;
-  }
-
-  GNUNET_assert (NULL != env.sf.s_add);
-  GNUNET_assert (NULL != env.sf.s_address_update_inuse);
-  GNUNET_assert (NULL != env.sf.s_address_update_property);
-  GNUNET_assert (NULL != env.sf.s_address_update_session);
-  GNUNET_assert (NULL != env.sf.s_address_update_network);
-  GNUNET_assert (NULL != env.sf.s_get);
-  GNUNET_assert (NULL != env.sf.s_get_stop);
-  GNUNET_assert (NULL != env.sf.s_pref);
-  GNUNET_assert (NULL != env.sf.s_feedback);
-  GNUNET_assert (NULL != env.sf.s_del);
-  GNUNET_assert (NULL != env.sf.s_bulk_start);
-  GNUNET_assert (NULL != env.sf.s_bulk_stop);
-
-
-  GAS_normalization_start (&normalized_preference_changed_cb, NULL,
-                           &normalized_property_changed_cb, NULL);
-
-  if (NULL == solver)
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                _("Failed to initialize solver!\n"));
-    return GNUNET_SYSERR;
-  }
-  /* up and running */
-  running = GNUNET_YES;
-
-  GNUNET_STATISTICS_set (stats,
-                         "# addresses",
-                         GNUNET_CONTAINER_multipeermap_size (addresses),
-                         GNUNET_NO);
-
-  return GNUNET_OK;
+  GSA_addresses = GNUNET_CONTAINER_multipeermap_create (128, GNUNET_NO);
+  update_addresses_stat ();
 }
 
 
@@ -1953,22 +716,6 @@ destroy_all_address_it (void *cls,
 {
   struct ATS_Address *aa = value;
 
-  /* Remove */
-  GNUNET_assert(GNUNET_YES ==
-               GNUNET_CONTAINER_multipeermap_remove (addresses,
-                                                      key,
-                                                      value));
-  /* Notify */
-  env.sf.s_del (solver, aa, GNUNET_NO);
-  /* Destroy */
-  GAS_performance_notify_all_clients (&aa->peer,
-                                      aa->plugin,
-                                      aa->addr,
-                                      aa->addr_len,
-                                      GNUNET_NO,
-                                      NULL, 0,
-                                      zero_bw,
-                                      zero_bw);
   free_address (aa);
   return GNUNET_OK;
 }
@@ -1980,17 +727,16 @@ destroy_all_address_it (void *cls,
 void
 GAS_addresses_destroy_all ()
 {
-  if (GNUNET_NO == running)
-    return;
-
   GNUNET_log (GNUNET_ERROR_TYPE_INFO,
               "Destroying all addresses\n");
-  env.sf.s_bulk_start (solver);
-  if (NULL != addresses)
-    GNUNET_CONTAINER_multipeermap_iterate (addresses,
-                                          &destroy_all_address_it,
-                                          NULL);
-  env.sf.s_bulk_start (solver);
+  if (0 ==
+      GNUNET_CONTAINER_multipeermap_size (GSA_addresses))
+    return;
+  GAS_plugin_solver_lock ();
+  GNUNET_CONTAINER_multipeermap_iterate (GSA_addresses,
+                                         &destroy_all_address_it,
+                                         NULL);
+  GAS_plugin_solver_unlock ();
 }
 
 
@@ -2000,41 +746,11 @@ GAS_addresses_destroy_all ()
 void
 GAS_addresses_done ()
 {
-  struct GAS_Addresses_Suggestion_Requests *cur;
-  struct GAS_Addresses_Preference_Clients *pcur;
-
   GNUNET_log(GNUNET_ERROR_TYPE_INFO,
              "Shutting down addresses\n");
- GAS_addresses_destroy_all ();
-  running = GNUNET_NO;
-  GNUNET_CONTAINER_multipeermap_destroy (addresses);
-  addresses = NULL;
-  while (NULL != (cur = pending_requests_head))
-  {
-    GNUNET_CONTAINER_DLL_remove (pending_requests_head,
-                                 pending_requests_tail,
-                                 cur);
-    GNUNET_free(cur);
-  }
-
-  while (NULL != (pcur = preference_clients_head))
-  {
-    GNUNET_CONTAINER_DLL_remove (preference_clients_head,
-                                 preference_clients_tail,
-                                 pcur);
-    GNUNET_assert (pref_clients > 0);
-    pref_clients --;
-    GNUNET_STATISTICS_set (stats,
-                           "# active performance clients",
-                           pref_clients,
-                           GNUNET_NO);
-    GNUNET_free (pcur);
-  }
-  GNUNET_PLUGIN_unload (plugin,
-                        solver);
-  GNUNET_free (plugin);
-  /* Stop configured solution method */
-  GAS_normalization_stop ();
+  GAS_addresses_destroy_all ();
+  GNUNET_CONTAINER_multipeermap_destroy (GSA_addresses);
+  GSA_addresses = NULL;
 }
 
 
@@ -2109,23 +825,249 @@ GAS_addresses_get_peer_info (const struct GNUNET_PeerIdentity *peer,
               (NULL == peer)
               ? "all peers"
               : GNUNET_i2s (peer),
-              (unsigned int) GNUNET_CONTAINER_multipeermap_size (addresses));
+              (unsigned int) GNUNET_CONTAINER_multipeermap_size (GSA_addresses));
   pi_ctx.it = pi_it;
   pi_ctx.it_cls = pi_it_cls;
   if (NULL == peer)
-    GNUNET_CONTAINER_multipeermap_iterate (addresses,
+    GNUNET_CONTAINER_multipeermap_iterate (GSA_addresses,
                                            &peerinfo_it,
                                            &pi_ctx);
   else
-    GNUNET_CONTAINER_multipeermap_get_multiple (addresses,
+    GNUNET_CONTAINER_multipeermap_get_multiple (GSA_addresses,
                                                 peer,
                                                 &peerinfo_it, &pi_ctx);
   pi_it (pi_it_cls,
          NULL, NULL, NULL, 0,
          GNUNET_NO,
          NULL, 0,
-         zero_bw,
-         zero_bw);
+         GNUNET_BANDWIDTH_ZERO,
+         GNUNET_BANDWIDTH_ZERO);
+}
+
+
+/**
+ * Information we need for the callbacks to return a list of addresses
+ * back to the client.
+ */
+struct AddressIteration
+{
+  /**
+   * Actual handle to the client.
+   */
+  struct GNUNET_SERVER_Client *client;
+
+  /**
+   * Are we sending all addresses, or only those that are active?
+   */
+  int all;
+
+  /**
+   * Which ID should be included in the response?
+   */
+  uint32_t id;
+
+};
+
+
+/**
+ * Send a #GNUNET_MESSAGE_TYPE_ATS_ADDRESSLIST_RESPONSE with the
+ * given address details to the client identified in @a ai.
+ *
+ * @param ai our address information context (identifies the client)
+ * @param id the peer id this address is for
+ * @param plugin_name name of the plugin that supports this address
+ * @param plugin_addr address
+ * @param plugin_addr_len length of @a plugin_addr
+ * @param active #GNUNET_YES if this address is actively used
+ * @param atsi ats performance information
+ * @param atsi_count number of ats performance elements in @a atsi
+ * @param bandwidth_out current outbound bandwidth assigned to address
+ * @param bandwidth_in current inbound bandwidth assigned to address
+ */
+static void
+transmit_req_addr (struct AddressIteration *ai,
+                   const struct GNUNET_PeerIdentity *id,
+                   const char *plugin_name,
+                   const void *plugin_addr,
+                   size_t plugin_addr_len,
+                   int active,
+                   const struct GNUNET_ATS_Information *atsi,
+                   uint32_t atsi_count,
+                   struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out,
+                   struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in)
+
+{
+  struct GNUNET_ATS_Information *atsp;
+  struct PeerInformationMessage *msg;
+  char *addrp;
+  size_t plugin_name_length;
+  size_t msize;
+  struct GNUNET_SERVER_NotificationContext *nc;
+
+  if (NULL != plugin_name)
+    plugin_name_length = strlen (plugin_name) + 1;
+  else
+    plugin_name_length = 0;
+  msize = sizeof (struct PeerInformationMessage) +
+          atsi_count * sizeof (struct GNUNET_ATS_Information) +
+          plugin_addr_len + plugin_name_length;
+  char buf[msize] GNUNET_ALIGN;
+
+  GNUNET_assert (msize < GNUNET_SERVER_MAX_MESSAGE_SIZE);
+  GNUNET_assert (atsi_count <
+                 GNUNET_SERVER_MAX_MESSAGE_SIZE /
+                 sizeof (struct GNUNET_ATS_Information));
+  msg = (struct PeerInformationMessage *) buf;
+  msg->header.size = htons (msize);
+  msg->header.type = htons (GNUNET_MESSAGE_TYPE_ATS_ADDRESSLIST_RESPONSE);
+  msg->ats_count = htonl (atsi_count);
+  msg->id = htonl (ai->id);
+  if (NULL != id)
+    msg->peer = *id;
+  else
+    memset (&msg->peer, '\0', sizeof (struct GNUNET_PeerIdentity));
+  msg->address_length = htons (plugin_addr_len);
+  msg->address_active = ntohl (active);
+  msg->plugin_name_length = htons (plugin_name_length);
+  msg->bandwidth_out = bandwidth_out;
+  msg->bandwidth_in = bandwidth_in;
+  atsp = (struct GNUNET_ATS_Information *) &msg[1];
+  memcpy (atsp, atsi, sizeof (struct GNUNET_ATS_Information) * atsi_count);
+  addrp = (char *) &atsp[atsi_count];
+  if (NULL != plugin_addr)
+    memcpy (addrp, plugin_addr, plugin_addr_len);
+  if (NULL != plugin_name)
+    strcpy (&addrp[plugin_addr_len], plugin_name);
+  nc = *GNUNET_SERVER_client_get_user_context (ai->client,
+                                               struct GNUNET_SERVER_NotificationContext *);
+  if (NULL == nc)
+  {
+    GNUNET_break (0);
+    return;
+  }
+  GNUNET_SERVER_notification_context_unicast (nc,
+                                              ai->client,
+                                              &msg->header,
+                                              GNUNET_NO);
 }
 
+
+/**
+ * Iterator for #GAS_addresses_get_peer_info(), called with peer-specific
+ * information to be passed back to the client.
+ *
+ * @param cls closure with our `struct AddressIteration *`
+ * @param id the peer id
+ * @param plugin_name plugin name
+ * @param plugin_addr address
+ * @param plugin_addr_len length of @a plugin_addr
+ * @param active is address actively used
+ * @param atsi ats performance information
+ * @param atsi_count number of ats performance elements in @a atsi
+ * @param bandwidth_out current outbound bandwidth assigned to address
+ * @param bandwidth_in current inbound bandwidth assigned to address
+ */
+static void
+req_addr_peerinfo_it (void *cls,
+                      const struct GNUNET_PeerIdentity *id,
+                      const char *plugin_name,
+                      const void *plugin_addr,
+                      size_t plugin_addr_len,
+                      int active,
+                      const struct GNUNET_ATS_Information *atsi,
+                      uint32_t atsi_count,
+                      struct GNUNET_BANDWIDTH_Value32NBO bandwidth_out,
+                      struct GNUNET_BANDWIDTH_Value32NBO bandwidth_in)
+{
+  struct AddressIteration *ai = cls;
+
+  if ( (NULL == id) &&
+       (NULL == plugin_name) &&
+       (NULL == plugin_addr) )
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Address iteration done for one peer\n");
+    return;
+  }
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Callback for %s peer `%s' plugin `%s' BW out %u, BW in %u\n",
+              (active == GNUNET_YES) ? "ACTIVE" : "INACTIVE",
+              GNUNET_i2s (id),
+              plugin_name,
+              (unsigned int) ntohl (bandwidth_out.value__),
+              (unsigned int) ntohl (bandwidth_in.value__));
+  /* Transmit result (either if address is active, or if
+     client wanted all addresses) */
+  if ( (GNUNET_YES != ai->all) &&
+       (GNUNET_YES != active))
+    return;
+  transmit_req_addr (ai,
+                     id,
+                     plugin_name,
+                     plugin_addr, plugin_addr_len,
+                     active,
+                     atsi,
+                     atsi_count,
+                     bandwidth_out,
+                     bandwidth_in);
+}
+
+
+/**
+ * Handle 'address list request' messages from clients.
+ *
+ * @param cls unused, NULL
+ * @param client client that sent the request
+ * @param message the request message
+ */
+void
+GAS_handle_request_address_list (void *cls,
+                                 struct GNUNET_SERVER_Client *client,
+                                 const struct GNUNET_MessageHeader *message)
+{
+  struct AddressIteration ai;
+  const struct AddressListRequestMessage *alrm;
+  struct GNUNET_PeerIdentity allzeros;
+
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Received ADDRESSLIST_REQUEST message\n");
+  alrm = (const struct AddressListRequestMessage *) message;
+  ai.all = ntohl (alrm->all);
+  ai.id = ntohl (alrm->id);
+  ai.client = client;
+
+  memset (&allzeros,
+          '\0',
+          sizeof (struct GNUNET_PeerIdentity));
+  if (0 == memcmp (&alrm->peer,
+                   &allzeros,
+                   sizeof (struct GNUNET_PeerIdentity)))
+  {
+    /* Return addresses for all peers */
+    GAS_addresses_get_peer_info (NULL,
+                                 &req_addr_peerinfo_it,
+                                 &ai);
+  }
+  else
+  {
+    /* Return addresses for a specific peer */
+    GAS_addresses_get_peer_info (&alrm->peer,
+                                 &req_addr_peerinfo_it,
+                                 &ai);
+  }
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+              "Finished handling `%s' message\n",
+              "ADDRESSLIST_REQUEST");
+  transmit_req_addr (&ai,
+                     NULL, NULL, NULL,
+                     0, GNUNET_NO,
+                     NULL, 0,
+                     GNUNET_BANDWIDTH_ZERO,
+                     GNUNET_BANDWIDTH_ZERO);
+  GNUNET_SERVER_receive_done (client,
+                              GNUNET_OK);
+}
+
+
+
 /* end of gnunet-service-ats_addresses.c */