mbox series

[rdma-next,v4,0/3] RDMA: handle pointless functions

Message ID 20180727182306.26664-1-kamalheib1@gmail.com (mailing list archive)
Headers show
Series RDMA: handle pointless functions | expand

Message

Kamal Heib July 27, 2018, 6:23 p.m. UTC
This patch set include three patches, the first remove {create,
destroy}_ah from the mandatory verbs, the second patch verify that the
verbs are implemented by the provideors before using them, the last one
remove the pointless function from the providers because no need for
them after the first two patches.

Changes from v3:
- Remove only {create, destroy}_ah from mandatory verbs check.

Changes from v2:
- Remove mandatory verbs check.

Changes from v1:
- Add patch #1.
- Remove check for destroy_ah and process_mad from patch #2.

Thanks,

Kamal Heib (3):
  RDMA/core: Remove {create,destroy}_ah from mandatory verbs
  RDMA/core: Check for verbs callbacks before using them
  RDMA/providers: Remove pointless functions

 drivers/infiniband/core/device.c                |  2 -
 drivers/infiniband/core/uverbs_cmd.c            |  3 +-
 drivers/infiniband/core/verbs.c                 |  3 ++
 drivers/infiniband/hw/cxgb3/iwch_provider.c     | 41 -----------------
 drivers/infiniband/hw/cxgb4/cq.c                |  5 ---
 drivers/infiniband/hw/cxgb4/iw_cxgb4.h          |  1 -
 drivers/infiniband/hw/cxgb4/provider.c          | 41 -----------------
 drivers/infiniband/hw/i40iw/i40iw_verbs.c       | 40 -----------------
 drivers/infiniband/hw/nes/nes_verbs.c           | 60 -------------------------
 drivers/infiniband/hw/ocrdma/ocrdma_ah.c        |  6 ---
 drivers/infiniband/hw/ocrdma/ocrdma_ah.h        |  1 -
 drivers/infiniband/hw/ocrdma/ocrdma_main.c      |  1 -
 drivers/infiniband/hw/vmw_pvrdma/pvrdma_cq.c    | 26 -----------
 drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c  |  3 --
 drivers/infiniband/hw/vmw_pvrdma/pvrdma_srq.c   |  7 ---
 drivers/infiniband/hw/vmw_pvrdma/pvrdma_verbs.h |  7 ---
 16 files changed, 5 insertions(+), 242 deletions(-)

Comments

Jason Gunthorpe July 31, 2018, 2:34 a.m. UTC | #1
On Fri, Jul 27, 2018 at 09:23:03PM +0300, Kamal Heib wrote:
> This patch set include three patches, the first remove {create,
> destroy}_ah from the mandatory verbs, the second patch verify that the
> verbs are implemented by the provideors before using them, the last one
> remove the pointless function from the providers because no need for
> them after the first two patches.
> 
> Changes from v3:
> - Remove only {create, destroy}_ah from mandatory verbs check.
> 
> Changes from v2:
> - Remove mandatory verbs check.
> 
> Changes from v1:
> - Add patch #1.
> - Remove check for destroy_ah and process_mad from patch #2.
> 
> Thanks,
> 
> Kamal Heib (3):
>   RDMA/core: Remove {create,destroy}_ah from mandatory verbs
>   RDMA/core: Check for verbs callbacks before using them
>   RDMA/providers: Remove pointless functions

Applied to for-next

I resolved trivial a merge conflict with the const patch from Bart,
please check here:

https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git/log/?h=wip/jgg-for-next

Thanks,
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html