From: David Gibson <david@gibson.dropbear.id.au>
To: Stefano Brivio <sbrivio@redhat.com>, passt-dev@passt.top
Cc: David Gibson <david@gibson.dropbear.id.au>
Subject: [PATCH] iov: Improve documentation of iov_skip_bytes()
Date: Fri, 1 Mar 2024 15:06:44 +1100 [thread overview]
Message-ID: <20240301040644.3680145-1-david@gibson.dropbear.id.au> (raw)
As pointed out in review, the documentation comments for iov_skip_bytes()
are more confusing than they should be. Reword them, including updating
parameter names, to make it clearer.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
iov.c | 28 ++++++++++++++--------------
iov.h | 2 +-
2 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/iov.c b/iov.c
index e3312628..8e850180 100644
--- a/iov.c
+++ b/iov.c
@@ -26,31 +26,31 @@
#include "iov.h"
-/* iov_skip_bytes() - Skip the first n bytes into an IO vector
+/* iov_skip_bytes() - Skip leading bytes of an IO vector
* @iov: IO vector
* @n: Number of entries in @iov
- * @vec_offset: Total byte offset into the IO vector
- * @buf_offset: Offset into a single buffer of the IO vector
+ * @skip: Number of leading bytes of @iov to skip
+ * @offset: Offset of first unskipped byte in its @iov entry
*
- * Return: index I of individual struct iovec which contains the byte at
- * @vec_offset bytes into the vector (as though all its buffers were
- * contiguous). If @buf_offset is non-NULL, update it to the offset of
- * that byte within @iov[I] (guaranteed to be less than @iov[I].iov_len)
- * If the whole vector has <= @vec_offset bytes, return @n.
+ * Return: index I of individual struct iovec which contains the byte at @skip
+ * bytes into the vector (as though all its buffers were contiguous).
+ * If @offset is non-NULL, update it to the offset of that byte within
+ * @iov[I] (guaranteed to be less than @iov[I].iov_len) If the whole
+ * vector has <= @skip bytes, return @n.
*/
size_t iov_skip_bytes(const struct iovec *iov, size_t n,
- size_t vec_offset, size_t *buf_offset)
+ size_t skip, size_t *offset)
{
- size_t offset = vec_offset, i;
+ size_t off = skip, i;
for (i = 0; i < n; i++) {
- if (offset < iov[i].iov_len)
+ if (off < iov[i].iov_len)
break;
- offset -= iov[i].iov_len;
+ off -= iov[i].iov_len;
}
- if (buf_offset)
- *buf_offset = offset;
+ if (offset)
+ *offset = off;
return i;
}
diff --git a/iov.h b/iov.h
index e1becdea..6058af77 100644
--- a/iov.h
+++ b/iov.h
@@ -19,7 +19,7 @@
#include <string.h>
size_t iov_skip_bytes(const struct iovec *iov, size_t n,
- size_t vec_offset, size_t *buf_offset);
+ size_t skip, size_t *offset);
size_t iov_from_buf(const struct iovec *iov, size_t iov_cnt,
size_t offset, const void *buf, size_t bytes);
size_t iov_to_buf(const struct iovec *iov, size_t iov_cnt,
--
@@ -19,7 +19,7 @@
#include <string.h>
size_t iov_skip_bytes(const struct iovec *iov, size_t n,
- size_t vec_offset, size_t *buf_offset);
+ size_t skip, size_t *offset);
size_t iov_from_buf(const struct iovec *iov, size_t iov_cnt,
size_t offset, const void *buf, size_t bytes);
size_t iov_to_buf(const struct iovec *iov, size_t iov_cnt,
--
2.44.0
next reply other threads:[~2024-03-01 4:06 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-01 4:06 David Gibson [this message]
2024-03-07 23:27 ` [PATCH] iov: Improve documentation of iov_skip_bytes() Stefano Brivio
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240301040644.3680145-1-david@gibson.dropbear.id.au \
--to=david@gibson.dropbear.id.au \
--cc=passt-dev@passt.top \
--cc=sbrivio@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://passt.top/passt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).