Re: [PATCH 00/27 5.10.y] Backport minmax.h updates from v6.17-rc6

Greg KH
2025-09-21T17:28:47Z

On Fri, Sep 19, 2025 at 10:17:00AM +0000, Eliav Farber wrote:
> This series includes a total of 27 patches, to align minmax.h of
> v5.15.y with v6.17-rc6.
>
> The set consists of 24 commits that directly update minmax.h:
> 1) 92d23c6e9415 ("overflow, tracing: Define the is_signed_type() macro
> once")

But this isn't in 5.15.y, so how is this syncing things up?

I'm all for this, but I got confused here, at the first commit :)

> 2) 5efcecd9a3b1 ("minmax: sanity check constant bounds when clamping")

> 3) 2122e2a4efc2 ("minmax: clamp more efficiently by avoiding extra
> comparison")
> 4) f9bff0e31881 ("minmax: add in_range() macro")
> 5) c952c748c7a9 ("minmax: Introduce {min,max}_array()")
> 6) 5e57418a2031 ("minmax: deduplicate __unconst_integer_typeof()")
> 7) f6e9d38f8eb0 ("minmax: fix header inclusions")
> 8) d03eba99f5bf ("minmax: allow min()/max()/clamp() if the arguments
> have the same signedness.")
> 9) f4b84b2ff851 ("minmax: fix indentation of __cmp_once() and
> __clamp_once()")
> 10) 4ead534fba42 ("minmax: allow comparisons of 'int' against 'unsigned
> char/short'")
> 11) 867046cc7027 ("minmax: relax check to allow comparison between
> unsigned arguments and signed constants")
> 12) 3a7e02c040b1 ("minmax: avoid overly complicated constant
> expressions in VM code")
> 14) 017fa3e89187 ("minmax: simplify and clarify min_t()/max_t()
> implementation")
> 15) 1a251f52cfdc ("minmax: make generic MIN() and MAX() macros
> available everywhere")
> 18) dc1c8034e31b ("minmax: simplify min()/max()/clamp()
> implementation")
> 19) 22f546873149 ("minmax: improve macro expansion and type
> checking")
> 20) 21b136cc63d2 ("minmax: fix up min3() and max3() too")
> 21) 71ee9b16251e ("minmax.h: add whitespace around operators and after
> commas")
> 22) 10666e992048 ("minmax.h: update some comments")
> 23) b280bb27a9f7 ("minmax.h: reduce the #define expansion of min(),
> max() and clamp()")
> 24) a5743f32baec ("minmax.h: use BUILD_BUG_ON_MSG() for the lo < hi
> test in clamp()")
> 25) c3939872ee4a ("minmax.h: move all the clamp() definitions after the
> min/max() ones")
> 26) 495bba17cdf9 ("minmax.h: simplify the variants of clamp()")
> 27) 2b97aaf74ed5 ("minmax.h: remove some #defines that are only
> expanded once")

Some of these are also only in newer kernels, which, as you know, is
generally a bad thing (i.e. I can't take patches only for older
kernels.)

I want these changes, as they are great, but can you perhaps provide
patch series for newer kernels first so that I can then take these?

thanks,

greg k-h