public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Eliav Farber <farbere@amazon.com>
To: <gregkh@linuxfoundation.org>, <sashal@kernel.org>,
	<mario.limonciello@amd.com>, <lijo.lazar@amd.com>,
	<David.Laight@ACULAB.COM>, <arnd@kernel.org>,
	<linux-kernel@vger.kernel.org>, <stable@vger.kernel.org>,
	<farbere@amazon.com>
Subject: [PATCH v2 00/12 6.6.y] Backport minmax.h updates from v6.17-rc7
Date: Mon, 29 Sep 2025 17:17:21 +0000	[thread overview]
Message-ID: <20250929171733.20671-1-farbere@amazon.com> (raw)

This series backports 15 patches to update minmax.h in the 6.6.y branch,
aligning it with v6.17-rc7.

The ultimate goal is to synchronize all longterm branches so that they
include the full set of minmax.h changes.

The key motivation is to bring in commit d03eba99f5bf ("minmax: allow
min()/max()/clamp() if the arguments have the same signedness"), which
is missing in older kernels.

In mainline, this change enables min()/max()/clamp() to accept mixed
argument types, provided both have the same signedness. Without it,
backported patches that use these forms may trigger compiler warnings,
which escalate to build failures when -Werror is enabled.

Changes between v1 and v2:
- v1 included 15 patches:
  https://lore.kernel.org/stable/20250922103241.16213-1-farbere@amazon.com/T/#t
- First 3 were pushed to the stable-tree.
- 4th cauded amd driver's build to fail.
- This change fixes it.
- Modified files:
   drivers/gpu/drm/amd/amdgpu/amdgpu.h
   drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
   drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c
   drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c

David Laight (7):
  minmax.h: add whitespace around operators and after commas
  minmax.h: update some comments
  minmax.h: reduce the #define expansion of min(), max() and clamp()
  minmax.h: use BUILD_BUG_ON_MSG() for the lo < hi test in clamp()
  minmax.h: move all the clamp() definitions after the min/max() ones
  minmax.h: simplify the variants of clamp()
  minmax.h: remove some #defines that are only expanded once

Linus Torvalds (5):
  minmax: make generic MIN() and MAX() macros available everywhere
  minmax: simplify min()/max()/clamp() implementation
  minmax: don't use max() in situations that want a C constant
    expression
  minmax: improve macro expansion and type checking
  minmax: fix up min3() and max3() too

 arch/um/drivers/mconsole_user.c               |   2 +
 drivers/edac/skx_common.h                     |   1 -
 drivers/gpu/drm/amd/amdgpu/amdgpu.h           |   2 +
 .../drm/amd/display/modules/hdcp/hdcp_ddc.c   |   2 +
 .../drm/amd/pm/powerplay/hwmgr/ppevvmath.h    |  14 +-
 .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c   |   2 +
 .../drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c  |   3 +
 .../drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c  |   3 +
 drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c        |   2 +-
 drivers/gpu/drm/radeon/evergreen_cs.c         |   2 +
 drivers/hwmon/adt7475.c                       |  24 +-
 drivers/input/touchscreen/cyttsp4_core.c      |   2 +-
 drivers/irqchip/irq-sun6i-r.c                 |   2 +-
 drivers/media/dvb-frontends/stv0367_priv.h    |   3 +
 .../net/can/usb/etas_es58x/es58x_devlink.c    |   2 +-
 drivers/net/fjes/fjes_main.c                  |   4 +-
 drivers/nfc/pn544/i2c.c                       |   2 -
 drivers/platform/x86/sony-laptop.c            |   1 -
 drivers/scsi/isci/init.c                      |   6 +-
 .../pci/hive_isp_css_include/math_support.h   |   5 -
 fs/btrfs/tree-checker.c                       |   2 +-
 include/linux/compiler.h                      |   9 +
 include/linux/minmax.h                        | 220 ++++++++++--------
 kernel/trace/preemptirq_delay_test.c          |   2 -
 lib/btree.c                                   |   1 -
 lib/decompress_unlzma.c                       |   2 +
 lib/vsprintf.c                                |   2 +-
 mm/zsmalloc.c                                 |   2 -
 tools/testing/selftests/mm/mremap_test.c      |   2 +
 tools/testing/selftests/seccomp/seccomp_bpf.c |   2 +
 30 files changed, 192 insertions(+), 136 deletions(-)

-- 
2.47.3


             reply	other threads:[~2025-09-29 17:17 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-29 17:17 Eliav Farber [this message]
2025-09-29 17:17 ` [PATCH v2 01/12 6.6.y] minmax: make generic MIN() and MAX() macros available everywhere Eliav Farber
2025-09-29 17:17 ` [PATCH v2 02/12 6.6.y] minmax: simplify min()/max()/clamp() implementation Eliav Farber
2025-09-29 17:17 ` [PATCH v2 03/12 6.6.y] minmax: don't use max() in situations that want a C constant expression Eliav Farber
2025-09-29 17:17 ` [PATCH v2 04/12 6.6.y] minmax: improve macro expansion and type checking Eliav Farber
2025-09-29 17:17 ` [PATCH v2 05/12 6.6.y] minmax: fix up min3() and max3() too Eliav Farber
2025-09-29 17:17 ` [PATCH v2 06/12 6.6.y] minmax.h: add whitespace around operators and after commas Eliav Farber
2025-09-29 17:17 ` [PATCH v2 07/12 6.6.y] minmax.h: update some comments Eliav Farber
2025-09-29 17:17 ` [PATCH v2 08/12 6.6.y] minmax.h: reduce the #define expansion of min(), max() and clamp() Eliav Farber
2025-09-29 17:17 ` [PATCH v2 09/12 6.6.y] minmax.h: use BUILD_BUG_ON_MSG() for the lo < hi test in clamp() Eliav Farber
2025-09-29 17:17 ` [PATCH v2 10/12 6.6.y] minmax.h: move all the clamp() definitions after the min/max() ones Eliav Farber
2025-09-29 17:17 ` [PATCH v2 11/12 6.6.y] minmax.h: simplify the variants of clamp() Eliav Farber
2025-09-29 17:17 ` [PATCH v2 12/12 6.6.y] minmax.h: remove some #defines that are only expanded once Eliav Farber
2025-09-30 14:04 ` [PATCH v2 00/12 6.6.y] Backport minmax.h updates from v6.17-rc7 Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250929171733.20671-1-farbere@amazon.com \
    --to=farbere@amazon.com \
    --cc=David.Laight@ACULAB.COM \
    --cc=arnd@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=lijo.lazar@amd.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    --cc=sashal@kernel.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox