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

Leon Romanovsky
2016-12-10T14:47:25Z

[-- 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 --]