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;
