diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2024-02-28 12:52:03 +1100 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2024-02-29 06:25:17 +0100 |
commit | 8bdb0883b441441590589aac46f991c80e4f80f3 (patch) | |
tree | 202c4a946b3ea9d365a5056d1bf044b873dfad6e /util.c | |
parent | 24410b37a44e819877bdb4449a3147567a866ffa (diff) | |
download | passt-8bdb0883b441441590589aac46f991c80e4f80f3.tar passt-8bdb0883b441441590589aac46f991c80e4f80f3.tar.gz passt-8bdb0883b441441590589aac46f991c80e4f80f3.tar.bz2 passt-8bdb0883b441441590589aac46f991c80e4f80f3.tar.lz passt-8bdb0883b441441590589aac46f991c80e4f80f3.tar.xz passt-8bdb0883b441441590589aac46f991c80e4f80f3.tar.zst passt-8bdb0883b441441590589aac46f991c80e4f80f3.zip |
util: Add write_remainder() helper
We have several places where we want to write(2) a buffer or buffers and we
handle short write()s by retrying until everything is successfully written.
Add a helper for this in util.c.
This version has some differences from the typical write_all() function.
First, take an IO vector rather than a single buffer, because that will be
useful for some of our cases. Second, allow it to take an parameter to
skip the first n bytes of the given buffers. This will be useful for some
of the cases we want, and also falls out quite naturally from the
implementation.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
[sbrivio: Minor formatting fixes in write_remainder()]
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'util.c')
-rw-r--r-- | util.c | 35 |
1 files changed, 35 insertions, 0 deletions
@@ -19,6 +19,7 @@ #include <arpa/inet.h> #include <net/ethernet.h> #include <sys/epoll.h> +#include <sys/uio.h> #include <fcntl.h> #include <string.h> #include <time.h> @@ -26,6 +27,7 @@ #include <stdbool.h> #include "util.h" +#include "iov.h" #include "passt.h" #include "packet.h" #include "log.h" @@ -574,3 +576,36 @@ int do_clone(int (*fn)(void *), char *stack_area, size_t stack_size, int flags, return clone(fn, stack_area + stack_size / 2, flags, arg); #endif } + +/* write_remainder() - write the tail of an IO vector to an fd + * @fd: File descriptor + * @iov: IO vector + * @iovcnt: Number of entries in @iov + * @skip: Number of bytes of the vector to skip writing + * + * Return: 0 on success, -1 on error (with errno set) + * + * #syscalls write writev + */ +int write_remainder(int fd, const struct iovec *iov, int iovcnt, size_t skip) +{ + int i; + + while ((i = iov_skip_bytes(iov, iovcnt, skip, &skip)) < iovcnt) { + ssize_t rc; + + if (skip) { + rc = write(fd, (char *)iov[i].iov_base + skip, + iov[i].iov_len - skip); + } else { + rc = writev(fd, &iov[i], iovcnt - i); + } + + if (rc < 0) + return -1; + + skip += rc; + } + + return 0; +} |