X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=blobdiff_plain;f=VNFs%2FDPPD-PROX%2Fhandle_swap.c;h=63c4dbde76976d9e3024968ffde187b831d622cd;hb=974e4b3123b7d926d548d8924b41e8b732e2732f;hp=516d3f637ff063eca803adf6750525298552333a;hpb=516aad7ecf88e968940d384a1f56dca4e42a279b;p=samplevnf.git

diff --git a/VNFs/DPPD-PROX/handle_swap.c b/VNFs/DPPD-PROX/handle_swap.c
index 516d3f63..63c4dbde 100644
--- a/VNFs/DPPD-PROX/handle_swap.c
+++ b/VNFs/DPPD-PROX/handle_swap.c
@@ -203,17 +203,18 @@ static void init_task_swap(struct task_base *tbase, struct task_args *targ)
 	struct ether_addr *src_addr, *dst_addr;
 
 	/*
-	 * Destination MAC can come from
-	 *    - pre-configured mac in case 'dst mac=xx:xx:xx:xx:xx:xx' in config file
-	 *    - src mac from the packet in case 'dst mac=packet' in config file
-	 *    - not written in case 'dst mac=no' in config file
-	 *    - (default - no 'dst mac') src mac from the packet
-	 * Source MAC can come from
-	 *    - pre-configured mac in case 'src mac=xx:xx:xx:xx:xx:xx' in config file
-	 *    - dst mac from the packet in case 'src mac=packet' in config file
-	 *    - not written in case 'src mac=no' in config file
-	 *    - (default - no 'src mac') if (tx_port) port mac
-	 *    - (default - no 'src mac') if (no tx_port) dst mac from the packet
+	 * The destination MAC of the outgoing packet is based on the config file:
+	 *    - 'dst mac=xx:xx:xx:xx:xx:xx' => the pre-configured mac will be used as dst mac
+	 *    - 'dst mac=packet'            => the src mac of the incoming packet is used as dst mac
+	 *    - (default - no 'dst mac')    => the src mac from the incoming packet is used as dst mac
+	 *
+	 * The source MAC of the outgoing packet is based on the config file:
+	 *    - 'src mac=xx:xx:xx:xx:xx:xx' => the pre-configured mac will be used as src mac
+	 *    - 'src mac=packet'            => the dst mac of the incoming packet is used as src mac
+	 *    - 'src mac=hw'                => the mac address of the tx port is used as src mac
+	 *                                     An error is returned if there are no physical tx ports
+	 *    - (default - no 'src mac')    => if there is physical tx port, the mac of that port is used as src mac
+	 *    - (default - no 'src mac')       if there are no physical tx ports the dst mac of the incoming packet
 	 */
 
 	if (targ->flags & TASK_ARG_DST_MAC_SET) {
@@ -221,14 +222,19 @@ static void init_task_swap(struct task_base *tbase, struct task_args *targ)
 		memcpy(&task->src_dst_mac[0], dst_addr, sizeof(*src_addr));
 	}
 
+	PROX_PANIC(targ->flags & TASK_ARG_DO_NOT_SET_SRC_MAC, "src mac must be set in swap mode, by definition => src mac=no is not supported\n");
+	PROX_PANIC(targ->flags & TASK_ARG_DO_NOT_SET_DST_MAC, "dst mac must be set in swap mode, by definition => dst mac=no is not supported\n");
+
 	if (targ->flags & TASK_ARG_SRC_MAC_SET) {
 		src_addr =  &targ->esaddr;
 		memcpy(&task->src_dst_mac[6], src_addr, sizeof(*dst_addr));
 		plog_info("\t\tCore %d: src mac set from config file\n", targ->lconf->id);
-	} else if (targ->nb_txports) {
-		src_addr = &prox_port_cfg[task->base.tx_params_hw.tx_port_queue[0].port].eth_addr;
-		memcpy(&task->src_dst_mac[6], src_addr, sizeof(*dst_addr));
-		if (targ->flags & TASK_ARG_HW_SRC_MAC){
+	} else {
+		if (targ->flags & TASK_ARG_HW_SRC_MAC)
+			PROX_PANIC(targ->nb_txports == 0, "src mac set to hw but no tx port\n");
+		if (targ->nb_txports) {
+			src_addr = &prox_port_cfg[task->base.tx_params_hw.tx_port_queue[0].port].eth_addr;
+			memcpy(&task->src_dst_mac[6], src_addr, sizeof(*dst_addr));
 			targ->flags |= TASK_ARG_SRC_MAC_SET;
 			plog_info("\t\tCore %d: src mac set from port\n", targ->lconf->id);
 		}
@@ -240,9 +246,8 @@ static struct task_init task_init_swap = {
 	.mode_str = "swap",
 	.init = init_task_swap,
 	.handle = handle_swap_bulk,
-	.flag_features = TASK_FEATURE_TXQ_FLAGS_NOOFFLOADS|TASK_FEATURE_TXQ_FLAGS_NOMULTSEGS,
+	.flag_features = TASK_FEATURE_TXQ_FLAGS_NOOFFLOADS,
 	.size = sizeof(struct task_swap),
-	.mbuf_size = 2048 + sizeof(struct rte_mbuf) + RTE_PKTMBUF_HEADROOM,
 };
 
 static struct task_init task_init_swap_arp = {
@@ -250,9 +255,8 @@ static struct task_init task_init_swap_arp = {
 	.sub_mode_str = "l3",
 	.init = init_task_swap,
 	.handle = handle_swap_bulk,
-	.flag_features = TASK_FEATURE_TXQ_FLAGS_NOOFFLOADS|TASK_FEATURE_TXQ_FLAGS_NOMULTSEGS,
+	.flag_features = TASK_FEATURE_TXQ_FLAGS_NOOFFLOADS,
 	.size = sizeof(struct task_swap),
-	.mbuf_size = 2048 + sizeof(struct rte_mbuf) + RTE_PKTMBUF_HEADROOM,
 };
 
 __attribute__((constructor)) static void reg_task_swap(void)