On Thu, Nov 02, 2017 at 01:41:39PM -0700, Ferruh Yigit wrote: > On 11/2/2017 9:42 AM, Matan Azrad wrote: > > v1: > > This series is a follow-up of the new datapath implementation introduced in > > RC1. > > It is dedicated to Tx and improves the performance. > > > > v2: > > Change uint32_t be_lkey to rte_be32_t lkey > > > > v3: > > Rebase to 17.11-rc2. > > Remove "net/mlx4: save lkey in big-endian format" patch which degrade > > performance. > > Merge separate segment cases patches "net/mlx4: improve performance of one > > Tx segment" and "net/mlx4: separate Tx for multi-segments". > > "net/mlx4: inline more Tx functions" renamed to "net/mlx4: associate MR to > > MP in a short function". > > Leave bytes counter as old version. > > Add memory barrier improvement patch. > > Remove empty segment support. > > > > v4: > > Remove "net/mlx4: remove empty Tx segment support" patch. > > Add "fix ring wraparound compiler hint" patch. > > Add "fix HW memory optimizations careless" patch. > > Remove unnecessary IO memory barrier in completion function. > > Remove inline declaration of completion function. > > Fix unnecessary empty lines. > > Fix indetations. > > Move max=0 check removal to patch "remove duplicate handling in Tx burst" > > and rename the patch. > > Add performace improvement in patch "mitigate Tx path memory barriers" > > > > v5: > > Fix compilation issue in debug mode for patch "net/mlx4: associate MR to MP > > in a short function". > > Add missing includes in "net/mlx4: associate MR to MP in a short function". > > Add cq_uar as volatile too in "net/mlx4: fix HW memory optimizations > > careless". > > Fix indentation issues in "net/mlx4: separate Tx segment cases". > > > > > > Matan Azrad (7): > > net/mlx4: remove error flows from Tx fast path > > net/mlx4: fix ring wraparound compiler hint > > net/mlx4: merge Tx path functions > > net/mlx4: remove duplicate handling in Tx burst > > net/mlx4: separate Tx segment cases > > net/mlx4: fix HW memory optimizations careless > > net/mlx4: mitigate Tx path memory barriers > > > > Ophir Munk (1): > > net/mlx4: associate MR to MP in a short function > > Patches has been sent and acked, so I already have my answer, but just to > double > check, these are not just fixes which normally rc3 should target, there are > data > path updates and performance improvements. > > Can you please confirm you want these patches in at rc3 phase? > I will wait for your explicit approval before getting them.
Yes, since they address a bunch of remaining performance issues, we'd like them included in v17.11 (otherwise we'll have to backport them for v17.11.1 anyway). Thanks Ferruh. -- Adrien Mazarguil 6WIND
