]> www.infradead.org Git - nvme.git/commitdiff
minmax: make generic MIN() and MAX() macros available everywhere
authorLinus Torvalds <torvalds@linux-foundation.org>
Sun, 28 Jul 2024 22:49:18 +0000 (15:49 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sun, 28 Jul 2024 22:49:18 +0000 (15:49 -0700)
This just standardizes the use of MIN() and MAX() macros, with the very
traditional semantics.  The goal is to use these for C constant
expressions and for top-level / static initializers, and so be able to
simplify the min()/max() macros.

These macro names were used by various kernel code - they are very
traditional, after all - and all such users have been fixed up, with a
few different approaches:

 - trivial duplicated macro definitions have been removed

   Note that 'trivial' here means that it's obviously kernel code that
   already included all the major kernel headers, and thus gets the new
   generic MIN/MAX macros automatically.

 - non-trivial duplicated macro definitions are guarded with #ifndef

   This is the "yes, they define their own versions, but no, the include
   situation is not entirely obvious, and maybe they don't get the
   generic version automatically" case.

 - strange use case #1

   A couple of drivers decided that the way they want to describe their
   versioning is with

#define MAJ 1
#define MIN 2
#define DRV_VERSION __stringify(MAJ) "." __stringify(MIN)

   which adds zero value and I just did my Alexander the Great
   impersonation, and rewrote that pointless Gordian knot as

#define DRV_VERSION "1.2"

   instead.

 - strange use case #2

   A couple of drivers thought that it's a good idea to have a random
   'MIN' or 'MAX' define for a value or index into a table, rather than
   the traditional macro that takes arguments.

   These values were re-written as C enum's instead. The new
   function-line macros only expand when followed by an open
   parenthesis, and thus don't clash with enum use.

Happily, there weren't really all that many of these cases, and a lot of
users already had the pattern of using '#ifndef' guarding (or in one
case just using '#undef MIN') before defining their own private version
that does the same thing. I left such cases alone.

Cc: David Laight <David.Laight@aculab.com>
Cc: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
20 files changed:
arch/um/drivers/mconsole_user.c
drivers/edac/skx_common.h
drivers/gpu/drm/amd/display/dc/core/dc_stream.c
drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c
drivers/gpu/drm/amd/pm/powerplay/hwmgr/ppevvmath.h
drivers/gpu/drm/radeon/evergreen_cs.c
drivers/hwmon/adt7475.c
drivers/media/dvb-frontends/stv0367_priv.h
drivers/net/fjes/fjes_main.c
drivers/nfc/pn544/i2c.c
drivers/platform/x86/sony-laptop.c
drivers/scsi/isci/init.c
drivers/staging/media/atomisp/pci/hive_isp_css_include/math_support.h
include/linux/minmax.h
kernel/trace/preemptirq_delay_test.c
lib/btree.c
lib/decompress_unlzma.c
mm/zsmalloc.c
tools/testing/selftests/mm/mremap_test.c
tools/testing/selftests/seccomp/seccomp_bpf.c

index e24298a734befd9023f534080bb23680b1f99c7d..a04cd13c6315a2af5f28ad83724d63004e322335 100644 (file)
@@ -71,7 +71,9 @@ static struct mconsole_command *mconsole_parse(struct mc_request *req)
        return NULL;
 }
 
+#ifndef MIN
 #define MIN(a,b) ((a)<(b) ? (a):(b))
+#endif
 
 #define STRINGX(x) #x
 #define STRING(x) STRINGX(x)
index 11faf1db4fa4820a1a2759b6a32d39281eba0b3d..473421ba7a18a56a330d00979186666caaf0a0c0 100644 (file)
@@ -45,7 +45,6 @@
 #define I10NM_NUM_CHANNELS     MAX(I10NM_NUM_DDR_CHANNELS, I10NM_NUM_HBM_CHANNELS)
 #define I10NM_NUM_DIMMS                MAX(I10NM_NUM_DDR_DIMMS, I10NM_NUM_HBM_DIMMS)
 
