*
* TODO:
* - path desirability score calculations are not done
- * (and will be tricky to have during path changes)
*/
#include "platform.h"
#include "gnunet-service-cadet-new_connection.h"
* Array of all the peers on the path. If @e hn is non-NULL, the
* last one is our owner.
*/
- struct CadetPeerPathEntry *entries;
+ struct CadetPeerPathEntry **entries;
/**
* Node of this path in the owner's heap. Used to update our position
*/
struct GNUNET_CONTAINER_HeapNode *hn;
- /**
- * Connections using this path, by destination peer
- * (each hop of the path could correspond to an
- * active connection).
- */
- struct GNUNET_CONTAINER_MultiPeerMap *connections;
-
/**
* Desirability of the path. How unique is it for the various peers
* on it?
};
+/**
+ * Calculate the path's desirability score.
+ *
+ * @param path path to calculate the score for
+ */
+static void
+recalculate_path_desirability (struct CadetPeerPath *path)
+{
+ /* FIXME: update path desirability! */
+ GNUNET_break (0); // not implemented
+}
+
+
/**
* Return how much we like keeping the path. This is an aggregate
* score based on various factors, including the age of the path
struct CadetPeerPathEntry *entry;
GNUNET_assert (off < path->entries_length);
- entry = &path->entries[off];
+ entry = path->entries[off];
GNUNET_assert (entry->peer == destination);
return entry->cc;
}
GCPP_2s (path),
off);
GNUNET_assert (off < path->entries_length);
- entry = &path->entries[off];
+ entry = path->entries[off];
GNUNET_assert (NULL == entry->cc);
entry->cc = cc;
}
GCPP_2s (path),
off);
GNUNET_assert (off < path->entries_length);
- entry = &path->entries[off];
+ entry = path->entries[off];
GNUNET_assert (cc == entry->cc);
entry->cc = NULL;
}
LOG (GNUNET_ERROR_TYPE_DEBUG,
"Destroying path %s\n",
GCPP_2s (path));
- GNUNET_assert (0 ==
- GNUNET_CONTAINER_multipeermap_size (path->connections));
- GNUNET_CONTAINER_multipeermap_destroy (path->connections);
+ for (unsigned int i=0;i<path->entries_length;i++)
+ GNUNET_free (path->entries[i]);
GNUNET_free (path->entries);
GNUNET_free (path);
}
"Owner releases path %s\n",
GCPP_2s (path));
path->hn = NULL;
- entry = &path->entries[path->entries_length - 1];
+ entry = path->entries[path->entries_length - 1];
while (1)
{
- /* cut 'off' end of path, verifying it is not in use */
- GNUNET_assert (NULL ==
- GNUNET_CONTAINER_multipeermap_get (path->connections,
- GCP_get_id (entry->peer)));
+ /* cut 'off' end of path */
GCP_path_entry_remove (entry->peer,
entry,
path->entries_length - 1);
path->entries_length--; /* We don't bother shrinking the 'entries' array,
as it's probably not worth it. */
+ GNUNET_free (entry);
if (0 == path->entries_length)
break; /* the end */
/* see if new peer at the end likes this path any better */
- entry = &path->entries[path->entries_length - 1];
+ entry = path->entries[path->entries_length - 1];
path->hn = GCP_attach_path (entry->peer,
path,
- path->entries_length,
+ path->entries_length - 1,
GNUNET_NO);
if (NULL != path->hn)
return; /* yep, got attached, we are done. */
struct CadetPeerPathEntry *entry;
GNUNET_assert (off < path->entries_length);
- entry = &path->entries[off];
+ entry = path->entries[off];
/* Add delta, with checks for overflows */
if (delta >= 0)
else
entry->score += delta;
}
-
- /* FIXME: update path desirability! */
+ recalculate_path_desirability (path);
}
*/
struct CadetPeer **cpath;
+ /**
+ * How long is the @e cpath array?
+ */
+ unsigned int cpath_length;
+
};
{
struct CheckMatchContext *cm_ctx = cls;
- if (path->entries_length > off)
- return GNUNET_YES; /* too long, cannot be useful */
+ GNUNET_assert (path->entries_length > off);
+ if ( (path->entries_length != off + 1) &&
+ (off + 1 != cm_ctx->cpath_length) )
+ {
+ LOG (GNUNET_ERROR_TYPE_DEBUG,
+ "check_match missmatch because path %s is too long (%u vs. %u vs. %u)\n",
+ GCPP_2s (path),
+ path->entries_length,
+ off + 1,
+ cm_ctx->cpath_length);
+ return GNUNET_YES; /* too long, goes somewhere else already, thus cannot be useful */
+ }
for (unsigned int i=0;i<off;i++)
if (cm_ctx->cpath[i] !=
GCPP_get_peer_at_offset (path,
i))
+ {
+ LOG (GNUNET_ERROR_TYPE_DEBUG,
+ "check_match path %s missmatches at offset %u\n",
+ GCPP_2s (path),
+ i);
return GNUNET_YES; /* missmatch, ignore */
+ }
+ LOG (GNUNET_ERROR_TYPE_DEBUG,
+ "check_match found match with path %s\n",
+ GCPP_2s (path));
cm_ctx->match = path;
return GNUNET_NO; /* match, we are done! */
}
struct GNUNET_CONTAINER_HeapNode *hn;
int i;
+ /* Expand path */
+ GNUNET_array_grow (path->entries,
+ path->entries_length,
+ old_len + num_peers);
+ for (i=num_peers-1;i >= 0;i--)
+ {
+ struct CadetPeerPathEntry *entry = GNUNET_new (struct CadetPeerPathEntry);
+
+ path->entries[old_len + i] = entry;
+ entry->peer = peers[i];
+ entry->path = path;
+ }
+ for (i=num_peers-1;i >= 0;i--)
+ {
+ struct CadetPeerPathEntry *entry = path->entries[old_len + i];
+
+ GCP_path_entry_add (entry->peer,
+ entry,
+ old_len + i);
+ }
+
/* If we extend an existing path, detach it from the
old owner and re-attach to the new one */
hn = NULL;
for (i=num_peers-1;i>=0;i--)
{
- /* FIXME: note that path->desirability is used, but not yet updated here! */
+ struct CadetPeerPathEntry *entry = path->entries[old_len + i];
+
+ path->entries_length = old_len + i + 1;
+ recalculate_path_desirability (path);
hn = GCP_attach_path (peers[i],
path,
old_len + (unsigned int) i,
GNUNET_YES);
if (NULL != hn)
break;
+ GCP_path_entry_remove (entry->peer,
+ entry,
+ old_len + i);
+ GNUNET_free (entry);
+ path->entries[old_len + i] = NULL;
}
if (NULL == hn)
- return; /* none of the peers is interested in this path */
- GCP_detach_path (path->entries[old_len-1].peer,
+ {
+ /* none of the peers is interested in this path;
+ shrink path back */
+ GNUNET_array_grow (path->entries,
+ path->entries_length,
+ old_len);
+ return;
+ }
+ GCP_detach_path (path->entries[old_len-1]->peer,
path,
path->hn);
path->hn = hn;
- GNUNET_array_grow (path->entries,
- path->entries_length,
- old_len + i);
- for (;i >= 0;i--)
- {
- struct CadetPeerPathEntry *entry = &path->entries[old_len + i];
-
- entry->peer = peers[i];
- entry->path = path;
- GCP_path_entry_add (entry->peer,
- entry,
- old_len + i);
- }
LOG (GNUNET_ERROR_TYPE_DEBUG,
"Extended path %s\n",
GCPP_2s (path));
/* First figure out if this path is a subset of an existing path, an
extension of an existing path, or a new path. */
+ cm_ctx.cpath_length = get_path_length + put_path_length;
cm_ctx.cpath = cpath;
cm_ctx.match = NULL;
for (i=get_path_length + put_path_length-1;i>=0;i--)
/* No match at all, create completely new path */
path = GNUNET_new (struct CadetPeerPath);
+ path->entries_length = get_path_length + put_path_length;
+ path->entries = GNUNET_new_array (path->entries_length,
+ struct CadetPeerPathEntry *);
+ for (i=path->entries_length-1;i>=0;i--)
+ {
+ struct CadetPeerPathEntry *entry = GNUNET_new (struct CadetPeerPathEntry);
- /* First, try to attach it */
+ path->entries[i] = entry;
+ entry->peer = cpath[i];
+ entry->path = path;
+ }
+ for (i=path->entries_length-1;i>=0;i--)
+ {
+ struct CadetPeerPathEntry *entry = path->entries[i];
+
+ GCP_path_entry_add (entry->peer,
+ entry,
+ i);
+ }
+
+ /* Finally, try to attach it */
hn = NULL;
for (i=get_path_length + put_path_length-1;i>=0;i--)
{
- path->entries_length = i;
- /* FIXME: note that path->desirability is used, but not yet initialized here! */
+ struct CadetPeerPathEntry *entry = path->entries[i];
+
+ path->entries_length = i + 1;
+ recalculate_path_desirability (path);
hn = GCP_attach_path (cpath[i],
path,
(unsigned int) i,
GNUNET_NO);
if (NULL != hn)
break;
+ GCP_path_entry_remove (entry->peer,
+ entry,
+ i);
+ GNUNET_free (entry);
+ path->entries[i] = NULL;
}
if (NULL == hn)
{
/* None of the peers on the path care about it. */
LOG (GNUNET_ERROR_TYPE_DEBUG,
"Path discovered from DHT is not interesting to us\n");
+ GNUNET_free (path->entries);
GNUNET_free (path);
return;
}
path->hn = hn;
- path->entries_length = i;
- path->entries = GNUNET_new_array (path->entries_length,
- struct CadetPeerPathEntry);
- for (;i>=0;i--)
- {
- struct CadetPeerPathEntry *entry = &path->entries[i];
-
- entry->peer = cpath[i];
- entry->path = path;
- GCP_path_entry_add (entry->peer,
- entry,
- i);
- }
+ /* Shrink path to actual useful length */
+ GNUNET_array_grow (path->entries,
+ path->entries_length,
+ i + 1);
LOG (GNUNET_ERROR_TYPE_DEBUG,
"Created new path %s based on information from DHT\n",
GCPP_2s (path));
/* First figure out if this path is a subset of an existing path, an
extension of an existing path, or a new path. */
cm_ctx.cpath = cpath;
+ cm_ctx.cpath_length = path_length;
cm_ctx.match = NULL;
for (int i=path_length-1;i>=0;i--)
{
GNUNET_assert (cm_ctx.match->entries_length == path_length);
return cm_ctx.match;
}
+ /* Eh, we found a match but couldn't use it? Something is wrong. */
+ GNUNET_break (0);
}
}
path = GNUNET_new (struct CadetPeerPath);
path->entries_length = path_length;
path->entries = GNUNET_new_array (path->entries_length,
- struct CadetPeerPathEntry);
+ struct CadetPeerPathEntry *);
for (int i=path_length-1;i>=0;i--)
{
- struct CadetPeerPathEntry *entry = &path->entries[i];
+ struct CadetPeerPathEntry *entry = GNUNET_new (struct CadetPeerPathEntry);
+ path->entries[i] = entry;
entry->peer = cpath[i];
entry->path = path;
+ }
+ for (int i=path_length-1;i>=0;i--)
+ {
+ struct CadetPeerPathEntry *entry = path->entries[i];
+
GCP_path_entry_add (entry->peer,
entry,
i);
}
+ recalculate_path_desirability (path);
+ LOG (GNUNET_ERROR_TYPE_DEBUG,
+ "Created new path %s to create inverse for incoming connection\n",
+ GCPP_2s (path));
path->hn = GCP_attach_path (cpath[path_length - 1],
path,
path_length - 1,
GNUNET_YES);
- LOG (GNUNET_ERROR_TYPE_DEBUG,
- "Created new path %s to create inverse for incoming connection\n",
- GCPP_2s (path));
return path;
}
unsigned int off)
{
GNUNET_assert (off < path->entries_length);
- return path->entries[off].peer;
+ return path->entries[off]->peer;
}
{
static char buf[2048];
size_t off;
- const unsigned int max_plen = sizeof(buf) / 5 - 2; /* 5 characters per entry */
+ const unsigned int max_plen = (sizeof(buf) - 16) / 5 - 2; /* 5 characters per entry */
off = 0;
for (unsigned int i = 0;
(i == max_plen / 2) )
off += GNUNET_snprintf (&buf[off],
sizeof (buf) - off,
- "... ");
+ "...-");
if ( (path->entries_length > max_plen) &&
(i > max_plen / 2) &&
(i < path->entries_length - max_plen / 2) )
continue;
off += GNUNET_snprintf (&buf[off],
sizeof (buf) - off,
- "%s ",
+ "%s%s",
GNUNET_i2s (GCP_get_id (GCPP_get_peer_at_offset (path,
- i))));
+ i))),
+ (i == path->entries_length -1) ? "" : "-");
}
+ GNUNET_snprintf (&buf[off],
+ sizeof (buf) - off,
+ "(%p)",
+ path);
return buf;
}
projects / oweals / gnunet.git / blobdiff