Hi Jianbo, Thanks for v3. Small compilation error. See inline comment. Otherwise it looks fine.
On Thu, 2017-05-11 at 17:25 +0800, Jianbo Liu wrote: > Use ARM NEON intrinsics to accelerate l3 fowarding. > > Signed-off-by: Jianbo Liu <jianbo....@linaro.org> > --- [...] > +/** > + * Process one packet: > + * Update source and destination MAC addresses in the ethernet > header. > + * Perform RFC1812 checks and updates for IPV4 packets. > + */ > +static inline void > +process_packet(struct rte_mbuf *pkt, uint16_t *dst_port) > +{ > + struct ether_hdr *eth_hdr; > + uint32x4_t te, ve; > + > + eth_hdr = rte_pktmbuf_mtod(pkt, struct ether_hdr *); > + > + te = vld1q_u32((uint32_t *)eth_hdr); > + ve = vreinterpretq_u32_s32(val_eth[dst_port[0]]); > + > + > + rfc1812_process((struct ipv4_hdr *)(eth_hdr + 1), dst_port, > + pkt->packet_type); > + > + ve = vcopyq_lane_u32(ve, 3, te, 3); Compilation error here. This should be vcopyq_laneq_u32 (Extra q after lane) > + vst1q_u32((uint32_t *)eth_hdr, ve); > +} > + [...]