On 7/6/2011 11:51 AM, Roland Dreier wrote: >> The patch that is not yet approved is >> https://patchwork.kernel.org/patch/523041/ >> >> The objections to date for the patch center on the use of reserved bits to >> key the existence of some vendor specific queries. > > Hmm, I'm not a huge fan of that patch since as you say it uses bits > that the IB spec says are reserved. I'd be more inclined to take Or's > patches which have gotten positive feedback from Sean, if we have to > choose. Hal (and/or anyone else who has more insight into the IBTA > intentions for management architecture), what do you think of the > patch linked above?
When the patch was originally issued, I had commented on this issue (http://www.spinics.net/lists/linux-rdma/msg07020.html). There were two issues I saw: 1. Two CapabilityMask2 bits are set for congestion statistics which appears to be a proprietary feature. #define IB_PMA_CLASS_CAP_PORT_CONGS cpu_to_be32(3 << 30) In general, it's dangerous to use IBA reserved bits as they may be claimed by errata to support new features. Should this feature be taken forward to the IBTA ? 2. A reserved field in some structure was being set for xmit wait rather than updating the structure for the spec errata for this. I think this has been fixed in the latest version of the patch. You had originally commented on this issue in http://www.spinics.net/lists/linux-rdma/msg07058.html I can rereview the patch intoto if this is needed. -- Hal > > - R. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
