Re: [OMPI devel] [OMPI svn] svn:open-mpi r32346 - trunk/opal/mca/btl/openib

2014-07-29 Thread Joshua Ladd
Nathan, can you take a look at https://svn.open-mpi.org/trac/ompi/changeset/32350 when you get a chance. Thanks, Josh On Tue, Jul 29, 2014 at 6:14 PM, Nathan Hjelm wrote: > On Tue, Jul 29, 2014 at 04:12:18PM -0600, Nathan Hjelm wrote: > > > > Yeah. Though it would be best to just check the so

Re: [OMPI devel] [OMPI svn] svn:open-mpi r32346 - trunk/opal/mca/btl/openib

2014-07-29 Thread Nathan Hjelm
On Tue, Jul 29, 2014 at 04:12:18PM -0600, Nathan Hjelm wrote: > > Yeah. Though it would be best to just check the source when you need to > see if it should come from the ini. Then if we need to set the value > from the ini either use mca_base_var_set_value or be sure to strdup when > changing the

Re: [OMPI devel] [OMPI svn] svn:open-mpi r32346 - trunk/opal/mca/btl/openib

2014-07-29 Thread Nathan Hjelm
Yeah. Though it would be best to just check the source when you need to see if it should come from the ini. Then if we need to set the value from the ini either use mca_base_var_set_value or be sure to strdup when changing the receive_queues value. The thing to remember is the use may do this: M

Re: [OMPI devel] [OMPI svn] svn:open-mpi r32346 - trunk/opal/mca/btl/openib

2014-07-29 Thread Joshua Ladd
Thanks, Nathan This is a user's patch. So, if I understand correctly, the best thing would be to: vari = mca_base_var_find ("opal", "btl", "openib", "receive_queues"); mca_base_var_get_value (vari, NULL, &source, NULL); mca_btl_openib_component. receive_queues_source = source Instead of doing

Re: [OMPI devel] [OMPI svn] svn:open-mpi r32346 - trunk/opal/mca/btl/openib

2014-07-29 Thread Nathan Hjelm
Josh, you can not free from a memory location that has been registered with the MCA variable system. It will likely SEGV when the component is unloaded. You should call mca_base_var_get_value to get the source of the value. The following should do it: vari = mca_base_var_find ("opal", "btl", "op