Good catch - the problem is that ompi_info_get_bool returns "success" if the
value isn't found, setting "flag" to false, but doesn't set the value of the
param itself. So if you don't specify "blocking_fence" in MPI_Info, then the
"blocking_fence" flag wasn't being set.
Fixed in r32812 and scheduled for 1.8.4
Thanks!
Ralph
On Sep 28, 2014, at 2:43 AM, Lisandro Dalcin <dalc...@gmail.com> wrote:
> Just built 1.8.3 for another round of testing with mpi4py. I'm getting
> the following valgrind warning:
>
> ==4718== Conditional jump or move depends on uninitialised value(s)
> ==4718== at 0xD0D9F4C: component_select (osc_sm_component.c:333)
> ==4718== by 0x4CF44F6: ompi_osc_base_select (osc_base_init.c:73)
> ==4718== by 0x4C68B69: ompi_win_allocate (win.c:182)
> ==4718== by 0x4CBB8C2: PMPI_Win_allocate (pwin_allocate.c:79)
> ==4718== by 0x400898: main (in /home/dalcinl/Devel/BUGS-MPI/openmpi/a.out)
>
> The offending code is in ompi/mca/osc/sm/osc_sm_component.c, it seems
> you forgot to initialize the "blocking_fence" to a default true or
> false value.
>
> bool blocking_fence;
> int flag;
>
> if (OMPI_SUCCESS != ompi_info_get_bool(info, "blocking_fence",
> &blocking_fence, &flag)) {
> goto error;
> }
>
> if (blocking_fence) {
>
>
> --
> Lisandro Dalcin
> ============
> Research Scientist
> Computer, Electrical and Mathematical Sciences & Engineering (CEMSE)
> Numerical Porous Media Center (NumPor)
> King Abdullah University of Science and Technology (KAUST)
> http://numpor.kaust.edu.sa/
>
> 4700 King Abdullah University of Science and Technology
> al-Khawarizmi Bldg (Bldg 1), Office # 4332
> Thuwal 23955-6900, Kingdom of Saudi Arabia
> http://www.kaust.edu.sa
>
> Office Phone: +966 12 808-0459
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post:
> http://www.open-mpi.org/community/lists/devel/2014/09/15925.php
