This web mail archive is frozen.
This page is part of a frozen web archive of this mailing list.
You can still navigate around this archive, but know that no new mails
have been added to it since July of 2016.
Click here to be taken to the new web archives of this list; it includes all the mails that are in this frozen archive plus all new mails that have been sent to the list since it was migrated to the new archives.
We talked about this on the call today.
A good suggestion was made: call show_help/opal_finalize/exit only when OPAL_ENABLE_DEBUG is true. Otherwise, return an error code.
If no one objects to this, I'll commit this tomorrow.
On Oct 31, 2011, at 4:16 PM, Jeff Squyres wrote:
> WHAT: what to do if registering an MCA param results in an error?
> WHERE: opal/mca/base/mca_base_param.c
> WHY: MCA param re-registration issues should be treated as OMPI developer errors
> WHEN: COB Friday, 4 Nov 2011
> Short version:
> Re-registering an MCA param to be a different type (e.g., it was initially registered to be a string, but was later re-registered to be an int) should be treated as an OMPI developer error, and should opal_finalize()/exit(1).
> More details:
> A mistaken MCA param re-registration recently caused an orted segv.
> The MCA param subsystem was fixed to avoid this segv, but silently convert the MCA param to the newly-registered type. Upon reflection and some discussion, this seems to be a bad idea. Instead, we should loudly complain via a show_help message and then exit(1).
> Specifically: this kind of behavior is clearly an error and should be fixed. Unfortunately, in most cases, we don't actually check the return value from MCA param registration functions, so if we change the MCA param function to simply return a non OPAL_SUCCESS status, it's unlikely that anyone will notice until some code tries to read the param value, likely still resulting in a segv.
> Does anyone have heartburn if I change the error behavior to opal_finalize()/exit(1)?
> Jeff Squyres
> For corporate legal information go to:
> devel mailing list
For corporate legal information go to: