Re: [dpdk-dev] [PATCH v2 0/8] mbuf: structure reorganization

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 4/5/2017 10:37 AM, Thomas Monjalon wrote:
> 2017-04-04 18:27, Olivier Matz:
>> Based on discussions done in [1] and in this thread, this patchset reorganizes
>> the mbuf.
>>
>> The main changes are:
>> - reorder structure to increase vector performance on some non-ia
>>   platforms.
>> - add a 64bits timestamp field in the 1st cache line. This timestamp
>>   is not normalized, i.e. no unit or time reference is enforced. A
>>   library may be added to do this job in the future.
>> - m->next, m->nb_segs, and m->refcnt are always initialized for mbufs
>>   in the pool, avoiding the need of setting m->next (located in the
>>   2nd cache line) in the Rx path for mono-segment packets.
>> - change port and nb_segs to 16 bits
>> - move seqn in the 2nd cache line
> 
> Applied, thanks for the long work

Hi Nelio, Adrien,

After this patch, mlx5 with debug enabled giving following build error
[1] with gcc, not really sure about reason of the error, can you please
check?

[1]
.../drivers/net/mlx5/mlx5_rxtx.c: In function ‘mlx5_rx_burst’:
.../drivers/net/mlx5/mlx5_rxtx.c:2082:17: error: ‘len’ may be used
uninitialized in this function [-Werror=maybe-uninitialized]
   DATA_LEN(seg) = len;