-#define MAX(a, b)      ((a) > (b) ? (a) : (b))
 #define NUM_IMC                MAX(SKX_NUM_IMC, I10NM_NUM_IMC)
 #define NUM_CHANNELS   MAX(SKX_NUM_CHANNELS, I10NM_NUM_CHANNELS)
 #define NUM_DIMMS      MAX(SKX_NUM_DIMMS, I10NM_NUM_DIMMS)
index be2638c763d7897031d651433d8c66890ffe2454..9a406d74c0dd7617948782ab12ef8de3e003ab7a 100644 (file)
 #include "dc_stream_priv.h"
 
 #define DC_LOGGER dc->ctx->logger
+#ifndef MIN
 #define MIN(X, Y) ((X) < (Y) ? (X) : (Y))
 #define MAX(x, y) ((x > y) ? x : y)
+#endif
 
 /*******************************************************************************
  * Private functions
index 7ecf76aea950514e12b4afd8599b147245d31285..6e064e6ae949f3979e06c33ff8597c27a2550422 100644 (file)
@@ -25,7 +25,9 @@
 
 #include "hdcp.h"
 
+#ifndef MIN
 #define MIN(a, b) ((a) < (b) ? (a) : (b))
+#endif
 #define HDCP_I2C_ADDR 0x3a     /* 0x74 >> 1*/
 #define KSV_READ_SIZE 0xf      /* 0x6803b - 0x6802c */
 #define HDCP_MAX_AUX_TRANSACTION_SIZE 16
index 6f54c410c2f9859392e78e15b1334d13b8713218..409aeec6baa92193db436878b1d0552b00c126b6 100644 (file)
  */
 #include <asm/div64.h>
 
-#define SHIFT_AMOUNT 16 /* We multiply all original integers with 2^SHIFT_AMOUNT to get the fInt representation */
+enum ppevvmath_constants {
+       /* We multiply all original integers with 2^SHIFT_AMOUNT to get the fInt representation */
+       SHIFT_AMOUNT    = 16,
 
-#define PRECISION 5 /* Change this value to change the number of decimal places in the final output - 5 is a good default */
+       /* Change this value to change the number of decimal places in the final output - 5 is a good default */
+       PRECISION       =  5,
 
-#define SHIFTED_2 (2 << SHIFT_AMOUNT)
-#define MAX (1 << (SHIFT_AMOUNT - 1)) - 1 /* 32767 - Might change in the future */
+       SHIFTED_2       = (2 << SHIFT_AMOUNT),
+
+       /* 32767 - Might change in the future */
+       MAX             = (1 << (SHIFT_AMOUNT - 1)) - 1,
+};
 
 /* -------------------------------------------------------------------------------
  * NEW TYPE - fINT
index 1fe6e0d883c79be07817a3cf582f128c9d5d97dd..e5577d2a19ef497b9da601a42dc241f9a460dd6a 100644 (file)
 #include "evergreen_reg_safe.h"
 #include "cayman_reg_safe.h"
 
+#ifndef MIN
 #define MAX(a, b)                   (((a) > (b)) ? (a) : (b))
 #define MIN(a, b)                   (((a) < (b)) ? (a) : (b))
+#endif
 
 #define REG_SAFE_BM_SIZE ARRAY_SIZE(evergreen_reg_safe_bm)
 
index bc186c61a2c0a575118f1c97bf28f39881da5c31..382a2bb9168a50ddde0fe2ecdd72d2d635a4dfce 100644 (file)
 #include <linux/util_macros.h>
 
 /* Indexes for the sysfs hooks */
-
-#define INPUT          0
-#define MIN            1
-#define MAX            2
-#define CONTROL                3
-#define OFFSET         3
-#define AUTOMIN                4
-#define THERM          5
-#define HYSTERSIS      6
-
+enum adt_sysfs_id {
+       INPUT           = 0,
+       MIN             = 1,
+       MAX             = 2,
+       CONTROL         = 3,
+       OFFSET          = 3,    // Dup
+       AUTOMIN         = 4,
+       THERM           = 5,
+       HYSTERSIS       = 6,
 /*
  * These are unique identifiers for the sysfs functions - unlike the
  * numbers above, these are not also indexes into an array
  */
+       ALARM           = 9,
+       FAULT           = 10,
+};
 
