123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459 |
- From b15b6cc56ac7764be17acbdbf96448f388992adc Mon Sep 17 00:00:00 2001
- From: Waldemar Brodkorb <wbx@openadk.org>
- Date: Wed, 23 Aug 2023 15:41:28 +0200
- Subject: [PATCH] f2fs-tools: convert to lseek() and kill lseek64
- This patch replaces lseek64 with lseek() having #define _FILE_OFFSET_BITS 64.
- Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
- Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
- Upstream: https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs-tools.git/commit/?id=b15b6cc56ac7764be17acbdbf96448f388992adc
- ---
- configure.ac | 2 --
- fsck/Makefile.am | 2 +-
- include/android_config.h | 4 ----
- lib/Makefile.am | 2 +-
- lib/libf2fs.c | 3 ---
- lib/libf2fs_io.c | 47 +++++++++++++++------------------------
- lib/libf2fs_zoned.c | 1 -
- mkfs/Makefile.am | 4 ++--
- mkfs/f2fs_format.c | 2 --
- mkfs/f2fs_format_main.c | 2 --
- mkfs/f2fs_format_utils.c | 10 ---------
- mkfs/f2fs_format_utils.h | 2 --
- tools/Makefile.am | 2 +-
- tools/f2fs_io/Makefile.am | 2 +-
- tools/f2fs_io/f2fs_io.c | 6 -----
- tools/f2fs_io_parse.c | 1 -
- tools/f2fscrypt.c | 9 --------
- tools/fibmap.c | 7 ------
- 18 files changed, 24 insertions(+), 84 deletions(-)
- diff --git a/configure.ac b/configure.ac
- index efab8d8..123ddbb 100644
- --- a/configure.ac
- +++ b/configure.ac
- @@ -187,8 +187,6 @@ AC_CHECK_FUNCS_ONCE([
- getmntent
- getuid
- keyctl
- - llseek
- - lseek64
- memset
- setmntent
- clock_gettime
- diff --git a/fsck/Makefile.am b/fsck/Makefile.am
- index 579dd26..40d31b8 100644
- --- a/fsck/Makefile.am
- +++ b/fsck/Makefile.am
- @@ -1,7 +1,7 @@
- ## Makefile.am
-
- AM_CPPFLAGS = ${libuuid_CFLAGS} -I$(top_srcdir)/include
- -AM_CFLAGS = -Wall
- +AM_CFLAGS = -Wall -D_FILE_OFFSET_BITS=64
- sbin_PROGRAMS = fsck.f2fs
- noinst_HEADERS = common.h dict.h dqblk_v2.h f2fs.h fsck.h node.h quotaio.h \
- quotaio_tree.h quotaio_v2.h xattr.h compress.h
- diff --git a/include/android_config.h b/include/android_config.h
- index 141fe06..da8abcb 100644
- --- a/include/android_config.h
- +++ b/include/android_config.h
- @@ -30,8 +30,6 @@
- #define HAVE_FSTAT 1
- #define HAVE_FSTAT64 1
- #define HAVE_GETMNTENT 1
- -#define HAVE_LLSEEK 1
- -#define HAVE_LSEEK64 1
- #define HAVE_MEMSET 1
- #define HAVE_SELINUX_ANDROID_H 1
- #define HAVE_SETMNTENT 1
- @@ -67,7 +65,6 @@
- #define HAVE_FSTAT 1
- #define HAVE_FSTAT64 1
- #define HAVE_GETMNTENT 1
- -#define HAVE_LLSEEK 1
- #define HAVE_MEMSET 1
- #define HAVE_SPARSE_SPARSE_H 1
- #define HAVE_LIBLZ4 1
- @@ -78,6 +75,5 @@
- #endif
-
- #if defined(_WIN32)
- -#define HAVE_LSEEK64
- #define HAVE_SPARSE_SPARSE_H 1
- #endif
- diff --git a/lib/Makefile.am b/lib/Makefile.am
- index 871d773..69d46f8 100644
- --- a/lib/Makefile.am
- +++ b/lib/Makefile.am
- @@ -3,7 +3,7 @@
- lib_LTLIBRARIES = libf2fs.la
-
- libf2fs_la_SOURCES = libf2fs.c libf2fs_io.c libf2fs_zoned.c nls_utf8.c
- -libf2fs_la_CFLAGS = -Wall
- +libf2fs_la_CFLAGS = -Wall -D_FILE_OFFSET_BITS=64
- libf2fs_la_CPPFLAGS = -I$(top_srcdir)/include
- libf2fs_la_LDFLAGS = -version-info $(LIBF2FS_CURRENT):$(LIBF2FS_REVISION):$(LIBF2FS_AGE)
-
- diff --git a/lib/libf2fs.c b/lib/libf2fs.c
- index 577a7e8..c3d5744 100644
- --- a/lib/libf2fs.c
- +++ b/lib/libf2fs.c
- @@ -6,9 +6,6 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#define _LARGEFILE64_SOURCE
- -#define _FILE_OFFSET_BITS 64
- -
- #include <f2fs_fs.h>
- #include <stdio.h>
- #include <stdlib.h>
- diff --git a/lib/libf2fs_io.c b/lib/libf2fs_io.c
- index 1a8167d..74e5f3a 100644
- --- a/lib/libf2fs_io.c
- +++ b/lib/libf2fs_io.c
- @@ -11,8 +11,6 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#define _LARGEFILE64_SOURCE
- -
- #include <stdio.h>
- #include <stdlib.h>
- #include <string.h>
- @@ -67,22 +65,13 @@ static int __get_device_fd(__u64 *offset)
- return -1;
- }
-
- -#ifndef HAVE_LSEEK64
- -typedef off_t off64_t;
- -
- -static inline off64_t lseek64(int fd, __u64 offset, int set)
- -{
- - return lseek(fd, offset, set);
- -}
- -#endif
- -
- /* ---------- dev_cache, Least Used First (LUF) policy ------------------- */
- /*
- * Least used block will be the first victim to be replaced when max hash
- * collision exceeds
- */
- static bool *dcache_valid; /* is the cached block valid? */
- -static off64_t *dcache_blk; /* which block it cached */
- +static off_t *dcache_blk; /* which block it cached */
- static uint64_t *dcache_lastused; /* last used ticks for cache entries */
- static char *dcache_buf; /* cached block data */
- static uint64_t dcache_usetick; /* current use tick */
- @@ -172,7 +161,7 @@ static int dcache_alloc_all(long n)
- {
- if (n <= 0)
- return -1;
- - if ((dcache_blk = (off64_t *) malloc(sizeof(off64_t) * n)) == NULL
- + if ((dcache_blk = (off_t *) malloc(sizeof(off_t) * n)) == NULL
- || (dcache_lastused = (uint64_t *)
- malloc(sizeof(uint64_t) * n)) == NULL
- || (dcache_buf = (char *) malloc (F2FS_BLKSIZE * n)) == NULL
- @@ -257,7 +246,7 @@ static inline long dcache_relocate(long entry, int n)
- dcache_config.num_cache_entry;
- }
-
- -static long dcache_find(off64_t blk)
- +static long dcache_find(off_t blk)
- {
- register long n = dcache_config.num_cache_entry;
- register unsigned m = dcache_config.max_hash_collision;
- @@ -278,10 +267,10 @@ static long dcache_find(off64_t blk)
- }
-
- /* Physical read into cache */
- -static int dcache_io_read(int fd, long entry, off64_t offset, off64_t blk)
- +static int dcache_io_read(int fd, long entry, off_t offset, off_t blk)
- {
- - if (lseek64(fd, offset, SEEK_SET) < 0) {
- - MSG(0, "\n lseek64 fail.\n");
- + if (lseek(fd, offset, SEEK_SET) < 0) {
- + MSG(0, "\n lseek fail.\n");
- return -1;
- }
- if (read(fd, dcache_buf + entry * F2FS_BLKSIZE, F2FS_BLKSIZE) < 0) {
- @@ -308,12 +297,12 @@ static int dcache_io_read(int fd, long entry, off64_t offset, off64_t blk)
- * 1: cache not available (uninitialized)
- * -1: error
- */
- -static int dcache_update_rw(int fd, void *buf, off64_t offset,
- +static int dcache_update_rw(int fd, void *buf, off_t offset,
- size_t byte_count, bool is_write)
- {
- - off64_t blk;
- + off_t blk;
- int addr_in_blk;
- - off64_t start;
- + off_t start;
-
- if (!dcache_initialized)
- dcache_init(); /* auto initialize */
- @@ -377,13 +366,13 @@ static int dcache_update_rw(int fd, void *buf, off64_t offset,
- * return value: 1: cache not available
- * 0: success, -1: I/O error
- */
- -int dcache_update_cache(int fd, void *buf, off64_t offset, size_t count)
- +int dcache_update_cache(int fd, void *buf, off_t offset, size_t count)
- {
- return dcache_update_rw(fd, buf, offset, count, true);
- }
-
- /* handles read into cache + read into buffer */
- -int dcache_read(int fd, void *buf, off64_t offset, size_t count)
- +int dcache_read(int fd, void *buf, off_t offset, size_t count)
- {
- return dcache_update_rw(fd, buf, offset, count, false);
- }
- @@ -395,7 +384,7 @@ int dev_read_version(void *buf, __u64 offset, size_t len)
- {
- if (c.sparse_mode)
- return 0;
- - if (lseek64(c.kd, (off64_t)offset, SEEK_SET) < 0)
- + if (lseek(c.kd, (off_t)offset, SEEK_SET) < 0)
- return -1;
- if (read(c.kd, buf, len) < 0)
- return -1;
- @@ -534,10 +523,10 @@ int dev_read(void *buf, __u64 offset, size_t len)
-
- /* err = 1: cache not available, fall back to non-cache R/W */
- /* err = 0: success, err=-1: I/O error */
- - err = dcache_read(fd, buf, (off64_t)offset, len);
- + err = dcache_read(fd, buf, (off_t)offset, len);
- if (err <= 0)
- return err;
- - if (lseek64(fd, (off64_t)offset, SEEK_SET) < 0)
- + if (lseek(fd, (off_t)offset, SEEK_SET) < 0)
- return -1;
- if (read(fd, buf, len) < 0)
- return -1;
- @@ -580,9 +569,9 @@ int dev_write(void *buf, __u64 offset, size_t len)
- * dcache_update_cache() just update cache, won't do I/O.
- * Thus even no error, we need normal non-cache I/O for actual write
- */
- - if (dcache_update_cache(fd, buf, (off64_t)offset, len) < 0)
- + if (dcache_update_cache(fd, buf, (off_t)offset, len) < 0)
- return -1;
- - if (lseek64(fd, (off64_t)offset, SEEK_SET) < 0)
- + if (lseek(fd, (off_t)offset, SEEK_SET) < 0)
- return -1;
- if (write(fd, buf, len) < 0)
- return -1;
- @@ -596,7 +585,7 @@ int dev_write_block(void *buf, __u64 blk_addr)
-
- int dev_write_dump(void *buf, __u64 offset, size_t len)
- {
- - if (lseek64(c.dump_fd, (off64_t)offset, SEEK_SET) < 0)
- + if (lseek(c.dump_fd, (off_t)offset, SEEK_SET) < 0)
- return -1;
- if (write(c.dump_fd, buf, len) < 0)
- return -1;
- @@ -618,7 +607,7 @@ int dev_fill(void *buf, __u64 offset, size_t len)
- /* Only allow fill to zero */
- if (*((__u8*)buf))
- return -1;
- - if (lseek64(fd, (off64_t)offset, SEEK_SET) < 0)
- + if (lseek(fd, (off_t)offset, SEEK_SET) < 0)
- return -1;
- if (write(fd, buf, len) < 0)
- return -1;
- diff --git a/lib/libf2fs_zoned.c b/lib/libf2fs_zoned.c
- index a0dd8bd..2ab2497 100644
- --- a/lib/libf2fs_zoned.c
- +++ b/lib/libf2fs_zoned.c
- @@ -6,7 +6,6 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#define _LARGEFILE64_SOURCE
-
- #include <f2fs_fs.h>
- #include <stdio.h>
- diff --git a/mkfs/Makefile.am b/mkfs/Makefile.am
- index af5b1c7..bfffd88 100644
- --- a/mkfs/Makefile.am
- +++ b/mkfs/Makefile.am
- @@ -1,7 +1,7 @@
- ## Makefile.am
-
- AM_CPPFLAGS = ${libuuid_CFLAGS} ${libblkid_CFLAGS} -I$(top_srcdir)/include
- -AM_CFLAGS = -Wall -DWITH_BLKDISCARD
- +AM_CFLAGS = -Wall -DWITH_BLKDISCARD -D_FILE_OFFSET_BITS=64
- sbin_PROGRAMS = mkfs.f2fs
- noinst_HEADERS = f2fs_format_utils.h
- include_HEADERS = $(top_srcdir)/include/f2fs_fs.h
- @@ -10,7 +10,7 @@ mkfs_f2fs_LDADD = ${libuuid_LIBS} ${libblkid_LIBS} $(top_builddir)/lib/libf2fs.l
-
- lib_LTLIBRARIES = libf2fs_format.la
- libf2fs_format_la_SOURCES = f2fs_format_main.c f2fs_format.c f2fs_format_utils.c
- -libf2fs_format_la_CFLAGS = -DWITH_BLKDISCARD
- +libf2fs_format_la_CFLAGS = -DWITH_BLKDISCARD -D_FILE_OFFSET_BITS=64
- libf2fs_format_la_LDFLAGS = ${libblkid_LIBS} ${libuuid_LIBS} -L$(top_builddir)/lib -lf2fs \
- -version-info $(FMT_CURRENT):$(FMT_REVISION):$(FMT_AGE)
-
- diff --git a/mkfs/f2fs_format.c b/mkfs/f2fs_format.c
- index 019fb3a..d937eda 100644
- --- a/mkfs/f2fs_format.c
- +++ b/mkfs/f2fs_format.c
- @@ -6,8 +6,6 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#define _LARGEFILE64_SOURCE
- -
- #include <stdio.h>
- #include <stdlib.h>
- #include <fcntl.h>
- diff --git a/mkfs/f2fs_format_main.c b/mkfs/f2fs_format_main.c
- index d8c9cea..08c1b25 100644
- --- a/mkfs/f2fs_format_main.c
- +++ b/mkfs/f2fs_format_main.c
- @@ -6,8 +6,6 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#define _LARGEFILE64_SOURCE
- -
- #include <stdio.h>
- #include <stdlib.h>
- #include <fcntl.h>
- diff --git a/mkfs/f2fs_format_utils.c b/mkfs/f2fs_format_utils.c
- index 448fbaa..437f113 100644
- --- a/mkfs/f2fs_format_utils.c
- +++ b/mkfs/f2fs_format_utils.c
- @@ -6,20 +6,10 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#ifndef _LARGEFILE_SOURCE
- -#define _LARGEFILE_SOURCE
- -#endif
- -#ifndef _LARGEFILE64_SOURCE
- -#define _LARGEFILE64_SOURCE
- -#endif
- #ifndef _GNU_SOURCE
- #define _GNU_SOURCE
- #endif
-
- -#ifndef _FILE_OFFSET_BITS
- -#define _FILE_OFFSET_BITS 64
- -#endif
- -
- #include <f2fs_fs.h>
-
- #include <stdio.h>
- diff --git a/mkfs/f2fs_format_utils.h b/mkfs/f2fs_format_utils.h
- index 807e7c3..6a7f687 100644
- --- a/mkfs/f2fs_format_utils.h
- +++ b/mkfs/f2fs_format_utils.h
- @@ -6,8 +6,6 @@
- *
- * Dual licensed under the GPL or LGPL version 2 licenses.
- */
- -#define _LARGEFILE64_SOURCE
- -
- #include "f2fs_fs.h"
-
- extern struct f2fs_configuration c;
- diff --git a/tools/Makefile.am b/tools/Makefile.am
- index 6b03814..7dfffb1 100644
- --- a/tools/Makefile.am
- +++ b/tools/Makefile.am
- @@ -1,7 +1,7 @@
- ## Makefile.am
-
- AM_CPPFLAGS = ${libuuid_CFLAGS} -I$(top_srcdir)/include
- -AM_CFLAGS = -Wall
- +AM_CFLAGS = -Wall -D_FILE_OFFSET_BITS=64
- sbin_PROGRAMS =
- if !WINDOWS
- sbin_PROGRAMS += fibmap.f2fs parse.f2fs
- diff --git a/tools/f2fs_io/Makefile.am b/tools/f2fs_io/Makefile.am
- index bc4f9d0..f5227ce 100644
- --- a/tools/f2fs_io/Makefile.am
- +++ b/tools/f2fs_io/Makefile.am
- @@ -2,7 +2,7 @@
-
- if LINUX
- AM_CPPFLAGS = -I$(top_srcdir)/include
- -AM_CFLAGS = -Wall
- +AM_CFLAGS = -Wall -D_FILE_OFFSET_BITS=64
- sbin_PROGRAMS = f2fs_io
- f2fs_io_SOURCES = f2fs_io.c
- endif
- diff --git a/tools/f2fs_io/f2fs_io.c b/tools/f2fs_io/f2fs_io.c
- index 771fd16..1f6549b 100644
- --- a/tools/f2fs_io/f2fs_io.c
- +++ b/tools/f2fs_io/f2fs_io.c
- @@ -9,12 +9,6 @@
- #ifndef _GNU_SOURCE
- #define _GNU_SOURCE
- #endif
- -#ifndef _LARGEFILE_SOURCE
- -#define _LARGEFILE_SOURCE
- -#endif
- -#ifndef _LARGEFILE64_SOURCE
- -#define _LARGEFILE64_SOURCE
- -#endif
- #ifndef O_LARGEFILE
- #define O_LARGEFILE 0
- #endif
- diff --git a/tools/f2fs_io_parse.c b/tools/f2fs_io_parse.c
- index 47f1194..c3c1005 100644
- --- a/tools/f2fs_io_parse.c
- +++ b/tools/f2fs_io_parse.c
- @@ -8,7 +8,6 @@
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- */
- -#define _LARGEFILE64_SOURCE
- #include <stdio.h>
- #include <stdlib.h>
- #include <unistd.h>
- diff --git a/tools/f2fscrypt.c b/tools/f2fscrypt.c
- index 8640ffa..d5222fb 100644
- --- a/tools/f2fscrypt.c
- +++ b/tools/f2fscrypt.c
- @@ -7,15 +7,6 @@
- * Authors: Michael Halcrow <mhalcrow@google.com>,
- * Ildar Muslukhov <ildarm@google.com>
- */
- -
- -#ifndef _LARGEFILE_SOURCE
- -#define _LARGEFILE_SOURCE
- -#endif
- -
- -#ifndef _LARGEFILE64_SOURCE
- -#define _LARGEFILE64_SOURCE
- -#endif
- -
- #ifndef _GNU_SOURCE
- #define _GNU_SOURCE
- #endif
- diff --git a/tools/fibmap.c b/tools/fibmap.c
- index bda8238..3acc77b 100644
- --- a/tools/fibmap.c
- +++ b/tools/fibmap.c
- @@ -1,13 +1,6 @@
- #if !defined(__FreeBSD__) && !defined(__NetBSD__) && !defined(__OpenBSD__)
- #define _XOPEN_SOURCE 600
- #define _DARWIN_C_SOURCE
- -#define _FILE_OFFSET_BITS 64
- -#ifndef _LARGEFILE_SOURCE
- -#define _LARGEFILE_SOURCE
- -#endif
- -#ifndef _LARGEFILE64_SOURCE
- -#define _LARGEFILE64_SOURCE
- -#endif
- #ifndef _GNU_SOURCE
- #define _GNU_SOURCE
- #endif
- --
- 2.39.2
|