#include "gnunet-service-xdht_routing.h"
#include "gnunet-service-xdht.h"
+
+/**
+ * FIXME: Check if its better to store pointer to friend rather than storing
+ * peer identity next_hop or prev_hop.
+ * keep entries in destnation and source peer also. so when we send the trail
+ * teardown message then we don't know the source but if source gets the message
+ * then it shold remove that trail id from its finger table. But how does
+ * source know what is the desination finger ? It will whenevr contact a trail
+ * will do a lookup in routing table and if no trail id present the remove
+ * that trail of the finger and if only one trail then remove the finger.
+ * because of this use case of trail teardown I think trail compression
+ * and trail teardown should not be merged.
+ * 2. store a pointer to friendInfo in place o peer identity.
+ */
/**
- * Maximum number of entries in routing table.
+ * Maximum number of entries in routing table.
*/
#define ROUTING_TABLE_THRESHOLD 64
-
/**
- * FIXME: do we need to store destination and source.
- * because in trail teardown we will reach destination but it will not find any
- * entry in routing table. so we should store destination and source.
* Routing table entry .
*/
struct RoutingTrail
* Global Unique identifier of the trail.
*/
struct GNUNET_HashCode trail_id;
-
+
/**
* The peer to which this request should be passed to.
*/
- struct GNUNET_PeerIdentity next_hop;
-
+ struct GNUNET_PeerIdentity next_hop;
+
/**
- * Peer just before next hop in the trail.
+ * Peer just before next hop in the trail.
*/
- struct GNUNET_PeerIdentity prev_hop;
+ struct GNUNET_PeerIdentity prev_hop;
};
/**
* your prev. hop.
* @param trail_id
* @return #GNUNET_OK success
- * #GNUNET_SYSERR in case no matching entry found in routing table.
+ * #GNUNET_SYSERR in case no matching entry found in routing table.
*/
int
GDS_ROUTING_update_trail_prev_hop (const struct GNUNET_HashCode trail_id,
struct GNUNET_PeerIdentity prev_hop)
{
struct RoutingTrail *trail;
-
+
trail = GNUNET_CONTAINER_multihashmap_get (routing_table, &trail_id);
-
+
if (NULL == trail)
return GNUNET_SYSERR;
-
+
trail->prev_hop = prev_hop;
return GNUNET_OK;
}
+
/**
* Get the next hop for trail corresponding to trail_id
- * @param trail_id Trail id to be searched.
+ * @param trail_id Trail id to be searched.
* @return Next_hop if found
- * NULL If next hop not found.
+ * NULL If next hop not found.
*/
struct GNUNET_PeerIdentity *
GDS_ROUTING_get_next_hop (const struct GNUNET_HashCode trail_id,
enum GDS_ROUTING_trail_direction trail_direction)
{
struct RoutingTrail *trail;
-
+
trail = GNUNET_CONTAINER_multihashmap_get (routing_table, &trail_id);
-
+
if (NULL == trail)
return NULL;
-
+
switch (trail_direction)
{
case GDS_ROUTING_SRC_TO_DEST:
/**
* Remove trail with trail_id
* @param trail_id Trail id to be removed
- * @return #GNUNET_YES success
+ * @return #GNUNET_YES success
* #GNUNET_NO if entry not found.
*/
int
GDS_ROUTING_remove_trail (const struct GNUNET_HashCode remove_trail_id)
{
struct RoutingTrail *remove_entry;
-
+
remove_entry = GNUNET_CONTAINER_multihashmap_get (routing_table, &remove_trail_id);
-
+
if (NULL == remove_entry)
return GNUNET_NO;
-
+
if (GNUNET_YES == GNUNET_CONTAINER_multihashmap_remove (routing_table,
- &remove_trail_id,
+ &remove_trail_id,
remove_entry))
{
GNUNET_free (remove_entry);
return GNUNET_YES;
- }
+ }
return GNUNET_NO;
}
/**
* Iterate over routing table and remove entries with value as part of any trail.
+ *
* @param cls closure
* @param key current public key
* @param value value in the hash map
void *value)
{
struct RoutingTrail *remove_trail = cls;
- struct GNUNET_PeerIdentity *peer = value;
+ struct GNUNET_PeerIdentity *disconnected_peer = value;
- if ((0 == GNUNET_CRYPTO_cmp_peer_identity (&remove_trail->next_hop, peer)) ||
- (0 == GNUNET_CRYPTO_cmp_peer_identity (&remove_trail->prev_hop, peer)))
+ /* If disconnected_peer is next_hop, then send a trail teardown message through
+ * prev_hop in direction from destination to source. */
+ if (0 == GNUNET_CRYPTO_cmp_peer_identity (&remove_trail->next_hop,
+ disconnected_peer))
{
- GNUNET_assert (GNUNET_YES ==
+ GDS_NEIGHBOURS_send_trail_teardown (remove_trail->trail_id,
+ GDS_ROUTING_DEST_TO_SRC,
+ &remove_trail->prev_hop);
+ }
+
+ /* If disconnected_peer is prev_hop, then send a trail teardown through
+ * next_hop in direction from Source to Destination. */
+ if (0 == GNUNET_CRYPTO_cmp_peer_identity (&remove_trail->prev_hop,
+ disconnected_peer))
+ {
+ GDS_NEIGHBOURS_send_trail_teardown (remove_trail->trail_id,
+ GDS_ROUTING_SRC_TO_DEST,
+ &remove_trail->next_hop);
+ }
+
+ GNUNET_assert (GNUNET_YES ==
GNUNET_CONTAINER_multihashmap_remove (routing_table,
&remove_trail->trail_id,
remove_trail));
- GNUNET_free (remove_trail);
- }
- return GNUNET_YES;
+ GNUNET_free (remove_trail);
+ return GNUNET_YES;
}
/**
- * * FIXME: when a friend gets disconnected, then we remove the entry from routing
- * table where this friend is either a next_hop or prev_hop. But we don't communicate
- * that the trail is broken to any one who is part of trail. Should we communicate or
- * not. And if not then the cases where trail setup fails because next_hop = NULL
- * or something like that. VERY URGENT.
- * Remove every trail where peer is either next_hop or prev_hop
- * @param peer Peer to be searched.
+ * Remove every trail where peer is either next_hop or prev_hop. Also send a
+ * trail teardown message in direction of hop which is not disconnected.
+ * @param peer Peer identity. Trail containing this peer should be removed.
*/
void
GDS_ROUTING_remove_trail_by_peer (const struct GNUNET_PeerIdentity *peer)
{
GNUNET_CONTAINER_multihashmap_iterate (routing_table, &remove_matching_trails,
- (void *)peer);
+ (void *)peer);
}
* but with different prev_hop/next_hop
*/
int
-GDS_ROUTING_add (struct GNUNET_HashCode new_trail_id,
- struct GNUNET_PeerIdentity *prev_hop,
- const struct GNUNET_PeerIdentity *next_hop)
+GDS_ROUTING_add (struct GNUNET_HashCode new_trail_id,
+ struct GNUNET_PeerIdentity prev_hop,
+ struct GNUNET_PeerIdentity next_hop)
{
struct RoutingTrail *new_entry;
-
+
new_entry = GNUNET_malloc (sizeof (struct RoutingTrail));
new_entry->trail_id = new_trail_id;
- new_entry->next_hop = *next_hop;
- new_entry->prev_hop = *prev_hop;
- return GNUNET_CONTAINER_multihashmap_put (routing_table,
+ new_entry->next_hop = next_hop;
+ new_entry->prev_hop = prev_hop;
+ return GNUNET_CONTAINER_multihashmap_put (routing_table,
&new_trail_id, new_entry,
GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY);
}
/**
- * Check if the size of routing table has crossed threshold.
+ * Check if the size of routing table has crossed threshold.
* @return #GNUNET_YES, if threshold crossed else #GNUNET_NO.
*/
int
GDS_ROUTING_threshold_reached (void)
{
- return (GNUNET_CONTAINER_multihashmap_size(routing_table) >
- ROUTING_TABLE_THRESHOLD) ? GNUNET_YES:GNUNET_NO;
+ return (GNUNET_CONTAINER_multihashmap_size(routing_table) >
+ ROUTING_TABLE_THRESHOLD) ? GNUNET_YES:GNUNET_NO;
}
*/
void
GDS_ROUTING_init (void)
-{
+{
routing_table = GNUNET_CONTAINER_multihashmap_create (ROUTING_TABLE_THRESHOLD * 4 / 3,
GNUNET_NO);
}
GNUNET_CONTAINER_multihashmap_destroy (routing_table);
}
-/* end of gnunet-service-xdht_routing.c */
\ No newline at end of file
+/* end of gnunet-service-xdht_routing.c */