-#define ALARM          9
-#define FAULT          10
 
 /* 7475 Common Registers */
 
index 617f605947b2c47e74d95eb3392d61d82ab9619b..7f056d1cce8228d97ce503905b3c673afdf153f8 100644 (file)
 #endif
 
 /* MACRO definitions */
+#ifndef MIN
 #define MAX(X, Y) ((X) >= (Y) ? (X) : (Y))
 #define MIN(X, Y) ((X) <= (Y) ? (X) : (Y))
+#endif
+
 #define INRANGE(X, Y, Z) \
        ((((X) <= (Y)) && ((Y) <= (Z))) || \
        (((Z) <= (Y)) && ((Y) <= (X))) ? 1 : 0)
index b3ddc9a629d9d95463977b7ae50e52683257c95a..fad5b65644645ded60601d4a15ef606ef19b3a13 100644 (file)
@@ -14,9 +14,7 @@
 #include "fjes.h"
 #include "fjes_trace.h"
 
-#define MAJ 1
-#define MIN 2
-#define DRV_VERSION __stringify(MAJ) "." __stringify(MIN)
+#define DRV_VERSION "1.2"
 #define DRV_NAME       "fjes"
 char fjes_driver_name[] = DRV_NAME;
 char fjes_driver_version[] = DRV_VERSION;
