net: add rte prefix to ARP defines
authorOlivier Matz <olivier.matz@6wind.com>
Tue, 21 May 2019 16:13:02 +0000 (18:13 +0200)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 24 May 2019 11:34:44 +0000 (13:34 +0200)
Add 'RTE_' prefix to defines:
- rename ARP_HRD_ETHER as RTE_ARP_HRD_ETHER.
- rename ARP_OP_REQUEST as RTE_ARP_OP_REQUEST.
- rename ARP_OP_REPLY as RTE_ARP_OP_REPLY.
- rename ARP_OP_REVREQUEST as RTE_ARP_OP_REVREQUEST.
- rename ARP_OP_REVREPLY as RTE_ARP_OP_REVREPLY.
- rename ARP_OP_INVREQUEST as RTE_ARP_OP_INVREQUEST.
- rename ARP_OP_INVREPLY as RTE_ARP_OP_INVREPLY.

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Reviewed-by: Stephen Hemminger <stephen@networkplumber.org>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
app/test-pmd/icmpecho.c
app/test/packet_burst_generator.c
app/test/test_link_bonding.c
drivers/net/bonding/rte_eth_bond_alb.c
drivers/net/bonding/rte_eth_bond_pmd.c
examples/bond/main.c
lib/librte_net/rte_arp.c
lib/librte_net/rte_arp.h

index 0f92d14..b0767f2 100644 (file)
@@ -39,17 +39,17 @@ static const char *
 arp_op_name(uint16_t arp_op)
 {
        switch (arp_op) {
-       case ARP_OP_REQUEST:
+       case RTE_ARP_OP_REQUEST:
                return "ARP Request";
-       case ARP_OP_REPLY:
+       case RTE_ARP_OP_REPLY:
                return "ARP Reply";
-       case ARP_OP_REVREQUEST:
+       case RTE_ARP_OP_REVREQUEST:
                return "Reverse ARP Request";
-       case ARP_OP_REVREPLY:
+       case RTE_ARP_OP_REVREPLY:
                return "Reverse ARP Reply";
-       case ARP_OP_INVREQUEST:
+       case RTE_ARP_OP_INVREQUEST:
                return "Peer Identify Request";
-       case ARP_OP_INVREPLY:
+       case RTE_ARP_OP_INVREPLY:
                return "Peer Identify Reply";
        default:
                break;
@@ -359,7 +359,7 @@ reply_to_icmp_echo_rqsts(struct fwd_stream *fs)
                                       arp_op_name(arp_op));
                        }
                        if ((RTE_BE_TO_CPU_16(arp_h->arp_hardware) !=
-                            ARP_HRD_ETHER) ||
+                            RTE_ARP_HRD_ETHER) ||
                            (arp_pro != ETHER_TYPE_IPv4) ||
                            (arp_h->arp_hlen != 6) ||
                            (arp_h->arp_plen != 4)
@@ -381,7 +381,7 @@ reply_to_icmp_echo_rqsts(struct fwd_stream *fs)
                                ipv4_addr_dump(" tip=", ip_addr);
                                printf("\n");
                        }
