Skip to content

Commit cb00866

Browse files
authored
Merge pull request open-mpi#1923 from edgargabriel/pr/aggregator-cleanup
io/ompio: move aggregator selection code to a separate file
2 parents 21e7f31 + c0bd872 commit cb00866

File tree

8 files changed

+1411
-1236
lines changed

8 files changed

+1411
-1236
lines changed

ompi/mca/common/ompio/common_ompio_file_open.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ int mca_common_ompio_file_open (ompi_communicator_t *comm,
110110

111111
ompio_fh->f_get_num_aggregators=mca_io_ompio_get_num_aggregators;
112112
ompio_fh->f_get_bytes_per_agg=mca_io_ompio_get_bytes_per_agg;
113-
ompio_fh->f_set_aggregator_props=ompi_io_ompio_set_aggregator_props;
113+
ompio_fh->f_set_aggregator_props=mca_io_ompio_set_aggregator_props;
114114

115115
/* This fix is needed for data seiving to work with
116116
two-phase collective I/O */

ompi/mca/io/ompio/Makefile.am

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,21 +31,25 @@ endif
3131

3232
mcacomponentdir = $(ompilibdir)
3333
mcacomponent_LTLIBRARIES = $(component_install)
34-
mca_io_ompio_la_SOURCES = $(sources)
34+
mca_io_ompio_la_SOURCES = $(headers) $(sources)
3535
mca_io_ompio_la_LDFLAGS = -module -avoid-version
3636
mca_io_ompio_la_LIBADD = $(io_ompio_LIBS) \
3737
$(OMPI_TOP_BUILDDIR)/ompi/mca/common/ompio/lib@OPAL_LIB_PREFIX@mca_common_ompio.la
3838

3939
noinst_LTLIBRARIES = $(component_noinst)
40-
libmca_io_ompio_la_SOURCES = $(sources)
40+
libmca_io_ompio_la_SOURCES = $(headers) $(sources)
4141
libmca_io_ompio_la_LDFLAGS = -module -avoid-version
4242

4343
# Source files
4444

45+
headers = \
46+
io_ompio.h \
47+
io_ompio_request.h \
48+
io_ompio_aggregators.h
49+
4550
sources = \
46-
io_ompio.h \
47-
io_ompio_request.h \
4851
io_ompio.c \
52+
io_ompio_aggregators.c \
4953
io_ompio_component.c \
5054
io_ompio_module.c \
5155
io_ompio_file_set_view.c \

0 commit comments

Comments
 (0)