Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-21 Thread Krishna Kumar2
Hi Roland Sean, What is your opinion on this patch set ? Anything else needs to be done for acceptance ? Thanks, - KK [EMAIL PROTECTED] wrote on 08/16/2006 11:42:35 AM: Hi James, Sorry for the delay, we had a long weekend. My opinion is that the create_qp taking generic parameters

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-21 Thread Roland Dreier
Krishna Hi Roland Sean, What is your opinion on this patch set Krishna ? Anything else needs to be done for acceptance ? It's a very low priority for me, since it's a pain to merge and a pain to maintain, and I don't see any urgency in renaming functions. I'll try to get to it after

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-21 Thread Sean Hefty
Krishna Kumar2 wrote: What is your opinion on this patch set ? Anything else needs to be done for acceptance ? I don't have any issues with it, but Roland would need to commit the changes to verbs as the first step. - Sean ___ openib-general

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-21 Thread Krishna Kumar2
Hi Roland, Krishna Hi Roland Sean, What is your opinion on this patch set Krishna ? Anything else needs to be done for acceptance ? It's a very low priority for me, since it's a pain to merge and a pain to maintain I understand. If you feel that I can reduce the pain by sending an

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-16 Thread Krishna Kumar2
Hi James, Sorry for the delay, we had a long weekend. My opinion is that the create_qp taking generic parameters is correct, only subsequent calls may need to use transport specific calls/arguments. Infact rdma_create_qp uses the ibv_create_qp (now changed to rdmav_create_qp) call

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-11 Thread James Lentini
On Wed, 9 Aug 2006, Krishna Kumar2 wrote: Hi James, Sorry for the late response, my system was down and I just got it fixed. Is there a benefit to having rdmav_create_qp() take generic parameters if the application needs to understand the type of QP (IB, iWARP, etc.) created and

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-11 Thread Tom Tucker
On Fri, 2006-08-11 at 10:20 -0400, James Lentini wrote: On Wed, 9 Aug 2006, Krishna Kumar2 wrote: Hi James, Sorry for the late response, my system was down and I just got it fixed. Is there a benefit to having rdmav_create_qp() take generic parameters if the application needs

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-11 Thread James Lentini
On Fri, 11 Aug 2006, Tom Tucker wrote: On Fri, 2006-08-11 at 10:20 -0400, James Lentini wrote: On Wed, 9 Aug 2006, Krishna Kumar2 wrote: Hi James, Sorry for the late response, my system was down and I just got it fixed. Is there a benefit to having rdmav_create_qp()

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-11 Thread Tom Tucker
[...snip...] I think the QP is associated with the transport type indirectly through the context. It can be queried with ibv_get_transport_type verb. A renamed rdma_get_transport type would probably suffice. We don't have a userspace ibv_get_transport_type() verb. There is a kernel

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-09 Thread Krishna Kumar2
Hi James, Sorry for the late response, my system was down and I just got it fixed. Is there a benefit to having rdmav_create_qp() take generic parameters if the application needs to understand the type of QP (IB, iWARP, etc.) created and the transport specific communication manager calls

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-07 Thread Krishna Kumar2
Hi James, Are the rdmav_ versions intended to be generic or are they intended for use with the native communications managers (IB CM and iWARP CM)? It is true that the rdmav_ version is generic. But using the rdmav_ routines means knowing the type of transport interface (eg, I need to do

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-07 Thread James Lentini
On Mon, 7 Aug 2006, Krishna Kumar2 wrote: Hi James, Are the rdmav_ versions intended to be generic or are they intended for use with the native communications managers (IB CM and iWARP CM)? It is true that the rdmav_ version is generic. But using the rdmav_ routines means knowing

[openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-03 Thread Krishna Kumar
This patchset is a proposal to create new API's and data structures with transport neutral names. The idea is to remove the old API once all libraries/applications/examples are gradually converted to use the new API. Patch 1/6 - Changes to libibverbs configuration file to build the libibverbs

[openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-03 Thread Krishna Kumar
[REPOSTING - as 2 patches in my first set didn't seem to have got to openib mailing list though I checked after 2 hours. Apologies in advance since people will get duplicates of atleast 4 of the successful patches] This patchset is a proposal to create new API's and data structures with transport

Re: [openib-general] [PATCH v3 0/6] Tranport Neutral Verbs Proposal.

2006-08-03 Thread James Lentini
On Thu, 3 Aug 2006, Krishna Kumar wrote: 2. Changed rdma_ to rdmav_. This also enabled to retain rdma_create_qp() and rdma_destroy_qp() routines. I'm glad to see that the generic RDMA verbs are becoming a reality. Exporting QP create/destroy functions from both the RDMA CM library and