-                       if (arp_op != ARP_OP_REQUEST) {
+                       if (arp_op != RTE_ARP_OP_REQUEST) {
                                rte_pktmbuf_free(pkt);
                                continue;
                        }
@@ -396,7 +396,7 @@ reply_to_icmp_echo_rqsts(struct fwd_stream *fs)
                        ether_addr_copy(&ports[fs->tx_port].eth_addr,
                                        &eth_h->s_addr);
 
-                       arp_h->arp_opcode = rte_cpu_to_be_16(ARP_OP_REPLY);
+                       arp_h->arp_opcode = rte_cpu_to_be_16(RTE_ARP_OP_REPLY);
                        ether_addr_copy(&arp_h->arp_data.arp_tha, &eth_addr);
                        ether_addr_copy(&arp_h->arp_data.arp_sha, &arp_h->arp_data.arp_tha);
                        ether_addr_copy(&eth_h->s_addr, &arp_h->arp_data.arp_sha);
index 0b12058..ccc0bd5 100644 (file)
@@ -78,7 +78,7 @@ initialize_arp_header(struct rte_arp_hdr *arp_hdr, struct ether_addr *src_mac,
                struct ether_addr *dst_mac, uint32_t src_ip, uint32_t dst_ip,
                uint32_t opcode)
 {
-       arp_hdr->arp_hardware = rte_cpu_to_be_16(ARP_HRD_ETHER);
+       arp_hdr->arp_hardware = rte_cpu_to_be_16(RTE_ARP_HRD_ETHER);
        arp_hdr->arp_protocol = rte_cpu_to_be_16(ETHER_TYPE_IPv4);
        arp_hdr->arp_hlen = ETHER_ADDR_LEN;
        arp_hdr->arp_plen = sizeof(uint32_t);
index 6ff0f5b..451c693 100644 (file)
@@ -4507,7 +4507,7 @@ test_alb_change_mac_in_reply_sent(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &bond_mac, &client_mac, ip_host, ip_client1,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        rte_eth_tx_burst(test_params->bonded_port_id, 0, &pkt, 1);
 
        pkt = rte_pktmbuf_alloc(test_params->mbuf_pool);
@@ -4518,7 +4518,7 @@ test_alb_change_mac_in_reply_sent(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &bond_mac, &client_mac, ip_host, ip_client2,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        rte_eth_tx_burst(test_params->bonded_port_id, 0, &pkt, 1);
 
        pkt = rte_pktmbuf_alloc(test_params->mbuf_pool);
@@ -4529,7 +4529,7 @@ test_alb_change_mac_in_reply_sent(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &bond_mac, &client_mac, ip_host, ip_client3,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        rte_eth_tx_burst(test_params->bonded_port_id, 0, &pkt, 1);
 
        pkt = rte_pktmbuf_alloc(test_params->mbuf_pool);
@@ -4540,7 +4540,7 @@ test_alb_change_mac_in_reply_sent(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &bond_mac, &client_mac, ip_host, ip_client4,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        rte_eth_tx_burst(test_params->bonded_port_id, 0, &pkt, 1);
 
        slave_mac1 =
@@ -4625,7 +4625,7 @@ test_alb_reply_from_client(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &client_mac, &bond_mac, ip_client1, ip_host,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        virtual_ethdev_add_mbufs_to_rx_queue(test_params->slave_port_ids[0], &pkt,
                        1);
 
@@ -4637,7 +4637,7 @@ test_alb_reply_from_client(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &client_mac, &bond_mac, ip_client2, ip_host,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        virtual_ethdev_add_mbufs_to_rx_queue(test_params->slave_port_ids[0], &pkt,
                        1);
 
@@ -4649,7 +4649,7 @@ test_alb_reply_from_client(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &client_mac, &bond_mac, ip_client3, ip_host,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        virtual_ethdev_add_mbufs_to_rx_queue(test_params->slave_port_ids[0], &pkt,
                        1);
 
@@ -4661,7 +4661,7 @@ test_alb_reply_from_client(void)
        arp_pkt = (struct rte_arp_hdr *)((char *)eth_pkt +
                                        sizeof(struct ether_hdr));
        initialize_arp_header(arp_pkt, &client_mac, &bond_mac, ip_client4, ip_host,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        virtual_ethdev_add_mbufs_to_rx_queue(test_params->slave_port_ids[0], &pkt,
                        1);
 
@@ -4761,7 +4761,7 @@ test_alb_receive_vlan_reply(void)
        vlan_pkt->eth_proto = rte_cpu_to_be_16(ETHER_TYPE_ARP);
        arp_pkt = (struct rte_arp_hdr *)((char *)(vlan_pkt + 1));
        initialize_arp_header(arp_pkt, &client_mac, &bond_mac, ip_client1, ip_host,
-                       ARP_OP_REPLY);
+                       RTE_ARP_OP_REPLY);
        virtual_ethdev_add_mbufs_to_rx_queue(test_params->slave_port_ids[0], &pkt,
                        1);
 
index 7032239..f16ae8f 100644 (file)
@@ -84,7 +84,7 @@ void bond_mode_alb_arp_recv(struct ether_hdr *eth_h, uint16_t offset,
        arp = (struct rte_arp_hdr *)((char *)(eth_h + 1) + offset);
 
        /* ARP Requests are forwarded to the application with no changes */
-       if (arp->arp_opcode != rte_cpu_to_be_16(ARP_OP_REPLY))
+       if (arp->arp_opcode != rte_cpu_to_be_16(RTE_ARP_OP_REPLY))
                return;
 
        /* From now on, we analyze only ARP Reply packets */
@@ -150,7 +150,7 @@ bond_mode_alb_arp_xmit(struct ether_hdr *eth_h, uint16_t offset,
        client_info = &hash_table[hash_index];
 
        rte_spinlock_lock(&internals->mode6.lock);
-       if (arp->arp_opcode == rte_cpu_to_be_16(ARP_OP_REPLY)) {
+       if (arp->arp_opcode == rte_cpu_to_be_16(RTE_ARP_OP_REPLY)) {
                if (client_info->in_use) {
                        if (client_info->app_ip == arp->arp_data.arp_sip &&
                                client_info->cli_ip == arp->arp_data.arp_tip) {
@@ -220,11 +220,11 @@ bond_mode_alb_arp_upd(struct client_data *client_info,
        ether_addr_copy(&client_info->cli_mac, &arp_h->arp_data.arp_tha);
        arp_h->arp_data.arp_tip = client_info->cli_ip;
 
-       arp_h->arp_hardware = rte_cpu_to_be_16(ARP_HRD_ETHER);
+       arp_h->arp_hardware = rte_cpu_to_be_16(RTE_ARP_HRD_ETHER);
        arp_h->arp_protocol = rte_cpu_to_be_16(ETHER_TYPE_IPv4);
        arp_h->arp_hlen = ETHER_ADDR_LEN;
        arp_h->arp_plen = sizeof(uint32_t);
-       arp_h->arp_opcode = rte_cpu_to_be_16(ARP_OP_REPLY);
+       arp_h->arp_opcode = rte_cpu_to_be_16(RTE_ARP_OP_REPLY);
 
        slave_idx = client_info->slave_idx;
        rte_spinlock_unlock(&internals->mode6.lock);
index 2833508..f04719e 100644 (file)
@@ -492,22 +492,22 @@ static void
 arp_op_name(uint16_t arp_op, char *buf, size_t buf_len)
 {
        switch (arp_op) {
-       case ARP_OP_REQUEST:
+       case RTE_ARP_OP_REQUEST:
                strlcpy(buf, "ARP Request", buf_len);
                return;
-       case ARP_OP_REPLY:
+       case RTE_ARP_OP_REPLY:
                strlcpy(buf, "ARP Reply", buf_len);
                return;
-       case ARP_OP_REVREQUEST:
+       case RTE_ARP_OP_REVREQUEST:
                strlcpy(buf, "Reverse ARP Request", buf_len);
                return;
-       case ARP_OP_REVREPLY:
+       case RTE_ARP_OP_REVREPLY:
                strlcpy(buf, "Reverse ARP Reply", buf_len);
                return;
-       case ARP_OP_INVREQUEST:
+       case RTE_ARP_OP_INVREQUEST:
                strlcpy(buf, "Peer Identify Request", buf_len);
                return;
-       case ARP_OP_INVREPLY:
+       case RTE_ARP_OP_INVREPLY:
                strlcpy(buf, "Peer Identify Reply", buf_len);
                return;
        default:
index 2447964..21a67e5 100644 (file)
@@ -385,8 +385,8 @@ static int lcore_main(__attribute__((unused)) void *arg1)
                                arp_hdr = (struct rte_arp_hdr *)(
                                        (char *)(eth_hdr + 1) + offset);
                                if (arp_hdr->arp_data.arp_tip == bond_ip) {
-                                       if (arp_hdr->arp_opcode == rte_cpu_to_be_16(ARP_OP_REQUEST)) {
-                                               arp_hdr->arp_opcode = rte_cpu_to_be_16(ARP_OP_REPLY);
+                                       if (arp_hdr->arp_opcode == rte_cpu_to_be_16(RTE_ARP_OP_REQUEST)) {
+                                               arp_hdr->arp_opcode = rte_cpu_to_be_16(RTE_ARP_OP_REPLY);
                                                /* Switch src and dst data and set bonding MAC */
                                                ether_addr_copy(&eth_hdr->s_addr, &eth_hdr->d_addr);
                                                rte_eth_macaddr_get(BOND_PORT, &eth_hdr->s_addr);
@@ -481,11 +481,11 @@ static void cmd_obj_send_parsed(void *parsed_result,
 
        arp_hdr = (struct rte_arp_hdr *)(
                (char *)eth_hdr + sizeof(struct ether_hdr));
-       arp_hdr->arp_hardware = rte_cpu_to_be_16(ARP_HRD_ETHER);
+       arp_hdr->arp_hardware = rte_cpu_to_be_16(RTE_ARP_HRD_ETHER);
        arp_hdr->arp_protocol = rte_cpu_to_be_16(ETHER_TYPE_IPv4);
        arp_hdr->arp_hlen = ETHER_ADDR_LEN;
        arp_hdr->arp_plen = sizeof(uint32_t);
-       arp_hdr->arp_opcode = rte_cpu_to_be_16(ARP_OP_REQUEST);
+       arp_hdr->arp_opcode = rte_cpu_to_be_16(RTE_ARP_OP_REQUEST);
 
        rte_eth_macaddr_get(BOND_PORT, &arp_hdr->arp_data.arp_sha);
        arp_hdr->arp_data.arp_sip = bond_ip;
index c80ebc7..877874a 100644 (file)
@@ -35,11 +35,11 @@ rte_net_make_rarp_packet(struct rte_mempool *mpool,
 
        /* RARP header. */
        rarp = (struct rte_arp_hdr *)(eth_hdr + 1);
-       rarp->arp_hardware = htons(ARP_HRD_ETHER);
+       rarp->arp_hardware = htons(RTE_ARP_HRD_ETHER);
        rarp->arp_protocol = htons(ETHER_TYPE_IPv4);
        rarp->arp_hlen = ETHER_ADDR_LEN;
        rarp->arp_plen = 4;
-       rarp->arp_opcode  = htons(ARP_OP_REVREQUEST);
+       rarp->arp_opcode  = htons(RTE_ARP_OP_REVREQUEST);
 
        ether_addr_copy(mac, &rarp->arp_data.arp_sha);
        ether_addr_copy(mac, &rarp->arp_data.arp_tha);
index c9b5cd4..a94fa6a 100644 (file)
@@ -33,18 +33,18 @@ struct rte_arp_ipv4 {
  */
 struct rte_arp_hdr {
        uint16_t arp_hardware;    /* format of hardware address */
-#define ARP_HRD_ETHER     1  /* ARP Ethernet address format */
+#define RTE_ARP_HRD_ETHER     1  /* ARP Ethernet address format */
 
        uint16_t arp_protocol;    /* format of protocol address */
        uint8_t  arp_hlen;    /* length of hardware address */
        uint8_t  arp_plen;    /* length of protocol address */
        uint16_t arp_opcode;     /* ARP opcode (command) */
-#define        ARP_OP_REQUEST    1 /* request to resolve address */
-#define        ARP_OP_REPLY      2 /* response to previous request */
-#define        ARP_OP_REVREQUEST 3 /* request proto addr given hardware */
-#define        ARP_OP_REVREPLY   4 /* response giving protocol address */
-#define        ARP_OP_INVREQUEST 8 /* request to identify peer */
-#define        ARP_OP_INVREPLY   9 /* response identifying peer */
+#define        RTE_ARP_OP_REQUEST    1 /* request to resolve address */
+#define        RTE_ARP_OP_REPLY      2 /* response to previous request */
+#define        RTE_ARP_OP_REVREQUEST 3 /* request proto addr given hardware */
+#define        RTE_ARP_OP_REVREPLY   4 /* response giving protocol address */
+#define        RTE_ARP_OP_INVREQUEST 8 /* request to identify peer */
+#define        RTE_ARP_OP_INVREPLY   9 /* response identifying peer */
 
        struct rte_arp_ipv4 arp_data;
 } __attribute__((__packed__));