Re: [PATCH V2] libibumad: add string functions for various enums

[Hal Rosenstock <hal-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>]

On 2/21/2013 6:02 PM, Ira Weiny wrote:
> 
> 
> Some of this code was originally proposed by Sean in 2010 as part of user space
> madeye: http://www.mail-archive.com/linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org/msg06185.html
> 
> Changes from V1:
> 	Move functions into library calls
> 	remove funky marco obfuscation
> 	remove public access to umad_[sm|sa|cm]_attr_str functions
> 
> Signed-off-by: Ira Weiny <weiny2-i2BcT+NCU+M@public.gmane.org>
> ---

This looks mostly good to me. Some minor comments embedded below.

>  Makefile.am                     |   12 +-
>  include/infiniband/umad_sm.h    |   31 ++--
>  include/infiniband/umad_str.h   |   56 +++++++
>  include/infiniband/umad_types.h |    4 +-
>  man/umad_attribute_str.3        |    3 +
>  man/umad_class_str.3            |   39 +++++
>  man/umad_mad_status_str.3       |    3 +
>  man/umad_method_str.3           |    3 +
>  src/libibumad.map               |    4 +
>  src/umad_str.c                  |  318 +++++++++++++++++++++++++++++++++++++++
>  10 files changed, 454 insertions(+), 19 deletions(-)
>  create mode 100644 include/infiniband/umad_str.h
>  create mode 100644 man/umad_attribute_str.3
>  create mode 100644 man/umad_class_str.3
>  create mode 100644 man/umad_mad_status_str.3
>  create mode 100644 man/umad_method_str.3
>  create mode 100644 src/umad_str.c
> 
> diff --git a/Makefile.am b/Makefile.am
> index 14e2389..dc34c3d 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -15,7 +15,12 @@ man_MANS = man/umad_debug.3 man/umad_get_ca.3 \
>  	   man/umad_get_pkey.3 \
>  	   man/umad_register.3 man/umad_register_oui.3 man/umad_unregister.3 \
>  	   man/umad_send.3 man/umad_recv.3 man/umad_poll.3 \
> -	   man/umad_get_issm_path.3
> +	   man/umad_get_issm_path.3 \
> +	   man/umad_attribute_str.3 \
> +	   man/umad_class_str.3 \
> +	   man/umad_mad_status_str.3 \
> +	   man/umad_method_str.3
> +
>  
>  lib_LTLIBRARIES = libibumad.la
>  
> @@ -27,7 +32,7 @@ else
>  libibumad_version_script =
>  endif
>  
> -libibumad_la_SOURCES = src/umad.c src/sysfs.c
> +libibumad_la_SOURCES = src/umad.c src/sysfs.c src/umad_str.c
>  libibumad_la_LDFLAGS = -version-info $(ibumad_api_version) \
>      -export-dynamic $(libibumad_version_script)
>  libibumad_la_DEPENDENCIES = $(srcdir)/src/libibumad.map
> @@ -38,7 +43,8 @@ libibumadinclude_HEADERS = $(srcdir)/include/infiniband/umad.h \
>  	$(srcdir)/include/infiniband/umad_types.h \
>  	$(srcdir)/include/infiniband/umad_sa.h \
>  	$(srcdir)/include/infiniband/umad_sm.h \
> -	$(srcdir)/include/infiniband/umad_cm.h
> +	$(srcdir)/include/infiniband/umad_cm.h \
> +	$(srcdir)/include/infiniband/umad_str.h
>  
>  EXTRA_DIST = $(srcdir)/include/infiniband/umad.h \
>  	libibumad.spec.in libibumad.spec \
> diff --git a/include/infiniband/umad_sm.h b/include/infiniband/umad_sm.h
> index 10348d4..f6ee171 100644
> --- a/include/infiniband/umad_sm.h
> +++ b/include/infiniband/umad_sm.h
> @@ -55,21 +55,22 @@ enum {
>  
>  /* Subnet management attributes */
>  enum {
> -	UMAD_SMP_ATTR_NODE_DESC		= 0x0010,
> -	UMAD_SMP_ATTR_NODE_INFO		= 0x0011,
> -	UMAD_SMP_ATTR_SWITCH_INFO	= 0x0012,
> -	UMAD_SMP_ATTR_GUID_INFO		= 0x0014,
> -	UMAD_SMP_ATTR_PORT_INFO		= 0x0015,
> -	UMAD_SMP_ATTR_PKEY_TABLE	= 0x0016,
> -	UMAD_SMP_ATTR_SLVL_TABLE	= 0x0017,
> -	UMAD_SMP_ATTR_VL_ARB_TABLE	= 0x0018,
> -	UMAD_SMP_ATTR_LINEAR_FT		= 0x0019,
> -	UMAD_SMP_ATTR_RANDOM_FT		= 0x001A,
> -	UMAD_SMP_ATTR_MCAST_FT		= 0x001B,
> -	UMAD_SMP_ATTR_SM_INFO		= 0x0020,
> -	UMAD_SMP_ATTR_VENDOR_DIAG	= 0x0030,
> -	UMAD_SMP_ATTR_LED_INFO		= 0x0031,
> -	UMAD_SMP_ATTR_VENDOR_MASK	= 0xFF00
> +	UMAD_SMP_ATTR_NODE_DESC			= 0x0010,
> +	UMAD_SMP_ATTR_NODE_INFO			= 0x0011,
> +	UMAD_SMP_ATTR_SWITCH_INFO		= 0x0012,
> +	UMAD_SMP_ATTR_GUID_INFO			= 0x0014,
> +	UMAD_SMP_ATTR_PORT_INFO			= 0x0015,
> +	UMAD_SMP_ATTR_PKEY_TABLE		= 0x0016,
> +	UMAD_SMP_ATTR_SLVL_TABLE		= 0x0017,
> +	UMAD_SMP_ATTR_VL_ARB_TABLE		= 0x0018,
> +	UMAD_SMP_ATTR_LINEAR_FT			= 0x0019,
> +	UMAD_SMP_ATTR_RANDOM_FT			= 0x001A,
> +	UMAD_SMP_ATTR_MCAST_FT			= 0x001B,
> +	UMAD_SMP_ATTR_LINK_SPD_WIDTH_TABLE	= 0x001C,
> +	UMAD_SMP_ATTR_SM_INFO			= 0x0020,
> +	UMAD_SMP_ATTR_VENDOR_DIAG		= 0x0030,
> +	UMAD_SMP_ATTR_LED_INFO			= 0x0031,
> +	UMAD_SMP_ATTR_VENDOR_MASK		= 0xFF00
>  };
>  
>  enum {
> diff --git a/include/infiniband/umad_str.h b/include/infiniband/umad_str.h
> new file mode 100644
> index 0000000..8c5fb9f
> --- /dev/null
> +++ b/include/infiniband/umad_str.h
> @@ -0,0 +1,56 @@
> +/*
> + * Copyright (c) 2004, 2005, 2010 Intel Corporation.  All rights reserved.
> + * Copyright (c) 2013 Lawrence Livermore National Security. All rights reserved.
> + *
> + * This software is available to you under a choice of one of two
> + * licenses.  You may choose to be licensed under the terms of the GNU
> + * General Public License (GPL) Version 2, available from the file
> + * COPYING in the main directory of this source tree, or the
> + * OpenIB.org BSD license below:
> + *
> + *     Redistribution and use in source and binary forms, with or
> + *     without modification, are permitted provided that the following
> + *     conditions are met:
> + *
> + *      - Redistributions of source code must retain the above
> + *        copyright notice, this list of conditions and the following
> + *        disclaimer.
> + *
> + *      - Redistributions in binary form must reproduce the above
> + *        copyright notice, this list of conditions and the following
> + *        disclaimer in the documentation and/or other materials
> + *        provided with the distribution.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
> + * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
> + * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> + * SOFTWARE.
> + *
> + */
> +#ifndef _UMAD_STR_H
> +#define _UMAD_STR_H
> +
> +#include <infiniband/umad.h>
> +
> +#ifdef __cplusplus
> +#  define BEGIN_C_DECLS extern "C" {
> +#  define END_C_DECLS   }
> +#else				/* !__cplusplus */
> +#  define BEGIN_C_DECLS
> +#  define END_C_DECLS
> +#endif				/* __cplusplus */
> +
> +BEGIN_C_DECLS
> +
> +char * umad_class_str(uint8_t mgmt_class);
> +char * umad_method_str(uint8_t method);
> +char * umad_mad_status_str(be16_t status, char * buf, size_t len);
> +
> +char * umad_attribute_str(uint8_t mgmt_class, be16_t attr_id);
> +
> +END_C_DECLS
> +#endif /* _UMAD_STR_H */
> diff --git a/include/infiniband/umad_types.h b/include/infiniband/umad_types.h
> index a722b8c..db13438 100644
> --- a/include/infiniband/umad_types.h
> +++ b/include/infiniband/umad_types.h
> @@ -64,10 +64,12 @@ enum {
>  	UMAD_CLASS_SNMP			= 0x08,
>  	UMAD_CLASS_VENDOR_RANGE1_START	= 0x09,
>  	UMAD_CLASS_VENDOR_RANGE1_END	= 0x0F,
> -	UMAD_CLASS_DEVICE_ADM		= 0x10,
> +	UMAD_CLASS_APPLICATION_START	= 0x10,
> +	UMAD_CLASS_DEVICE_ADM		= UMAD_CLASS_APPLICATION_START,
>  	UMAD_CLASS_BOOT_MGMT		= 0x11,
>  	UMAD_CLASS_BIS			= 0x12,
>  	UMAD_CLASS_CONG_MGMT		= 0x21,
> +	UMAD_CLASS_APPLICATION_END	= 0x2F,
>  	UMAD_CLASS_VENDOR_RANGE2_START	= 0x30,
>  	UMAD_CLASS_VENDOR_RANGE2_END	= 0x4F
>  };
> diff --git a/man/umad_attribute_str.3 b/man/umad_attribute_str.3
> new file mode 100644
> index 0000000..f5e6b02
> --- /dev/null
> +++ b/man/umad_attribute_str.3
> @@ -0,0 +1,3 @@
> +.\" -*- nroff -*-
> +.\"
> +.so man3/umad_class_str.3
> diff --git a/man/umad_class_str.3 b/man/umad_class_str.3
> new file mode 100644
> index 0000000..761f034
> --- /dev/null
> +++ b/man/umad_class_str.3
> @@ -0,0 +1,39 @@
> +.\" -*- nroff -*-
> +.\"
> +.TH UMAD_CLASS_STR 3  "Feb 15, 2013" "OpenIB" "OpenIB Programmer\'s Manual"
> +.SH "NAME"
> +umad_*_str \- class of inline functions to return string representations of enums

These functions are no longer inline, right ?

> +
> +.SH "SYNOPSIS"
> +.nf
> +.B #include <infiniband/umad_str.h>
> +.sp
> +.BI "char * umad_class_str(uint8_t mgmt_class)"
> +.BI "char * umad_method_str(uint8_t method)"
> +.BI "char * umad_mad_status_str(be16_t status, char * buf, size_t len)"
> +
> +.B "char * umad_attribute_str(uint8_t mgmt_class, be16_t attr_id)"
> +
> +.SH "DESCRIPTION"
> +
> +.B "char * umad_class_str(uint8_t mgmt_class)"
> +Return string value of management class enum
> +
> +.B "char * umad_method_str(uint8_t method)"
> +Return string value of method enum
> +
> +.B "char * umad_mad_status_str(be16_t status, char * buf, size_t len)"
> +Print MAD status into buf and return buf
> +
> +.B "char * umad_attribute_str(uint8_t mgmt_class, be16_t attr_id)"
> +Return string value of attribute specified in attr_id based on mgmt_class specified.
> +
> +Currently SM, SA, and CM classes are supported.
> +
> +.SH "RETURN VALUE"
> +
> +Returns a string representations of the fields specified.
> +
> +.SH "AUTHOR"
> +.TP
> +Ira Weiny <weiny2-i2BcT+NCU+M@public.gmane.org>
> diff --git a/man/umad_mad_status_str.3 b/man/umad_mad_status_str.3
> new file mode 100644
> index 0000000..f5e6b02
> --- /dev/null
> +++ b/man/umad_mad_status_str.3
> @@ -0,0 +1,3 @@
> +.\" -*- nroff -*-
> +.\"
> +.so man3/umad_class_str.3
> diff --git a/man/umad_method_str.3 b/man/umad_method_str.3
> new file mode 100644
> index 0000000..f5e6b02
> --- /dev/null
> +++ b/man/umad_method_str.3
> @@ -0,0 +1,3 @@
> +.\" -*- nroff -*-
> +.\"
> +.so man3/umad_class_str.3
> diff --git a/src/libibumad.map b/src/libibumad.map
> index 0154b7f..af68d11 100644
> --- a/src/libibumad.map
> +++ b/src/libibumad.map
> @@ -30,5 +30,9 @@ IBUMAD_1.0 {
>  		umad_debug;
>  		umad_addr_dump;
>  		umad_dump;
> +		umad_class_str;
> +		umad_method_str;
> +		umad_mad_status_str;
> +		umad_attribute_str;
>  	local: *;
>  };
> diff --git a/src/umad_str.c b/src/umad_str.c
> new file mode 100644
> index 0000000..a9e8336
> --- /dev/null
> +++ b/src/umad_str.c
> @@ -0,0 +1,318 @@
> +/*
> + * Copyright (c) 2004, 2005, 2010 Intel Corporation.  All rights reserved.
> + * Copyright (c) 2013 Lawrence Livermore National Security. All rights reserved.
> + *
> + * This software is available to you under a choice of one of two
> + * licenses.  You may choose to be licensed under the terms of the GNU
> + * General Public License (GPL) Version 2, available from the file
> + * COPYING in the main directory of this source tree, or the
> + * OpenIB.org BSD license below:
> + *
> + *     Redistribution and use in source and binary forms, with or
> + *     without modification, are permitted provided that the following
> + *     conditions are met:
> + *
> + *      - Redistributions of source code must retain the above
> + *        copyright notice, this list of conditions and the following
> + *        disclaimer.
> + *
> + *      - Redistributions in binary form must reproduce the above
> + *        copyright notice, this list of conditions and the following
> + *        disclaimer in the documentation and/or other materials
> + *        provided with the distribution.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
> + * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
> + * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> + * SOFTWARE.
> + *
> + */
> +
> +#include <stdio.h>
> +#include <arpa/inet.h>
> +#include <infiniband/umad.h>
> +#include <infiniband/umad_types.h>
> +#include <infiniband/umad_sm.h>
> +#include <infiniband/umad_sa.h>
> +#include <infiniband/umad_cm.h>
> +
> +char * umad_class_str(uint8_t mgmt_class)
> +{
> +	switch (mgmt_class) {
> +		case UMAD_CLASS_SUBN_LID_ROUTED:
> +		case UMAD_CLASS_SUBN_DIRECTED_ROUTE:
> +			return("Subn");
> +		case UMAD_CLASS_SUBN_ADM:
> +			return("SubnAdm");
> +		case UMAD_CLASS_PERF_MGMT:
> +			return("Perf");
> +		case UMAD_CLASS_BM:
> +			return("BM");
> +		case UMAD_CLASS_DEVICE_MGMT:
> +			return("DevMgt");
> +		case UMAD_CLASS_CM:
> +			return("ComMgt");
> +		case UMAD_CLASS_SNMP:
> +			return("SNMP");
> +		case UMAD_CLASS_DEVICE_ADM:
> +			return("DevAdm");
> +		case UMAD_CLASS_BOOT_MGMT:
> +			return("BootMgt");
> +		case UMAD_CLASS_BIS:
> +			return("BIS");
> +		case UMAD_CLASS_CONG_MGMT:
> +			return("CongestionManagment");
> +		default:
> +			break;
> +	}
> +
> +	if ((UMAD_CLASS_VENDOR_RANGE1_START <= mgmt_class
> +		&& mgmt_class <= UMAD_CLASS_VENDOR_RANGE1_END)
> +	    || (UMAD_CLASS_VENDOR_RANGE2_START <= mgmt_class
> +		&& mgmt_class <= UMAD_CLASS_VENDOR_RANGE2_END))
> +		return("Vendor");
> +
> +	if (UMAD_CLASS_APPLICATION_START <= mgmt_class
> +	    && mgmt_class <= UMAD_CLASS_APPLICATION_END) {
> +		return("Application");
> +	}
> +	return ("<unknown>");
> +}
> +
> +char * umad_method_str(uint8_t method)

mgmt_class should also be passed here; see below.

> +{
> +	switch(method) {
> +		case UMAD_METHOD_GET:
> +			return ("Get");
> +		case UMAD_METHOD_SET:
> +			return ("Set");
> +		case UMAD_METHOD_GET_RESP:
> +			return ("GetResp");
> +		case UMAD_METHOD_SEND:
> +			return ("Send");
> +		case UMAD_METHOD_TRAP:
> +			return ("Trap");
> +		case UMAD_METHOD_REPORT:
> +			return ("Report");
> +		case UMAD_METHOD_REPORT_RESP:
> +			return ("ReportResp");
> +		case UMAD_METHOD_TRAP_REPRESS:
> +			return ("TrapRepress");
> +		case UMAD_METHOD_RESP:
> +			return ("Resp");

Shouldn't this be removed from here ?

> +		case UMAD_SA_METHOD_GET_TABLE:
> +			return ("GetTable");
> +		case UMAD_SA_METHOD_GET_TABLE_RESP:
> +			return ("GetTableResp");
> +		case UMAD_SA_METHOD_DELETE:
> +			return ("Delete");
> +		case UMAD_SA_METHOD_DELETE_RESP:
> +			return ("DeleteResp");
> +		case UMAD_SA_METHOD_GET_MULTI:
> +			return ("GetMulti");
> +		case UMAD_SA_METHOD_GET_MULTI_RESP:
> +			return ("GetMultiResp");
> +		case UMAD_SA_METHOD_GET_TRACE_TABLE:
> +			return ("GetTraceTable");

SA methods should be conditionalized on SA mgmt_class.

> +	}
> +	return ("<unknown>");
> +}
> +
> +char * umad_mad_status_str(be16_t status, char * buf, size_t len)
> +{
> +	size_t n = len;
> +	int rc = 0;
> +
> +	status = ntohs(status);
> +
> +	if (status & 0x0001) {

Use UMAD_STATUS_BUSY rather than 0x0001

> +		rc = snprintf(buf, n, "busy");
> +		if (rc < 0 || rc >= n)
> +			goto done;
> +		n -= rc;
> +		buf += rc;
> +	}
> +
> +	if (status & 0x0002) {

Use UMAD_STATUS_REDIRECT rather than 0x0002

> +		rc = snprintf(buf, n, " redirection required");
> +		if (rc < 0 || rc >= n)
> +			goto done;
> +		n -= rc;
> +		buf += rc;
> +	}
> +
> +	switch((status & 0x001C) >> 2) {
> +	case 1:
> +		rc = snprintf(buf, n, " bad version");
> +		break;
> +	case 2:
> +		rc = snprintf(buf, n, " method not supported");
> +		break;
> +	case 3:
> +		rc = snprintf(buf, n, " method/attribute combo not supported");
> +		break;
> +	case 7:
> +		rc = snprintf(buf, n, " invalid attribute/modifier value");
> +		break;
> +	}

Use UMAD_STATUS_xxxx for Invalid fields above ?

> +
> +	if (rc < 0 || rc >= n)
> +		goto done;
> +	n -= rc;
> +	buf += rc;
> +
> +	if (n == len)
> +		snprintf(buf, n, "Success");
> +done:
> +	return (buf);
> +}
> +
> +static char *umad_common_attr_str(be16_t attr_id)
> +{
> +	switch(ntohs(attr_id)) {
> +		case UMAD_ATTR_CLASS_PORT_INFO:
> +			return "Class Port Info";
> +		case UMAD_ATTR_NOTICE:
> +			return "Notice";
> +		case UMAD_ATTR_INFORM_INFO:
> +			return "Inform Info";
> +		default:
> +			return "<unknown>";
> +	}
> +}
> +
> +static char * umad_sm_attr_str(be16_t attr_id)
> +{
> +	switch(ntohs(attr_id)) {
> +		case UMAD_SMP_ATTR_NODE_DESC:
> +			return ("NodeDescription");
> +		case UMAD_SMP_ATTR_NODE_INFO:
> +			return ("NodeInfo");
> +		case UMAD_SMP_ATTR_SWITCH_INFO:
> +			return ("SwitchInfo");
> +		case UMAD_SMP_ATTR_GUID_INFO:
> +			return ("GUIDInfo");
> +		case UMAD_SMP_ATTR_PORT_INFO:
> +			return ("PortInfo");
> +		case UMAD_SMP_ATTR_PKEY_TABLE:
> +			return ("P_KeyTable");
> +		case UMAD_SMP_ATTR_SLVL_TABLE:
> +			return ("SLtoVLMappingTable");
> +		case UMAD_SMP_ATTR_VL_ARB_TABLE:
> +			return ("VLArbitrationTable");
> +		case UMAD_SMP_ATTR_LINEAR_FT:
> +			return ("LinearForwardingTable");
> +		case UMAD_SMP_ATTR_RANDOM_FT:
> +			return ("RandomForwardingTable");
> +		case UMAD_SMP_ATTR_MCAST_FT:
> +			return ("MulticastForwardingTable");
> +		case UMAD_SMP_ATTR_SM_INFO:
> +			return ("SMInfo");
> +		case UMAD_SMP_ATTR_VENDOR_DIAG:
> +			return ("VendorDiag");
> +		case UMAD_SMP_ATTR_LED_INFO:
> +			return ("LedInfo");
> +		case UMAD_SMP_ATTR_LINK_SPD_WIDTH_TABLE:
> +			return ("LinkSpeedWidthPairsTable");
> +		default:
> +			return (umad_common_attr_str(attr_id));
> +	}
> +	return ("<unknown>");
> +}
> +
> +static char * umad_sa_attr_str(be16_t attr_id)
> +{
> +	switch(ntohs(attr_id)) {
> +		case UMAD_SA_ATTR_NODE_REC:
> +			return ("NodeRecord");
> +		case UMAD_SA_ATTR_PORT_INFO_REC:
> +			return ("PortInfoRecord");
> +		case UMAD_SA_ATTR_SLVL_REC:
> +			return ("SLtoVLMappingTableRecord");
> +		case UMAD_SA_ATTR_SWITCH_INFO_REC:
> +			return ("SwitchInfoRecord");
> +		case UMAD_SA_ATTR_LINEAR_FT_REC:
> +			return ("LinearForwardingTableRecord");
> +		case UMAD_SA_ATTR_RANDOM_FT_REC:
> +			return ("RandomForwardingTableRecord");
> +		case UMAD_SA_ATTR_MCAST_FT_REC:
> +			return ("MulticastForwardingTableRecord");
> +		case UMAD_SA_ATTR_SM_INFO_REC:
> +			return ("SMInfoRecord");
> +		case UMAD_SA_ATTR_INFORM_INFO_REC:
> +			return ("InformInfoRecord");
> +		case UMAD_SA_ATTR_LINK_REC:
> +			return ("LinkRecord");
> +		case UMAD_SA_ATTR_GUID_INFO_REC:
> +			return ("GuidInfoRecord");
> +		case UMAD_SA_ATTR_SERVICE_REC:
> +			return ("ServiceRecord");
> +		case UMAD_SA_ATTR_PKEY_TABLE_REC:
> +			return ("P_KeyTableRecord");
> +		case UMAD_SA_ATTR_PATH_REC:
> +			return ("PathRecord");
> +		case UMAD_SA_ATTR_VL_ARB_REC:
> +			return ("VLArbitrationTableRecord");
> +		case UMAD_SA_ATTR_MCMEMBER_REC:
> +			return ("MCMemberRecord");
> +		case UMAD_SA_ATTR_TRACE_REC:
> +			return ("TraceRecord");
> +		case UMAD_SA_ATTR_MULTI_PATH_REC:
> +			return ("MultiPathRecord");
> +		case UMAD_SA_ATTR_SERVICE_ASSOC_REC:
> +			return ("ServiceAssociationRecord");
> +		default:
> +			return (umad_common_attr_str(attr_id));
> +	}
> +	return ("<unknown>");
> +}
> +
> +static char * umad_cm_attr_str(be16_t attr_id)
> +{
> +	switch(ntohs(attr_id)) {
> +		case UMAD_CM_ATTR_REQ:
> +			return "ConnectRequest";
> +		case UMAD_CM_ATTR_MRA:
> +			return "MsgRcptAck";
> +		case UMAD_CM_ATTR_REJ:
> +			return "ConnectReject";
> +		case UMAD_CM_ATTR_REP:
> +			return "ConnectReply";
> +		case UMAD_CM_ATTR_RTU:
> +			return "ReadyToUse";
> +		case UMAD_CM_ATTR_DREQ:
> +			return "DisconnectRequest";
> +		case UMAD_CM_ATTR_DREP:
> +			return "DisconnectReply";
> +		case UMAD_CM_ATTR_SIDR_REQ:
> +			return "ServiceIDResReq";
> +		case UMAD_CM_ATTR_SIDR_REP:
> +			return "ServiceIDResReqResp";
> +		case UMAD_CM_ATTR_LAP:
> +			return "LoadAlternatePath";
> +		case UMAD_CM_ATTR_APR:
> +			return "AlternatePathResponse";
> +		default:
> +			return (umad_common_attr_str(attr_id));
> +	}
> +	return ("<unknown>");
> +}
> +
> +char * umad_attribute_str(uint8_t mgmt_class, be16_t attr_id)
> +{
> +	switch (mgmt_class) {
> +		case UMAD_CLASS_SUBN_LID_ROUTED:
> +		case UMAD_CLASS_SUBN_DIRECTED_ROUTE:
> +			return(umad_sm_attr_str(attr_id));
> +		case UMAD_CLASS_SUBN_ADM:
> +			return(umad_sa_attr_str(attr_id));
> +		case UMAD_CLASS_CM:
> +			return(umad_cm_attr_str(attr_id));
> +	}
> +
> +	return (umad_common_attr_str(attr_id));
> +}

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html