test/mbuf: fix virtual address conversion
Checks
Commit Message
Seen with address sanitizer.
rte_mempool_virt2iova() can only be used on mempool elements. In this case,
it is incorrect, and rte_mem_virt2iova() has to be used.
Bugzilla ID: 737
Fixes: 7b295dceea07 ("test/mbuf: add unit test cases")
Cc: stable@dpdk.org
Reported-by: Zhihong Peng <zhihongx.peng@intel.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
app/test/test_mbuf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Mon, Jul 5, 2021 at 9:28 AM Olivier Matz <olivier.matz@6wind.com> wrote:
>
> Seen with address sanitizer.
>
> rte_mempool_virt2iova() can only be used on mempool elements. In this case,
> it is incorrect, and rte_mem_virt2iova() has to be used.
>
> Bugzilla ID: 737
> Fixes: 7b295dceea07 ("test/mbuf: add unit test cases")
> Cc: stable@dpdk.org
>
> Reported-by: Zhihong Peng <zhihongx.peng@intel.com>
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Reviewed-by: David Marchand <david.marchand@redhat.com>
On Tue, Jul 6, 2021 at 1:50 PM David Marchand <david.marchand@redhat.com> wrote:
>
> On Mon, Jul 5, 2021 at 9:28 AM Olivier Matz <olivier.matz@6wind.com> wrote:
> >
> > Seen with address sanitizer.
> >
> > rte_mempool_virt2iova() can only be used on mempool elements. In this case,
> > it is incorrect, and rte_mem_virt2iova() has to be used.
> >
> > Bugzilla ID: 737
> > Fixes: 7b295dceea07 ("test/mbuf: add unit test cases")
> > Cc: stable@dpdk.org
> >
> > Reported-by: Zhihong Peng <zhihongx.peng@intel.com>
> > Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> Reviewed-by: David Marchand <david.marchand@redhat.com>
Applied, thanks.
@@ -2363,7 +2363,7 @@ test_pktmbuf_ext_shinfo_init_helper(struct rte_mempool *pktmbuf_pool)
if (rte_mbuf_refcnt_read(m) != 1)
GOTO_FAIL("%s: Invalid refcnt in mbuf\n", __func__);
- buf_iova = rte_mempool_virt2iova(ext_buf_addr);
+ buf_iova = rte_mem_virt2iova(ext_buf_addr);
rte_pktmbuf_attach_extbuf(m, ext_buf_addr, buf_iova, buf_len,
ret_shinfo);
if (m->ol_flags != EXT_ATTACHED_MBUF)