On 4/7/2017 4:13 PM, Ferruh Yigit wrote: > On 4/4/2017 11:29 AM, Konstantin Ananyev wrote: >> Pretty much the same as one from Bruce: >> http://dpdk.org/ml/archives/dev/2017-April/062936.html >> but now for ixgbe. >> Based on Olivier's mbuf rework patchset, and makes some >> improvement to the ixgbe driver taking account of the rework. >> It also removes a build-time option that seems unnecessary. >> >> Depends on: http://dpdk.org/ml/archives/dev/2017-March/059693.html >> >> Konstantin Ananyev (2): >> net/ixgbe: eliminate mbuf write on rearm >> net/ixgbe: remove option to disable offload flags >> >> config/common_base | 1 - >> doc/guides/nics/ixgbe.rst | 18 ---------- >> drivers/net/ixgbe/ixgbe_rxtx_vec_common.h | 7 ---- >> drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 11 ------ >> drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 58 >> +++++++++++++------------------ >> 5 files changed, 24 insertions(+), 71 deletions(-) > > Series applied to dpdk-next-net/master, thanks.
Hi Konstantin, I talked a little early, getting following build error [1] with "default" machine type. Patches dropped from tree for now. [1] ...drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c:217:11: error: implicit declaration of function '_mm_blend_epi16' is invalid in C99 [-Werror,-Wimplicit-function-declaration] rearm0 = _mm_blend_epi16(mbuf_init, _mm_slli_si128(vtag1, 8), 0x10); ^ ...drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c:217:9: error: assigning to '__m128i' (vector of 2 'long long' values) from incompatible type 'int' rearm0 = _mm_blend_epi16(mbuf_init, _mm_slli_si128(vtag1, 8), 0x10); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c:218:9: error: assigning to '__m128i' (vector of 2 'long long' values) from incompatible type 'int' rearm1 = _mm_blend_epi16(mbuf_init, _mm_slli_si128(vtag1, 6), 0x10); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c:219:9: error: assigning to '__m128i' (vector of 2 'long long' values) from incompatible type 'int' rearm2 = _mm_blend_epi16(mbuf_init, _mm_slli_si128(vtag1, 4), 0x10); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ...drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c:220:9: error: assigning to '__m128i' (vector of 2 'long long' values) from incompatible type 'int' rearm3 = _mm_blend_epi16(mbuf_init, _mm_slli_si128(vtag1, 2), 0x10); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~