From nobody Fri Apr 24 20:36:32 2026 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4g2PrJ4wwyz6bD4d for ; Fri, 24 Apr 2026 20:36:32 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R13" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4g2PrJ4Bpyz3Tsw for ; Fri, 24 Apr 2026 20:36:32 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1777062992; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=2mxZgTwU291O3PucAstOGvwxYFkgcb9bfem8Lrg86F8=; b=viCZPr+ylgEF4r7fgmk9RKaADIHWOrF+62HUelrMyQtZ/uN2KGLwrohw+m9YKLiw0+AOo2 gkgTMJMr41Q+RKU4eeYjnenDhCdfR+ykAgpkcua5i6Rq0kqeTH8RkB91dkk8SWDgwDj548 Yl9VN8MJem2Aa8DBiubEUXSRhulQTOkPwjGub6nPk7dEerIJpzGVmX4V+r+PFlvc2/hU/A EmpkHqmhuE2blqup3++7RBZLsvyIch3e7sSXu3BsqBb1wPw3CC+R6S0/qUCuFITa0qB5Bk UIWA1fRpvNvtGQyEXNWEqXSOQwY0BAdVOA2trl5JBhLXWzELqSPS8jS/cO6HoA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1777062992; a=rsa-sha256; cv=none; b=N2EhrGqexOjwBEqkixLk+xxasd3NQsA2+HIo5hgno2zfepzfeOeqsTTqLBPisuFk6F4XMl PNo/sxwVgVqMPRronkwnbmS632Xv6tY6zLl7FZ4WCFog7fujGqNchDvcz9/Y6injx4d4A/ Je0vvkrvXXZPOm9ceM7MkDS9zJ6XAKVActkjqhQ6UdHDaAozQFhvFvbXYNvdLOZWbybEi6 MsMSkLe00v0k0x3MWkfNz7B8PafncDSkzgW8ph2i/ITi8rfQ1wCZBEPaOp5BOQZMeLagqz yQhPMmJiGyh47cw95OPBhmQma3f5iu53/XHbjfLowjndUx3g0039kejafexscQ== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1777062992; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=2mxZgTwU291O3PucAstOGvwxYFkgcb9bfem8Lrg86F8=; b=ihfg0ROTMt1dGlPFoSbEOrXONKPXKTHREBhLcFV+gObhZ7nM0MAOzIjYUdbTLGZpG+YxgU mLN4Sq+cI0HDCSqvt1fm04Mws31u2IdRT5lASjmItqa8Ep5CgkNryJIwdeaIwIfKuJYA1J n37dOdO/MJ3mufdzXH8lVmi9SQbgdm+2uLa98Zy5i8Rs9bQxJwsTNFMBbkMfmQXIMfFH2i JEZOy8UuKVkfkCHj6FRPcUlgFZV0mywo1NcgGe4siARw7YzvGmcHniupbPrCTkNaIPbc4N h90cKwZKSjJTeNzkNLA2csU5a+LSieJynnDth6uPLlyrog0kJmXfjh7fk5wx2Q== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4g2PrJ3msjz1HSV for ; Fri, 24 Apr 2026 20:36:32 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 36574 by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Fri, 24 Apr 2026 20:36:32 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Bjoern A. Zeeb Subject: git: 93d301d95ab2 - main - Remove -fms-extensions throughout the tree List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: bz X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 93d301d95ab230ea159b21b5412aac2ce5362ac5 Auto-Submitted: auto-generated Date: Fri, 24 Apr 2026 20:36:32 +0000 Message-Id: <69ebd450.36574.1b31c920@gitrepo.freebsd.org> The branch main has been updated by bz: URL: https://cgit.FreeBSD.org/src/commit/?id=93d301d95ab230ea159b21b5412aac2ce5362ac5 commit 93d301d95ab230ea159b21b5412aac2ce5362ac5 Author: Bjoern A. Zeeb AuthorDate: 2026-02-03 01:51:40 +0000 Commit: Bjoern A. Zeeb CommitDate: 2026-04-24 20:36:15 +0000 Remove -fms-extensions throughout the tree During a discussion about using -fms-extensions jhb pointed out that we have them enabled in the kernel for gcc by default (even multiple times in one part). I had missed all that and clang still failed on my use case (needing another option). The original cause for enabling them for our tree back then was that we needed to support C11 anonymous struct/unions. Our in-tree gcc 4.2.1, despite later patches, needed the -fms-extensions to support these even though this was not the expected use case for that option ( cc4a90c445aa0 enabled it globally for the kernel). clang at that time (or at least when it became default for 10.0) already was fine (with C11). Any later gcc (4.6.0 onwards) did not need that option anymore, even when compiled for -std=iso9899:1990 (which does not support anonymous structs/unions) unless one would add -pedantic (see gcc git 4bdd0a60b27a). This is also the reason why userland cddl sources now compile with the option removed despite CSTD=c99. The only driver which needed the option recently was ccp, but that was fixed in 8d3f41dbcb2a by jhb. So cleanup all uses cases of -fms-extensions for the moment as they are no longer needed given all compilers currently supported seem to be fine without them and gcc-4.2.1 was removed from the tree in stable/13 in 2020 (a9854bc3812b). Reported by: jhb (all this but possibly the world CDDL parts) Sponsored by: The FreeBSD Foundation MFC after: 3 days Reviewed by: emaste (earlier), imp, jhb, glebius Differential Revision: https://reviews.freebsd.org/D55072 --- cddl/lib/libzpool/Makefile | 1 - cddl/usr.bin/ztest/Makefile | 1 - cddl/usr.sbin/zdb/Makefile | 1 - sys/conf/kern.pre.mk | 3 +-- sys/conf/kmod.mk | 1 - sys/modules/iser/Makefile | 1 - sys/modules/pms/Makefile | 1 - sys/powerpc/conf/dpaa/config.dpaa | 2 +- 8 files changed, 2 insertions(+), 9 deletions(-) diff --git a/cddl/lib/libzpool/Makefile b/cddl/lib/libzpool/Makefile index 9361d352f7c0..0c680367956f 100644 --- a/cddl/lib/libzpool/Makefile +++ b/cddl/lib/libzpool/Makefile @@ -281,7 +281,6 @@ CFLAGS+= -DWANTS_MUTEX_OWNED CFLAGS+= -I${SRCTOP}/lib/libpthread/thread CFLAGS+= -I${SRCTOP}/lib/libpthread/sys CFLAGS+= -I${SRCTOP}/lib/libthr/arch/${MACHINE_CPUARCH}/include -CFLAGS.gcc+= -fms-extensions LIBADD= md pthread z spl icp nvpair avl umem diff --git a/cddl/usr.bin/ztest/Makefile b/cddl/usr.bin/ztest/Makefile index 83c7b0eee13a..89ee381f38ed 100644 --- a/cddl/usr.bin/ztest/Makefile +++ b/cddl/usr.bin/ztest/Makefile @@ -28,7 +28,6 @@ CSTD= c99 # Since there are many asserts in this program, it makes no sense to compile # it without debugging. CFLAGS+= -g -DDEBUG=1 -Wno-format -DZFS_DEBUG=1 -CFLAGS.gcc+= -fms-extensions HAS_TESTS= SUBDIR.${MK_TESTS}+= tests diff --git a/cddl/usr.sbin/zdb/Makefile b/cddl/usr.sbin/zdb/Makefile index 8bf07cb4c06a..239299aa1d67 100644 --- a/cddl/usr.sbin/zdb/Makefile +++ b/cddl/usr.sbin/zdb/Makefile @@ -24,7 +24,6 @@ CFLAGS+= \ LIBADD= nvpair umem zdb zfs_core zfs spl avl zutil zpool crypto pthread -CFLAGS.gcc+= -fms-extensions # Since there are many asserts in this program, it makes no sense to compile # it without debugging. CFLAGS+= -g -DDEBUG=1 -DZFS_DEBUG=1 diff --git a/sys/conf/kern.pre.mk b/sys/conf/kern.pre.mk index 3dc1efe3d5d2..a5b21cdbe843 100644 --- a/sys/conf/kern.pre.mk +++ b/sys/conf/kern.pre.mk @@ -78,10 +78,9 @@ CFLAGS= ${COPTFLAGS} ${DEBUG} CFLAGS+= ${INCLUDES} -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h CFLAGS_PARAM_INLINE_UNIT_GROWTH?=100 CFLAGS_PARAM_LARGE_FUNCTION_GROWTH?=1000 -CFLAGS.gcc+= -fms-extensions -finline-limit=${INLINE_LIMIT} +CFLAGS.gcc+= -finline-limit=${INLINE_LIMIT} CFLAGS.gcc+= --param inline-unit-growth=${CFLAGS_PARAM_INLINE_UNIT_GROWTH} CFLAGS.gcc+= --param large-function-growth=${CFLAGS_PARAM_LARGE_FUNCTION_GROWTH} -CFLAGS.gcc+= -fms-extensions .if defined(CFLAGS_ARCH_PARAMS) CFLAGS.gcc+=${CFLAGS_ARCH_PARAMS} .endif diff --git a/sys/conf/kmod.mk b/sys/conf/kmod.mk index aacd7a17ef99..c35116dfcaa8 100644 --- a/sys/conf/kmod.mk +++ b/sys/conf/kmod.mk @@ -139,7 +139,6 @@ CFLAGS+= -include ${.OBJDIR}/opt_global.h CFLAGS+= -I. -I${SYSDIR} -I${SYSDIR}/contrib/ck/include CFLAGS.gcc+= -finline-limit=${INLINE_LIMIT} -CFLAGS.gcc+= -fms-extensions CFLAGS.gcc+= --param inline-unit-growth=100 CFLAGS.gcc+= --param large-function-growth=1000 diff --git a/sys/modules/iser/Makefile b/sys/modules/iser/Makefile index ff08ae6f346a..2c00b3a4027b 100644 --- a/sys/modules/iser/Makefile +++ b/sys/modules/iser/Makefile @@ -17,7 +17,6 @@ CFLAGS+= -I${SYSDIR}/ofed/include/uapi CFLAGS+= ${LINUXKPI_INCLUDES} CFLAGS+= -DCONFIG_INFINIBAND_USER_MEM CFLAGS+= -DINET6 -DINET -CFLAGS+= -fms-extensions CFLAGS+=-DICL_KERNEL_PROXY diff --git a/sys/modules/pms/Makefile b/sys/modules/pms/Makefile index 8bd21fc4b502..81599e0a3ceb 100644 --- a/sys/modules/pms/Makefile +++ b/sys/modules/pms/Makefile @@ -17,7 +17,6 @@ ${SRCTOP}/sys/dev/pms/freebsd/driver/common CFLAGS+=-fno-builtin -CFLAGS+=-fms-extensions CFLAGS+=-Wredundant-decls CFLAGS+=-Wunused-variable diff --git a/sys/powerpc/conf/dpaa/config.dpaa b/sys/powerpc/conf/dpaa/config.dpaa index 6a923baed894..0b7c8fcf3b29 100644 --- a/sys/powerpc/conf/dpaa/config.dpaa +++ b/sys/powerpc/conf/dpaa/config.dpaa @@ -2,7 +2,7 @@ files "dpaa/files.dpaa" makeoptions DPAA_COMPILE_CMD="${LINUXKPI_C} ${DPAAWARNFLAGS} \ - -Wno-cast-qual -Wno-unused-function -Wno-init-self -fms-extensions \ + -Wno-cast-qual -Wno-unused-function -Wno-init-self \ -include $S/contrib/ncsw/build/dflags.h \ -Wno-error=missing-prototypes \ -I$S/contrib/ncsw/build/ \