]> www.infradead.org Git - users/hch/block.git/commit
minmax: add a few more MIN_T/MAX_T users
authorLinus Torvalds <torvalds@linux-foundation.org>
Sun, 28 Jul 2024 20:03:48 +0000 (13:03 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sun, 28 Jul 2024 20:41:14 +0000 (13:41 -0700)
commit4477b39c32fdc03363affef4b11d48391e6dc9ff
tree29fdc4e2d7b47dd23deb73f0660f358669d5c037
parent7e2d0ba732ad649c557b49b0a3ef69d54df256d0
minmax: add a few more MIN_T/MAX_T users

Commit 3a7e02c040b1 ("minmax: avoid overly complicated constant
expressions in VM code") added the simpler MIN_T/MAX_T macros in order
to avoid some excessive expansion from the rather complicated regular
min/max macros.

The complexity of those macros stems from two issues:

 (a) trying to use them in situations that require a C constant
     expression (in static initializers and for array sizes)

 (b) the type sanity checking

and MIN_T/MAX_T avoids both of these issues.

Now, in the whole (long) discussion about all this, it was pointed out
that the whole type sanity checking is entirely unnecessary for
min_t/max_t which get a fixed type that the comparison is done in.

But that still leaves min_t/max_t unnecessarily complicated due to
worries about the C constant expression case.

However, it turns out that there really aren't very many cases that use
min_t/max_t for this, and we can just force-convert those.

This does exactly that.

Which in turn will then allow for much simpler implementations of
min_t()/max_t().  All the usual "macros in all upper case will evaluate
the arguments multiple times" rules apply.

We should do all the same things for the regular min/max() vs MIN/MAX()
cases, but that has the added complexity of various drivers defining
their own local versions of MIN/MAX, so that needs another level of
fixes first.

Link: https://lore.kernel.org/all/b47fad1d0cf8449886ad148f8c013dae@AcuMS.aculab.com/
Cc: David Laight <David.Laight@aculab.com>
Cc: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/x86/mm/pgtable.c
drivers/edac/sb_edac.c
drivers/gpu/drm/drm_color_mgmt.c
drivers/md/dm-integrity.c
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
net/ipv4/proc.c
net/ipv6/proc.c