[dpdk-dev] [PATCH 00/24] Refactor mlx5 to improve performance

[Nélio Laranjeiro]

On Mon, Jun 20, 2016 at 04:03:00PM +0100, Ferruh Yigit wrote:
> On 6/20/2016 8:38 AM, Nélio Laranjeiro wrote:
> > On Fri, Jun 17, 2016 at 05:09:43PM +0100, Ferruh Yigit wrote:
> >> On 6/8/2016 10:47 AM, Nelio Laranjeiro wrote:
> >>> Enhance mlx5 with a data path that bypasses Verbs.
> >>>
> >>> The first half of this patchset removes support for functionality completely
> >>> rewritten in the second half (scatter/gather, inline send), while the data
> >>> path is refactored without Verbs.
> >>>
> >>> The PMD remains usable during the transition.
> >>>
> >>> This patchset must be applied after "Miscellaneous fixes for mlx4 and mlx5".
> >>>
> >>> Adrien Mazarguil (8):
> >>>   mlx5: replace countdown with threshold for TX completions
> >>>   mlx5: add debugging information about TX queues capabilities
> >>>   mlx5: check remaining space while processing TX burst
> >>>   mlx5: resurrect TX gather support
> >>>   mlx5: work around spurious compilation errors
> >>>   mlx5: remove redundant RX queue initialization code
> >>>   mlx5: make RX queue reinitialization safer
> >>>   mlx5: resurrect RX scatter support
> >>>
> >>> Nelio Laranjeiro (15):
> >>>   mlx5: split memory registration function for better performance
> >>>   mlx5: remove TX gather support
> >>>   mlx5: remove RX scatter support
> >>>   mlx5: remove configuration variable for maximum number of segments
> >>>   mlx5: remove inline TX support
> >>>   mlx5: split TX queue structure
> >>>   mlx5: split RX queue structure
> >>>   mlx5: update prerequisites for upcoming enhancements
> >>>   mlx5: add definitions for data path without Verbs
> >>>   mlx5: add support for configuration through kvargs
> >>>   mlx5: add TX/RX burst function selection wrapper
> >>>   mlx5: refactor RX data path
> >>>   mlx5: refactor TX data path
> >>>   mlx5: handle RX CQE compression
> >>>   mlx5: add support for multi-packet send
> >>>
> >>> Yaacov Hazan (1):
> >>>   mlx5: add support for inline send
> >>>
> >>
> >> I run basic checks to the patchset:
> >>
> >> There are various checkpatch warnings, all are warning or check level
> >>
> >> Patch 8 and 13 failed to apply with via git, -looks line line numbers
> >> shifted a little, this is not a problem since eventually it applies but
> >> just for your information.
> >>
> >> check-git-log is giving following errors, it is mainly case issue in Rx/Tx:
> >> Wrong headline lowercase:
> >>         mlx5: resurrect RX scatter support
> >>         mlx5: make RX queue reinitialization safer
> >>         mlx5: remove redundant RX queue initialization code
> >>         mlx5: resurrect TX gather support
> >>         mlx5: check remaining space while processing TX burst
> >>         mlx5: add debugging information about TX queues capabilities
> >>         mlx5: replace countdown with threshold for TX completions
> >>         mlx5: handle RX CQE compression
> >>         mlx5: refactor RX data path
> >>         mlx5: add TX/RX burst function selection wrapper
> >>         mlx5: split RX queue structure
> >>         mlx5: split TX queue structure
> >>         mlx5: remove inline TX support
> >>         mlx5: remove RX scatter support
> >>         mlx5: remove TX gather support
> >> Headline too long:
> >>         mlx5: remove configuration variable for maximum number of segments
> >>         mlx5: split memory registration function for better performance
> >>
> >>
> >> It compiles fine.
> >>
> >> Regards,
> >> ferruh
> > 
> > Hi ferruh,
> > 
> > In fact, It does not apply well on top the current DPDK master branch.
> > 
> 
> I did able to apply on top of rel_16_07 branch using "patch" binary. but
> if you think it doesn't apply well, any plan to send a new version?
> 
> Thanks,
> ferruh
 
I am finishing the V2, with some small fixes (it will be more detailed
in the cover letter).

It will be sent in few minutes the necessary time to run the check-*
scripts on it.

Thanks,

-- 
Nélio Laranjeiro
6WIND