View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006773ParaView(No Category)public2008-04-08 13:002011-01-13 17:00
ReporterKen Moreland 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.4Fixed in Version3.4 
Summary0006773: Use Broadcast to propagate commands from root sever node to other server nodes
DescriptionAs we all know, the PV client connects to node 0 of servers and sends commands to that node. That node propagates commands to the other server nodes (as appropriate).

This is implemented using the ProcessRMIs method in vtkMultiProcessController. This means that each node waits in a MPI_Recv from any command. Node 0 then iteratively sends a message to each of the other nodes.

This is not a very efficient way to broadcast messages. Instead, everyone should use the Broadcast method to propagate the requests. (Broadcast is now part of vtkMultiProcessController.)
Additional InformationWhen working correctly, the current method isn't a huge deal. However, there is a bug in the buffer management code of OpenMPI that is causing node 0 to eat up a bunch of memory. This is a killer for one of our important uses and the main driver for this bug.
TagsNo tags attached.
Project
Topic Name
Type
Attached Files

 Relationships
has duplicate 0006035closedBerk Geveci (MARS-immediate) 0th process using too much memory 

  Notes
(0012672)
Utkarsh Ayachit (administrator)
2008-07-09 10:34

Add global "use-ssend-for-trigger-RMI" and then set it in paraview since there are complications with broadcast. We should investigate using broadcast after 3.4.
(0012723)
Utkarsh Ayachit (administrator)
2008-07-16 12:56

ParaView now uses Ssend for all TriggerRMI requests.

/cvsroot/ParaView3/ParaView3/Servers/Common/vtkProcessModule.cxx,v <-- Servers
/Common/vtkProcessModule.cxx
new revision: 1.85; previous revision: 1.84
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkMPICommunicator.cxx,v <-- VTK/Par
allel/vtkMPICommunicator.cxx
new revision: 1.48; previous revision: 1.47
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkMPICommunicator.h,v <-- VTK/Paral
lel/vtkMPICommunicator.h
new revision: 1.37; previous revision: 1.36
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkMPIController.cxx,v <-- VTK/Paral
lel/vtkMPIController.cxx
new revision: 1.25; previous revision: 1.24
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkMPIController.h,v <-- VTK/Paralle
l/vtkMPIController.h
new revision: 1.24; previous revision: 1.23
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkMultiProcessController.cxx,v <--
VTK/Parallel/vtkMultiProcessController.cxx
new revision: 1.30; previous revision: 1.29
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkMultiProcessController.h,v <-- VT
K/Parallel/vtkMultiProcessController.h
new revision: 1.37; previous revision: 1.36
(0012832)
Alan Scott (manager)
2008-07-25 13:40

Accepting utkarsh's word for it...

 Issue History
Date Modified Username Field Change
2008-04-08 13:00 Ken Moreland New Issue
2008-04-08 13:01 Ken Moreland Relationship added has duplicate 0006035
2008-04-10 18:21 Ken Moreland Status backlog => tabled
2008-04-10 18:21 Ken Moreland Assigned To => Berk Geveci
2008-07-09 10:23 Utkarsh Ayachit Assigned To Berk Geveci => Utkarsh Ayachit
2008-07-09 10:34 Utkarsh Ayachit Note Added: 0012672
2008-07-16 12:56 Utkarsh Ayachit Status tabled => @80@
2008-07-16 12:56 Utkarsh Ayachit Resolution open => fixed
2008-07-16 12:56 Utkarsh Ayachit Note Added: 0012723
2008-07-25 13:40 Alan Scott Status @80@ => closed
2008-07-25 13:40 Alan Scott Note Added: 0012832
2009-05-13 13:58 Utkarsh Ayachit Target Version => 3.4
2009-05-13 13:59 Utkarsh Ayachit Fixed in Version => 3.4
2011-01-13 17:00 Source_changeset_attached => VTK master a2bd8391
2011-01-13 17:00 Source_changeset_attached => VTK master 020ef709
2011-06-16 13:10 Zack Galbreath Category => (No Category)


Copyright © 2000 - 2018 MantisBT Team