1
1
openmpi/ompi/debuggers
David Solt 50aa143ab6 Major structural changes to data types: .super infosubscriber
ompi_communicator_t, ompi_win_t, ompi_file_t all have a super class of type opal_infosubscriber_t instead of a base/super type of opal_object_t (in previous code comm used c_base, but file used super).  It may be a bit bold to say that being a subscriber of MPI_Info is the foundational piece that ties these three things together, but if you object, then I would prefer to turn infosubscriber into a more general name that encompasses other common features rather than create a different super class.  The key here is that we want to be able to pass comm, win and file objects as if they were opal_infosubscriber_t, so that one routine can heandle all 3 types of objects being passed to it.

MPI_INFO_NULL is still an ompi_predefined_info_t type since an MPI_Info is part of ompi but the internal details of the underlying information concept is part of opal.

An ompi_info_t type still exists for exposure to the user, but it is simply a wrapper for the opal object.

Routines such as ompi_info_dup, etc have all been moved to opal_info_dup and related to the opal directory.

Fortran to C translation tables are only used for MPI_Info that is exposed to the application and are therefore part of the ompi_info_t and not the opal_info_t

The data structure changes are primarily in the following files:

    communicator/communicator.h
    ompi/info/info.h
    ompi/win/win.h
    ompi/file/file.h

The following new files were created:

    opal/util/info.h
    opal/util/info.c
    opal/util/info_subscriber.h
    opal/util/info_subscriber.c

This infosubscriber concept is that communicators, files and windows can have subscribers that subscribe to any changes in the info associated with the comm/file/window.  When xxx_set_info is called, the new info is presented to each subscriber who can modify the info in any way they want.  The new value is presented to the next subscriber and so on until all subscribers have had a chance to modify the value.  Therefore, the order of subscribers can make a difference but we hope that there is generally only one subscriber that cares or modifies any given key/value pair.  The final info is then stored and returned by a call to xxx_get_info.

The new model can be seen in the following files:

    ompi/mpi/c/comm_get_info.c
    ompi/mpi/c/comm_set_info.c
    ompi/mpi/c/file_get_info.c
    ompi/mpi/c/file_set_info.c
    ompi/mpi/c/win_get_info.c
    ompi/mpi/c/win_set_info.c

The current subscribers where changed as follows:

    mca/io/ompio/io_ompio_file_open.c
    mca/io/ompio/io_ompio_module.c
    mca/osc/rmda/osc_rdma_component.c (This one actually subscribes to "no_locks")
    mca/osc/sm/osc_sm_component.c (This one actually subscribes to "blocking_fence" and "alloc_shared_contig")

Signed-off-by: Mark Allen <markalle@us.ibm.com>

Conflicts:
	AUTHORS
	ompi/communicator/comm.c
	ompi/debuggers/ompi_mpihandles_dll.c
	ompi/file/file.c
	ompi/file/file.h
	ompi/info/info.c
	ompi/mca/io/ompio/io_ompio.h
	ompi/mca/io/ompio/io_ompio_file_open.c
	ompi/mca/io/ompio/io_ompio_file_set_view.c
	ompi/mca/osc/pt2pt/osc_pt2pt.h
	ompi/mca/sharedfp/addproc/sharedfp_addproc.h
	ompi/mca/sharedfp/addproc/sharedfp_addproc_file_open.c
	ompi/mca/topo/treematch/topo_treematch_dist_graph_create.c
	ompi/mpi/c/lookup_name.c
	ompi/mpi/c/publish_name.c
	ompi/mpi/c/unpublish_name.c
	opal/mca/mpool/base/mpool_base_alloc.c
	opal/util/Makefile.am
2017-05-12 14:41:05 -04:00
..
debuggers.h Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
dlopen_test.c Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
Makefile.am Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
MPI_Handles_interface.txt Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
mpihandles_interface.h Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
msgq_interface.h Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
ompi_common_dll_defs.h ompi/debuggers: make the binary bit reproducible 2016-12-08 13:46:43 +09:00
ompi_common_dll.c ompi/debuggers: make the binary bit reproducible 2016-12-08 13:46:43 +09:00
ompi_debugger_canary.c Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
ompi_debuggers.c Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
ompi_mpihandles_dll_defs.h Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
ompi_mpihandles_dll.c Major structural changes to data types: .super infosubscriber 2017-05-12 14:41:05 -04:00
ompi_msgq_dll_defs.h Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
ompi_msgq_dll.c Merge pull request #2448 from thananon/remove_request_lock 2017-01-03 19:31:46 -08:00
predefined_gap_test.c Major structural changes to data types: .super infosubscriber 2017-05-12 14:41:05 -04:00
predefined_pad_test.c Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00
tv-debugger-attach.txt Revert "Update to sync with OMPI master and cleanup to build" 2016-11-22 15:03:20 -08:00