Skip to content

Commit 176da51

Browse files
committed
mca_base_var: fix output bug about settable vars
Fix the test that determined whether we output "writeable" or "read-only" for MCA vars (it was checking the wrong flag). Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
1 parent 3f598e9 commit 176da51

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

opal/mca/base/mca_base_var.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
* University of Stuttgart. All rights reserved.
1111
* Copyright (c) 2004-2005 The Regents of the University of California.
1212
* All rights reserved.
13-
* Copyright (c) 2008-2015 Cisco Systems, Inc. All rights reserved.
13+
* Copyright (c) 2008-2018 Cisco Systems, Inc. All rights reserved
1414
* Copyright (c) 2012-2018 Los Alamos National Security, LLC. All rights
1515
* reserved.
1616
* Copyright (c) 2014-2016 Intel, Inc. All rights reserved.
@@ -2153,7 +2153,8 @@ int mca_base_var_dump(int vari, char ***out, mca_base_var_dump_type_t output_typ
21532153
asprintf(out[0] + line++, "%ssource:%s", tmp, source_string);
21542154

21552155
/* Output whether it's read only or writable */
2156-
asprintf(out[0] + line++, "%sstatus:%s", tmp, VAR_IS_DEFAULT_ONLY(var[0]) ? "read-only" : "writeable");
2156+
asprintf(out[0] + line++, "%sstatus:%s", tmp,
2157+
VAR_IS_SETTABLE(var[0]) ? "writeable" : "read-only");
21572158

21582159
/* Output the info level of this parametere */
21592160
asprintf(out[0] + line++, "%slevel:%d", tmp, var->mbv_info_lvl + 1);

0 commit comments

Comments
 (0)