On 3/12/2019 9:22 AM, Yangchao Zhou wrote: > va2pa depends on the physical address and virtual address offset of > current mbuf. It may get the wrong physical address of next mbuf which > allocated in another hugepage segment. > > In rte_mempool_populate_default(), trying to allocate whole block of > contiguous memory could be failed. Then, it would reserve memory in > several memzones that have different physical address and virtual address > offsets. The rte_mempool_populate_default() is used by > rte_pktmbuf_pool_create(). > > Signed-off-by: Yangchao Zhou <zhouya...@gmail.com> > --- > v2: Add an explanation that causes this problem. > Use m->next to store physical address.
Overall code looks good to me, thanks for the work, it looks pretty clean. I would like to do some test before giving an ack. Meanwhile, can you please update kni documentation, to document for the mbufs sent to kernel has mbuf->next fields as physical address. It is OK to send as a new version of the patch with documentation. Thanks, ferruh
