From 4d2e8310cbeacf1c20ec0adb3dc6cf06abcdcbb8 Mon Sep 17 00:00:00 2001 From: Michele Papalini Date: Mon, 16 Jan 2023 12:00:38 +0100 Subject: feat: forward local interest if PIT entry exists in case of local remote strategy Ref: HICN-823 Signed-off-by: Michele Papalini Change-Id: I9e4b5da6ae1011e29e128d7811e29b3020a4a3cc (cherry picked from commit f3a58a1ea5f4b99b5e788aec1a5afdcb19c42074) --- hicn-light/src/hicn/core/msgbuf.h | 5 +++- hicn-light/src/hicn/core/packet_cache.c | 44 +++++++++++++++++++++++++++++++++ 2 files changed, 48 insertions(+), 1 deletion(-) diff --git a/hicn-light/src/hicn/core/msgbuf.h b/hicn-light/src/hicn/core/msgbuf.h index ee9337225..ef2b38c51 100644 --- a/hicn-light/src/hicn/core/msgbuf.h +++ b/hicn-light/src/hicn/core/msgbuf.h @@ -37,7 +37,10 @@ typedef struct { unsigned connection_id; // ingress Ticks recv_ts; // timestamp unsigned refs; // refcount - unsigned path_label; // XXX what is this ? + unsigned path_label; // original path label of the received message. used + // as a base for the path label computation when the + // same data packet needs to be forwarded on multiple + // face. // XXX Cache storage union { diff --git a/hicn-light/src/hicn/core/packet_cache.c b/hicn-light/src/hicn/core/packet_cache.c index 7fa18b48f..937b0480b 100644 --- a/hicn-light/src/hicn/core/packet_cache.c +++ b/hicn-light/src/hicn/core/packet_cache.c @@ -68,6 +68,7 @@ * */ +#include #include "packet_cache.h" const char *_pkt_cache_verdict_str[] = { @@ -565,6 +566,49 @@ bool pkt_cache_try_aggregate_in_pit(pkt_cache_t *pkt_cache, // in the PIT entry (i.e. it is a retransmission) unsigned connection_id = msgbuf_get_connection_id(msgbuf); bool is_aggregated = !pit_entry_ingress_contains(pit_entry, connection_id); + + // If the strategy to use for this packet is STRATEGY_TYPE_LOCAL_REMOTE we may + // want to forward the packet even if it should be aggregated. This is useful + // when a local consumer socket and a remote one are asking for the same + // content. If the interest coming from the remote socket is received before + // the one from the local socket, the second interest is aggregated in the PIT + // and no interest is satisfied (IRIS app). + // Forward the interest if: 1) the strategy in the fib_entry stored in the pit + // entry is STRATEGY_TYPE_LOCAL_REMOTE 2) the ingress connection of the + // interest if a local connection 3) the ingress list of pit entry has no + // other local ingress connection if all these conditions are satisfied, send + // the interest. if is_aggregated = false avoid the check (it will be useless) + + if (is_aggregated) { + fib_entry_t *fib_entry = pit_entry_get_fib_entry(pit_entry); + if (fib_entry && + (fib_entry_strategy_type(fib_entry) == STRATEGY_TYPE_LOCAL_REMOTE)) { + // the strategy is STRATEGY_TYPE_LOCAL_REMOTE, check the input face + connection_table_t *table = + forwarder_get_connection_table(fib_entry->forwarder); + if (table) { + connection_t *msg_conn = + connection_table_get_by_id(table, connection_id); + if (msg_conn && connection_is_local(msg_conn)) { + // The face is local, check that no other input face is local. With + // this check only the first local consumer is able to send + // the interest, while the other interests will be aggregated + // (however this is quite inefficient) + nexthops_t *ingressIdSet = pit_entry_get_ingress(pit_entry); + bool in_local_connextion_exists = false; + nexthops_enumerate(ingressIdSet, i, nexthop, { + connection_t *in_conn = connection_table_get_by_id(table, nexthop); + if (in_conn && connection_is_local(in_conn)) { + in_local_connextion_exists = true; + break; + } + }); + if (!in_local_connextion_exists) is_aggregated = false; + } + } + } + } + if (is_aggregated) pit_entry_ingress_add(pit_entry, connection_id); WITH_DEBUG({ -- cgit 1.2.3-korg