gso: fix mbuf freeing responsibility
authorYi Yang <yangyi01@inspur.com>
Mon, 26 Oct 2020 06:47:13 +0000 (14:47 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Tue, 3 Nov 2020 21:45:02 +0000 (22:45 +0100)
commitc0d002aed98d6d1d38d6bb318a5bd2ed5cdc01b1
tree2b67e2c1fb879854b835ebb14811194cf40ae7e3
parentbd117d62a6ac105bcb7062cef2aa141f50057787
gso: fix mbuf freeing responsibility

rte_gso_segment decreased refcnt of pkt by one, but
it is wrong if pkt is external mbuf, pkt won't be
freed because of incorrect refcnt, the result is
application can't allocate mbuf from mempool because
mbufs in mempool are run out of.

One correct way is application should call
rte_pktmbuf_free after calling rte_gso_segment to free
pkt explicitly. rte_gso_segment must not handle it, this
should be responsibility of application.

This commit changed rte_gso_segment in functional behavior
and return value, so the application must take appropriate
actions according to return values, "ret < 0" means it
should free and drop 'pkt', "ret == 0" means 'pkt' isn't
GSOed but 'pkt' can be transmitted as a normal packet,
"ret > 0" means 'pkt' has been GSOed into two or multiple
segments, it should use "pkts_out" to transmit these
segments. The application must free 'pkt' after call
rte_gso_segment when return value isn't equal to 0.

Fixes: 119583797b6a ("gso: support TCP/IPv4 GSO")
Cc: stable@dpdk.org
Signed-off-by: Yi Yang <yangyi01@inspur.com>
Acked-by: Jiayu Hu <jiayu.hu@intel.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
app/test-pmd/csumonly.c
doc/guides/prog_guide/generic_segmentation_offload_lib.rst
doc/guides/rel_notes/release_20_11.rst
drivers/net/tap/rte_eth_tap.c
lib/librte_gso/gso_tcp4.c
lib/librte_gso/gso_tunnel_tcp4.c
lib/librte_gso/gso_udp4.c
lib/librte_gso/rte_gso.c
lib/librte_gso/rte_gso.h