index 9fe664960b38032d74fb6544b998ab1c92e198fc..e2a6575b9ff7f8d55aaec1f683b93e703844a2dd 100644 (file)
@@ -126,8 +126,6 @@ struct pn544_i2c_fw_secure_blob {
 #define PN544_FW_CMD_RESULT_COMMAND_REJECTED 0xE0
 #define PN544_FW_CMD_RESULT_CHUNK_ERROR 0xE6
 
-#define MIN(X, Y) ((X) < (Y) ? (X) : (Y))
-
 #define PN544_FW_WRITE_BUFFER_MAX_LEN 0x9f7
 #define PN544_FW_I2C_MAX_PAYLOAD PN544_HCI_I2C_LLC_MAX_SIZE
 #define PN544_FW_I2C_WRITE_FRAME_HEADER_LEN 8
index 3e94fdd1ea52e035588847b6578b43209aa93bc5..3197aaa69da73dbde21be0c3da2d0bd8011cdcf3 100644 (file)
@@ -757,7 +757,6 @@ static union acpi_object *__call_snc_method(acpi_handle handle, char *method,
        return result;
 }
 
-#define MIN(a, b)      (a > b ? b : a)
 static int sony_nc_buffer_call(acpi_handle handle, char *name, u64 *value,
                void *buffer, size_t buflen)
 {
index d31884f82f2ad5daa4a0657059377824e609c8b4..73085d2f5c4344681aaa5adef25cb4ef82a211b3 100644 (file)
 #include "task.h"
 #include "probe_roms.h"
 
-#define MAJ 1
-#define MIN 2
-#define BUILD 0
-#define DRV_VERSION __stringify(MAJ) "." __stringify(MIN) "." \
-       __stringify(BUILD)
+#define DRV_VERSION "1.2.0"
 
 MODULE_VERSION(DRV_VERSION);
 
index 7349943bba2b469cae3a5a177711f5b307459c03..160c496784b7fc775f99fbdd52784493c3bfce9d 100644 (file)
 /* force a value to a lower even value */
 #define EVEN_FLOOR(x)        ((x) & ~1)
 
-/* for preprocessor and array sizing use MIN and MAX
-   otherwise use min and max */
-#define MAX(a, b)            (((a) > (b)) ? (a) : (b))
-#define MIN(a, b)            (((a) < (b)) ? (a) : (b))
-
 #define CEIL_DIV(a, b)       (((b) != 0) ? ((a) + (b) - 1) / (b) : 0)
 #define CEIL_MUL(a, b)       (CEIL_DIV(a, b) * (b))
 #define CEIL_MUL2(a, b)      (((a) + (b) - 1) & ~((b) - 1))
index 9c2848abc80496de84a23e2805163550ecd84599..fc384714da4509ea0605adf8da2c6df6adfdee94 100644 (file)
@@ -277,6 +277,8 @@ static inline bool in_range32(u32 val, u32 start, u32 len)
  * Use these carefully: no type checking, and uses the arguments
  * multiple times. Use for obvious constants only.
  */
+#define MIN(a,b) __cmp(min,a,b)
+#define MAX(a,b) __cmp(max,a,b)
 #define MIN_T(type,a,b) __cmp(min,(type)(a),(type)(b))
 #define MAX_T(type,a,b) __cmp(max,(type)(a),(type)(b))
 
index cb0871fbdb07f034465c51c59ef9ca06b84768de..314ffc143039c53da1c0612448901a8104cdd3d5 100644 (file)
@@ -34,8 +34,6 @@ MODULE_PARM_DESC(cpu_affinity, "Cpu num test is running on");
 
 static struct completion done;
 
-#define MIN(x, y) ((x) < (y) ? (x) : (y))
-
 static void busy_wait(ulong time)
 {
        u64 start, end;
index 49420cae3a8330ac650ff3114ee98cb2ed8e088a..bb81d3393ac5c52cd2a80e2712380e6dfe8b8e47 100644 (file)
@@ -43,7 +43,6 @@
 #include <linux/slab.h>
 #include <linux/module.h>
 
-#define MAX(a, b) ((a) > (b) ? (a) : (b))
 #define NODESIZE MAX(L1_CACHE_BYTES, 128)
 
 struct btree_geo {
index 20a858031f12b7600f906a90584d73aa21e2311e..9d34d35908daa89c74bf4e53864b49e4434e18c5 100644 (file)
@@ -37,7 +37,9 @@
 
 #include <linux/decompress/mm.h>
 
+#ifndef MIN
 #define        MIN(a, b) (((a) < (b)) ? (a) : (b))
+#endif
 
 static long long INIT read_int(unsigned char *ptr, int size)
 {
index 5d6581ab7c073d5c1cb933bcd8ef7991d4a77d5f..2d3163e4da96a565f7c2f8bcc6ac122ed64aa2f0 100644 (file)
 #define CLASS_BITS     8
 #define MAGIC_VAL_BITS 8
 
-#define MAX(a, b) ((a) >= (b) ? (a) : (b))
-
 #define ZS_MAX_PAGES_PER_ZSPAGE        (_AC(CONFIG_ZSMALLOC_CHAIN_SIZE, UL))
 
 /* ZS_MIN_ALLOC_SIZE must be multiple of ZS_ALIGN */
index 1b03bcfaefdfa2be6131220088ef4e9ce3fce6bb..5a3a9bcba6404853678a8840d9b9f7860b6179ee 100644 (file)
 #define VALIDATION_DEFAULT_THRESHOLD 4 /* 4MB */
 #define VALIDATION_NO_THRESHOLD 0      /* Verify the entire region */
 
+#ifndef MIN
 #define MIN(X, Y) ((X) < (Y) ? (X) : (Y))
 #define MAX(X, Y) ((X) > (Y) ? (X) : (Y))
+#endif
 #define SIZE_MB(m) ((size_t)m * (1024 * 1024))
 #define SIZE_KB(k) ((size_t)k * 1024)
 
index e3f97f90d8db0f6fb8864d9641da44f9138c8b6b..8c3a73461475bae52436e311d4f0b723c059b664 100644 (file)
@@ -60,7 +60,9 @@
 #define SKIP(s, ...)   XFAIL(s, ##__VA_ARGS__)
 #endif
 
+#ifndef MIN
 #define MIN(X, Y) ((X) < (Y) ? (X) : (Y))
+#endif
 
 #ifndef PR_SET_PTRACER
 # define PR_SET_PTRACER 0x59616d61