RE: [PATCH v2] libibverbs: add path record definitions to sa.h
Hello Steve, I want to add a change preventing creation of the L2 RAW_QPT from user priviledge (uid = 0 will be able to do such operation) What is the best place to do such change: ibv_create_qp in libibverbs(verbs.c) or allowing to decide for NIC vendors if they want to enable such API to user or root. In that case the change is requested only for libnes library? Regards, Mirek -Original Message- From: Steve Wise [mailto:sw...@opengridcomputing.com] Sent: Wednesday, May 19, 2010 6:00 PM To: Walukiewicz, Miroslaw Cc: Roland Dreier; Hefty, Sean; linux-rdma Subject: Re: [PATCH v2] libibverbs: add path record definitions to sa.h Walukiewicz, Miroslaw wrote: Hello Steve, Do you plan some changes in the core code related to RAW_QPT? The only changes I see needed to the kernel core is the mcast change you already proposed to allow mcast attach/detach for RAW_ETY qps... Could you explain me better what means priviledged interface for you? I just mean that allocating these raw qps should only be allowed by effective UID 0. This is analogous to PF_PACKET sockets which are privileged as well. Steve. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v2] libibverbs: add path record definitions to sa.h
Hello Steve, Do you plan some changes in the core code related to RAW_QPT? Could you explain me better what means priviledged interface for you? Regards, Mirek -Original Message- From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-ow...@vger.kernel.org] On Behalf Of Steve Wise Sent: Tuesday, May 18, 2010 4:04 PM To: Roland Dreier Cc: Hefty, Sean; linux-rdma Subject: Re: [PATCH v2] libibverbs: add path record definitions to sa.h Roland Dreier wrote: Can you add the RAW_ETY qp type in this release as well? To be honest I haven't looked at the iWARP datagram stuff at all. I'm not sure overloading the RAW_ETY QP type is necessarily the right thing to do -- it has quite different (never implemented) semantics in the IB case. Is there any overview of what you guys are planning as far as how work requests are created for such QPs? The RAW_ETY qp would be just that: A kernel-bypass/user mode qp that allows sending/receiving ethernet packets. It would also provide a way for user applications to join/leave ethernet mcast groups (which requires an rdma core kernel change that Intel posted too). What the iWARP vendors are doing on top of that is implementing some form of UDP in user mode. The main goal here is to provide an ultra low latency UDP multicast and unicast channel for important market segments that desire this paradigm. Also, due to the nature of this (send/recv raw eth frames), the interface would be privileged. If you want to wait, then later I'll post patches on how this is being done for cxgb4. But I thought adding the RAW_ETY was definitely a common requirement for Intel and Chelsio. Steve. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
Walukiewicz, Miroslaw wrote: Hello Steve, Do you plan some changes in the core code related to RAW_QPT? The only changes I see needed to the kernel core is the mcast change you already proposed to allow mcast attach/detach for RAW_ETY qps... Could you explain me better what means priviledged interface for you? I just mean that allocating these raw qps should only be allowed by effective UID 0. This is analogous to PF_PACKET sockets which are privileged as well. Steve. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
Steve Wise wrote: Walukiewicz, Miroslaw wrote: Hello Steve, Do you plan some changes in the core code related to RAW_QPT? The only changes I see needed to the kernel core is the mcast change you already proposed to allow mcast attach/detach for RAW_ETY qps... Also, There is a rdmacm kernel change to pass up iwarp L2 addresses once a cm_id has resolved the addresses. I posted it earlier and I think Sean is going to integrate it for 2.6.36. Steve. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v2] libibverbs: add path record definitions to sa.h
Also, There is a rdmacm kernel change to pass up iwarp L2 addresses once a cm_id has resolved the addresses. I posted it earlier and I think Sean is going to integrate it for 2.6.36. I believe that the patch you posted earlier is sufficient for 2.6.35. I just need to update my patch set for AF_IB support, which were written assuming that iWarp devices did not return L2 addresses. - Sean -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
Roland Dreier wrote: Can you add the RAW_ETY qp type in this release as well? To be honest I haven't looked at the iWARP datagram stuff at all. I'm not sure overloading the RAW_ETY QP type is necessarily the right thing to do -- it has quite different (never implemented) semantics in the IB case. Is there any overview of what you guys are planning as far as how work requests are created for such QPs? The RAW_ETY qp would be just that: A kernel-bypass/user mode qp that allows sending/receiving ethernet packets. It would also provide a way for user applications to join/leave ethernet mcast groups (which requires an rdma core kernel change that Intel posted too). What the iWARP vendors are doing on top of that is implementing some form of UDP in user mode. The main goal here is to provide an ultra low latency UDP multicast and unicast channel for important market segments that desire this paradigm. Also, due to the nature of this (send/recv raw eth frames), the interface would be privileged. If you want to wait, then later I'll post patches on how this is being done for cxgb4. But I thought adding the RAW_ETY was definitely a common requirement for Intel and Chelsio. Steve. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
Roland Dreier wrote: thanks, applied this. Will roll a new libibverbs release early next week. Can you add the RAW_ETY qp type in this release as well? -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
Can you add the RAW_ETY qp type in this release as well? To be honest I haven't looked at the iWARP datagram stuff at all. I'm not sure overloading the RAW_ETY QP type is necessarily the right thing to do -- it has quite different (never implemented) semantics in the IB case. Is there any overview of what you guys are planning as far as how work requests are created for such QPs? -- Roland Dreier rola...@cisco.com || For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/index.html -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
thanks, applied this. Will roll a new libibverbs release early next week. -- Roland Dreier rola...@cisco.com || For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/index.html -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v2] libibverbs: add path record definitions to sa.h
Roland, I'd like to release a new version of librdmacm that can support the user space SA query feature in 2.6.33, which will also be part of OFED 1.5.2. Currently, there's a dependency on the path record definition being part of libibverbs. Do you have any opinions on the best way to handle this? I guess, ideally, I'd like to see a released version of libibverbs include this, but I can think of ways around this. - Sean -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] libibverbs: add path record definitions to sa.h
I'd like to release a new version of librdmacm that can support the user space SA query feature in 2.6.33, which will also be part of OFED 1.5.2. Currently, there's a dependency on the path record definition being part of libibverbs. Do you have any opinions on the best way to handle this? I guess, ideally, I'd like to see a released version of libibverbs include this, but I can think of ways around this. I can release an updated version of libibverbs (I have enough stuff pending that this is probably a good idea anyway). However could you do some autoconf stuff so librdmacm works against older libibverbs (but doesn't enable the stuff that can't be done without the missing stuff)? Or maybe it's not worth it. -- Roland Dreier rola...@cisco.com || For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/index.html -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v2] libibverbs: add path record definitions to sa.h
I can release an updated version of libibverbs (I have enough stuff pending that this is probably a good idea anyway). However could you do some autoconf stuff so librdmacm works against older libibverbs (but doesn't enable the stuff that can't be done without the missing stuff)? Or maybe it's not worth it. I'll see if I can figure out how to update the autoconf stuff correctly. Thanks. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html