Project

General

Profile

Bug #1441

Lies at startup about "The comm. thread both sends and receives messages"

Added by Jim Phillips 2 months ago. Updated 11 days ago.

Status:
Merged
Priority:
Urgent
Assignee:
Category:
Machine Layers
Target version:
Start date:
02/21/2017
Due date:
% Done:

0%


Description

There is logic for setting and reporting Cmi_smp_mode_setting at startup, but this only reflects the behavior of the mpi-smp and gni-smp layers.

jim@sunnyvale$grep -r Cmi_smp_mode_setting .
./arch/util/machine-common-core.c:static enum MACHINE_SMP_MODE Cmi_smp_mode_setting = COMM_THREAD_SEND_RECV;
./arch/util/machine-common-core.c:      if (Cmi_smp_mode_setting == COMM_THREAD_SEND_RECV) {
./arch/util/machine-common-core.c:      } else if (Cmi_smp_mode_setting == COMM_THREAD_ONLY_RECV) {
./arch/util/machine-common-core.c:      } else if (Cmi_smp_mode_setting == COMM_WORK_THREADS_SEND_RECV) {
./arch/util/machine-common-core.c:      } else if (Cmi_smp_mode_setting == COMM_THREAD_NOT_EXIST) {
./arch/util/machine-common-core.c:    if(Cmi_smp_mode_setting == COMM_THREAD_ONLY_RECV
./arch/util/machine-common-core.c:       || Cmi_smp_mode_setting == COMM_THREAD_NOT_EXIST)
./arch/mpi/machine.c:    if (Cmi_smp_mode_setting == COMM_THREAD_SEND_RECV) {
./arch/mpi/machine.c:  if (Cmi_smp_mode_setting == COMM_THREAD_ONLY_RECV) {
./arch/mpi/machine.c:    if(Cmi_smp_mode_setting == COMM_THREAD_SEND_RECV){
./arch/mpi/machine.c:    }else if(Cmi_smp_mode_setting == COMM_THREAD_ONLY_RECV) {
./arch/mpi/machine.c:      Cmi_smp_mode_setting = COMM_THREAD_ONLY_RECV;
./arch/mpi/machine.c:    if (Cmi_smp_mode_setting == COMM_THREAD_SEND_RECV)
./arch/mpi/machine.c:    if (Cmi_smp_mode_setting == COMM_THREAD_ONLY_RECV && _thread_provided != MPI_THREAD_MULTIPLE) {
./arch/mpi/machine.c:        Cmi_smp_mode_setting = COMM_THREAD_SEND_RECV; 
./arch/gni/machine.c:    Cmi_smp_mode_setting = COMM_WORK_THREADS_SEND_RECV;

History

#1 Updated by Jim Phillips 2 months ago

More specifically, verbs and net-ibverbs only use the comm thread for receives.

#2 Updated by Sam White about 2 months ago

  • Target version set to 6.8.0
  • Assignee set to Nitin Bhat

#3 Updated by Phil Miller about 1 month ago

  • Assignee changed from Nitin Bhat to Jaemin Choi

#4 Updated by Phil Miller 19 days ago

  • Priority changed from Normal to Urgent

Setting priority higher because these are easy to fix things that act as paper-cuts to users

#5 Updated by Jaemin Choi 18 days ago

  • Status changed from New to Implemented

#6 Updated by Sam White 11 days ago

  • Status changed from Implemented to Merged
  • Subject changed from lies at startup about "The comm. thread both sends and receives messages" to Lies at startup about "The comm. thread both sends and receives messages"

Also available in: Atom PDF