Skip to content

Commit

Permalink
Merge pull request #9553 from shasson5/prot_res_test
Browse files Browse the repository at this point in the history
TEST/PROTO: Reset request tests for offset = 0
  • Loading branch information
yosefe authored Jan 7, 2024
2 parents f93e4d5 + bca427b commit 1dbcdf0
Show file tree
Hide file tree
Showing 20 changed files with 449 additions and 45 deletions.
3 changes: 2 additions & 1 deletion src/ucp/core/ucp_context.c
Original file line number Diff line number Diff line change
Expand Up @@ -726,7 +726,8 @@ void ucp_config_release(ucp_config_t *config)
ucs_status_t ucp_config_modify_internal(ucp_config_t *config, const char *name,
const char *value)
{
return ucs_config_parser_set_value(config, ucp_config_table, name, value);
return ucs_config_parser_set_value(config, ucp_config_table, NULL, name,
value);
}

ucs_status_t ucp_config_modify(ucp_config_t *config, const char *name,
Expand Down
7 changes: 3 additions & 4 deletions src/ucs/config/global_opts.c
Original file line number Diff line number Diff line change
Expand Up @@ -326,16 +326,15 @@ ucs_status_t ucs_global_opts_set_value(const char *name, const char *value)
}

return ucs_config_parser_set_value(&ucs_global_opts,
ucs_global_opts_read_only_table,
ucs_global_opts_read_only_table, NULL,
name, value);
}

ucs_status_t ucs_global_opts_set_value_modifiable(const char *name,
const char *value)
{
return ucs_config_parser_set_value(&ucs_global_opts,
ucs_global_opts_table, name,
value);
return ucs_config_parser_set_value(&ucs_global_opts, ucs_global_opts_table,
NULL, name, value);
}

ucs_status_t ucs_global_opts_get_value(const char *name, char *value,
Expand Down
6 changes: 4 additions & 2 deletions src/ucs/config/parser.c
Original file line number Diff line number Diff line change
Expand Up @@ -1560,9 +1560,11 @@ ucs_config_parser_fill_opts(void *opts, ucs_config_global_list_entry_t *entry,
}

ucs_status_t ucs_config_parser_set_value(void *opts, ucs_config_field_t *fields,
const char *name, const char *value)
const char *prefix, const char *name,
const char *value)
{
return ucs_config_parser_set_value_internal(opts, fields, name, value, NULL, 1);
return ucs_config_parser_set_value_internal(opts, fields, name, value,
prefix, 1);
}

ucs_status_t ucs_config_parser_get_value(void *opts, ucs_config_field_t *fields,
Expand Down
4 changes: 3 additions & 1 deletion src/ucs/config/parser.h
Original file line number Diff line number Diff line change
Expand Up @@ -522,11 +522,13 @@ ucs_status_t ucs_config_parser_get_value(void *opts, ucs_config_field_t *fields,
*
* @param opts User-defined options structure.
* @param fields Array of fields which define how to parse.
* @param prefix Configuration table prefix.
* @param name Option name to modify.
* @param value Value to assign.
*/
ucs_status_t ucs_config_parser_set_value(void *opts, ucs_config_field_t *fields,
const char *name, const char *value);
const char *prefix, const char *name,
const char *value);

/**
* Wrapper for `ucs_config_parser_print_env_vars`
Expand Down
3 changes: 2 additions & 1 deletion src/uct/base/uct_md.c
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,8 @@ ucs_status_t uct_config_get(void *config, const char *name, char *value,
ucs_status_t uct_config_modify(void *config, const char *name, const char *value)
{
uct_config_bundle_t *bundle = (uct_config_bundle_t *)config - 1;
return ucs_config_parser_set_value(bundle->data, bundle->table, name, value);
return ucs_config_parser_set_value(bundle->data, bundle->table,
bundle->table_prefix, name, value);
}

static ucs_status_t
Expand Down
Loading

0 comments on commit 1dbcdf0

Please sign in to comment.