topjohnwu/ndk-busybox
1
0
Fork 0
mirror of https://github.com/topjohnwu/ndk-busybox.git synced 2024-11-27 13:40:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fsync,sync: make them similar

Browse Source 
sync: add O_NOCTTY
fsync: drop O_NOATIME, add O_NONBLOCK, set exitcode to 1 if fsync() fails,
    update --help message to be similar to sync.
both: reformat code to minimize "diff -u sync.c fsync.c":
    in particular, they use same open() flags now

function                                             old     new   delta
fsync_main                                           126     130      +4
packed_usage                                       33316   33317      +1
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 2/0 up/down: 5/0)                 Total: 5 bytes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2019-04-02 14:39:56 +02:00
parent e48559eae3
commit 4f74bb6059
2 changed files with 25 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
coreutils/fsync.c
View File

@ -12,6 +12,7 @@
//config: help //config: help
//config: fsync is used to flush file-related cached blocks to disk. //config: fsync is used to flush file-related cached blocks to disk.
// APPLET_NOFORK:name main location suid_type help
//applet:IF_FSYNC(APPLET_NOFORK(fsync, fsync, BB_DIR_BIN, BB_SUID_DROP, fsync)) //applet:IF_FSYNC(APPLET_NOFORK(fsync, fsync, BB_DIR_BIN, BB_SUID_DROP, fsync))
//kbuild:lib-$(CONFIG_FSYNC) += fsync.o //kbuild:lib-$(CONFIG_FSYNC) += fsync.o
@ -19,20 +20,17 @@
//usage:#define fsync_trivial_usage //usage:#define fsync_trivial_usage
//usage: "[-d] FILE..." //usage: "[-d] FILE..."
//usage:#define fsync_full_usage "\n\n" //usage:#define fsync_full_usage "\n\n"
//usage: "Write files' buffered blocks to disk\n" //usage: "Write all buffered blocks in FILEs to disk\n"
//usage: "\n -d Avoid syncing metadata" //usage: "\n -d Avoid syncing metadata"
#include "libbb.h" #include "libbb.h"
#ifndef O_NOATIME
# define O_NOATIME 0
#endif
/* This is a NOFORK applet. Be very careful! */ /* This is a NOFORK applet. Be very careful! */
int fsync_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int fsync_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int fsync_main(int argc UNUSED_PARAM, char **argv) int fsync_main(int argc UNUSED_PARAM, char **argv)
{ {
int status; int ret;
int opts; int opts;
opts = getopt32(argv, "d"); /* fdatasync */ opts = getopt32(argv, "d"); /* fdatasync */
@ -41,20 +39,24 @@ int fsync_main(int argc UNUSED_PARAM, char **argv)
bb_show_usage(); bb_show_usage();
} }
status = EXIT_SUCCESS; ret = EXIT_SUCCESS;
do { do {
int fd = open_or_warn(*argv, O_NOATIME | O_NOCTTY | O_RDONLY); /* GNU "sync FILE" uses O_NONBLOCK open */
int fd = open_or_warn(*argv, /*O_NOATIME |*/ O_NOCTTY | O_RDONLY | O_NONBLOCK);
/* open(NOATIME) can only be used by owner or root, don't use NOATIME here */
if (fd == -1) { if (fd < 0) {
status = EXIT_FAILURE; ret = EXIT_FAILURE;
continue; goto next;
} }
if ((opts ? fdatasync(fd) : fsync(fd))) { if ((opts ? fdatasync(fd) : fsync(fd)) != 0) {
//status = EXIT_FAILURE; - do we want this?
bb_simple_perror_msg(*argv); bb_simple_perror_msg(*argv);
ret = EXIT_FAILURE;
} }
close(fd); close(fd);
} while (*++argv); next:
argv++;
} while (*argv);
return status; return ret;
} }

22
coreutils/sync.c
View File

@ -20,6 +20,7 @@
//config: sync -d FILE... executes fdatasync() on each FILE. //config: sync -d FILE... executes fdatasync() on each FILE.
//config: sync -f FILE... executes syncfs() on each FILE. //config: sync -f FILE... executes syncfs() on each FILE.
// APPLET_NOFORK:name main location suid_type help
//applet:IF_SYNC(APPLET_NOFORK(sync, sync, BB_DIR_BIN, BB_SUID_DROP, sync)) //applet:IF_SYNC(APPLET_NOFORK(sync, sync, BB_DIR_BIN, BB_SUID_DROP, sync))
//kbuild:lib-$(CONFIG_SYNC) += sync.o //kbuild:lib-$(CONFIG_SYNC) += sync.o
@ -52,7 +53,7 @@ int sync_main(int argc UNUSED_PARAM, char **argv IF_NOT_DESKTOP(UNUSED_PARAM))
return EXIT_SUCCESS; return EXIT_SUCCESS;
#else #else
unsigned opts; unsigned opts;
int ret = EXIT_SUCCESS; int ret;
enum { enum {
OPT_DATASYNC = (1 << 0), OPT_DATASYNC = (1 << 0),
@ -66,35 +67,30 @@ int sync_main(int argc UNUSED_PARAM, char **argv IF_NOT_DESKTOP(UNUSED_PARAM))
if (!argv[0]) if (!argv[0])
sync(); sync();
ret = EXIT_SUCCESS;
while (*argv) { while (*argv) {
int fd = open_or_warn(*argv, O_RDONLY); /* GNU "sync FILE" uses O_NONBLOCK open */
int fd = open_or_warn(*argv, /*O_NOATIME |*/ O_NOCTTY | O_RDONLY | O_NONBLOCK);
/* open(NOATIME) can only be used by owner or root, don't use NOATIME here */
if (fd < 0) { if (fd < 0) {
ret = EXIT_FAILURE; ret = EXIT_FAILURE;
goto next; goto next;
} }
if (opts & OPT_DATASYNC) {
if (fdatasync(fd))
goto err;
goto do_close;
}
if (opts & OPT_SYNCFS) { if (opts & OPT_SYNCFS) {
/* /*
* syncfs is documented to only fail with EBADF, * syncfs is documented to only fail with EBADF,
* which can't happen here. So, no error checks. * which can't happen here. So, no error checks.
*/ */
syncfs(fd); syncfs(fd);
goto do_close; } else
} if (((opts & OPT_DATASYNC) ? fdatasync(fd) : fsync(fd)) != 0) {
if (fsync(fd)) {
err:
bb_simple_perror_msg(*argv); bb_simple_perror_msg(*argv);
ret = EXIT_FAILURE; ret = EXIT_FAILURE;
} }
do_close:
close(fd); close(fd);
next: next:
++argv; argv++;
} }
return ret; return ret;