Commit c757dea8 authored by Michael Wang's avatar Michael Wang Committed by Doug Ledford

IB/Verbs: Use management helper rdma_cap_ib_mad()

Introduce helper rdma_cap_ib_mad() to help us check if the port of an
IB device support Infiniband Management Datagrams.
Signed-off-by: default avatarMichael Wang <yun.wang@profitbricks.com>
Reviewed-by: default avatarIra Weiny <ira.weiny@intel.com>
Tested-by: default avatarIra Weiny <ira.weiny@intel.com>
Reviewed-by: default avatarSean Hefty <sean.hefty@intel.com>
Reviewed-by: default avatarJason Gunthorpe <jgunthorpe@obsidianresearch.com>
Tested-by: default avatarDoug Ledford <dledford@redhat.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent fef60902
...@@ -3066,7 +3066,7 @@ static void ib_mad_init_device(struct ib_device *device) ...@@ -3066,7 +3066,7 @@ static void ib_mad_init_device(struct ib_device *device)
} }
for (i = start; i <= end; i++) { for (i = start; i <= end; i++) {
if (!rdma_ib_or_iboe(device, i)) if (!rdma_cap_ib_mad(device, i))
continue; continue;
if (ib_mad_port_open(device, i)) { if (ib_mad_port_open(device, i)) {
...@@ -3087,7 +3087,7 @@ static void ib_mad_init_device(struct ib_device *device) ...@@ -3087,7 +3087,7 @@ static void ib_mad_init_device(struct ib_device *device)
error: error:
while (--i >= start) { while (--i >= start) {
if (!rdma_ib_or_iboe(device, i)) if (!rdma_cap_ib_mad(device, i))
continue; continue;
if (ib_agent_port_close(device, i)) if (ib_agent_port_close(device, i))
...@@ -3111,7 +3111,7 @@ static void ib_mad_remove_device(struct ib_device *device) ...@@ -3111,7 +3111,7 @@ static void ib_mad_remove_device(struct ib_device *device)
} }
for (i = start; i <= end; i++) { for (i = start; i <= end; i++) {
if (!rdma_ib_or_iboe(device, i)) if (!rdma_cap_ib_mad(device, i))
continue; continue;
if (ib_agent_port_close(device, i)) if (ib_agent_port_close(device, i))
......
...@@ -1294,7 +1294,7 @@ static void ib_umad_add_one(struct ib_device *device) ...@@ -1294,7 +1294,7 @@ static void ib_umad_add_one(struct ib_device *device)
umad_dev->end_port = e; umad_dev->end_port = e;
for (i = s; i <= e; ++i) { for (i = s; i <= e; ++i) {
if (!rdma_ib_or_iboe(device, i)) if (!rdma_cap_ib_mad(device, i))
continue; continue;
umad_dev->port[i - s].umad_dev = umad_dev; umad_dev->port[i - s].umad_dev = umad_dev;
...@@ -1315,7 +1315,7 @@ static void ib_umad_add_one(struct ib_device *device) ...@@ -1315,7 +1315,7 @@ static void ib_umad_add_one(struct ib_device *device)
err: err:
while (--i >= s) { while (--i >= s) {
if (!rdma_ib_or_iboe(device, i)) if (!rdma_cap_ib_mad(device, i))
continue; continue;
ib_umad_kill_port(&umad_dev->port[i - s]); ib_umad_kill_port(&umad_dev->port[i - s]);
...@@ -1333,7 +1333,7 @@ static void ib_umad_remove_one(struct ib_device *device) ...@@ -1333,7 +1333,7 @@ static void ib_umad_remove_one(struct ib_device *device)
return; return;
for (i = 0; i <= umad_dev->end_port - umad_dev->start_port; ++i) { for (i = 0; i <= umad_dev->end_port - umad_dev->start_port; ++i) {
if (rdma_ib_or_iboe(device, i)) if (rdma_cap_ib_mad(device, i))
ib_umad_kill_port(&umad_dev->port[i]); ib_umad_kill_port(&umad_dev->port[i]);
} }
......
...@@ -1774,6 +1774,21 @@ static inline bool rdma_ib_or_iboe(struct ib_device *device, u8 port_num) ...@@ -1774,6 +1774,21 @@ static inline bool rdma_ib_or_iboe(struct ib_device *device, u8 port_num)
return (pt == RDMA_PROTOCOL_IB || pt == RDMA_PROTOCOL_IBOE); return (pt == RDMA_PROTOCOL_IB || pt == RDMA_PROTOCOL_IBOE);
} }
/**
* rdma_cap_ib_mad - Check if the port of device has the capability Infiniband
* Management Datagrams.
*
* @device: Device to be checked
* @port_num: Port number of the device
*
* Return false when port of the device don't support Infiniband
* Management Datagrams.
*/
static inline bool rdma_cap_ib_mad(struct ib_device *device, u8 port_num)
{
return rdma_ib_or_iboe(device, port_num);
}
int ib_query_gid(struct ib_device *device, int ib_query_gid(struct ib_device *device,
u8 port_num, int index, union ib_gid *gid); u8 port_num, int index, union ib_gid *gid);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment