examples/l3fwd-acl: update MAC addresses

Introduces two changes into l3fwd-acl behaviour to make
it behave in the same way as l3fwd:
- Add a command-line parameter to allow the user to specify the
  destination mac address for each ethernet port used.
- While forwarding the packet update source and destination mac
  addresses.

Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
This commit is contained in:
Konstantin Ananyev 2020-10-16 14:12:42 +01:00 committed by David Marchand
parent 3d98f921fb
commit 20f76bb666
3 changed files with 71 additions and 6 deletions

View File

@ -330,6 +330,12 @@ New Features
* Added scatter gather support.
* Added NIST GCMVS complaint GMAC test method support.
* **Updated l3wfd-acl sample application.**
* Added new optional parameter ``--eth-dest`` for the ``l3fwd-acl`` to allow
the user to specify the destination mac address for each ethernet port
used.
Removed Items
-------------

View File

@ -236,7 +236,7 @@ The application has a number of command line options:
.. code-block:: console
./build/l3fwd-acl [EAL options] -- -p PORTMASK [-P] --config(port,queue,lcore)[,(port,queue,lcore)] --rule_ipv4 FILENAME rule_ipv6 FILENAME [--scalar] [--enable-jumbo [--max-pkt-len PKTLEN]] [--no-numa]
./build/l3fwd-acl [EAL options] -- -p PORTMASK [-P] --config(port,queue,lcore)[,(port,queue,lcore)] --rule_ipv4 FILENAME rule_ipv6 FILENAME [--scalar] [--enable-jumbo [--max-pkt-len PKTLEN]] [--no-numa] [--eth-dest=X,MM:MM:MM:MM:MM:MM]
where,
@ -260,6 +260,8 @@ where,
* --no-numa: optional, disables numa awareness
* --eth-dest=X,MM:MM:MM:MM:MM:MM: optional, ethernet destination for port X
For example, consider a dual processor socket platform with 8 physical cores, where cores 0-7 and 16-23 appear on socket 0,
while cores 8-15 and 24-31 appear on socket 1.

View File

