public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: Ira Weiny <weiny2-i2BcT+NCU+M@public.gmane.org>
To: "Chu, Al" <chu11-i2BcT+NCU+M@public.gmane.org>
Cc: "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH] infiniband-diags: Support ibccconfig, congestion control setting tool
Date: Mon, 16 Jan 2012 15:35:38 -0800	[thread overview]
Message-ID: <20120116153538.0d800fcf.weiny2@llnl.gov> (raw)
In-Reply-To: <1323382190.6368.103.camel-akkeaxHeDKRliZ7u+bvwcg@public.gmane.org>



On Thu, 8 Dec 2011 14:09:50 -0800
Albert Chu <chu11-i2BcT+NCU+M@public.gmane.org> wrote:

> Signed-off-by: Albert Chu <chu11-i2BcT+NCU+M@public.gmane.org>

Thanks, applied with the following warning added to the man page and help message:

WARNING -- You should understand what you are doing before using this tool.  Misuse of this tool could result in a broken fabric.

Ira

> ---
>  Makefile.am      |    6 +-
>  man/ibccconfig.8 |   85 +++++++
>  src/ibccconfig.c |  644 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 733 insertions(+), 2 deletions(-)
>  create mode 100644 man/ibccconfig.8
>  create mode 100644 src/ibccconfig.c
>
> diff --git a/Makefile.am b/Makefile.am
> index 388b491..950f95b 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -14,7 +14,8 @@ sbin_PROGRAMS = src/ibaddr src/ibnetdiscover src/ibping src/ibportstate \
>                 src/ibroute src/ibstat src/ibsysstat src/ibtracert \
>                 src/perfquery src/sminfo src/smpdump src/smpquery \
>                 src/saquery src/vendstat src/iblinkinfo \
> -               src/ibqueryerrors src/ibcacheedit src/ibccquery
> +               src/ibqueryerrors src/ibcacheedit src/ibccquery \
> +               src/ibccconfig
>
>  if ENABLE_TEST_UTILS
>  sbin_PROGRAMS += src/ibsendtrap src/mcm_rereg_test
> @@ -61,6 +62,7 @@ src_vendstat_SOURCES = src/vendstat.c
>  src_mcm_rereg_test_SOURCES = src/mcm_rereg_test.c
>  src_iblinkinfo_SOURCES = src/iblinkinfo.c
>  src_ibccquery_SOURCES = src/ibccquery.c
> +src_ibccconfig_SOURCES = src/ibccconfig.c
>  src_iblinkinfo_LDFLAGS = -L$(top_builddir)/libibnetdisc -libnetdisc
>  src_ibqueryerrors_SOURCES = src/ibqueryerrors.c
>  src_ibqueryerrors_LDFLAGS = -L$(top_builddir)/libibnetdisc -libnetdisc
> @@ -81,7 +83,7 @@ man_MANS = man/ibaddr.8 man/ibcheckerrors.8 man/ibcheckerrs.8 \
>         man/ibdatacounts.8 man/ibdatacounters.8 \
>         man/ibrouters.8 man/ibprintrt.8 man/ibidsverify.8 \
>         man/check_lft_balance.8 man/ibcacheedit.8 \
> -       man/infiniband-diags.8 man/ibccquery.8
> +       man/infiniband-diags.8 man/ibccquery.8 man/ibccconfig.8
>
>  BUILT_SOURCES = ibdiag_version
>  ibdiag_version:
> diff --git a/man/ibccconfig.8 b/man/ibccconfig.8
> new file mode 100644
> index 0000000..cfdd709
> --- /dev/null
> +++ b/man/ibccconfig.8
> @@ -0,0 +1,85 @@
> +.TH IBCCCONFIG 8 "Dec 1, 2011" "OpenIB" "OpenIB Diagnostics"
> +
> +.SH NAME
> +ibccconfig \- configure congestion control settings
> +
> +.SH SYNOPSIS
> +.B ibccconfig
> +[\-d(ebug)] [\-G(uid)] [\-C ca_name] [\-P ca_port] [\-t(imeout) timeout_ms]
> +[\-V(ersion)] [\-h(elp)] [\-c cckey] <op> <lid|guid> [port]
> +
> +.SH DESCRIPTION
> +.PP
> +.B ibccconfig
> +supports the configuration of congestion control settings on switches
> +and HCAs.
> +
> +.SH OPTIONS
> +
> +.PP
> +.TP
> +Current supported operations and their parameters:
> +  CongestionKeyInfo (CK) <lid|guid> <cckey> <cckeyprotectbit> <cckeyleaseperiod> <cckeyviolations>
> +  SwitchCongestionSetting (SS) <lid|guid> <controlmap> <victimmask> <creditmask> <threshold> <packetsize> <csthreshold> <csreturndelay> <markingrate>
> +  SwitchPortCongestionSetting (SP) <lid|guid> <portnum> <valid> <control_type> <threshold> <packet_size> <cong_parm_marking_rate>
> +  CACongestionSetting (CS) <lid|guid> <port_control> <control_map> <sl> <ccti_timer> <ccti_increase> <trigger_threshold> <ccti_min>
> +  CongestionControlTable (CT) <lid|guid> <cctilimit> <index> <cctentry> <cctentry> ...
> +
> +.TP
> +\fB\-c\-fR, \fB\-\-cckey\fR <cckey>
> +Specify a congestion control (CC) key.  If none is specified, a key of 0 is used.
> +
> +.SH COMMON OPTIONS
> +
> +Most OpenIB diagnostics take the following common flags. The exact list of
> +supported flags per utility can be found in the usage message and can be shown
> +using the util_name -h syntax.
> +
> +# Debugging flags
> +.PP
> +\-d      raise the IB debugging level.
> +        May be used several times (-ddd or -d -d -d).
> +.PP
> +\-e      show send and receive errors (timeouts and others)
> +.PP
> +\-h      show the usage message
> +.PP
> +\-v      increase the application verbosity level.
> +        May be used several times (-vv or -v -v -v)
> +.PP
> +\-V      show the version info.
> +
> +# Addressing flags
> +.PP
> +\-G      use GUID address argument. In most cases, it is the Port GUID.
> +        Example:
> +        "0x08f1040023"
> +.PP
> +\-s <smlid>      use 'smlid' as the target lid for SM/SA queries.
> +
> +# Other common flags:
> +.PP
> +\-C <ca_name>    use the specified ca_name.
> +.PP
> +\-P <ca_port>    use the specified ca_port.
> +.PP
> +\-t <timeout_ms> override the default timeout for the solicited mads.
> +
> +If a port and/or CA name is specified, the user request is
> +attempted to be fulfilled, and will fail if it is not possible.
> +
> +.SH EXAMPLES
> +
> +.PP
> +ibccconfig SwitchCongestionSetting 2 0x1F 0x1FFFFFFFFF 0x0 0xF 8 0 0:0 1    # Configure Switch Congestion Settings
> +.PP
> +ibccconfig CACongestionSetting 1 0 0x3 0 150 1 0 0          # Configure CA Congestion Settings
> +.PP
> +ibccconfig CongestionControlTable 1 63 0 0:0 0:1 ...        # Configure first block of Congestion Control Table
> +.PP
> +ibccconfig CongestionControlTable 1 127 0 0:64 0:65 ...     # Configure second block of Congestion Control Table
> +
> +.SH AUTHOR
> +.TP
> +Albert Chu
> +.RI < chu11-i2BcT+NCU+M@public.gmane.org >
> diff --git a/src/ibccconfig.c b/src/ibccconfig.c
> new file mode 100644
> index 0000000..296bc47
> --- /dev/null
> +++ b/src/ibccconfig.c
> @@ -0,0 +1,644 @@
> +/*
> + * Copyright (c) 2004-2009 Voltaire Inc.  All rights reserved.
> + * Copyright (c) 2011 Mellanox Technologies LTD.  All rights reserved.
> + * Copyright (c) 2011 Lawrence Livermore National Lab.  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.
> + *
> + */
> +
> +#if HAVE_CONFIG_H
> +#  include <config.h>
> +#endif                         /* HAVE_CONFIG_H */
> +
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <unistd.h>
> +#include <string.h>
> +#include <getopt.h>
> +#include <errno.h>
> +#include <netinet/in.h>
> +#include <limits.h>
> +#include <ctype.h>
> +
> +#define __STDC_FORMAT_MACROS
> +#include <inttypes.h>
> +
> +#include <infiniband/mad.h>
> +
> +#include "ibdiag_common.h"
> +
> +struct ibmad_port *srcport;
> +
> +typedef char *(op_fn_t) (ib_portid_t * dest, char **argv, int argc);
> +
> +typedef struct match_rec {
> +       const char *name, *alias;
> +       op_fn_t *fn;
> +       unsigned opt_portnum;
> +       char *ops_extra;
> +} match_rec_t;
> +
> +static op_fn_t congestion_key_info;
> +static op_fn_t switch_congestion_setting;
> +static op_fn_t switch_port_congestion_setting;
> +static op_fn_t ca_congestion_setting;
> +static op_fn_t congestion_control_table;
> +
> +static const match_rec_t match_tbl[] = {
> +       {"CongestionKeyInfo", "CK", congestion_key_info, 0,
> +        "<cckey> <cckeyprotectbit> <cckeyleaseperiod> <cckeyviolations>"},
> +       {"SwitchCongestionSetting", "SS", switch_congestion_setting, 0,
> +        "<controlmap> <victimmask> <creditmask> <threshold> <packetsize> "
> +        "<csthreshold> <csreturndelay> <markingrate>"},
> +       {"SwitchPortCongestionSetting", "SP", switch_port_congestion_setting, 1,
> +        "<valid> <control_type> <threshold> <packet_size> <cong_parm_marking_rate>"},
> +       {"CACongestionSetting", "CS", ca_congestion_setting, 0,
> +        "<port_control> <control_map> <sl> <ccti_timer> <ccti_increase> "
> +        "<trigger_threshold> <ccti_min>"},
> +       {"CongestionControlTable", "CT", congestion_control_table, 0,
> +        "<cctilimit> <index> <cctentry> <cctentry> ..."},
> +       {0}
> +};
> +
> +uint64_t cckey = 0;
> +
> +/*******************************************/
> +static char *parselonglongint(char *arg, uint64_t *val)
> +{
> +       char *endptr = NULL;
> +
> +       errno = 0;
> +       *val = strtoull(arg, &endptr, 0);
> +       if ((endptr && *endptr != '\0')
> +           || errno != 0) {
> +               if (errno == ERANGE)
> +                       return "value out of range";
> +               return "invalid integer input";
> +       }
> +
> +       return NULL;
> +}
> +
> +static char *parseint(char *arg, uint32_t *val, int hexonly)
> +{
> +       char *endptr = NULL;
> +
> +       errno = 0;
> +       *val = strtoul(arg, &endptr, hexonly ? 16 : 0);
> +       if ((endptr && *endptr != '\0')
> +           || errno != 0) {
> +               if (errno == ERANGE)
> +                       return "value out of range";
> +               return "invalid integer input";
> +       }
> +
> +       return NULL;
> +}
> +
> +static char *congestion_key_info(ib_portid_t * dest, char **argv, int argc)
> +{
> +       uint8_t rcv[IB_CC_DATA_SZ] = { 0 };
> +       uint8_t payload[IB_CC_DATA_SZ] = { 0 };
> +       uint64_t cc_key;
> +       uint32_t cc_keyprotectbit;
> +       uint32_t cc_keyleaseperiod;
> +       uint32_t cc_keyviolations;
> +       char *errstr;
> +
> +       if (argc != 4)
> +               return "invalid number of parameters for CongestionKeyInfo";
> +
> +       if ((errstr = parselonglongint(argv[0], &cc_key)))
> +               return errstr;
> +       if ((errstr = parseint(argv[1], &cc_keyprotectbit, 0)))
> +               return errstr;
> +       if ((errstr = parseint(argv[2], &cc_keyleaseperiod, 0)))
> +               return errstr;
> +       if ((errstr = parseint(argv[3], &cc_keyviolations, 0)))
> +               return errstr;
> +
> +       if (cc_keyprotectbit != 0 && cc_keyprotectbit != 1)
> +               return "invalid cc_keyprotectbit value";
> +
> +       if (cc_keyleaseperiod > USHRT_MAX)
> +               return "invalid cc_keyleaseperiod value";
> +
> +       if (cc_keyviolations > USHRT_MAX)
> +               return "invalid cc_keyviolations value";
> +
> +       mad_set_field64(payload,
> +                       0,
> +                       IB_CC_CONGESTION_KEY_INFO_CC_KEY_F,
> +                       cc_key);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_CONGESTION_KEY_INFO_CC_KEY_PROTECT_BIT_F,
> +                        &cc_keyprotectbit);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_CONGESTION_KEY_INFO_CC_KEY_LEASE_PERIOD_F,
> +                        &cc_keyleaseperiod);
> +
> +       /* spec says "setting the counter to a value other than zero results
> +        * in the counter being left unchanged.  So if user wants no change,
> +        * they gotta input non-zero
> +        */
> +        mad_encode_field(payload,
> +                        IB_CC_CONGESTION_KEY_INFO_CC_KEY_VIOLATIONS_F,
> +                        &cc_keyviolations);
> +
> +       if (!cc_config_status_via(payload, rcv, dest, IB_CC_ATTR_CONGESTION_KEY_INFO,
> +                                 0, 0, NULL, srcport, cckey))
> +               return "congestion key info config failed";
> +
> +       return NULL;
> +}
> +
> +
> +/* parse like it's a hypothetical 256 bit hex code */
> +static char *parse256(char *arg, uint8_t *buf)
> +{
> +       int numdigits = 0;
> +       int startindex;
> +       char *ptr;
> +       int i;
> +
> +       if (!strncmp(arg, "0x", 2) || !strncmp(arg, "0X", 2))
> +               arg += 2;
> +
> +       for (ptr = arg; *ptr; ptr++) {
> +               if (!isxdigit(*ptr))
> +                       return "invalid hex digit read";
> +               numdigits++;
> +       }
> +
> +       if (numdigits > 64)
> +               return "hex code too long";
> +
> +       /* we need to imagine that this is like a 256-bit int stored
> +        * in big endian.  So we need to find the first index
> +        * point where the user's input would start in our array.
> +        */
> +       startindex = 32 - ((numdigits - 1) / 2) - 1;
> +
> +       for (i = startindex; i <= 31; i++) {
> +               char tmp[3] = { 0 };
> +               uint32_t tmpint;
> +               char *errstr;
> +
> +               /* I can't help but think there is a strtoX that
> +                * will do this for me, but I can't find it.
> +                */
> +               if (i == startindex && numdigits % 2) {
> +                       memcpy(tmp, arg, 1);
> +                       arg++;
> +               }
> +               else {
> +                       memcpy(tmp, arg, 2);
> +                       arg += 2;
> +               }
> +
> +               if ((errstr = parseint(tmp, &tmpint, 1)))
> +                       return errstr;
> +               buf[i] = tmpint;
> +       }
> +
> +       return NULL;
> +}
> +
> +static char *parsecct(char *arg, uint32_t *shift, uint32_t *multiplier)
> +{
> +       char buf[1024] = { 0 };
> +       char *errstr;
> +       char *ptr;
> +
> +       strcpy(buf, arg);
> +
> +       if (!(ptr = strchr(buf, ':')))
> +               return "ccts are formatted shift:multiplier";
> +
> +       *ptr = '\0';
> +       ptr++;
> +
> +       if ((errstr = parseint(buf, shift, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(ptr, multiplier, 0)))
> +               return errstr;
> +
> +       return NULL;
> +}
> +
> +static char *switch_congestion_setting(ib_portid_t * dest, char **argv, int argc)
> +{
> +       uint8_t rcv[IB_CC_DATA_SZ] = { 0 };
> +       uint8_t payload[IB_CC_DATA_SZ] = { 0 };
> +       uint32_t control_map;
> +       uint8_t victim_mask[32] = { 0 };
> +       uint8_t credit_mask[32] = { 0 };
> +       uint32_t threshold;
> +       uint32_t packet_size;
> +       uint32_t cs_threshold;
> +       uint32_t cs_returndelay_s;
> +       uint32_t cs_returndelay_m;
> +       uint32_t cs_returndelay;
> +       uint32_t marking_rate;
> +       char *errstr;
> +
> +       if (argc != 8)
> +               return "invalid number of parameters for SwitchCongestionSetting";
> +
> +       if ((errstr = parseint(argv[0], &control_map, 0)))
> +               return errstr;
> +
> +       if ((errstr = parse256(argv[1], victim_mask)))
> +               return errstr;
> +
> +       if ((errstr = parse256(argv[2], credit_mask)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[3], &threshold, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[4], &packet_size, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[5], &cs_threshold, 0)))
> +               return errstr;
> +
> +       if ((errstr = parsecct(argv[6], &cs_returndelay_s, &cs_returndelay_m)))
> +               return errstr;
> +
> +       cs_returndelay = cs_returndelay_m;
> +       cs_returndelay |= (cs_returndelay_s << 14);
> +
> +       if ((errstr = parseint(argv[7], &marking_rate, 0)))
> +               return errstr;
> +
> +       mad_encode_field(payload,
> +                        IB_CC_SWITCH_CONGESTION_SETTING_CONTROL_MAP_F,
> +                        &control_map);
> +
> +       mad_set_array(payload,
> +                     0,
> +                     IB_CC_SWITCH_CONGESTION_SETTING_VICTIM_MASK_F,
> +                     victim_mask);
> +
> +       mad_set_array(payload,
> +                     0,
> +                     IB_CC_SWITCH_CONGESTION_SETTING_CREDIT_MASK_F,
> +                     credit_mask);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_SWITCH_CONGESTION_SETTING_THRESHOLD_F,
> +                        &threshold);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_SWITCH_CONGESTION_SETTING_PACKET_SIZE_F,
> +                        &packet_size);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_SWITCH_CONGESTION_SETTING_CS_THRESHOLD_F,
> +                        &cs_threshold);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_SWITCH_CONGESTION_SETTING_CS_RETURN_DELAY_F,
> +                        &cs_returndelay);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_SWITCH_CONGESTION_SETTING_MARKING_RATE_F,
> +                        &marking_rate);
> +
> +       if (!cc_config_status_via(payload, rcv, dest, IB_CC_ATTR_SWITCH_CONGESTION_SETTING,
> +                                 0, 0, NULL, srcport, cckey))
> +               return "switch congestion setting config failed";
> +
> +       return NULL;
> +}
> +
> +static char *switch_port_congestion_setting(ib_portid_t * dest, char **argv, int argc)
> +{
> +       uint8_t rcv[IB_CC_DATA_SZ] = { 0 };
> +       uint8_t payload[IB_CC_DATA_SZ] = { 0 };
> +       uint8_t data[IB_CC_DATA_SZ] = { 0 };
> +       uint32_t portnum;
> +       uint32_t valid;
> +       uint32_t control_type;
> +       uint32_t threshold;
> +       uint32_t packet_size;
> +       uint32_t cong_parm_marking_rate;
> +       uint32_t type;
> +       uint32_t numports;
> +       uint8_t *ptr;
> +       char *errstr;
> +
> +       if (argc != 6)
> +               return "invalid number of parameters for SwitchPortCongestion";
> +
> +       if ((errstr = parseint(argv[0], &portnum, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[1], &valid, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[2], &control_type, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[3], &threshold, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[4], &packet_size, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[5], &cong_parm_marking_rate, 0)))
> +               return errstr;
> +
> +       if (!portnum)
> +               return "invalid port number specified";
> +
> +       /* Figure out number of ports first */
> +       if (!smp_query_via(data, dest, IB_ATTR_NODE_INFO, 0, 0, srcport))
> +               return "node info config failed";
> +
> +       mad_decode_field((uint8_t *)data, IB_NODE_TYPE_F, &type);
> +       mad_decode_field((uint8_t *)data, IB_NODE_NPORTS_F, &numports);
> +
> +       if (type != IB_NODE_SWITCH)
> +               return "destination not a switch";
> +
> +       if (portnum > numports)
> +               return "invalid port number specified";
> +
> +       /* We are modifying only 1 port, so get the current config */
> +       if (!cc_query_status_via(payload, dest, IB_CC_ATTR_SWITCH_PORT_CONGESTION_SETTING,
> +                                (portnum - 1) / 32, 0, NULL, srcport, cckey))
> +               return "switch congestion setting query failed";
> +
> +       ptr = payload + (((portnum % 32 - 1) * 4));
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_SWITCH_PORT_CONGESTION_SETTING_ELEMENT_VALID_F,
> +                        &valid);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_SWITCH_PORT_CONGESTION_SETTING_ELEMENT_CONTROL_TYPE_F,
> +                        &control_type);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_SWITCH_PORT_CONGESTION_SETTING_ELEMENT_THRESHOLD_F,
> +                        &threshold);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_SWITCH_PORT_CONGESTION_SETTING_ELEMENT_PACKET_SIZE_F,
> +                        &packet_size);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_SWITCH_PORT_CONGESTION_SETTING_ELEMENT_CONG_PARM_MARKING_RATE_F,
> +                        &cong_parm_marking_rate);
> +
> +       if (!cc_config_status_via(payload, rcv, dest, IB_CC_ATTR_SWITCH_PORT_CONGESTION_SETTING,
> +                                 (portnum - 1) / 32, 0, NULL, srcport, cckey))
> +               return "switch congestion setting config failed";
> +
> +       return NULL;
> +}
> +
> +static char *ca_congestion_setting(ib_portid_t * dest, char **argv, int argc)
> +{
> +       uint8_t rcv[IB_CC_DATA_SZ] = { 0 };
> +       uint8_t payload[IB_CC_DATA_SZ] = { 0 };
> +       uint32_t port_control;
> +       uint32_t control_map;
> +       uint32_t sl;
> +       uint32_t ccti_timer;
> +       uint32_t ccti_increase;
> +       uint32_t trigger_threshold;
> +       uint32_t ccti_min;
> +       uint8_t *ptr;
> +       char *errstr;
> +
> +       if (argc != 7)
> +               return "invalid number of parameters for CACongestionSetting";
> +
> +       if ((errstr = parseint(argv[0], &port_control, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[1], &control_map, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[2], &sl, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[3], &ccti_timer, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[4], &ccti_increase, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[5], &trigger_threshold, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[6], &ccti_min, 0)))
> +               return errstr;
> +
> +       if (sl > 15)
> +               return "invalid SL specified";
> +
> +       /* We are modifying only 1 SL at a time, so get the current config */
> +       if (!cc_query_status_via(payload, dest, IB_CC_ATTR_CA_CONGESTION_SETTING,
> +                                0, 0, NULL, srcport, cckey))
> +               return "ca congestion setting query failed";
> +
> +       mad_encode_field(payload,
> +                        IB_CC_CA_CONGESTION_SETTING_PORT_CONTROL_F,
> +                        &port_control);
> +
> +       mad_encode_field(payload,
> +                        IB_CC_CA_CONGESTION_SETTING_CONTROL_MAP_F,
> +                        &control_map);
> +
> +       ptr = payload + 2 + 2 + sl * 8;
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_CA_CONGESTION_ENTRY_CCTI_TIMER_F,
> +                        &ccti_timer);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_CA_CONGESTION_ENTRY_CCTI_INCREASE_F,
> +                        &ccti_increase);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_CA_CONGESTION_ENTRY_TRIGGER_THRESHOLD_F,
> +                        &trigger_threshold);
> +
> +       mad_encode_field(ptr,
> +                        IB_CC_CA_CONGESTION_ENTRY_CCTI_MIN_F,
> +                        &ccti_min);
> +
> +       if (!cc_config_status_via(payload, rcv, dest, IB_CC_ATTR_CA_CONGESTION_SETTING,
> +                                 0, 0, NULL, srcport, cckey))
> +               return "ca congestion setting config failed";
> +
> +       return NULL;
> +}
> +
> +static char *congestion_control_table(ib_portid_t * dest, char **argv, int argc)
> +{
> +       uint8_t rcv[IB_CC_DATA_SZ] = { 0 };
> +       uint8_t payload[IB_CC_DATA_SZ] = { 0 };
> +       uint32_t ccti_limit;
> +       uint32_t index;
> +       uint32_t cctshifts[64];
> +       uint32_t cctmults[64];
> +       char *errstr;
> +       int i;
> +
> +       if (argc < 2 || argc > 66)
> +               return "invalid number of parameters for CongestionControlTable";
> +
> +       if ((errstr = parseint(argv[0], &ccti_limit, 0)))
> +               return errstr;
> +
> +       if ((errstr = parseint(argv[1], &index, 0)))
> +               return errstr;
> +
> +       if (ccti_limit && (ccti_limit + 1) != (index * 64 + (argc - 2)))
> +               return "invalid number of cct entries input given ccti_limit and index";
> +
> +       for (i = 0; i < (argc - 2); i++) {
> +               if ((errstr = parsecct(argv[i + 2], &cctshifts[i], &cctmults[i])))
> +                       return errstr;
> +       }
> +
> +       mad_encode_field(payload,
> +                        IB_CC_CONGESTION_CONTROL_TABLE_CCTI_LIMIT_F,
> +                        &ccti_limit);
> +
> +       for (i = 0; i < (argc - 2); i++) {
> +               mad_encode_field(payload + 4 + i * 2,
> +                                IB_CC_CONGESTION_CONTROL_TABLE_ENTRY_CCT_SHIFT_F,
> +                                &cctshifts[i]);
> +
> +               mad_encode_field(payload + 4 + i * 2,
> +                                IB_CC_CONGESTION_CONTROL_TABLE_ENTRY_CCT_MULTIPLIER_F,
> +                                &cctmults[i]);
> +       }
> +
> +       if (!cc_config_status_via(payload, rcv, dest, IB_CC_ATTR_CONGESTION_CONTROL_TABLE,
> +                                 index, 0, NULL, srcport, cckey))
> +               return "congestion control table config failed";
> +
> +       return NULL;
> +}
> +
> +static op_fn_t *match_op(char *name)
> +{
> +       const match_rec_t *r;
> +       unsigned len = strlen(name);
> +       for (r = match_tbl; r->name; r++)
> +               if (!strncasecmp(r->name, name, len) ||
> +                   (r->alias && !strncasecmp(r->alias, name, len)))
> +                       return r->fn;
> +       return NULL;
> +}
> +
> +static int process_opt(void *context, int ch, char *optarg)
> +{
> +       switch (ch) {
> +       case 'c':
> +               cckey = (uint64_t) strtoull(optarg, 0, 0);
> +               break;
> +       default:
> +               return -1;
> +       }
> +       return 0;
> +}
> +
> +int main(int argc, char **argv)
> +{
> +       char usage_args[1024];
> +       int mgmt_classes[4] =
> +           { IB_SMI_CLASS, IB_SMI_DIRECT_CLASS, IB_SA_CLASS, IB_CC_CLASS };
> +       ib_portid_t portid = { 0 };
> +       char *err;
> +       op_fn_t *fn;
> +       const match_rec_t *r;
> +       int n;
> +
> +       const struct ibdiag_opt opts[] = {
> +               {"cckey", 'c', 1, "<key>", "CC key"},
> +               {0}
> +       };
> +       const char *usage_examples[] = {
> +               "SwitchCongestionSetting 2 0x1F 0x1FFFFFFFFF 0x0 0xF 8 0 0:0 1\t# Configure Switch Congestion Settings",
> +               "CACongestionSetting 1 0 0x3 0 150 1 0 0\t\t# Configure CA Congestion Settings",
> +               "CongestionControlTable 1 63 0 0:0 0:1 ...\t# Configure first block of Congestion Control Table",
> +               "CongestionControlTable 1 127 0 0:64 0:65 ...\t# Configure second block of Congestion Control Table",
> +               NULL
> +       };
> +
> +       n = sprintf(usage_args, "[-c key] <op> <lid|guid>\n"
> +                   "\nSupported ops (and aliases, case insensitive):\n");
> +       for (r = match_tbl; r->name; r++) {
> +               n += snprintf(usage_args + n, sizeof(usage_args) - n,
> +                             "  %s (%s) <lid|guid>%s%s%s\n", r->name,
> +                             r->alias ? r->alias : "",
> +                             r->opt_portnum ? " <portnum>" : "",
> +                             r->ops_extra ? " " : "",
> +                             r->ops_extra ? r->ops_extra : "");
> +               if (n >= sizeof(usage_args))
> +                       exit(-1);
> +       }
> +
> +       ibdiag_process_opts(argc, argv, NULL, "D", opts, process_opt,
> +                           usage_args, usage_examples);
> +
> +       argc -= optind;
> +       argv += optind;
> +
> +       if (argc < 2)
> +               ibdiag_show_usage();
> +
> +       if (!(fn = match_op(argv[0])))
> +               IBERROR("operation '%s' not supported", argv[0]);
> +
> +       srcport = mad_rpc_open_port(ibd_ca, ibd_ca_port, mgmt_classes, 4);
> +       if (!srcport)
> +               IBERROR("Failed to open '%s' port '%d'", ibd_ca, ibd_ca_port);
> +
> +       if (ib_resolve_portid_str_via(&portid, argv[1], ibd_dest_type,
> +                                     ibd_sm_id, srcport) < 0)
> +               IBERROR("can't resolve destination %s", argv[1]);
> +       if ((err = fn(&portid, argv + 2, argc - 2)))
> +               IBERROR("operation %s: %s", argv[0], err);
> +
> +       mad_rpc_close_port(srcport);
> +       exit(0);
> +}
> --
> 1.7.1
>
>
>
> --
> 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


--
Ira Weiny
Member of Technical Staff
Lawrence Livermore National Lab
925-423-8008
weiny2-i2BcT+NCU+M@public.gmane.org
--
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

      parent reply	other threads:[~2012-01-16 23:35 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-08 22:09 [PATCH] infiniband-diags: Support ibccconfig, congestion control setting tool Albert Chu
     [not found] ` <1323382190.6368.103.camel-akkeaxHeDKRliZ7u+bvwcg@public.gmane.org>
2012-01-16 23:35   ` Ira Weiny [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120116153538.0d800fcf.weiny2@llnl.gov \
    --to=weiny2-i2bct+ncu+m@public.gmane.org \
    --cc=chu11-i2BcT+NCU+M@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox