Re: [PATCH V2 00/10] i40iw: Fixes and optimizations

[Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1743 bytes --]

On Fri, Dec 09, 2016 at 11:54:54AM -0600, Tatyana Nikolova wrote:
> The patch series includes fixes, optimizations and
> clean up to the user space i40iw.
> 
> V2 Changes:
> 
> Use do/while(0) in i40iw_debug macro.
> 
> Mustafa Ismail (4):
>   i40iw: Optimize setting fragments
>   i40iw: Remove unnecessary parameter
>   i40iw: Optimize inline data copy
>   i40iw: Remove unnecessary check for moving CQ head
> 
> Shiraz Saleem (6):
>   i40iw: Fix incorrect assignment of SQ head
>   i40iw: Return correct error codes for destroy QP and CQ
>   i40iw: Do not destroy QP/CQ if lock is held
>   i40iw: Control debug error prints using env variable
>   i40iw: Use 2M huge pages for CQ/QP memory if available
>   i40iw: Remove SQ size constraint

Hi Tatyana,

I merged all patches except "i40iw: Control debug error prints using
env variable" and "i40iw: Use 2M huge pages for CQ/QP memory if available".

For the first patch, I gave rationale by responding to that patch. The
seond patch adds new environmental variable, which is better to be in
common place.

Thanks

> 
>  providers/i40iw/i40iw_d.h      |   6 +-
>  providers/i40iw/i40iw_uk.c     |  62 ++++++++------------
>  providers/i40iw/i40iw_umain.c  |  21 ++++++-
>  providers/i40iw/i40iw_umain.h  |  11 ++++
>  providers/i40iw/i40iw_user.h   |   2 +-
>  providers/i40iw/i40iw_uverbs.c | 128 ++++++++++++++++++++++++++++++-----------
>  6 files changed, 153 insertions(+), 77 deletions(-)
> 
> -- 
> 1.8.5.2
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]