From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 00A07A04BC; Sun, 27 Sep 2020 06:56:13 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 8A90B1D92E; Sun, 27 Sep 2020 06:56:11 +0200 (CEST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id F07481D92C for ; Sun, 27 Sep 2020 06:56:09 +0200 (CEST) IronPort-SDR: Oc59gi9jECfHulibHiCkIxwn0HiHmYEDzUB5gvLNVoJyMZXON0LbQidOGlSa5AC3NI3i5Vzvcl L5SmjCPIJRLw== X-IronPort-AV: E=McAfee;i="6000,8403,9756"; a="179985495" X-IronPort-AV: E=Sophos;i="5.77,308,1596524400"; d="scan'208";a="179985495" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Sep 2020 21:56:06 -0700 IronPort-SDR: 3hgwvV6qJ+qyejFWwaowNqnc7j3Uk9dshxmbSF1x6volfV5tdXjzxhUFSpGXtFY7pgnEIobgxc 2A42QZXnK6eA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,308,1596524400"; d="scan'208";a="349439082" Received: from fmsmsx606.amr.corp.intel.com ([10.18.126.86]) by FMSMGA003.fm.intel.com with ESMTP; 26 Sep 2020 21:56:06 -0700 Received: from shsmsx605.ccr.corp.intel.com (10.109.6.215) by fmsmsx606.amr.corp.intel.com (10.18.126.86) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Sat, 26 Sep 2020 21:56:05 -0700 Received: from shsmsx606.ccr.corp.intel.com (10.109.6.216) by SHSMSX605.ccr.corp.intel.com (10.109.6.215) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Sun, 27 Sep 2020 12:56:04 +0800 Received: from shsmsx606.ccr.corp.intel.com ([10.109.6.216]) by SHSMSX606.ccr.corp.intel.com ([10.109.6.216]) with mapi id 15.01.1713.004; Sun, 27 Sep 2020 12:56:04 +0800 From: "Hu, Jiayu" To: "yang_y_yi@163.com" , "dev@dpdk.org" CC: "thomas@monjalon.net" , "yangyi01@inspur.com" Thread-Topic: [PATCH v7 1/3] gro: add UDP/IPv4 GRO support Thread-Index: AQHWklDKyDG6tt1fTUime6RyzvDlsql77/9Q Date: Sun, 27 Sep 2020 04:56:03 +0000 Message-ID: <127869031ab743a6946a459530594834@intel.com> References: <20200924085740.270192-1-yang_y_yi@163.com> <20200924085740.270192-2-yang_y_yi@163.com> In-Reply-To: <20200924085740.270192-2-yang_y_yi@163.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.5.1.3 dlp-product: dlpe-windows x-originating-ip: [10.239.127.36] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v7 1/3] gro: add UDP/IPv4 GRO support X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Acked-by: Jiayu Hu > -----Original Message----- > From: yang_y_yi@163.com > Sent: Thursday, September 24, 2020 4:58 PM > To: dev@dpdk.org > Cc: Hu, Jiayu ; thomas@monjalon.net; > yangyi01@inspur.com; yang_y_yi@163.com > Subject: [PATCH v7 1/3] gro: add UDP/IPv4 GRO support >=20 > From: Yi Yang >=20 > UDP/IPv4 GRO can help improve VM-to-VM UDP performance > when UFO or GSO is enabled in VM, GRO must be supported > if UFO or GSO is enabled, otherwise, performance can't > get big improvement if only GSO is there. >=20 > With this enabled in DPDK, OVS DPDK can leverage it > to improve VM-to-VM UDP performance, it will reassemble > UDP fragments immediate after they are received from > a physical NIC. It is very helpful in OVS DPDK VLAN use > case. >=20 > Signed-off-by: Yi Yang > --- > lib/librte_gro/gro_udp4.c | 433 > +++++++++++++++++++++++++++++++++++++++++++++ > lib/librte_gro/gro_udp4.h | 281 +++++++++++++++++++++++++++++ > lib/librte_gro/meson.build | 2 +- > lib/librte_gro/rte_gro.c | 94 ++++++++-- > lib/librte_gro/rte_gro.h | 5 +- > 5 files changed, 798 insertions(+), 17 deletions(-) > create mode 100644 lib/librte_gro/gro_udp4.c > create mode 100644 lib/librte_gro/gro_udp4.h >=20 > diff --git a/lib/librte_gro/gro_udp4.c b/lib/librte_gro/gro_udp4.c > new file mode 100644 > index 0000000..b830129 > --- /dev/null > +++ b/lib/librte_gro/gro_udp4.c > @@ -0,0 +1,433 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(c) 2020 Inspur Corporation > + */ > + > +#include > +#include > +#include > +#include > + > +#include "gro_udp4.h" > + > +void * > +gro_udp4_tbl_create(uint16_t socket_id, > + uint16_t max_flow_num, > + uint16_t max_item_per_flow) > +{ > + struct gro_udp4_tbl *tbl; > + size_t size; > + uint32_t entries_num, i; > + > + entries_num =3D max_flow_num * max_item_per_flow; > + entries_num =3D RTE_MIN(entries_num, > GRO_UDP4_TBL_MAX_ITEM_NUM); > + > + if (entries_num =3D=3D 0) > + return NULL; > + > + tbl =3D rte_zmalloc_socket(__func__, > + sizeof(struct gro_udp4_tbl), > + RTE_CACHE_LINE_SIZE, > + socket_id); > + if (tbl =3D=3D NULL) > + return NULL; > + > + size =3D sizeof(struct gro_udp4_item) * entries_num; > + tbl->items =3D rte_zmalloc_socket(__func__, > + size, > + RTE_CACHE_LINE_SIZE, > + socket_id); > + if (tbl->items =3D=3D NULL) { > + rte_free(tbl); > + return NULL; > + } > + tbl->max_item_num =3D entries_num; > + > + size =3D sizeof(struct gro_udp4_flow) * entries_num; > + tbl->flows =3D rte_zmalloc_socket(__func__, > + size, > + RTE_CACHE_LINE_SIZE, > + socket_id); > + if (tbl->flows =3D=3D NULL) { > + rte_free(tbl->items); > + rte_free(tbl); > + return NULL; > + } > + /* INVALID_ARRAY_INDEX indicates an empty flow */ > + for (i =3D 0; i < entries_num; i++) > + tbl->flows[i].start_index =3D INVALID_ARRAY_INDEX; > + tbl->max_flow_num =3D entries_num; > + > + return tbl; > +} > + > +void > +gro_udp4_tbl_destroy(void *tbl) > +{ > + struct gro_udp4_tbl *udp_tbl =3D tbl; > + > + if (udp_tbl) { > + rte_free(udp_tbl->items); > + rte_free(udp_tbl->flows); > + } > + rte_free(udp_tbl); > +} > + > +static inline uint32_t > +find_an_empty_item(struct gro_udp4_tbl *tbl) > +{ > + uint32_t i; > + uint32_t max_item_num =3D tbl->max_item_num; > + > + for (i =3D 0; i < max_item_num; i++) > + if (tbl->items[i].firstseg =3D=3D NULL) > + return i; > + return INVALID_ARRAY_INDEX; > +} > + > +static inline uint32_t > +find_an_empty_flow(struct gro_udp4_tbl *tbl) > +{ > + uint32_t i; > + uint32_t max_flow_num =3D tbl->max_flow_num; > + > + for (i =3D 0; i < max_flow_num; i++) > + if (tbl->flows[i].start_index =3D=3D INVALID_ARRAY_INDEX) > + return i; > + return INVALID_ARRAY_INDEX; > +} > + > +static inline uint32_t > +insert_new_item(struct gro_udp4_tbl *tbl, > + struct rte_mbuf *pkt, > + uint64_t start_time, > + uint32_t prev_idx, > + uint16_t frag_offset, > + uint8_t is_last_frag) > +{ > + uint32_t item_idx; > + > + item_idx =3D find_an_empty_item(tbl); > + if (unlikely(item_idx =3D=3D INVALID_ARRAY_INDEX)) > + return INVALID_ARRAY_INDEX; > + > + tbl->items[item_idx].firstseg =3D pkt; > + tbl->items[item_idx].lastseg =3D rte_pktmbuf_lastseg(pkt); > + tbl->items[item_idx].start_time =3D start_time; > + tbl->items[item_idx].next_pkt_idx =3D INVALID_ARRAY_INDEX; > + tbl->items[item_idx].frag_offset =3D frag_offset; > + tbl->items[item_idx].is_last_frag =3D is_last_frag; > + tbl->items[item_idx].nb_merged =3D 1; > + tbl->item_num++; > + > + /* if the previous packet exists, chain them together. */ > + if (prev_idx !=3D INVALID_ARRAY_INDEX) { > + tbl->items[item_idx].next_pkt_idx =3D > + tbl->items[prev_idx].next_pkt_idx; > + tbl->items[prev_idx].next_pkt_idx =3D item_idx; > + } > + > + return item_idx; > +} > + > +static inline uint32_t > +delete_item(struct gro_udp4_tbl *tbl, uint32_t item_idx, > + uint32_t prev_item_idx) > +{ > + uint32_t next_idx =3D tbl->items[item_idx].next_pkt_idx; > + > + /* NULL indicates an empty item */ > + tbl->items[item_idx].firstseg =3D NULL; > + tbl->item_num--; > + if (prev_item_idx !=3D INVALID_ARRAY_INDEX) > + tbl->items[prev_item_idx].next_pkt_idx =3D next_idx; > + > + return next_idx; > +} > + > +static inline uint32_t > +insert_new_flow(struct gro_udp4_tbl *tbl, > + struct udp4_flow_key *src, > + uint32_t item_idx) > +{ > + struct udp4_flow_key *dst; > + uint32_t flow_idx; > + > + flow_idx =3D find_an_empty_flow(tbl); > + if (unlikely(flow_idx =3D=3D INVALID_ARRAY_INDEX)) > + return INVALID_ARRAY_INDEX; > + > + dst =3D &(tbl->flows[flow_idx].key); > + > + rte_ether_addr_copy(&(src->eth_saddr), &(dst->eth_saddr)); > + rte_ether_addr_copy(&(src->eth_daddr), &(dst->eth_daddr)); > + dst->ip_src_addr =3D src->ip_src_addr; > + dst->ip_dst_addr =3D src->ip_dst_addr; > + dst->ip_id =3D src->ip_id; > + > + tbl->flows[flow_idx].start_index =3D item_idx; > + tbl->flow_num++; > + > + return flow_idx; > +} > + > +/* > + * update the packet length for the flushed packet. > + */ > +static inline void > +update_header(struct gro_udp4_item *item) > +{ > + struct rte_ipv4_hdr *ipv4_hdr; > + struct rte_mbuf *pkt =3D item->firstseg; > + uint16_t frag_offset; > + > + ipv4_hdr =3D (struct rte_ipv4_hdr *)(rte_pktmbuf_mtod(pkt, char *) + > + pkt->l2_len); > + ipv4_hdr->total_length =3D rte_cpu_to_be_16(pkt->pkt_len - > + pkt->l2_len); > + > + /* Clear MF bit if it is last fragment */ > + if (item->is_last_frag) { > + frag_offset =3D rte_be_to_cpu_16(ipv4_hdr->fragment_offset); > + ipv4_hdr->fragment_offset =3D > + rte_cpu_to_be_16(frag_offset & > ~RTE_IPV4_HDR_MF_FLAG); > + } > +} > + > +int32_t > +gro_udp4_reassemble(struct rte_mbuf *pkt, > + struct gro_udp4_tbl *tbl, > + uint64_t start_time) > +{ > + struct rte_ether_hdr *eth_hdr; > + struct rte_ipv4_hdr *ipv4_hdr; > + uint16_t ip_dl; > + uint16_t ip_id, hdr_len; > + uint16_t frag_offset =3D 0; > + uint8_t is_last_frag; > + > + struct udp4_flow_key key; > + uint32_t cur_idx, prev_idx, item_idx; > + uint32_t i, max_flow_num, remaining_flow_num; > + int cmp; > + uint8_t find; > + > + eth_hdr =3D rte_pktmbuf_mtod(pkt, struct rte_ether_hdr *); > + ipv4_hdr =3D (struct rte_ipv4_hdr *)((char *)eth_hdr + pkt->l2_len); > + hdr_len =3D pkt->l2_len + pkt->l3_len; > + > + /* > + * Don't process non-fragment packet. > + */ > + if (!is_ipv4_fragment(ipv4_hdr)) > + return -1; > + > + /* > + * Don't process the packet whose payload length is less than or > + * equal to 0. > + */ > + if (pkt->pkt_len <=3D hdr_len) > + return -1; > + > + ip_dl =3D rte_be_to_cpu_16(ipv4_hdr->total_length); > + if (ip_dl <=3D pkt->l3_len) > + return -1; > + > + ip_dl -=3D pkt->l3_len; > + ip_id =3D rte_be_to_cpu_16(ipv4_hdr->packet_id); > + frag_offset =3D rte_be_to_cpu_16(ipv4_hdr->fragment_offset); > + is_last_frag =3D ((frag_offset & RTE_IPV4_HDR_MF_FLAG) =3D=3D 0) ? 1 : = 0; > + frag_offset =3D (uint16_t)(frag_offset & RTE_IPV4_HDR_OFFSET_MASK) > << 3; > + > + rte_ether_addr_copy(&(eth_hdr->s_addr), &(key.eth_saddr)); > + rte_ether_addr_copy(&(eth_hdr->d_addr), &(key.eth_daddr)); > + key.ip_src_addr =3D ipv4_hdr->src_addr; > + key.ip_dst_addr =3D ipv4_hdr->dst_addr; > + key.ip_id =3D ip_id; > + > + /* Search for a matched flow. */ > + max_flow_num =3D tbl->max_flow_num; > + remaining_flow_num =3D tbl->flow_num; > + find =3D 0; > + for (i =3D 0; i < max_flow_num && remaining_flow_num; i++) { > + if (tbl->flows[i].start_index !=3D INVALID_ARRAY_INDEX) { > + if (is_same_udp4_flow(tbl->flows[i].key, key)) { > + find =3D 1; > + break; > + } > + remaining_flow_num--; > + } > + } > + > + /* > + * Fail to find a matched flow. Insert a new flow and store the > + * packet into the flow. > + */ > + if (find =3D=3D 0) { > + item_idx =3D insert_new_item(tbl, pkt, start_time, > + INVALID_ARRAY_INDEX, frag_offset, > + is_last_frag); > + if (unlikely(item_idx =3D=3D INVALID_ARRAY_INDEX)) > + return -1; > + if (insert_new_flow(tbl, &key, item_idx) =3D=3D > + INVALID_ARRAY_INDEX) { > + /* > + * Fail to insert a new flow, so delete the > + * stored packet. > + */ > + delete_item(tbl, item_idx, INVALID_ARRAY_INDEX); > + return -1; > + } > + return 0; > + } > + > + /* > + * Check all packets in the flow and try to find a neighbor for > + * the input packet. > + */ > + cur_idx =3D tbl->flows[i].start_index; > + prev_idx =3D cur_idx; > + do { > + cmp =3D udp4_check_neighbor(&(tbl->items[cur_idx]), > + frag_offset, ip_dl, 0); > + if (cmp) { > + if (merge_two_udp4_packets(&(tbl->items[cur_idx]), > + pkt, cmp, frag_offset, > + is_last_frag, 0)) > + return 1; > + /* > + * Fail to merge the two packets, as the packet > + * length is greater than the max value. Store > + * the packet into the flow. > + */ > + if (insert_new_item(tbl, pkt, start_time, prev_idx, > + frag_offset, is_last_frag) =3D=3D > + INVALID_ARRAY_INDEX) > + return -1; > + return 0; > + } > + > + /* Ensure inserted items are ordered by frag_offset */ > + if (frag_offset > + < tbl->items[cur_idx].frag_offset) { > + break; > + } > + > + prev_idx =3D cur_idx; > + cur_idx =3D tbl->items[cur_idx].next_pkt_idx; > + } while (cur_idx !=3D INVALID_ARRAY_INDEX); > + > + /* Fail to find a neighbor, so store the packet into the flow. */ > + if (cur_idx =3D=3D tbl->flows[i].start_index) { > + /* Insert it before the first packet of the flow */ > + item_idx =3D insert_new_item(tbl, pkt, start_time, > + INVALID_ARRAY_INDEX, frag_offset, > + is_last_frag); > + if (unlikely(item_idx =3D=3D INVALID_ARRAY_INDEX)) > + return -1; > + tbl->items[item_idx].next_pkt_idx =3D cur_idx; > + tbl->flows[i].start_index =3D item_idx; > + } else { > + if (insert_new_item(tbl, pkt, start_time, prev_idx, > + frag_offset, is_last_frag) > + =3D=3D INVALID_ARRAY_INDEX) > + return -1; > + } > + > + return 0; > +} > + > +static int > +gro_udp4_merge_items(struct gro_udp4_tbl *tbl, > + uint32_t start_idx) > +{ > + uint16_t frag_offset; > + uint8_t is_last_frag; > + int16_t ip_dl; > + struct rte_mbuf *pkt; > + int cmp; > + uint32_t item_idx; > + uint16_t hdr_len; > + > + item_idx =3D tbl->items[start_idx].next_pkt_idx; > + while (item_idx !=3D INVALID_ARRAY_INDEX) { > + pkt =3D tbl->items[item_idx].firstseg; > + hdr_len =3D pkt->l2_len + pkt->l3_len; > + ip_dl =3D pkt->pkt_len - hdr_len; > + frag_offset =3D tbl->items[item_idx].frag_offset; > + is_last_frag =3D tbl->items[item_idx].is_last_frag; > + cmp =3D udp4_check_neighbor(&(tbl->items[start_idx]), > + frag_offset, ip_dl, 0); > + if (cmp) { > + if (merge_two_udp4_packets( > + &(tbl->items[start_idx]), > + pkt, cmp, frag_offset, > + is_last_frag, 0)) { > + item_idx =3D delete_item(tbl, item_idx, > + > INVALID_ARRAY_INDEX); > + tbl->items[start_idx].next_pkt_idx > + =3D item_idx; > + } else > + return 0; > + } else > + return 0; > + } > + > + return 0; > +} > + > +uint16_t > +gro_udp4_tbl_timeout_flush(struct gro_udp4_tbl *tbl, > + uint64_t flush_timestamp, > + struct rte_mbuf **out, > + uint16_t nb_out) > +{ > + uint16_t k =3D 0; > + uint32_t i, j; > + uint32_t max_flow_num =3D tbl->max_flow_num; > + > + for (i =3D 0; i < max_flow_num; i++) { > + if (unlikely(tbl->flow_num =3D=3D 0)) > + return k; > + > + j =3D tbl->flows[i].start_index; > + while (j !=3D INVALID_ARRAY_INDEX) { > + if (tbl->items[j].start_time <=3D flush_timestamp) { > + gro_udp4_merge_items(tbl, j); > + out[k++] =3D tbl->items[j].firstseg; > + if (tbl->items[j].nb_merged > 1) > + update_header(&(tbl->items[j])); > + /* > + * Delete the packet and get the next > + * packet in the flow. > + */ > + j =3D delete_item(tbl, j, INVALID_ARRAY_INDEX); > + tbl->flows[i].start_index =3D j; > + if (j =3D=3D INVALID_ARRAY_INDEX) > + tbl->flow_num--; > + > + if (unlikely(k =3D=3D nb_out)) > + return k; > + } else > + /* > + * Flushing packets does not strictly follow > + * timestamp. It does not flush left packets of > + * the flow this time once it finds one item > + * whose start_time is greater than > + * flush_timestamp. So go to check other > flows. > + */ > + break; > + } > + } > + return k; > +} > + > +uint32_t > +gro_udp4_tbl_pkt_count(void *tbl) > +{ > + struct gro_udp4_tbl *gro_tbl =3D tbl; > + > + if (gro_tbl) > + return gro_tbl->item_num; > + > + return 0; > +} > diff --git a/lib/librte_gro/gro_udp4.h b/lib/librte_gro/gro_udp4.h > new file mode 100644 > index 0000000..0a078e4 > --- /dev/null > +++ b/lib/librte_gro/gro_udp4.h > @@ -0,0 +1,281 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(c) 2020 Inspur Corporation > + */ > + > +#ifndef _GRO_UDP4_H_ > +#define _GRO_UDP4_H_ > + > +#include > +#include > + > +#define INVALID_ARRAY_INDEX 0xffffffffUL > +#define GRO_UDP4_TBL_MAX_ITEM_NUM (1024UL * 1024UL) > + > +/* > + * The max length of a IPv4 packet, which includes the length of the L3 > + * header, the L4 header and the data payload. > + */ > +#define MAX_IPV4_PKT_LENGTH UINT16_MAX > + > +/* Header fields representing a UDP/IPv4 flow */ > +struct udp4_flow_key { > + struct rte_ether_addr eth_saddr; > + struct rte_ether_addr eth_daddr; > + uint32_t ip_src_addr; > + uint32_t ip_dst_addr; > + > + /* IP fragment for UDP does not contain UDP header > + * except the first one. But IP ID must be same. > + */ > + uint16_t ip_id; > +}; > + > +struct gro_udp4_flow { > + struct udp4_flow_key key; > + /* > + * The index of the first packet in the flow. > + * INVALID_ARRAY_INDEX indicates an empty flow. > + */ > + uint32_t start_index; > +}; > + > +struct gro_udp4_item { > + /* > + * The first MBUF segment of the packet. If the value > + * is NULL, it means the item is empty. > + */ > + struct rte_mbuf *firstseg; > + /* The last MBUF segment of the packet */ > + struct rte_mbuf *lastseg; > + /* > + * The time when the first packet is inserted into the table. > + * This value won't be updated, even if the packet is merged > + * with other packets. > + */ > + uint64_t start_time; > + /* > + * next_pkt_idx is used to chain the packets that > + * are in the same flow but can't be merged together > + * (e.g. caused by packet reordering). > + */ > + uint32_t next_pkt_idx; > + /* offset of IP fragment packet */ > + uint16_t frag_offset; > + /* is last IP fragment? */ > + uint8_t is_last_frag; > + /* the number of merged packets */ > + uint16_t nb_merged; > +}; > + > +/* > + * UDP/IPv4 reassembly table structure. > + */ > +struct gro_udp4_tbl { > + /* item array */ > + struct gro_udp4_item *items; > + /* flow array */ > + struct gro_udp4_flow *flows; > + /* current item number */ > + uint32_t item_num; > + /* current flow num */ > + uint32_t flow_num; > + /* item array size */ > + uint32_t max_item_num; > + /* flow array size */ > + uint32_t max_flow_num; > +}; > + > +/** > + * This function creates a UDP/IPv4 reassembly table. > + * > + * @param socket_id > + * Socket index for allocating the UDP/IPv4 reassemble table > + * @param max_flow_num > + * The maximum number of flows in the UDP/IPv4 GRO table > + * @param max_item_per_flow > + * The maximum number of packets per flow > + * > + * @return > + * - Return the table pointer on success. > + * - Return NULL on failure. > + */ > +void *gro_udp4_tbl_create(uint16_t socket_id, > + uint16_t max_flow_num, > + uint16_t max_item_per_flow); > + > +/** > + * This function destroys a UDP/IPv4 reassembly table. > + * > + * @param tbl > + * Pointer pointing to the UDP/IPv4 reassembly table. > + */ > +void gro_udp4_tbl_destroy(void *tbl); > + > +/** > + * This function merges a UDP/IPv4 packet. > + * > + * This function does not check if the packet has correct checksums and > + * does not re-calculate checksums for the merged packet. It returns the > + * packet if it isn't UDP fragment or there is no available space in > + * the table. > + * > + * @param pkt > + * Packet to reassemble > + * @param tbl > + * Pointer pointing to the UDP/IPv4 reassembly table > + * @start_time > + * The time when the packet is inserted into the table > + * > + * @return > + * - Return a positive value if the packet is merged. > + * - Return zero if the packet isn't merged but stored in the table. > + * - Return a negative value for invalid parameters or no available > + * space in the table. > + */ > +int32_t gro_udp4_reassemble(struct rte_mbuf *pkt, > + struct gro_udp4_tbl *tbl, > + uint64_t start_time); > + > +/** > + * This function flushes timeout packets in a UDP/IPv4 reassembly table, > + * and without updating checksums. > + * > + * @param tbl > + * UDP/IPv4 reassembly table pointer > + * @param flush_timestamp > + * Flush packets which are inserted into the table before or at the > + * flush_timestamp. > + * @param out > + * Pointer array used to keep flushed packets > + * @param nb_out > + * The element number in 'out'. It also determines the maximum number > of > + * packets that can be flushed finally. > + * > + * @return > + * The number of flushed packets > + */ > +uint16_t gro_udp4_tbl_timeout_flush(struct gro_udp4_tbl *tbl, > + uint64_t flush_timestamp, > + struct rte_mbuf **out, > + uint16_t nb_out); > + > +/** > + * This function returns the number of the packets in a UDP/IPv4 > + * reassembly table. > + * > + * @param tbl > + * UDP/IPv4 reassembly table pointer > + * > + * @return > + * The number of packets in the table > + */ > +uint32_t gro_udp4_tbl_pkt_count(void *tbl); > + > +/* > + * Check if two UDP/IPv4 packets belong to the same flow. > + */ > +static inline int > +is_same_udp4_flow(struct udp4_flow_key k1, struct udp4_flow_key k2) > +{ > + return (rte_is_same_ether_addr(&k1.eth_saddr, &k2.eth_saddr) && > + rte_is_same_ether_addr(&k1.eth_daddr, > &k2.eth_daddr) && > + (k1.ip_src_addr =3D=3D k2.ip_src_addr) && > + (k1.ip_dst_addr =3D=3D k2.ip_dst_addr) && > + (k1.ip_id =3D=3D k2.ip_id)); > +} > + > +/* > + * Merge two UDP/IPv4 packets without updating checksums. > + * If cmp is larger than 0, append the new packet to the > + * original packet. Otherwise, pre-pend the new packet to > + * the original packet. > + */ > +static inline int > +merge_two_udp4_packets(struct gro_udp4_item *item, > + struct rte_mbuf *pkt, > + int cmp, > + uint16_t frag_offset, > + uint8_t is_last_frag, > + uint16_t l2_offset) > +{ > + struct rte_mbuf *pkt_head, *pkt_tail, *lastseg; > + uint16_t hdr_len, l2_len; > + uint32_t ip_len; > + > + if (cmp > 0) { > + pkt_head =3D item->firstseg; > + pkt_tail =3D pkt; > + } else { > + pkt_head =3D pkt; > + pkt_tail =3D item->firstseg; > + } > + > + /* check if the IPv4 packet length is greater than the max value */ > + hdr_len =3D l2_offset + pkt_head->l2_len + pkt_head->l3_len; > + l2_len =3D l2_offset > 0 ? pkt_head->outer_l2_len : pkt_head->l2_len; > + ip_len =3D pkt_head->pkt_len - l2_len > + + pkt_tail->pkt_len - hdr_len; > + if (unlikely(ip_len > MAX_IPV4_PKT_LENGTH)) > + return 0; > + > + /* remove the packet header for the tail packet */ > + rte_pktmbuf_adj(pkt_tail, hdr_len); > + > + /* chain two packets together */ > + if (cmp > 0) { > + item->lastseg->next =3D pkt; > + item->lastseg =3D rte_pktmbuf_lastseg(pkt); > + } else { > + lastseg =3D rte_pktmbuf_lastseg(pkt); > + lastseg->next =3D item->firstseg; > + item->firstseg =3D pkt; > + item->frag_offset =3D frag_offset; > + } > + item->nb_merged++; > + if (is_last_frag) > + item->is_last_frag =3D is_last_frag; > + > + /* update MBUF metadata for the merged packet */ > + pkt_head->nb_segs +=3D pkt_tail->nb_segs; > + pkt_head->pkt_len +=3D pkt_tail->pkt_len; > + > + return 1; > +} > + > +/* > + * Check if two UDP/IPv4 packets are neighbors. > + */ > +static inline int > +udp4_check_neighbor(struct gro_udp4_item *item, > + uint16_t frag_offset, > + uint16_t ip_dl, > + uint16_t l2_offset) > +{ > + struct rte_mbuf *pkt_orig =3D item->firstseg; > + uint16_t len; > + > + /* check if the two packets are neighbors */ > + len =3D pkt_orig->pkt_len - l2_offset - pkt_orig->l2_len - > + pkt_orig->l3_len; > + if (frag_offset =3D=3D item->frag_offset + len) > + /* append the new packet */ > + return 1; > + else if (frag_offset + ip_dl =3D=3D item->frag_offset) > + /* pre-pend the new packet */ > + return -1; > + > + return 0; > +} > + > +static inline int > +is_ipv4_fragment(const struct rte_ipv4_hdr *hdr) > +{ > + uint16_t flag_offset, ip_flag, ip_ofs; > + > + flag_offset =3D rte_be_to_cpu_16(hdr->fragment_offset); > + ip_ofs =3D (uint16_t)(flag_offset & RTE_IPV4_HDR_OFFSET_MASK); > + ip_flag =3D (uint16_t)(flag_offset & RTE_IPV4_HDR_MF_FLAG); > + > + return ip_flag !=3D 0 || ip_ofs !=3D 0; > +} > +#endif > diff --git a/lib/librte_gro/meson.build b/lib/librte_gro/meson.build > index 501668c..0d18dc2 100644 > --- a/lib/librte_gro/meson.build > +++ b/lib/librte_gro/meson.build > @@ -1,6 +1,6 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2017 Intel Corporation >=20 > -sources =3D files('rte_gro.c', 'gro_tcp4.c', 'gro_vxlan_tcp4.c') > +sources =3D files('rte_gro.c', 'gro_tcp4.c', 'gro_udp4.c', 'gro_vxlan_tc= p4.c') > headers =3D files('rte_gro.h') > deps +=3D ['ethdev'] > diff --git a/lib/librte_gro/rte_gro.c b/lib/librte_gro/rte_gro.c > index 6618f4d..ac23df1 100644 > --- a/lib/librte_gro/rte_gro.c > +++ b/lib/librte_gro/rte_gro.c > @@ -9,6 +9,7 @@ >=20 > #include "rte_gro.h" > #include "gro_tcp4.h" > +#include "gro_udp4.h" > #include "gro_vxlan_tcp4.h" >=20 > typedef void *(*gro_tbl_create_fn)(uint16_t socket_id, > @@ -18,17 +19,23 @@ > typedef uint32_t (*gro_tbl_pkt_count_fn)(void *tbl); >=20 > static gro_tbl_create_fn tbl_create_fn[RTE_GRO_TYPE_MAX_NUM] =3D { > - gro_tcp4_tbl_create, gro_vxlan_tcp4_tbl_create, NULL}; > + gro_tcp4_tbl_create, gro_vxlan_tcp4_tbl_create, > + gro_udp4_tbl_create, NULL}; > static gro_tbl_destroy_fn tbl_destroy_fn[RTE_GRO_TYPE_MAX_NUM] =3D { > gro_tcp4_tbl_destroy, gro_vxlan_tcp4_tbl_destroy, > + gro_udp4_tbl_destroy, > NULL}; > static gro_tbl_pkt_count_fn tbl_pkt_count_fn[RTE_GRO_TYPE_MAX_NUM] =3D > { > gro_tcp4_tbl_pkt_count, > gro_vxlan_tcp4_tbl_pkt_count, > + gro_udp4_tbl_pkt_count, > NULL}; >=20 > #define IS_IPV4_TCP_PKT(ptype) (RTE_ETH_IS_IPV4_HDR(ptype) && \ > ((ptype & RTE_PTYPE_L4_TCP) =3D=3D RTE_PTYPE_L4_TCP)) >=20 > +#define IS_IPV4_UDP_PKT(ptype) (RTE_ETH_IS_IPV4_HDR(ptype) && \ > + ((ptype & RTE_PTYPE_L4_UDP) =3D=3D RTE_PTYPE_L4_UDP)) > + > #define IS_IPV4_VXLAN_TCP4_PKT(ptype) (RTE_ETH_IS_IPV4_HDR(ptype) > && \ > ((ptype & RTE_PTYPE_L4_UDP) =3D=3D RTE_PTYPE_L4_UDP) && \ > ((ptype & RTE_PTYPE_TUNNEL_VXLAN) =3D=3D \ > @@ -40,6 +47,7 @@ > RTE_PTYPE_INNER_L3_IPV4_EXT | \ > RTE_PTYPE_INNER_L3_IPV4_EXT_UNKNOWN)) !=3D 0)) >=20 > + > /* > * GRO context structure. It keeps the table structures, which are > * used to merge packets, for different GRO types. Before using > @@ -123,20 +131,26 @@ struct gro_ctx { > struct gro_tcp4_flow tcp_flows[RTE_GRO_MAX_BURST_ITEM_NUM]; > struct gro_tcp4_item tcp_items[RTE_GRO_MAX_BURST_ITEM_NUM] > =3D {{0} }; >=20 > - /* Allocate a reassembly table for VXLAN GRO */ > + /* allocate a reassembly table for UDP/IPv4 GRO */ > + struct gro_udp4_tbl udp_tbl; > + struct gro_udp4_flow > udp_flows[RTE_GRO_MAX_BURST_ITEM_NUM]; > + struct gro_udp4_item udp_items[RTE_GRO_MAX_BURST_ITEM_NUM] > =3D {{0} }; > + > + /* Allocate a reassembly table for VXLAN TCP GRO */ > struct gro_vxlan_tcp4_tbl vxlan_tbl; > struct gro_vxlan_tcp4_flow > vxlan_flows[RTE_GRO_MAX_BURST_ITEM_NUM]; > - struct gro_vxlan_tcp4_item > vxlan_items[RTE_GRO_MAX_BURST_ITEM_NUM] =3D { > - {{0}, 0, 0} }; > + struct gro_vxlan_tcp4_item > vxlan_items[RTE_GRO_MAX_BURST_ITEM_NUM] > + =3D {{{0}, 0, 0} }; >=20 > struct rte_mbuf *unprocess_pkts[nb_pkts]; > uint32_t item_num; > int32_t ret; > uint16_t i, unprocess_num =3D 0, nb_after_gro =3D nb_pkts; > - uint8_t do_tcp4_gro =3D 0, do_vxlan_gro =3D 0; > + uint8_t do_tcp4_gro =3D 0, do_vxlan_gro =3D 0, do_udp4_gro =3D 0; >=20 > if (unlikely((param->gro_types & (RTE_GRO_IPV4_VXLAN_TCP_IPV4 | > - RTE_GRO_TCP_IPV4)) =3D=3D 0)) > + RTE_GRO_TCP_IPV4 | > + RTE_GRO_UDP_IPV4)) =3D=3D 0)) > return nb_pkts; >=20 > /* Get the maximum number of packets */ > @@ -170,6 +184,20 @@ struct gro_ctx { > do_tcp4_gro =3D 1; > } >=20 > + if (param->gro_types & RTE_GRO_UDP_IPV4) { > + for (i =3D 0; i < item_num; i++) > + udp_flows[i].start_index =3D INVALID_ARRAY_INDEX; > + > + udp_tbl.flows =3D udp_flows; > + udp_tbl.items =3D udp_items; > + udp_tbl.flow_num =3D 0; > + udp_tbl.item_num =3D 0; > + udp_tbl.max_flow_num =3D item_num; > + udp_tbl.max_item_num =3D item_num; > + do_udp4_gro =3D 1; > + } > + > + > for (i =3D 0; i < nb_pkts; i++) { > /* > * The timestamp is ignored, since all packets > @@ -177,7 +205,8 @@ struct gro_ctx { > */ > if (IS_IPV4_VXLAN_TCP4_PKT(pkts[i]->packet_type) && > do_vxlan_gro) { > - ret =3D gro_vxlan_tcp4_reassemble(pkts[i], &vxlan_tbl, > 0); > + ret =3D gro_vxlan_tcp4_reassemble(pkts[i], > + &vxlan_tbl, 0); > if (ret > 0) > /* Merge successfully */ > nb_after_gro--; > @@ -191,27 +220,43 @@ struct gro_ctx { > nb_after_gro--; > else if (ret < 0) > unprocess_pkts[unprocess_num++] =3D pkts[i]; > + } else if (IS_IPV4_UDP_PKT(pkts[i]->packet_type) && > + do_udp4_gro) { > + ret =3D gro_udp4_reassemble(pkts[i], &udp_tbl, 0); > + if (ret > 0) > + /* merge successfully */ > + nb_after_gro--; > + else if (ret < 0) > + unprocess_pkts[unprocess_num++] =3D pkts[i]; > } else > unprocess_pkts[unprocess_num++] =3D pkts[i]; > } >=20 > - if (nb_after_gro < nb_pkts) { > + if ((nb_after_gro < nb_pkts) > + || (unprocess_num < nb_pkts)) { > i =3D 0; > /* Flush all packets from the tables */ > if (do_vxlan_gro) { > i =3D gro_vxlan_tcp4_tbl_timeout_flush(&vxlan_tbl, > 0, pkts, nb_pkts); > } > + > if (do_tcp4_gro) { > i +=3D gro_tcp4_tbl_timeout_flush(&tcp_tbl, 0, > &pkts[i], nb_pkts - i); > } > + > + if (do_udp4_gro) { > + i +=3D gro_udp4_tbl_timeout_flush(&udp_tbl, 0, > + &pkts[i], nb_pkts - i); > + } > /* Copy unprocessed packets */ > if (unprocess_num > 0) { > memcpy(&pkts[i], unprocess_pkts, > sizeof(struct rte_mbuf *) * > unprocess_num); > } > + nb_after_gro =3D i + unprocess_num; > } >=20 > return nb_after_gro; > @@ -224,22 +269,26 @@ struct gro_ctx { > { > struct rte_mbuf *unprocess_pkts[nb_pkts]; > struct gro_ctx *gro_ctx =3D ctx; > - void *tcp_tbl, *vxlan_tbl; > + void *tcp_tbl, *udp_tbl, *vxlan_tbl; > uint64_t current_time; > uint16_t i, unprocess_num =3D 0; > - uint8_t do_tcp4_gro, do_vxlan_gro; > + uint8_t do_tcp4_gro, do_vxlan_gro, do_udp4_gro; >=20 > if (unlikely((gro_ctx->gro_types & (RTE_GRO_IPV4_VXLAN_TCP_IPV4 > | > - RTE_GRO_TCP_IPV4)) =3D=3D 0)) > + RTE_GRO_TCP_IPV4 | > + RTE_GRO_UDP_IPV4)) =3D=3D 0)) > return nb_pkts; >=20 > tcp_tbl =3D gro_ctx->tbls[RTE_GRO_TCP_IPV4_INDEX]; > vxlan_tbl =3D gro_ctx->tbls[RTE_GRO_IPV4_VXLAN_TCP_IPV4_INDEX]; > + udp_tbl =3D gro_ctx->tbls[RTE_GRO_UDP_IPV4_INDEX]; >=20 > do_tcp4_gro =3D (gro_ctx->gro_types & RTE_GRO_TCP_IPV4) =3D=3D > RTE_GRO_TCP_IPV4; > do_vxlan_gro =3D (gro_ctx->gro_types & > RTE_GRO_IPV4_VXLAN_TCP_IPV4) =3D=3D > RTE_GRO_IPV4_VXLAN_TCP_IPV4; > + do_udp4_gro =3D (gro_ctx->gro_types & RTE_GRO_UDP_IPV4) =3D=3D > + RTE_GRO_UDP_IPV4; >=20 > current_time =3D rte_rdtsc(); >=20 > @@ -254,6 +303,11 @@ struct gro_ctx { > if (gro_tcp4_reassemble(pkts[i], tcp_tbl, > current_time) < 0) > unprocess_pkts[unprocess_num++] =3D pkts[i]; > + } else if (IS_IPV4_UDP_PKT(pkts[i]->packet_type) && > + do_udp4_gro) { > + if (gro_udp4_reassemble(pkts[i], udp_tbl, > + current_time) < 0) > + unprocess_pkts[unprocess_num++] =3D pkts[i]; > } else > unprocess_pkts[unprocess_num++] =3D pkts[i]; > } > @@ -275,6 +329,7 @@ struct gro_ctx { > struct gro_ctx *gro_ctx =3D ctx; > uint64_t flush_timestamp; > uint16_t num =3D 0; > + uint16_t left_nb_out =3D max_nb_out; >=20 > gro_types =3D gro_types & gro_ctx->gro_types; > flush_timestamp =3D rte_rdtsc() - timeout_cycles; > @@ -282,16 +337,25 @@ struct gro_ctx { > if (gro_types & RTE_GRO_IPV4_VXLAN_TCP_IPV4) { > num =3D gro_vxlan_tcp4_tbl_timeout_flush(gro_ctx->tbls[ > RTE_GRO_IPV4_VXLAN_TCP_IPV4_INDEX], > - flush_timestamp, out, max_nb_out); > - max_nb_out -=3D num; > + flush_timestamp, out, left_nb_out); > + left_nb_out =3D max_nb_out - num; > } >=20 > /* If no available space in 'out', stop flushing. */ > - if ((gro_types & RTE_GRO_TCP_IPV4) && max_nb_out > 0) { > + if ((gro_types & RTE_GRO_TCP_IPV4) && left_nb_out > 0) { > num +=3D gro_tcp4_tbl_timeout_flush( > gro_ctx->tbls[RTE_GRO_TCP_IPV4_INDEX], > flush_timestamp, > - &out[num], max_nb_out); > + &out[num], left_nb_out); > + left_nb_out =3D max_nb_out - num; > + } > + > + /* If no available space in 'out', stop flushing. */ > + if ((gro_types & RTE_GRO_UDP_IPV4) && left_nb_out > 0) { > + num +=3D gro_udp4_tbl_timeout_flush( > + gro_ctx->tbls[RTE_GRO_UDP_IPV4_INDEX], > + flush_timestamp, > + &out[num], left_nb_out); > } >=20 > return num; > diff --git a/lib/librte_gro/rte_gro.h b/lib/librte_gro/rte_gro.h > index 8d781b5..470f3ed 100644 > --- a/lib/librte_gro/rte_gro.h > +++ b/lib/librte_gro/rte_gro.h > @@ -31,7 +31,10 @@ > /**< TCP/IPv4 GRO flag */ > #define RTE_GRO_IPV4_VXLAN_TCP_IPV4_INDEX 1 > #define RTE_GRO_IPV4_VXLAN_TCP_IPV4 (1ULL << > RTE_GRO_IPV4_VXLAN_TCP_IPV4_INDEX) > -/**< VxLAN GRO flag. */ > +/**< VxLAN TCP/IPv4 GRO flag. */ > +#define RTE_GRO_UDP_IPV4_INDEX 2 > +#define RTE_GRO_UDP_IPV4 (1ULL << RTE_GRO_UDP_IPV4_INDEX) > +/**< UDP/IPv4 GRO flag */ >=20 > /** > * Structure used to create GRO context objects or used to pass > -- > 1.8.3.1