On 06/27/2016 10:21 AM, Ananyev, Konstantin wrote: > > >> -----Original Message----- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Keith Wiles >> Sent: Saturday, June 25, 2016 4:56 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf chain >> friendly >> >> Move the next pointer to the first cacheline of the rte_mbuf structure >> and move the offload values to the second cacheline to give better >> performance to applications using chained mbufs. >> >> Enabled by a configuration option CONFIG_RTE_MBUF_CHAIN_FRIENDLY default >> is set to No. > > First, it would make ixgbe and i40e vector RX functions to work incorrectly. > Second, I don't think we can afford to allow people swap mbuf fields in the > way they like. > Otherwise we'll end-up with totally unmaintainable code pretty soon. > So NACK.
+1
