public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
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


             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).