git: e098d24b5290 - main - nvmecontrol: Have to truncate on all 32-bit architectures

From: Warner Losh <imp_at_FreeBSD.org>
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;