Re: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-15 Thread Hal Rosenstock
On 4/13/2015 3:50 PM, Jason Gunthorpe wrote: > Less clear is how rocee vs ib work within a device... Can you APM > between those two kinds of ports? The specs allow this to work but AFAIK it's not implemented. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body o

Re: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-14 Thread Michael Wang
On 04/14/2015 05:50 PM, ira.weiny wrote: > On Tue, Apr 14, 2015 at 10:35:34AM +0200, Michael Wang wrote: >> >> >> On 04/13/2015 09:25 PM, Hefty, Sean wrote: @@ -1037,17 +1033,13 @@ void rdma_destroy_id(struct rdma_cm_id *id) mutex_unlock(&id_priv->handler_mutex); if (id_

Re: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-14 Thread ira.weiny
On Tue, Apr 14, 2015 at 10:35:34AM +0200, Michael Wang wrote: > > > On 04/13/2015 09:25 PM, Hefty, Sean wrote: > >> @@ -1037,17 +1033,13 @@ void rdma_destroy_id(struct rdma_cm_id *id) > >>mutex_unlock(&id_priv->handler_mutex); > >> > >>if (id_priv->cma_dev) { > >> - switch (rdma_

Re: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-14 Thread Michael Wang
On 04/13/2015 09:25 PM, Hefty, Sean wrote: >> @@ -1037,17 +1033,13 @@ void rdma_destroy_id(struct rdma_cm_id *id) >> mutex_unlock(&id_priv->handler_mutex); >> >> if (id_priv->cma_dev) { >> -switch (rdma_node_get_transport(id_priv->id.device- >>> node_type)) { >> -

RE: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-13 Thread Hefty, Sean
> On Mon, Apr 13, 2015 at 07:25:48PM +, Hefty, Sean wrote: > > > @@ -1037,17 +1033,13 @@ void rdma_destroy_id(struct rdma_cm_id *id) > > > mutex_unlock(&id_priv->handler_mutex); > > > > > > if (id_priv->cma_dev) { > > > - switch (rdma_node_get_transport(id_priv->id.device- > > > >no

Re: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-13 Thread Jason Gunthorpe
On Mon, Apr 13, 2015 at 07:25:48PM +, Hefty, Sean wrote: > > @@ -1037,17 +1033,13 @@ void rdma_destroy_id(struct rdma_cm_id *id) > > mutex_unlock(&id_priv->handler_mutex); > > > > if (id_priv->cma_dev) { > > - switch (rdma_node_get_transport(id_priv->id.device- > > >node_type

RE: [PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-13 Thread Hefty, Sean
> @@ -1037,17 +1033,13 @@ void rdma_destroy_id(struct rdma_cm_id *id) > mutex_unlock(&id_priv->handler_mutex); > > if (id_priv->cma_dev) { > - switch (rdma_node_get_transport(id_priv->id.device- > >node_type)) { > - case RDMA_TRANSPORT_IB: > + if (rd

[PATCH v3 10/28] IB/Verbs: Reform cm related part in IB-core cma

2015-04-13 Thread Michael Wang
Use raw management helpers to reform cm related part in IB-core cma. Cc: Steve Wise Cc: Tom Talpey Cc: Jason Gunthorpe Cc: Doug Ledford Cc: Ira Weiny Cc: Sean Hefty Signed-off-by: Michael Wang --- drivers/infiniband/core/cma.c | 79 ++- 1 file chang