lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <51DD714D.4080001@profitbricks.com>
Date:	Wed, 10 Jul 2013 16:35:57 +0200
From:	Sebastian Riemer <sebastian.riemer@...fitbricks.com>
To:	Roland Dreier <roland@...nel.org>
CC:	Linus Torvalds <torvalds@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, linux-rdma@...r.kernel.org
Subject: Re: [GIT PULL] please pull infiniband.git

Hi Roland,

I've checked the commits on that tag and the following commit is not
what we've agreed on:

commit 086f44f58855ae18bab19fb794cce6c6d2c6143b
Author: Bart Van Assche <bvanassche@....org>

    IB/srp: Avoid skipping srp_reset_host() after a transport error

Together with the following commit this results in the wrong return code
if transport_offline (SUCCESS instead of FAST_IO_FAIL):

commit 99e1c1398f44a056b16e78122133988c82b66d97
Author: Bart Van Assche <bvanassche@....org>

    IB/srp: Fail I/O fast if target offline


Resulting code:
1755         if (srp_send_tsk_mgmt(target, req->index, scmnd->device->lun,
1756                               SRP_TSK_ABORT_TASK) == 0 ||
1757             target->transport_offline)
1758                 ret = SUCCESS;
1759         else if (target->transport_offline)
1760                 ret = FAST_IO_FAIL;
1761         else
1762                 ret = FAILED;

This doesn't make sense. The check for transport_offline has to be
removed for the SUCCESS case! I've already told that on linux-rdma.

Cheers,
Sebastian


On 09.07.2013 19:36, Roland Dreier wrote:
> Hi Linus,
> 
> Please pull from
> 
>     git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git tags/rdma-for-linus
> 
> ----------------------------------------------------------------
> 
> Main batch of InfiniBand/RDMA changes for 3.11 merge window:
> 
>  - AF_IB (native IB addressing) for CMA from Sean Hefty
>  - New mlx5 driver for Mellanox Connect-IB adapters
>  - SRP fixes from Bart Van Assche
>  - qib HW driver updates
>  - Resurrection of ocrdma HW driver development
>  - uverbs conversion to create fds with O_CLOEXEC set
>  - Other small changes and fixes
> 
> ----------------------------------------------------------------
> Bart Van Assche (5):
>       IB/srp: Avoid skipping srp_reset_host() after a transport error
>       IB/srp: Skip host settle delay
>       IB/srp: Fail I/O fast if target offline
>       IB/srp: Maintain a single connection per I_T nexus
>       IB/srp: Make HCA completion vector configurable
> 
> Dan Carpenter (1):
>       RDMA/cxgb3: Timeout condition is never true
> 
> Dotan Barak (1):
>       IB/srp: Fix remove_one crash due to resource exhaustion
> 
> Eli Cohen (1):
>       mlx5: Add driver for Mellanox Connect-IB adapters
> 
> Gottumukkala, Naresh (1):
>       RDMA/ocrdma: Remove use_cnt for queues
> 
> Jack Morgenstein (1):
>       IB/core: Add reserved values to enums for low-level driver use
> 
> Mike Marciniszyn (6):
>       IB/qib: Add DCA support
>       IB/qib: Remove atomic_inc_not_zero() from QP RCU
>       IB/qib: Optimize CQ callbacks
>       IB/qib: Convert opcode counters to per-context
>       IB/qib: Add per-context stats interface
>       IB/qib: Add qp_stats debug file
> 
> Mitko Haralanov (1):
>       IB/qib: New transmitter tunning settings for Dell 1.1 backplane
> 
> Naresh Gottumukkala (5):
>       RDMA/ocrdma: Use MCC_CREATE_EXT_V1 for MCC create
>       RDMA/ocrdma: Replace ocrdma_err with pr_err
>       RDMA/ocrdma: Set bad_wr in error case
>       RDMA/ocrdma: Change macros to inline funtions
>       RDMA/ocrdma: Reorg structures to avoid padding
> 
> Ramkrishna Vepa (2):
>       IB/qib: Add optional NUMA affinity
>       IB/qib: Add dual-rail NUMA awareness for PSM processes
> 
> Roland Dreier (5):
>       mlx5: Fix parameter type of health_handler_t
>       IB/mlx5: Make profile[] static in main.c
>       mlx5_core: Fixes for sparse warnings
>       IB/uverbs: Use get_unused_fd_flags(O_CLOEXEC) instead of get_unused_fd()
>       Merge branches 'af_ib', 'cxgb4', 'misc', 'mlx5', 'ocrdma', 'qib' and 'srp' into for-next
> 
> Sean Hefty (28):
>       RDMA/cma: Define native IB address
>       RDMA/cma: Allow enabling reuseaddr in any state
>       RDMA/cma: Include AF_IB in loopback and any address checks
>       IB/addr: Add AF_IB support to ip_addr_size
>       RDMA/cma: Update port reservation to support AF_IB
>       RDMA/cma: Allow user to specify AF_IB when binding
>       RDMA/cma: Do not modify sa_family when setting loopback address
>       RDMA/cma: Add helper functions to return id address information
>       RDMA/cma: Restrict AF_IB loopback to binding to IB devices only
>       RDMA/cma: Verify that source and dest sa_family are the same
>       RDMA/cma: Add support for AF_IB to rdma_resolve_addr()
>       RDMA/cma: Add support for AF_IB to rdma_resolve_route()
>       RDMA/cma: Add support for AF_IB to cma_get_service_id()
>       RDMA/cma: Remove unused SDP related code
>       RDMA/cma: Merge cma_get/save_net_info
>       RDMA/cma: Expose private data when using AF_IB
>       RDMA/cma: Set qkey for AF_IB
>       RDMA/cma: Only listen on IB devices when using AF_IB
>       RDMA/ucma: Support querying for AF_IB addresses
>       IB/sa: Export function to pack a path record into wire format
>       RDMA/ucma: Support querying when IB paths are not reversible
>       RDMA/cma: Export cma_get_service_id()
>       RDMA/ucma: Add ability to query GID addresses
>       RDMA/ucma: Name changes to indicate only IP addresses supported
>       RDMA/ucma: Allow user space to bind to AF_IB
>       RDMA/ucma: Allow user space to pass AF_IB into resolve
>       RDMA/ucma: Allow user space to specify AF_IB when joining multicast
>       RDMA/cma: Export AF_IB statistics
> 
> Vinit Agnihotri (1):
>       IB/qib: Update minor version number
> 
> Vu Pham (1):
>       IB/srp: Bump driver version and release date
> 
> Wei Yongjun (3):
>       IB/ehca: Fix error return code in ehca_create_slab_caches()
>       RDMA/ocrdma: Fix error return code in ocrdma_set_create_qp_rq_cmd()
>       IB/core: Fix error return code in add_port()
> 
>  Documentation/ABI/stable/sysfs-driver-ib_srp        |    7 +
>  MAINTAINERS                                         |   22 ++
>  drivers/infiniband/Kconfig                          |    1 +
>  drivers/infiniband/Makefile                         |    1 +
>  drivers/infiniband/core/addr.c                      |   20 +-
>  drivers/infiniband/core/cma.c                       |  906 ++++++++++++++++++++++++++++++++++-------------------------
>  drivers/infiniband/core/sa_query.c                  |    6 +
>  drivers/infiniband/core/sysfs.c                     |    8 +-
>  drivers/infiniband/core/ucma.c                      |  321 ++++++++++++++++++---
>  drivers/infiniband/core/uverbs_cmd.c                |    4 +-
>  drivers/infiniband/hw/cxgb3/iwch_qp.c               |    3 +-
>  drivers/infiniband/hw/ehca/ehca_main.c              |    1 +
>  drivers/infiniband/hw/mlx5/Kconfig                  |   10 +
>  drivers/infiniband/hw/mlx5/Makefile                 |    3 +
>  drivers/infiniband/hw/mlx5/ah.c                     |   92 ++++++
>  drivers/infiniband/hw/mlx5/cq.c                     |  843 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/infiniband/hw/mlx5/doorbell.c               |  100 +++++++
>  drivers/infiniband/hw/mlx5/mad.c                    |  139 +++++++++
>  drivers/infiniband/hw/mlx5/main.c                   | 1504 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/infiniband/hw/mlx5/mem.c                    |  162 +++++++++++
>  drivers/infiniband/hw/mlx5/mlx5_ib.h                |  545 +++++++++++++++++++++++++++++++++++
>  drivers/infiniband/hw/mlx5/mr.c                     | 1007 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/infiniband/hw/mlx5/qp.c                     | 2524 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/infiniband/hw/mlx5/srq.c                    |  473 +++++++++++++++++++++++++++++++
>  drivers/infiniband/hw/mlx5/user.h                   |  121 ++++++++
>  drivers/infiniband/hw/ocrdma/ocrdma.h               |   63 +++--
>  drivers/infiniband/hw/ocrdma/ocrdma_hw.c            |   86 +++---
>  drivers/infiniband/hw/ocrdma/ocrdma_main.c          |    6 +-
>  drivers/infiniband/hw/ocrdma/ocrdma_sli.h           |   35 +--
>  drivers/infiniband/hw/ocrdma/ocrdma_verbs.c         |  135 ++++-----
>  drivers/infiniband/hw/qib/Kconfig                   |    8 +
>  drivers/infiniband/hw/qib/Makefile                  |    1 +
>  drivers/infiniband/hw/qib/qib.h                     |   61 ++--
>  drivers/infiniband/hw/qib/qib_common.h              |    2 +-
>  drivers/infiniband/hw/qib/qib_cq.c                  |   67 ++++-
>  drivers/infiniband/hw/qib/qib_debugfs.c             |  283 +++++++++++++++++++
>  drivers/infiniband/hw/qib/qib_debugfs.h             |   45 +++
>  drivers/infiniband/hw/qib/qib_driver.c              |    1 -
>  drivers/infiniband/hw/qib/qib_file_ops.c            |  176 ++++++++----
>  drivers/infiniband/hw/qib/qib_iba6120.c             |   10 +
>  drivers/infiniband/hw/qib/qib_iba7220.c             |   10 +
>  drivers/infiniband/hw/qib/qib_iba7322.c             |  393 +++++++++++++++++++++++---
>  drivers/infiniband/hw/qib/qib_init.c                |  139 +++++++--
>  drivers/infiniband/hw/qib/qib_qp.c                  |  123 +++++++-
>  drivers/infiniband/hw/qib/qib_verbs.c               |    8 +-
>  drivers/infiniband/hw/qib/qib_verbs.h               |   33 ++-
>  drivers/infiniband/ulp/srp/ib_srp.c                 |   90 +++++-
>  drivers/infiniband/ulp/srp/ib_srp.h                 |    1 +
>  drivers/net/ethernet/mellanox/Kconfig               |    1 +
>  drivers/net/ethernet/mellanox/Makefile              |    1 +
>  drivers/net/ethernet/mellanox/mlx5/core/Kconfig     |   18 ++
>  drivers/net/ethernet/mellanox/mlx5/core/Makefile    |    5 +
>  drivers/net/ethernet/mellanox/mlx5/core/alloc.c     |  238 ++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/cmd.c       | 1515 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/cq.c        |  224 +++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/debugfs.c   |  587 ++++++++++++++++++++++++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/eq.c        |  521 ++++++++++++++++++++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/fw.c        |  185 ++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/health.c    |  227 +++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/mad.c       |   78 ++++++
>  drivers/net/ethernet/mellanox/mlx5/core/main.c      |  475 +++++++++++++++++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/mcg.c       |  106 +++++++
>  drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.h |   73 +++++
>  drivers/net/ethernet/mellanox/mlx5/core/mr.c        |  136 +++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/pagealloc.c |  435 ++++++++++++++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/pd.c        |  101 +++++++
>  drivers/net/ethernet/mellanox/mlx5/core/port.c      |  104 +++++++
>  drivers/net/ethernet/mellanox/mlx5/core/qp.c        |  301 ++++++++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/srq.c       |  223 +++++++++++++++
>  drivers/net/ethernet/mellanox/mlx5/core/uar.c       |  223 +++++++++++++++
>  include/linux/mlx5/cmd.h                            |   51 ++++
>  include/linux/mlx5/cq.h                             |  165 +++++++++++
>  include/linux/mlx5/device.h                         |  893 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/mlx5/doorbell.h                       |   79 ++++++
>  include/linux/mlx5/driver.h                         |  769 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/mlx5/qp.h                             |  467 ++++++++++++++++++++++++++++++
>  include/linux/mlx5/srq.h                            |   41 +++
>  include/linux/socket.h                              |    2 +
>  include/rdma/ib.h                                   |   89 ++++++
>  include/rdma/ib_addr.h                              |    6 +-
>  include/rdma/ib_sa.h                                |    7 +
>  include/rdma/ib_verbs.h                             |   35 ++-
>  include/rdma/rdma_cm.h                              |   13 +
>  include/uapi/rdma/rdma_user_cm.h                    |   73 ++++-
>  84 files changed, 18267 insertions(+), 799 deletions(-)
>  create mode 100644 drivers/infiniband/hw/mlx5/Kconfig
>  create mode 100644 drivers/infiniband/hw/mlx5/Makefile
>  create mode 100644 drivers/infiniband/hw/mlx5/ah.c
>  create mode 100644 drivers/infiniband/hw/mlx5/cq.c
>  create mode 100644 drivers/infiniband/hw/mlx5/doorbell.c
>  create mode 100644 drivers/infiniband/hw/mlx5/mad.c
>  create mode 100644 drivers/infiniband/hw/mlx5/main.c
>  create mode 100644 drivers/infiniband/hw/mlx5/mem.c
>  create mode 100644 drivers/infiniband/hw/mlx5/mlx5_ib.h
>  create mode 100644 drivers/infiniband/hw/mlx5/mr.c
>  create mode 100644 drivers/infiniband/hw/mlx5/qp.c
>  create mode 100644 drivers/infiniband/hw/mlx5/srq.c
>  create mode 100644 drivers/infiniband/hw/mlx5/user.h
>  create mode 100644 drivers/infiniband/hw/qib/qib_debugfs.c
>  create mode 100644 drivers/infiniband/hw/qib/qib_debugfs.h
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/Kconfig
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/Makefile
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/alloc.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/cmd.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/cq.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/debugfs.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/eq.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/fw.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/health.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/mad.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/main.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/mcg.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.h
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/mr.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/pagealloc.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/pd.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/port.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/qp.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/srq.c
>  create mode 100644 drivers/net/ethernet/mellanox/mlx5/core/uar.c
>  create mode 100644 include/linux/mlx5/cmd.h
>  create mode 100644 include/linux/mlx5/cq.h
>  create mode 100644 include/linux/mlx5/device.h
>  create mode 100644 include/linux/mlx5/doorbell.h
>  create mode 100644 include/linux/mlx5/driver.h
>  create mode 100644 include/linux/mlx5/qp.h
>  create mode 100644 include/linux/mlx5/srq.h
>  create mode 100644 include/rdma/ib.h
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