Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 11 May 2024 19:05:44 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 3d966ae7895b - main - nvmecontrol: Appease a meaningless signed error warning thing
Message-ID:  <202405111905.44BJ5iFR094517@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=3d966ae7895b40c7a4985f61ff4819c25c8618c9

commit 3d966ae7895b40c7a4985f61ff4819c25c8618c9
Author:     Warner Losh <imp@FreeBSD.org>
AuthorDate: 2024-05-11 19:01:10 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2024-05-11 19:03:29 +0000

    nvmecontrol: Appease a meaningless signed error warning thing
    
    Consistently use the weird ssize_t type for things I know are unsigned,
    but which none-the-less need to be compared to signed things to prevent
    a bogus warning. Sigh, these aren't bugs, can't possibly be bugs and
    a waste of time to fix.
    
    Sponsored by:           Netflix
---
 sbin/nvmecontrol/telemetry.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sbin/nvmecontrol/telemetry.c b/sbin/nvmecontrol/telemetry.c
index df83382d50ed..72c304229a80 100644
--- a/sbin/nvmecontrol/telemetry.c
+++ b/sbin/nvmecontrol/telemetry.c
@@ -96,9 +96,9 @@ telemetry_log(const struct cmd *f, int argc, char *argv[])
 	int				fd, fdout;
 	char				*path;
 	uint32_t			nsid;
-	uint64_t			size;
+	ssize_t				size;
 	uint64_t			off;
-	uint32_t			chunk;
+	ssize_t				chunk;
 	struct nvme_controller_data	cdata;
 	bool				can_telemetry;
 	struct nvme_telemetry_log_page  tlp, buf;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202405111905.44BJ5iFR094517>