diff options
| author | Leah Rowe <leah@libreboot.org> | 2026-03-14 21:30:20 +0000 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2026-03-14 21:30:20 +0000 |
| commit | d6f3aadff169657ddd8542c6c10a38dfacb1ace3 (patch) | |
| tree | b4aded5fb6721b67a6e5ce8d31fee0932cc9df7c /util/nvmutil/nvmutil.c | |
| parent | 74d26d446e9cf31dd80e893b4c47dc438334df1b (diff) | |
util/nvmutil: remove rw_file_once
we don't need it anymore.
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'util/nvmutil/nvmutil.c')
| -rw-r--r-- | util/nvmutil/nvmutil.c | 54 |
1 files changed, 2 insertions, 52 deletions
diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c index ba4ee1e3..376ab117 100644 --- a/util/nvmutil/nvmutil.c +++ b/util/nvmutil/nvmutil.c @@ -405,9 +405,6 @@ static ssize_t rw_gbe_file_exact(int fd, u8 *mem, size_t nrw, static ssize_t rw_file_exact(int fd, u8 *mem, size_t len, off_t off, int rw_type, int loop_eagain, int loop_eintr, size_t max_retries); -static ssize_t rw_file_once(int fd, u8 *mem, size_t len, - off_t off, int rw_type, int loop_eagain, - int loop_eintr, size_t max_retries); static ssize_t prw(int fd, void *mem, size_t nrw, off_t off, int rw_type, int loop_eagain, int loop_eintr); static int check_file(int fd, struct stat *st); @@ -1883,9 +1880,9 @@ rw_file_exact(int fd, u8 *mem, size_t nrw, if ((size_t)rc >= nrw) break; - if ((rv = rw_file_once(fd, + if ((rv = prw(fd, mem + rc, nrw - rc, off + rc, rw_type, - loop_eagain, loop_eintr, max_retries)) < 0) + loop_eagain, loop_eintr)) < 0) return -1; /* Prevent theoretical overflow */ @@ -1912,53 +1909,6 @@ err_rw_file_exact: } /* - * rw_file_once() - Read less than perfectly - * (and possibly die) - * - * Read/write, but don't insist on an - * absolute read; e.g. if 100 bytes are - * requested, this may return 80 <-- fine - * - * This function will never return zero. - * It will only return below (error), - * or above (success). On error, -1 is - * returned and errno is set accordingly. - * - * Zero-byte returns are not allowed. - */ -static ssize_t -rw_file_once(int fd, u8 *mem, size_t nrw, - off_t off, int rw_type, int loop_eagain, - int loop_eintr, size_t max_retries) -{ - ssize_t rv; - size_t retries_on_zero = 0; - - if (mem == NULL) - goto err_rw_file_once; - -read_again: - rv = prw(fd, mem, nrw, off, rw_type, - loop_eagain, loop_eintr); - - if (rv < 0) - return -1; - - if ((size_t)rv > nrw)/* don't overflow */ - goto err_rw_file_once; - - if (rv != 0) - return rv; - - if (retries_on_zero++ < max_retries) - goto read_again; - -err_rw_file_once: - errno = EIO; - return -1; -} - -/* * prw() - portable read-write * * This implements a portable analog of pwrite() |
