Skip to content

Commit 2ef5e72

Browse files
committed
io/ompio: change default for sharedfp_lazy_open parameter
Revert the logic of io_ompio_sharedfp_lazy_open. The user now has to explicitely disable shared fp in order for the structures not to be allocated. Otherwise, resetting the shared fp e.g. in case the file was opened in append mode will not work correctly, the code could deadlock. Signed-off-by: Edgar Gabriel <[email protected]>
1 parent 739923c commit 2ef5e72

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ompi/mca/io/ompio/io_ompio_component.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ int mca_io_ompio_bytes_per_agg = OMPIO_PREALLOC_MAX_BUF_SIZE;
4949
int mca_io_ompio_num_aggregators = -1;
5050
int mca_io_ompio_record_offset_info = 0;
5151
int mca_io_ompio_coll_timing_info = 0;
52-
int mca_io_ompio_sharedfp_lazy_open = 1;
52+
int mca_io_ompio_sharedfp_lazy_open = 0;
5353

5454
int mca_io_ompio_grouping_option=5;
5555

@@ -206,7 +206,7 @@ static int register_component(void)
206206
&mca_io_ompio_num_aggregators);
207207

208208

209-
mca_io_ompio_sharedfp_lazy_open = 1;
209+
mca_io_ompio_sharedfp_lazy_open = 0;
210210
(void) mca_base_component_var_register(&mca_io_ompio_component.io_version,
211211
"sharedfp_lazy_open",
212212
"lazy allocation of internal shared file pointer structures",

0 commit comments

Comments
 (0)