mbuf: fix external buffer pool boundaries
authorAlexander Kozyrev <akozyrev@mellanox.com>
Mon, 1 Jun 2020 15:24:16 +0000 (15:24 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Thu, 11 Jun 2020 07:51:46 +0000 (09:51 +0200)
commitd6eb247371216315bfc8e7b088bc38f67462bd48
tree2467d9fa6b0434f9d836aacd5dd764e9ef749625
parent72ea8d4050c12cf04bb4ac6ac16c3c084b811426
mbuf: fix external buffer pool boundaries

Memzones are created in testpmd in order to test external data
buffers functionality. Each memzone is 2Mb in size and divided among
the pool of external memory buffers.

Memzone may not always be fully utilized because mbufs size can vary
and some space can be left unused at the tail of a memzone. This is
not handled properly and mbuf can get the address of this leftover
space since this address is still valid (part of memzone), but there
is not enough space to fit the whole packet data. As a result packet
data may overflow and cause the memory corruption.

Take mbuf size into account when distributing memory addresses from
a memzone to external mbufs. Skip the remaining tail in case there
is not enough room for a packet and move to a next memzone instead.

Fixes: 6c8e50c2e5 ("mbuf: create pool with external memory buffers")
Cc: stable@dpdk.org
Signed-off-by: Alexander Kozyrev <akozyrev@mellanox.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
lib/librte_mbuf/rte_mbuf.c