git: e098d24b5290 - main - nvmecontrol: Have to truncate on all 32-bit architectures
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 17 Apr 2024 03:33:43 UTC
The branch main has been updated by imp:
URL: https://cgit.FreeBSD.org/src/commit/?id=e098d24b5290a9f59734587ded783c19d4dc6e31
commit e098d24b5290a9f59734587ded783c19d4dc6e31
Author: Warner Losh <imp@FreeBSD.org>
AuthorDate: 2024-04-16 22:35:53 +0000
Commit: Warner Losh <imp@FreeBSD.org>
CommitDate: 2024-04-17 03:30:17 +0000
nvmecontrol: Have to truncate on all 32-bit architectures
armv7, powerpc, powerpcspe and i386 all lack 128-bit integer
types. Adjust the comment and #ifdef. I don't think we support nvme on
any of these other architectures at the moment, but it won't hurt to be
more precise.
Sponsored by: Netflix
Reviewed by: chuck
Differential Revision: https://reviews.freebsd.org/D44650
---
sbin/nvmecontrol/nvmecontrol.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sbin/nvmecontrol/nvmecontrol.h b/sbin/nvmecontrol/nvmecontrol.h
index a098da4c9c45..ed331327ae9e 100644
--- a/sbin/nvmecontrol/nvmecontrol.h
+++ b/sbin/nvmecontrol/nvmecontrol.h
@@ -92,12 +92,12 @@ void print_intel_add_smart(const struct nvme_controller_data *cdata __unused, vo
/* Utility Routines */
/*
* 128-bit integer augments to standard values. On i386 this
- * doesn't exist, so we use 64-bit values. So, on 32-bit i386,
+ * doesn't exist, so we use 64-bit values. So, on 32-bit systems,
* you'll get truncated values until someone implement 128bit
* ints in software.
*/
#define UINT128_DIG 39
-#ifdef __i386__
+#ifdef __ILP32__
typedef uint64_t uint128_t;
#else
typedef __uint128_t uint128_t;