[Rpm-maint] [rpm-software-management/rpm] Add macro to force fsync() on close() (#187)

Jeff Johnson notifications at github.com
Sun Apr 9 21:50:30 UTC 2017


Here is an updated patch, including the bare minimum changes with a %_periodic_fsync "opt-out" macro to the "production" rpmfiArchiveReadToFilePsm() to use fdatasync+posix_fadvise+fsync.

I've left all the debugging gook in the patch to document the --rpmiodebug trace and to illustrate the mincore(2) etc verification testing in the previous comment.

The important & necessary parts of the patch should be obvious.

The patch passes "make check" and isn't going to kill your kittens or steal your girl friend. ;-)

Experience with the cost of always (including rotating DASD) installing with fsync is needed before undertaking further steps.

[rpm+fsync.patch.gz](https://github.com/rpm-software-management/rpm/files/908578/rpm.fsync.patch.gz)


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/187#issuecomment-292816086
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20170409/6349af77/attachment.html>


More information about the Rpm-maint mailing list