splatink/lbmk
1
0
Fork 0
mirror of https://codeberg.org/libreboot/lbmk.git synced 2026-03-25 13:29:03 +02:00
Code Issues Packages Projects Releases Wiki Activity

Revert "util/nvmutil: don't use fsync()"

Browse Source 
This reverts commit bdb43afac6.
This commit is contained in:
Leah Rowe
2026-03-16 16:39:59 +00:00
parent 9ef92b3b25
commit fb4f263005
1 changed files with 19 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
util/nvmutil/nvmutil.c
View File

@@ -1109,7 +1109,12 @@ copy_gbe(void)
if (gbe_tmp_size != gbe_file_size)
err(EIO, "%s: %s: not the same size", fname, tname);
sync();
/*
* fsync tmp gbe file, because we will compare
* its contents to what was read (for safety)
*/
if (fsync(tmp_fd) == -1)
err(errno, "%s: fsync (tmpfile copy)", tname);
r = rw_file_exact(tmp_fd, bufcmp, gbe_file_size,
0, IO_PREAD, NO_LOOP_EAGAIN, LOOP_EINTR,
@@ -1796,7 +1801,13 @@ write_to_gbe_bin(void)
write_gbe_file();
sync();
/*
* We may otherwise read from
* cache, so we must sync.
*/
if (fsync(tmp_fd) == -1)
err(errno, "%s: fsync (pre-verification)",
tname);
check_written_part(0);
check_written_part(1);
@@ -2033,7 +2044,8 @@ gbe_mv(void)
if (r < 0)
goto ret_gbe_mv;
sync();
if (fsync(dest_fd) == -1)
goto ret_gbe_mv;
if (x_i_close(dest_fd) == -1)
goto ret_gbe_mv;
@@ -2147,7 +2159,9 @@ fsync_dir(const char *path)
goto err_fsync_dir;
}
sync();
/* sync file on disk */
if (fsync(dfd) == -1)
goto err_fsync_dir;
if (x_i_close(dfd) == -1)
goto err_fsync_dir;
@@ -3090,7 +3104,7 @@ x_i_rename(const char *src, const char *dst)
return -1;
}
sync();
fsync(dfd);
x_i_close(sfd);
x_i_close(dfd);