@ -39,6 +39,9 @@
#include <rte_string_fns.h>
#include <rte_acl.h>
#include <cmdline_parse.h>
#include <cmdline_parse_etheraddr.h>
#if RTE_LOG_DP_LEVEL >= RTE_LOG_DEBUG
#define L3FWDACL_DEBUG
#endif
@ -81,9 +84,6 @@
static uint16_t nb_rxd = RTE_TEST_RX_DESC_DEFAULT;
static uint16_t nb_txd = RTE_TEST_TX_DESC_DEFAULT;
/* ethernet addresses of ports */
static struct rte_ether_addr ports_eth_addr[RTE_MAX_ETHPORTS];
/* mask of enabled ports */
static uint32_t enabled_port_mask;
static int promiscuous_on; /**< Ports set in promiscuous mode off by default. */
@ -143,6 +143,9 @@ static struct rte_eth_conf port_conf = {
static struct rte_mempool *pktmbuf_pool[NB_SOCKETS];
/* ethernet addresses of ports */
static struct rte_ether_hdr port_l2hdr[RTE_MAX_ETHPORTS];
/***********************start of ACL part******************************/
#ifdef DO_RFC_1812_CHECKS
static inline int
@ -164,6 +167,7 @@ send_single_packet(struct rte_mbuf *m, uint16_t port);
#define OPTION_RULE_IPV4 "rule_ipv4"
#define OPTION_RULE_IPV6 "rule_ipv6"
#define OPTION_SCALAR "scalar"
#define OPTION_ETH_DEST "eth-dest"
#define ACL_DENY_SIGNATURE 0xf0000000
#define RTE_LOGTYPE_L3FWDACL RTE_LOGTYPE_USER3
#define acl_log(format, ...) RTE_LOG(ERR, L3FWDACL, format, ##__VA_ARGS__)
@ -1275,9 +1279,14 @@ send_single_packet(struct rte_mbuf *m, uint16_t port)
{
uint32_t lcore_id;
struct lcore_conf *qconf;
struct rte_ether_hdr *eh;
lcore_id = rte_lcore_id();
/* update src and dst mac*/
eh = rte_pktmbuf_mtod(m, struct rte_ether_hdr *);
memcpy(eh, &port_l2hdr[port], sizeof(eh->d_addr) + sizeof(eh->s_addr));
qconf = &lcore_conf[lcore_id];
rte_eth_tx_buffer(port, qconf->tx_queue_id[port],
qconf->tx_buffer[port], m);
@ -1627,6 +1636,26 @@ parse_config(const char *q_arg)
return 0;
}
static const char *
parse_eth_dest(const char *optarg)
{
unsigned long portid;
char *port_end;
errno = 0;
portid = strtoul(optarg, &port_end, 0);
if (errno != 0 || port_end == optarg || *port_end++ != ',')
return "Invalid format";
else if (portid >= RTE_MAX_ETHPORTS)
return "port value exceeds RTE_MAX_ETHPORTS("
RTE_STR(RTE_MAX_ETHPORTS) ")";
if (cmdline_parse_etheraddr(NULL, port_end, &port_l2hdr[portid].d_addr,
sizeof(port_l2hdr[portid].d_addr)) < 0)
return "Invalid ethernet address";
return NULL;
}
/* Parse the argument given in the command line of the application */
static int
parse_args(int argc, char **argv)
@ -1642,6 +1671,7 @@ parse_args(int argc, char **argv)
{OPTION_RULE_IPV4, 1, 0, 0},
{OPTION_RULE_IPV6, 1, 0, 0},
{OPTION_SCALAR, 0, 0, 0},
{OPTION_ETH_DEST, 1, 0, 0},
{NULL, 0, 0, 0}
};
@ -1737,6 +1767,16 @@ parse_args(int argc, char **argv)
OPTION_SCALAR, sizeof(OPTION_SCALAR)))
parm_config.scalar = 1;
if (!strncmp(lgopts[option_index].name, OPTION_ETH_DEST,
sizeof(OPTION_ETH_DEST))) {
const char *serr = parse_eth_dest(optarg);
if (serr != NULL) {
printf("invalid %s value:\"%s\": %s\n",
OPTION_ETH_DEST, optarg, serr);
print_usage(prgname);
return -1;
}
}
break;
@ -1862,6 +1902,20 @@ check_all_ports_link_status(uint32_t port_mask)
}
}
/*
* build-up default vaues for dest MACs.
*/
static void
set_default_dest_mac(void)
{
uint32_t i;
for (i = 0; i != RTE_DIM(port_l2hdr); i++) {
port_l2hdr[i].d_addr.addr_bytes[0] = RTE_ETHER_LOCAL_ADMIN_ADDR;
port_l2hdr[i].d_addr.addr_bytes[5] = i;
}
}
int
main(int argc, char **argv)
{
@ -1883,6 +1937,8 @@ main(int argc, char **argv)
argc -= ret;
argv += ret;
set_default_dest_mac();
/* parse application arguments (after the EAL ones) */
ret = parse_args(argc, argv);
if (ret < 0)
@ -1962,13 +2018,14 @@ main(int argc, char **argv)
"rte_eth_dev_adjust_nb_rx_tx_desc: err=%d, port=%d\n",
ret, portid);
ret = rte_eth_macaddr_get(portid, &ports_eth_addr[portid]);
ret = rte_eth_macaddr_get(portid, &port_l2hdr[portid].s_addr);
if (ret < 0)
rte_exit(EXIT_FAILURE,
"rte_eth_macaddr_get: err=%d, port=%d\n",
ret, portid);
print_ethaddr(" Address:", &ports_eth_addr[portid]);
print_ethaddr("Dst MAC:", &port_l2hdr[portid].d_addr);
print_ethaddr(", Src MAC:", &port_l2hdr[portid].s_addr);
printf(", ");
/* init memory */