Re: [PATCH v1 00/22] iser patches for 4.3

[Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

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

On 08/06/2015 11:32 AM, Sagi Grimberg wrote:
> This set is a resend that includes some extra patches that
> piled up in the meantime.
> 
> I still have some patches in the pipe (including initiator/target
> support for remote invalidate) but I'm targeting those to 4.4
> 
> I postponed the comment from Steve on using system PAGE_SIZE
> in patch 20 until I have access to a relevant arch to test on.
> 
> This patch set includes:
> - Small fixes for bugs encountered in testing
> - Small fixes detected by static checkers
> - Memory registration code path rework (consolidate to
>   a single code path that branches only at the actual registration
>   FRWR vs. FMR). This reduces code duplication that exists in current code.
> - Larger IO transfer size support (up to 8MB at the moment) depending on
>   the device capabilities.
> - Optimize Io path by chaining send work requests and posting them
>   only once.
> 
> Changes from v0:
> - Added Fixes: tags to patches 4,6
> - Modified change logs for patche 20,22 to add some more details
>   (comments from Or)
> 
> Adir Lev (1):
>   IB/iser: Maintain connection fmr_pool under a single registration
>     descriptor
> 
> Jenny Falkovich (1):
>   IB/iser: Change some module parameters to be RO
> 
> Sagi Grimberg (20):
>   IB/iser: Change minor assignments and logging prints
>   IB/iser: Remove '.' from log message
>   IB/iser: Fix missing return status check in iser_send_data_out
>   IB/iser: Get rid of un-maintained counters
>   IB/iser: Fix possible bogus DMA unmapping
>   IB/iser: Remove a redundant always-false condition
>   IB/iser: Remove an unneeded print for unaligned memory
>   IB/iser: Introduce struct iser_reg_resources
>   IB/iser: Rename struct fast_reg_descriptor -> iser_fr_desc
>   IB/iser: Remove dead code in fmr_pool alloc/free
>   IB/iser: Introduce iser_reg_ops
>   IB/iser: Move fastreg descriptor allocation to
>     iser_create_fastreg_desc
>   IB/iser: Introduce iser registration pool struct
>   IB/iser: Rename iser_reg_page_vec to iser_fast_reg_fmr
>   IB/iser: Make reg_desc_get a per device routine
>   IB/iser: Unify fast memory registration flows
>   IB/iser: Pass registration pool a size parameter
>   IB/iser: Support up to 8MB data transfer in a single command
>   IB/iser: Add debug prints to the various memory registration methods
>   IB/iser: Chain all iser transaction send work requests
> 
>  drivers/infiniband/ulp/iser/iscsi_iser.c     |  89 +++--
>  drivers/infiniband/ulp/iser/iscsi_iser.h     | 206 ++++++++----
>  drivers/infiniband/ulp/iser/iser_initiator.c |  34 +-
>  drivers/infiniband/ulp/iser/iser_memory.c    | 480 +++++++++++++++------------
>  drivers/infiniband/ulp/iser/iser_verbs.c     | 328 ++++++++++--------
>  5 files changed, 645 insertions(+), 492 deletions(-)
> 

I've pulled these in as a replacement for the v0 patches.

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]