On Fri, 22 Feb 2013 18:47:37 +0000 "Hefty, Sean" <[email protected]> wrote:
> > +char * umad_class_str(uint8_t mgmt_class); > > +char * umad_method_str(uint8_t mgmt_class, uint8_t method); > > +char * umad_attribute_str(uint8_t mgmt_class, be16_t attr_id); > > +char * umad_mad_status_str(be16_t status, char * buf, size_t len); > > Why is umad_mad_status_str() different? Which difference are you speaking of, the lack of mgmt_class or the required buffer? I did not originally intend to include class specific status decoding. But it would probably be worth including. Especially with your last patch. I will add the mgmt_class parameter. The required buffer allows for printing multiple strings for various fields which may be set. This is done because status is not defined to be a strict enum. (Although it might be that all current use cases result in unique values being returned.) Ira > -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 [email protected] -- 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
