[dpdk-dev] [memnic PATCH 3/7] pmd: use helper macros

[Hiroshi Shimamoto]

> Subject: Re: [dpdk-dev] [memnic PATCH 3/7] pmd: use helper macros
> 
> 2014-09-11 07:47, Hiroshi Shimamoto:
> > Do not touch pktmbuf directly.
> >
> > Instead of direct access, use rte_pktmbuf_pkt_len() and 
> > rte_pktmbuf_data_len()
> > to access the property.
> 
> I guess this change is for compatibility with DPDK 1.8.

Yep, I had the thought need to prepare upcoming code.

> Does it have an impact on performance?

No, it must not.

thanks,
Hiroshi

> 
> --
> Thomas

[dpdk-dev] [memnic PATCH 3/7] pmd: use helper macros

[Thomas Monjalon]

2014-09-11 07:47, Hiroshi Shimamoto:
> Do not touch pktmbuf directly.
> 
> Instead of direct access, use rte_pktmbuf_pkt_len() and rte_pktmbuf_data_len()
> to access the property.

I guess this change is for compatibility with DPDK 1.8.
Does it have an impact on performance?

-- 
Thomas

[dpdk-dev] [memnic PATCH 3/7] pmd: use helper macros

[Hiroshi Shimamoto]

From: Hiroshi Shimamoto 

Do not touch pktmbuf directly.

Instead of direct access, use rte_pktmbuf_pkt_len() and rte_pktmbuf_data_len()
to access the property.

Signed-off-by: Hiroshi Shimamoto 
Reviewed-by: Hayato Momma 
---
 pmd/pmd_memnic.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pmd/pmd_memnic.c b/pmd/pmd_memnic.c
index bbb5380..8341da7 100644
--- a/pmd/pmd_memnic.c
+++ b/pmd/pmd_memnic.c
@@ -308,8 +308,8 @@ static uint16_t memnic_recv_pkts(void *rx_queue,

rte_memcpy(rte_pktmbuf_mtod(mb, void *), p->data, p->len);
mb->pkt.in_port = q->port_id;
-   mb->pkt.pkt_len = p->len;
-   mb->pkt.data_len = p->len;
+   rte_pktmbuf_pkt_len(mb) = p->len;
+   rte_pktmbuf_data_len(mb) = p->len;
rx_pkts[nr] = mb;

pkts++;
-- 
1.8.3.1