The YAML format used for configuring and showing networks was not
consistent. This patch makes both formats consistent.
EX:
net:
- net: tcp
nid: 192.168.206.130@tcp
status: up
interfaces:
0: eth0
tunables:
peer_timeout: 180
peer_credits: 8
peer_buffer_credits: 0
credits: 256
CPT: [0,1]
Signed-off-by: Amir Shehata <amir.shehata@intel.com>
Change-Id: Id4314679930709ac43104f1ba544bb6d1ca8cb0a
Reviewed-on: http://review.whamcloud.com/12556
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
*item = NULL, *first_seq = NULL;
int str_buf_len = LNET_MAX_SHOW_NUM_CPT * 2;
char str_buf[str_buf_len];
*item = NULL, *first_seq = NULL;
int str_buf_len = LNET_MAX_SHOW_NUM_CPT * 2;
char str_buf[str_buf_len];
char err_str[LNET_MAX_STR_LEN];
snprintf(err_str, sizeof(err_str), "\"out of memory\"");
char err_str[LNET_MAX_STR_LEN];
snprintf(err_str, sizeof(err_str), "\"out of memory\"");
goto out;
for (i = 0;; i++) {
goto out;
for (i = 0;; i++) {
+ pos = str_buf;
+
+ memset(buf, 0, sizeof(*data) + sizeof(*net_config));
+
LIBCFS_IOC_INIT_V2(*data, cfg_hdr);
/*
* set the ioc_len to the proper value since INIT assumes
LIBCFS_IOC_INIT_V2(*data, cfg_hdr);
/*
* set the ioc_len to the proper value since INIT assumes
first_seq = item;
if (cYAML_create_string(item,
first_seq = item;
if (cYAML_create_string(item,
+ "net",
+ libcfs_net2str(
+ LNET_NIDNET(data->cfg_nid)))
+ == NULL)
+ goto out;
+
+ if (cYAML_create_string(item, "nid",
libcfs_nid2str(data->cfg_nid)) == NULL)
goto out;
libcfs_nid2str(data->cfg_nid)) == NULL)
goto out;
net_max_tx_credits) == NULL)
goto out;
net_max_tx_credits) == NULL)
goto out;
+ /* out put the CPTs in the format: "[x,x,x,...]" */
+ pos += snprintf(pos, str_buf + str_buf_len - pos, "[");
for (j = 0 ; data->cfg_ncpts > 1 &&
for (j = 0 ; data->cfg_ncpts > 1 &&
- j < data->cfg_ncpts; j++) {
- pos += snprintf(str_buf,
+ j < data->cfg_ncpts; j++) {
+ pos += snprintf(pos,
str_buf + str_buf_len - pos,
str_buf + str_buf_len - pos,
- " %d", net_config->ni_cpts[j]);
+ "%d", net_config->ni_cpts[j]);
+ if ((j + 1) < data->cfg_ncpts)
+ pos += snprintf(pos,
+ str_buf +
+ str_buf_len - pos,
+ ",");
+ if (str_buf + str_buf_len - pos <= 0)
+ pos += snprintf(str_buf + str_buf_len - 2,
+ 2, "]");
+ else
+ pos += snprintf(pos,
+ str_buf + str_buf_len - pos,
+ "]");
if (data->cfg_ncpts > 1 &&
if (data->cfg_ncpts > 1 &&
- cYAML_create_string(tunables, "CPTs",
+ cYAML_create_string(tunables, "CPT",
str_buf) == NULL)
goto out;
}
str_buf) == NULL)
goto out;
}
peer_buf_cr = cYAML_get_object_item(tunables,
"peer_buffer_credits");
credits = cYAML_get_object_item(tunables, "credits");
peer_buf_cr = cYAML_get_object_item(tunables,
"peer_buffer_credits");
credits = cYAML_get_object_item(tunables, "credits");
- smp = cYAML_get_object_item(tunables, "SMP");
+ smp = cYAML_get_object_item(tunables, "CPT");
}
seq_no = cYAML_get_object_item(tree, "seq_no");
}
seq_no = cYAML_get_object_item(tree, "seq_no");