Thanks, Ralph. I quickly checked the fix. It worked fine for me.
Tetsuya Mishima
> I fixed that in today's final cleanup
>
> On Jan 27, 2014, at 3:17 PM, tmish...@jcity.maeda.co.jp wrote:
>
>
>
> As for the NEWS - it is actually already correct. We default to map-by
> core, not slot, as of 1.7.4.
>
> Is it correct? As far as I browse the source code, map-by slot is used if
> np <=2.
>
> [mishima@manage openmpi-1.7.4rc2r30425]$ cat -n
> orte/mca/rmaps/base/rmaps_base_map_job.c
> ...
> 107 /* default based on number of procs */
> 108 if (nprocs <= 2) {
> 109 opal_output_verbose(5,
> orte_rmaps_base_framework.framework_output,
> 110 "mca:rmaps mapping not given -
> using byslot");
> 111 ORTE_SET_MAPPING_POLICY(map->mapping,
> ORTE_MAPPING_BYSLOT);
> 112 } else {
> 113 opal_output_verbose(5,
> orte_rmaps_base_framework.framework_output,
> 114 "mca:rmaps mapping not given -
> using bysocket");
> 115 ORTE_SET_MAPPING_POLICY(map->mapping,
> ORTE_MAPPING_BYSOCKET);
> 116 }
>
> Regards,
> Tetsuya Mishima
>
> On Jan 26, 2014, at 3:02 PM, tmish...@jcity.maeda.co.jp wrote:
>
>
> Hi Ralph,
>
> I tried latest nightly snapshots of openmpi-1.7.4rc2r30425.tar.gz.
> Almost everything works fine, except that the unexpected output appears
> as below:
>
> [mishima@node04 ~]$ mpirun -cpus-per-proc 4 ~/mis/openmpi/demos/myprog
> App launch reported: 3 (out of 3) daemons - 8 (out of 12) procs
> ...
>
> You dropped the if-statement checking "orte_report_launch_progress" in
> plm_base_receive.c @ r30423, which causes the problem.
>
> --- orte/mca/plm/base/plm_base_receive.c.org2014-01-25
> 11:51:59.000000000 +0900
> +++ orte/mca/plm/base/plm_base_receive.c2014-01-26
> 12:20:10.000000000
> +0900
> @@ -315,9 +315,11 @@
> /* record that we heard back from a daemon during app
> launch
> */
> if (running && NULL != jdata) {
> jdata->num_daemons_reported++;
> - if (0 == jdata->num_daemons_reported % 100 ||
> - jdata->num_daemons_reported ==
> orte_process_info.num_procs) {
> - ORTE_ACTIVATE_JOB_STATE(jdata,
> ORTE_JOB_STATE_REPORT_PROGRESS);
> + if (orte_report_launch_progress) {
> + if (0 == jdata->num_daemons_reported % 100 ||
> + jdata->num_daemons_reported ==
> orte_process_info.num_procs) {
> + ORTE_ACTIVATE_JOB_STATE(jdata,
> ORTE_JOB_STATE_REPORT_PROGRESS);
> + }
> }
> }
> /* prepare for next job */
>
> Regards,
> Tetsuya Mishima
>
> P.S. It's also better to change the line 65 in NEWS.
>
> ...
> 64 * Mapping:
> 65 * if #procs <= 2, default to map-by core -> map-by slot
> ^^^^^^^^^^^
> 66 * if #procs > 2, default to map-by socket
> ...
>
> _______________________________________________
> users mailing list
> us...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/users
>
> _______________________________________________
> users mailing list
> us...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/users
>
> _______________________________________________
> users mailing list
> us...@open-mpi.org
>
http://www.open-mpi.org/mailman/listinfo.cgi/users_______________________________________________
> users mailing list
> users@open-mpi.orghttp://www.open-mpi.org/mailman/listinfo.cgi/users
