From: Justin M. Forbes <jfor...@fedoraproject.org>

Turn off forced debug builds

For many years, rawhide kernels have forced users to run debug builds on
git snapshots by not building a non debug kernel as an option.  While
this has served us well, in finding occasional bugs that are less likely
to surface otherwise, the performance of debug kernels has gotten
considerably worse over time. After evaluating the debug configs to see
if performance could be improved, it has become clear that we would have
to trade off too much to regain performance. We would be better served
by leaving more debug options enabled and offering a non debug kernel
for all users on all builds.

Signed-off-by: Justin M. Forbes <jfor...@fedoraproject.org>

diff --git a/redhat/kernel.spec.template b/redhat/kernel.spec.template
index blahblah..blahblah 100755
--- a/redhat/kernel.spec.template
+++ b/redhat/kernel.spec.template
@@ -122,7 +122,7 @@ Summary: The Linux kernel
 # Set debugbuildsenabled to 0 to not build a separate debug kernel, but
 #  to build the base kernel using the debug configuration. (Specifying
 #  the --with-release option overrides this setting.)
-%define debugbuildsenabled %%SPECDEBUG_BUILDS_ENABLED%%
+%define debugbuildsenabled 1
 %%SPECBUILDID%%
 %define specversion %%SPECVERSION%%
 %define patchversion %%SPECKVERSION%%.%%SPECKPATCHLEVEL%%
diff --git a/redhat/scripts/genspec/genspec.sh 
b/redhat/scripts/genspec/genspec.sh
index blahblah..blahblah 100755
--- a/redhat/scripts/genspec/genspec.sh
+++ b/redhat/scripts/genspec/genspec.sh
@@ -4,15 +4,6 @@
 UPSTREAM=$(git rev-parse -q --verify origin/"${UPSTREAM_BRANCH}" || \
           git rev-parse -q --verify "${UPSTREAM_BRANCH}")
 
-if [ "$SNAPSHOT" = 0 ]; then
-       # This is based off a tag on Linus's tree (e.g. v5.5 or v5.5-rc5).
-       # Two kernels are built, one with debug configuration and one without.
-       SPECDEBUG_BUILDS_ENABLED=1
-else
-       # All kernels are built with debug configurations.
-       SPECDEBUG_BUILDS_ENABLED=0
-fi
-
 if [ -n "$DISTLOCALVERSION" ]; then
        SPECBUILDID=$(printf "%%define buildid %s" "$DISTLOCALVERSION")
 else
@@ -59,7 +50,6 @@ test -f "$SOURCES/$SPECFILE" &&
        s/%%SPECBUILD%%/$SPECBUILD/
        s/%%SPECRELEASE%%/$SPECRELEASE/
        s/%%SPECRELEASED_KERNEL%%/$SPECRELEASED_KERNEL/
-       s/%%SPECDEBUG_BUILDS_ENABLED%%/$SPECDEBUG_BUILDS_ENABLED/
        s/%%SPECINCLUDE_FEDORA_FILES%%/$SPECINCLUDE_FEDORA_FILES/
        s/%%SPECINCLUDE_RHEL_FILES%%/$SPECINCLUDE_RHEL_FILES/
        s/%%SPECPATCHLIST_CHANGELOG%%/$SPECPATCHLIST_CHANGELOG/

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263
_______________________________________________
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to