I've fixed the reporting flag - thanks!
As for the NEWS - it is actually already correct. We default to map-by core,
not slot, as of 1.7.4. However, if cpus-per-proc is given, we should probably
fall back to map-by slot, so I'll make that change
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.org 2014-01-25
> 11:51:59.000000000 +0900
> +++ orte/mca/plm/base/plm_base_receive.c 2014-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
