From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753269AbbC3QX0 (ORCPT ); Mon, 30 Mar 2015 12:23:26 -0400 Received: from mail-wi0-f180.google.com ([209.85.212.180]:33525 "EHLO mail-wi0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752858AbbC3QXU (ORCPT ); Mon, 30 Mar 2015 12:23:20 -0400 Message-ID: <55197874.3080807@profitbricks.com> Date: Mon, 30 Mar 2015 18:23:16 +0200 From: Michael Wang User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Doug Ledford CC: Roland Dreier , Sean Hefty , Hal Rosenstock , Ira Weiny , linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, linux-nfs@vger.kernel.org, netdev@vger.kernel.org, "J. Bruce Fields" , Trond Myklebust , "David S. Miller" , Or Gerlitz , Moni Shoua , PJ Waskiewicz , Tatyana Nikolova , Yan Burman , Jack Morgenstein , Bart Van Assche , Yann Droneaud , Colin Ian King , Majd Dibbiny , Jiri Kosina , Matan Barak , Alex Estrin , Eric Dumazet , Erez Shitrit , Sagi Grimberg , Haggai Eran , Shachar Raindel , Mike Marciniszyn , Steve Wise , Tom Tucker , Chuck Lever Subject: Re: [RFC PATCH 02/11] IB/Verbs: Use management helper tech_iboe() for iboe-check References: <551579CA.4030901@profitbricks.com> <55157A4F.1090301@profitbricks.com> <1427732250.21101.202.camel@redhat.com> In-Reply-To: <1427732250.21101.202.camel@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/30/2015 06:17 PM, Doug Ledford wrote: > On Fri, 2015-03-27 at 16:42 +0100, Michael Wang wrote: >> Introduce helper tech_iboe() to help us check if the port of an IB >> device is using RoCE/IBoE technology. > Just use rdma_transport_is_roce() instead. Sounds good :-) will be in next version. Regards, Michael Wang > >> Cc: Jason Gunthorpe >> Cc: Doug Ledford >> Cc: Ira Weiny >> Cc: Sean Hefty >> Signed-off-by: Michael Wang >> --- >> drivers/infiniband/core/cma.c | 6 ++---- >> include/rdma/ib_verbs.h | 16 ++++++++++++++++ >> 2 files changed, 18 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/infiniband/core/cma.c b/drivers/infiniband/core/cma.c >> index 668e955..280cfe3 100644 >> --- a/drivers/infiniband/core/cma.c >> +++ b/drivers/infiniband/core/cma.c >> @@ -375,8 +375,7 @@ static int cma_acquire_dev(struct rdma_id_private *id_priv, >> listen_id_priv->id.port_num) == dev_ll) { >> cma_dev = listen_id_priv->cma_dev; >> port = listen_id_priv->id.port_num; >> - if (rdma_transport_is_ib(cma_dev->device) && >> - rdma_port_ll_is_eth(cma_dev->device, port)) >> + if (tech_iboe(cma_dev->device, port)) >> ret = ib_find_cached_gid(cma_dev->device, &iboe_gid, >> &found_port, NULL); >> else >> @@ -395,8 +394,7 @@ static int cma_acquire_dev(struct rdma_id_private *id_priv, >> listen_id_priv->id.port_num == port) >> continue; >> if (rdma_port_get_link_layer(cma_dev->device, port) == dev_ll) { >> - if (rdma_transport_is_ib(cma_dev->device) && >> - rdma_port_ll_is_eth(cma_dev->device, port)) >> + if (tech_iboe(cma_dev->device, port)) >> ret = ib_find_cached_gid(cma_dev->device, &iboe_gid, &found_port, NULL); >> else >> ret = ib_find_cached_gid(cma_dev->device, &gid, &found_port, NULL); >> diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h >> index 2bf9094..ca6d6bc 100644 >> --- a/include/rdma/ib_verbs.h >> +++ b/include/rdma/ib_verbs.h >> @@ -1767,6 +1767,22 @@ static inline int rdma_port_ll_is_eth(struct ib_device *device, u8 port_num) >> == IB_LINK_LAYER_ETHERNET; >> } >> >> +/** >> + * tech_iboe - Check if the port of device using technology >> + * RoCE/IBoE. >> + * >> + * @device: Device to be checked >> + * @port_num: Port number of the device >> + * >> + * Return 0 when port of the device is not using technology >> + * RoCE/IBoE. >> + */ >> +static inline int tech_iboe(struct ib_device *device, u8 port_num) >> +{ >> + return rdma_transport_is_ib(device) && >> + rdma_port_ll_is_eth(device, port_num); >> +} >> + >> int ib_query_gid(struct ib_device *device, >> u8 port_num, int index, union ib_gid *gid); >> >