Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the VOL for HDF5 2.0.0 changes. #73

Merged
merged 16 commits into from
Nov 14, 2024
Merged

Update the VOL for HDF5 2.0.0 changes. #73

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
32b09d9
removed the use of H5VLpeek_connector_id_by_name as it was removed in…
brtnfld Oct 14, 2024
250be3c
installed uuid-dev for ubuntu
brtnfld Oct 14, 2024
569adde
switch to testing the state of info for calling H5VLclose
brtnfld Oct 14, 2024
cf2b1de
disable iotest
brtnfld Oct 15, 2024
0fe899b
fixed warnings
brtnfld Nov 1, 2024
ea07c0c
added library context
brtnfld Nov 7, 2024
689cc7c
updated test to use newest HDF5 code
brtnfld Nov 12, 2024
0d0554e
enable hdf5-iotest
brtnfld Nov 13, 2024
0042b10
changed when to use h5pcc for compiling hdf5-iotest
brtnfld Nov 13, 2024
3aca8f7
update patch
brtnfld Nov 13, 2024
368b5fd
testing
brtnfld Nov 13, 2024
0ffd842
testing
brtnfld Nov 13, 2024
a8b7940
testing
brtnfld Nov 13, 2024
936b353
removed testing statements
brtnfld Nov 13, 2024
c077ef9
updated doc. link
brtnfld Nov 13, 2024
a50b135
removed CI testing checks
brtnfld Nov 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 17 additions & 3 deletions src/H5VL_log_file.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ void *H5VL_log_file_create (
htri_t ret;
ret = H5VLis_connector_registered_by_name ("native");
if (ret != 1) { ERR_OUT ("Native VOL not found") }
uvlid = H5VLpeek_connector_id_by_name ("native");
uvlid = H5VLget_connector_id_by_name("native");
CHECK_ID (uvlid)
under_vol_info = NULL;
}
Expand Down Expand Up @@ -173,6 +173,10 @@ void *H5VL_log_file_create (
H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VLFILE_CREATE);
H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VL_LOG_FILE_CREATE_FILE);
H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VL_LOG_FILE_CREATE);
if (under_vol_info == NULL) {
err = H5VLclose(uvlid);
CHECK_ERR
}
}
H5VL_LOGI_EXP_CATCH

Expand Down Expand Up @@ -236,7 +240,7 @@ void *H5VL_log_file_open (
htri_t ret;
ret = H5VLis_connector_registered_by_name ("native");
if (ret != 1) { ERR_OUT ("Native VOL not found") }
uvlid = H5VLpeek_connector_id_by_name ("native");
uvlid = H5VLget_connector_id_by_name("native");
CHECK_ID (uvlid)
under_vol_info = NULL;
// return NULL;
Expand Down Expand Up @@ -304,6 +308,12 @@ void *H5VL_log_file_open (
H5VL_log_filei_register (fp);

H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VL_LOG_FILE_OPEN);

if (under_vol_info == NULL) {
err = H5VLclose(uvlid);
CHECK_ERR
}

}
H5VL_LOGI_EXP_CATCH

Expand Down Expand Up @@ -430,7 +440,7 @@ herr_t H5VL_log_file_specific (void *file,
htri_t ret;
ret = H5VLis_connector_registered_by_name ("native");
if (ret != 1) { ERR_OUT ("Native VOL not found") }
uvlid = H5VLpeek_connector_id_by_name ("native");
uvlid = H5VLget_connector_id_by_name("native");
CHECK_ID (uvlid)
under_vol_info = NULL;
}
Expand All @@ -440,6 +450,10 @@ herr_t H5VL_log_file_specific (void *file,
err = H5VLfile_specific (NULL, uvlid, args, dxpl_id, req);
CHECK_ERR
H5Pclose (under_fapl_id);
if (under_vol_info == NULL) {
err = H5VLclose(uvlid);
CHECK_ERR
}
} break;
case H5VL_FILE_FLUSH: {
H5VL_LOGI_PROFILING_TIMER_START;
Expand Down
20 changes: 16 additions & 4 deletions utils/h5ldump/h5ldump.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ const char *hdf5sig = "\211HDF\r\n\032\n";
const char ncsig[] = {'C', 'D', 'F'};

inline std::string get_file_signature (std::string &path) {
herr_t err = 0;
hid_t fid = -1; // File ID
hid_t faplid = -1; // File access property ID
hid_t nativevlid = -1; // Native VOL ID
Expand All @@ -50,9 +51,14 @@ inline std::string get_file_signature (std::string &path) {
fin.close ();
if (!memcmp (hdf5sig, sig, 8)) {
// Always use native VOL
nativevlid = H5VLpeek_connector_id_by_name ("native");
nativevlid = H5VLget_connector_id_by_name("native");
CHECK_ID (nativevlid)
faplid = H5Pcreate (H5P_FILE_ACCESS);
H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ID (faplid)
err = H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ERR
err = H5VLclose(nativevlid);
CHECK_ERR

// Open the input file
fid = H5Fopen (path.c_str (), H5F_ACC_RDONLY, faplid);
Expand Down Expand Up @@ -215,9 +221,15 @@ void h5ldump_file (std::string path,
});

// Always use native VOL
nativevlid = H5VLpeek_connector_id_by_name ("native");
nativevlid = H5VLget_connector_id_by_name("native");
CHECK_ID (nativevlid)
faplid = H5Pcreate (H5P_FILE_ACCESS);
H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ID (faplid)
err = H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ERR
err = H5VLclose(nativevlid);
CHECK_ERR


// Open the input file
fid = H5Fopen (path.c_str (), H5F_ACC_RDONLY, faplid);
Expand Down
9 changes: 7 additions & 2 deletions utils/h5ldump/h5ldump_visit.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,14 @@ void h5ldump_visit (std::string path, std::vector<H5VL_log_dset_info_t> &dsets)
});

// Always use native VOL
nativevlid = H5VLpeek_connector_id_by_name ("native");
nativevlid = H5VLget_connector_id_by_name("native");
CHECK_ID (nativevlid)
faplid = H5Pcreate (H5P_FILE_ACCESS);
H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ID (faplid)
err = H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ERR
err = H5VLclose(nativevlid);
CHECK_ERR

// Open the input file
fid = H5Fopen (path.c_str (), H5F_ACC_RDONLY, faplid);
Expand Down
7 changes: 5 additions & 2 deletions utils/h5lreplay/h5lreplay.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,16 +176,19 @@ void h5lreplay_core (std::string &inpath, std::string &outpath, int rank, int np
});

// Open the input and output file
nativevlid = H5VLpeek_connector_id_by_name ("native");
nativevlid = H5VLget_connector_id_by_name("native");
CHECK_ID (nativevlid)
faplid = H5Pcreate (H5P_FILE_ACCESS);
CHECK_ID (faplid)
err = H5Pset_fapl_mpio (faplid, MPI_COMM_WORLD, MPI_INFO_NULL);
CHECK_ERR
err = H5Pset_vol (faplid, nativevlid, NULL);
CHECK_ERR
err = H5VLclose(nativevlid);
CHECK_ERR

finid = H5Fopen (inpath.c_str (), H5F_ACC_RDONLY, faplid);
CHECK_ID (faplid)
CHECK_ID (finid)
foutid = H5Fcreate (outpath.c_str (), H5F_ACC_TRUNC, H5P_DEFAULT, faplid);
CHECK_ID (foutid)
mpierr = MPI_File_open (MPI_COMM_WORLD, inpath.c_str (), MPI_MODE_RDONLY, MPI_INFO_NULL, &fin);
Expand Down
Loading