Re: [OS-BUILD PATCH 0/3] Add clang config framework

2023-09-28 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1582704141 Agreed, I tried to make sure nothing changed for users. This is just an internal thing. ___ kernel mailing list -- kernel

[OS-BUILD PATCH 3/3] Add clang config framework

2023-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add clang config framework Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework (which uses priority files) to setup clang to be easier to update their configs. This leads to strange cases where

Re: [OS-BUILD PATCH 0/3] Add clang config framework

2023-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1579247973 @tales-aparecida pushed fix. thanks! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

[OS-BUILD PATCH 0/3] Add clang config framework

2023-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654 Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework to setup clang to be easier to update their configs

[OS-BUILD PATCH 2/3] Apply partial snippet configs to all configs

2023-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus Apply partial snippet configs to all configs There is a macro all_arch_configs that tries to optimize what configs we apply the partial snippets to. While it speeds up things up a bit, it creates conflicts with CLANG options. The process_configs.sh script still processes

[OS-BUILD PATCH 1/3] Remove unpackaged kgcov config files

2023-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unpackaged kgcov config files The kgcov option is applied as a partial snippet in the spec file. The generated kernel-*-kgcov.config are not packaged and therefore unused. My motivation here is because they conflict with the CLANG options when I overlay the CLANG

[OS-BUILD PATCHv2 1/2] Add target_branch for gen_config_patches.sh

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add target_branch for gen_config_patches.sh To support new config updates in other brances besides os-build, we need to be more flexible with the staging branch. Currently new configs get added to configs/$date/. This won't work when new branches like os-build-rt-devel and os

[OS-BUILD PATCHv2 0/2] Add new os-build targets: rt-devel and automotive-devel

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2732 This is an attempt to automate the rt and automotive devel branches using the upstream linux-rt-devel tree as the base combined with os-build. The overall process isn't too complicated

[OS-BUILD PATCHv2 2/2] Add new os-build targets: rt-devel and automotive-devel

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add new os-build targets: rt-devel and automotive-devel This is an attempt to automate the rt and automotive devel branches using the upstream linux-rt-devel tree as the base combined with os-build. The overall process isn't too complicated but there are enough steps to make

[OS-BUILD PATCHv4 2/3] Apply partial snippet configs to all configs

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Apply partial snippet configs to all configs There is a macro all_arch_configs that tries to optimize what configs we apply the partial snippets to. While it speeds up things up a bit, it creates conflicts with CLANG options. The process_configs.sh script still processes

[OS-BUILD PATCHv4 3/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add clang config framework Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework (which uses priority files) to setup clang to be easier to update their configs. This leads to strange cases where

[OS-BUILD PATCHv4 0/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654 Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework to setup clang to be easier to update their configs

[OS-BUILD PATCHv4 1/3] Remove unpackaged kgcov config files

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unpackaged kgcov config files The kgcov option is applied as a partial snippet in the spec file. The generated kernel-*-kgcov.config are not packaged and therefore unused. My motivation here is because they conflict with the CLANG options when I overlay the CLANG

Re: [OS-BUILD PATCHv4 0/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1590158249 rebased to latest code to see if that fixes the strange x86 build issues ___ kernel mailing list -- kernel@lists.fedoraproject.org

[OS-BUILD PATCHv5 0/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654 Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework to setup clang to be easier to update their configs

[OS-BUILD PATCHv5 1/3] Remove unpackaged kgcov config files

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unpackaged kgcov config files The kgcov option is applied as a partial snippet in the spec file. The generated kernel-*-kgcov.config are not packaged and therefore unused. My motivation here is because they conflict with the CLANG options when I overlay the CLANG

[OS-BUILD PATCHv5 2/3] Apply partial snippet configs to all configs

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Apply partial snippet configs to all configs There is a macro all_arch_configs that tries to optimize what configs we apply the partial snippets to. While it speeds up things up a bit, it creates conflicts with CLANG options. The process_configs.sh script still processes

[OS-BUILD PATCHv5 3/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add clang config framework Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework (which uses priority files) to setup clang to be easier to update their configs. This leads to strange cases where

Re: [OS-BUILD PATCHv5 0/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1590162968 updated with Jan's suggestion about 'cp %{SOURCE3001} .' ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send

Re: [OS-BUILD PATCH 3/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1590160321 Yeah, I wasn't sure if folks had a script that relied on a kernel-local or not so I left it. But will will happily update to take your suggestion

Re: [OS-BUILD PATCHv5 0/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1590319086 @jstancek with your ack, can I assume this is resolved? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

Re: [OS-BUILD PATCHv5 0/3] Add clang config framework

2023-10-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1590321109 @ptalbert what does Merge Warning mean? I rebased this to top of tree for os- build. ___ kernel mailing list -- kernel

[OS-BUILD PATCH] Add target_branch for gen_config_patches.sh

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add target_branch for gen_config_patches.sh To support new config updates in other brances besides os-build, we need to be more flexible with the staging branch. Currently new configs get added to configs/$date/. This won't work when new branches like os-build-rt-devel and os

Re: [OS-BUILD PATCH 3/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1584560878 It appears I can only 'cp file1 file2' and not multiple files. I am going to skip this suggestion unless someone can point out what I am messing up

[OS-BUILD PATCHv3 0/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654 Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework to setup clang to be easier to update their configs

[OS-BUILD PATCHv3 3/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add clang config framework Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework (which uses priority files) to setup clang to be easier to update their configs. This leads to strange cases where

[OS-BUILD PATCHv3 1/3] Remove unpackaged kgcov config files

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unpackaged kgcov config files The kgcov option is applied as a partial snippet in the spec file. The generated kernel-*-kgcov.config are not packaged and therefore unused. My motivation here is because they conflict with the CLANG options when I overlay the CLANG

[OS-BUILD PATCHv3 2/3] Apply partial snippet configs to all configs

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Apply partial snippet configs to all configs There is a macro all_arch_configs that tries to optimize what configs we apply the partial snippets to. While it speeds up things up a bit, it creates conflicts with CLANG options. The process_configs.sh script still processes

[OS-BUILD PATCH 1/2] Add target_branch for gen_config_patches.sh

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add target_branch for gen_config_patches.sh To support new config updates in other brances besides os-build, we need to be more flexible with the staging branch. Currently new configs get added to configs/$date/. This won't work when new branches like os-build-rt-devel and os

[OS-BUILD PATCH 0/2] Add new os-build targets: rt-devel and automotive-devel

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2732 This is an attempt to automate the rt and automotive devel branches using the upstream linux-rt-devel tree as the base combined with os-build. The overall process isn't too complicated

[OS-BUILD PATCH 2/2] Add new os-build targets: rt-devel and automotive-devel

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add new os-build targets: rt-devel and automotive-devel This is an attempt to automate the rt and automotive devel branches using the upstream linux-rt-devel tree as the base combined with os-build. The overall process isn't too complicated but there are enough steps to make

Re: [OS-BUILD PATCHv3 0/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1584576862 Yeah, I am scratching my head on that too. I got this working as a starter. Open to ideas. ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH 0/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654#note_1584466006 Resolving the thread as it appears stan agreed with Jan's reply below. ___ kernel mailing list -- kernel@lists.fedoraproject.org

[OS-BUILD PATCHv2 0/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2654 Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework to setup clang to be easier to update their configs

[OS-BUILD PATCHv2 1/3] Remove unpackaged kgcov config files

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unpackaged kgcov config files The kgcov option is applied as a partial snippet in the spec file. The generated kernel-*-kgcov.config are not packaged and therefore unused. My motivation here is because they conflict with the CLANG options when I overlay the CLANG

[OS-BUILD PATCHv2 2/3] Apply partial snippet configs to all configs

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Apply partial snippet configs to all configs There is a macro all_arch_configs that tries to optimize what configs we apply the partial snippets to. While it speeds up things up a bit, it creates conflicts with CLANG options. The process_configs.sh script still processes

[OS-BUILD PATCHv2 3/3] Add clang config framework

2023-09-29 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add clang config framework Compiling for clang is getting trickier to do because the configs are different than gcc in some cases. Mimic the kgcov framework (which uses priority files) to setup clang to be easier to update their configs. This leads to strange cases where

Re: [OS-BUILD PATCH] Fix binutils breakage

2023-10-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2749#note_1607846588 @codonell-rh @vmalikrh - patching samples/bpf/Makefile (add quotes around TPROGS_LDFLAGS) allowed me to use TPROGS_LDFLAGS correctly. @vmalikrh - who to send this patch to upstream? I

Re: [OS-BUILD PATCH] Fix binutils breakage

2023-10-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2749#note_1609196658 I guess I spoke too soon, moving the TPROGS_xxx before make still causes rpm to fail. hmm... ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH] Fix binutils breakage

2023-10-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2749#note_1609190998 @hertonrk-rh @vmalikrh - moving TPROGS_xxx to before make resolved my missing includes issue. But I see what @vmalikrh means for that one target. The rpm still builds, so I think we

Re: [OS-BUILD PATCH 0/2] random: Add hook to override device reads and getrandom(2)

2023-10-19 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2757#note_1611091360 @herbert.xu2 thanks! Is there a reason why this isn't pushed upstream and needs to be RHEL-only? (We ask on all RHEL-only patches

Re: [OS-BUILD PATCH] Fix binutils breakage

2023-10-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2749#note_1612950237 @vmalikrh thanks for the fixes! I tried your suggestions and got this new error: ```plaintext + /usr/bin/make -s 'HOSTCFLAGS=-O2 -fexceptions -g -grecord-gcc-switches -pipe -Wall

[OS-BUILD PATCH] Fix binutils breakage

2023-10-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus Fix binutils breakage The binutils package in Rawhide has stricter checks about PIE/PIC code and will start reporting errors if it detects mixing and matching no-PIE with PIE binaries (especially around bpf binaries). Example errors look like: /usr/bin/ld: /tmp/ccL7dkfR.o

Re: [OS-BUILD PATCH] Fix binutils breakage

2023-10-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2749#note_1602503582 @vmalikrh - FYI as the approval rules won't automatically add you. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe

Re: [OS-BUILD PATCH] Fix binutils breakage

2023-10-24 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2749#note_1617007832 @vmalikrh - were you able to push patches upstream? we were looking to bring them in prematurely for ARK until upstream merges them. This helps move binutils forward

Re: [OS-BUILD PATCHv2] redhat/configs: disable CONFIG_ROCKCHIP_ERRATUM_3588001 for RHEL

2023-08-21 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2526#note_1521767186 @mlangsdorf @jlinton - review? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCH] redhat/configs: Enable CONFIG_DEVICE_PRIVATE on aarch64

2023-08-21 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2639#note_1521766040 @raquini @ddutile - mm review? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

[OS-BUILD PATCH] Fix tarball creation logic

2023-08-28 Thread Don Zickus (via Email Bridge)
From: Don Zickus Fix tarball creation logic The create tarball logic tried to determine if the tarball existed or not by using a git sha. However the check always failed because the current sha was shortened and the tarball sha was the full 40 digits. Correct this by converting the shortened

Re: [OS-BUILD PATCH 0/2] Makefile: only depend on dist-buildreq-check when building actual RPMs

2022-05-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1781#note_949736929 I used a macro to address this. This seems to work. Thoughts? https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786 ___ kernel

[OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus [redhat] Explicitly set srpm requirements It isn't clear what packages are necessary to be installed to generate the srpm for the kernel. Use another spec macro to call out the exact packages needed. This allows a makefile target to be used to verify the right packages have

[OS-BUILD PATCH 2/3] [redhat] Makefile: drop duplicate dist-sources deps

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Ondrej Mosnacek [redhat] Makefile: drop duplicate dist-sources deps do-rpmbuild itself already depends on dist-sources, thus no need to specify when we depend on do-rpmbuild already. Signed-off-by: Ondrej Mosnacek diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah

[OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786 Developers want to build a srpm locally and submit it to a build server like Koji. The current process checks the BuildRequires in the kernel.spec file to ensure all the right packages

[OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus [redhat] Add dist-buildreq-check for binary rpm builds By simplifying the package check to srpm only, we neglect to check the packaging for local binary rpm building. Add that Makefile dependency for those cases. Signed-off-by: Don Zickus diff --git a/redhat/Makefile b

Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951720936 @hertonrk-rh @prarit @jmflinuxtx To answer a variety of questions. Yes srpm processes the configs. It kinda has to otherwise what is rpmbuild going to use to build the kernel? :-) Why

Re: [OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951705198 I can't prove it mathematically, but in order to build the SRPM, you need to build the configs. What does dist-configs-check do that regular dist-configs does not? And if it is missing

Re: [OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951723323 Sure, but it doesn't describe why some rpmbuild commands get it, while others do not. Hence the -binary part. If you are building binaries you needed the extra suffix. This makes

Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951732268 Oh and @omos filed this https://gitlab.com/cki-project/kernel- ark/-/merge_requests/1781 because he got hit with my buildreqs change from last week. That MR kinda undoes the check

Re: [OS-BUILD PATCHv2 0/4] redhat/self-tests: Add self-tests to gitlab-ci.yaml

2022-05-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1741#note_954969180 I don't know who or what was originally blocking this thread, but I believe it to be resolved (at least Prarit and Veronika appear to think things are good). So I am clicking the resolve

Re: [OS-BUILD PATCHv2 0/4] redhat/self-tests: Add self-tests to gitlab-ci.yaml

2022-05-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1741#note_954971065 @jmflinuxtx - up to you if you want to sneak this in before Monday's merge window opening. The last commit in this series is the piece that enables the cki part (run_self_test: true

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: remove stray *.hardlink-temporary files, if any

2022-05-27 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1811#note_962560082 @ptalbert @jmflinuxtx @prarit - so this breaks the self-tests and if included will break self-tests for all future configs. Hopefully this doesn't get merged until that is resolved

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: remove stray *.hardlink-temporary files, if any

2022-05-27 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1811#note_962630004 It does. Though I have a patch to reduce it down to changes to the %%SPEC lines, which is more realistic. I have to wait for @prarit to come back from PTO to review it. But for this MR

[OS-BUILD PATCH] Fix nightly merge CI

2022-05-12 Thread Don Zickus (via Email Bridge)
From: Don Zickus Fix nightly merge CI My recent MR for builddeps broke the nightly merge CI scripts because the script was using the wrong supported container. Update the CI script to use the correct supported CKI container image for ARK. Signed-off-by: Don Zickus diff --git a/.gitlab

Re: [OS-BUILD PATCHv2 0/17] redhat/Makefile: General improvements and fixes

2022-05-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1757#note_948427545 It definitely depends on the repo. I think I had so more garbage that it inflated things. 'git gc' removed enough junk that my 'git log --oneline' is down to 12. Looking into this more

Re: [OS-BUILD PATCHv2 0/17] redhat/Makefile: General improvements and fixes

2022-05-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1757#note_948374003 maybe it is just me, but a 'git log --oneline' shows 13 chars already. git log --oneline --abbrev=8|grep -Ev "^[0-9a-f]{,10} " shows me 9 commits that are 11 chars l

Re: [OS-BUILD PATCH 0/2] Makefile: only depend on dist-buildreq-check when building actual RPMs

2022-05-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1781#note_948589022 @hertonrk-rh agreed. There are srpm deps: https://cki- project.gitlab.io/kernel-ark/#cloning-the-repository Now I will agree that dist-buildreq-check might have swung to far in one

Re: [OS-BUILD PATCH] Use new HOME path in ssh command

2022-06-23 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1887#note_1002826830 with containers!412 merged, this MR is unnecessary. Closing. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send

[OS-BUILD PATCHv3] Test: Broken code, do not merge

2022-06-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Test: Broken code, do not merge diff --git a/redhat/docs/index.rst b/redhat/docs/index.rst index blahblah..blahblah 100644 --- a/redhat/docs/index.rst +++ b/redhat/docs/index.rst @@ -117,7 +117,7 @@ Contributor Guide Maintainer Guide -.. toctree

[OS-BUILD PATCH] Use new HOME path in ssh command

2022-06-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Use new HOME path in ssh command CKI recently updated the HOME variable to point to /cki instead of /root. The .gitlab-ci.yml file applies all ssh tweaks to ~/.ssh/ which is /cki/.ssh. However, ssh still uses /root/.ssh. So all the tweaks are ignored and ssh connections

[OS-BUILD PATCH] Enable CKI on os-build MRs only

2022-06-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus Enable CKI on os-build MRs only Not all the old Fedora branches have the latest Makefile and specfile changes to support the self-test check. Disable the Fedora branches for now. Signed-off-by: Don Zickus diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index blahblah..blahblah

Re: [OS-BUILD PATCH 0/2] create-data.sh: Reduce specfile data output

2022-06-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1878#note_990962307 Thank you! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org

[OS-BUILD PATCHv2] Test: Broken code, do not merge

2022-06-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus Test: Broken code, do not merge diff --git a/redhat/docs/index.rst b/redhat/docs/index.rst index blahblah..blahblah 100644 --- a/redhat/docs/index.rst +++ b/redhat/docs/index.rst @@ -117,7 +117,7 @@ Contributor Guide Maintainer Guide -.. toctree

[OS-BUILD PATCH] Test: Broken code, do not merge

2022-06-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus Test: Broken code, do not merge diff --git a/redhat/docs/index.rst b/redhat/docs/index.rst index blahblah..blahblah 100644 --- a/redhat/docs/index.rst +++ b/redhat/docs/index.rst @@ -117,7 +117,7 @@ Contributor Guide Maintainer Guide -.. toctree

Re: [OS-BUILD PATCH] Add system-sb-certs for RHEL-9

2022-05-10 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1774#note_941572465 Hmm, I remembered centos-stream, but forgot about centos. I will push that update. Not sure if that change or just using '!{?fedora}' is easier

[OS-BUILD PATCHv2] Add system-sb-certs for RHEL-9

2022-05-10 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add system-sb-certs for RHEL-9 Copy the RHEL-9 signing requirement over to ARK. This change is wrapped with "{?rhel}{?centos} && !{?eln}" to allow Fedora and ELN to continue using their signing technology. The system-sb-certs is just a metapackage for ei

[OS-BUILD PATCH] Fix dist-buildcheck-reqs

2022-05-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus Fix dist-buildcheck-reqs The check for rpmspec buildrequires wasn't working because of a Makefile typo. Verified by deleting known buildreqs. Signed-off-by: Don Zickus diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah 100644 --- a/redhat/Makefile +++ b

[OS-BUILD PATCH] Add system-sb-certs for RHEL-9

2022-05-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add system-sb-certs for RHEL-9 Copy the RHEL-9 signing requirement over to ARK. This change is wrapped with "{?rhel} && !{?eln}" to allow Fedora and ELN to continue using their signing technology. The system-sb-certs is just a metapackage for either rhel-sb-

Re: [OS-BUILD PATCH] Add system-sb-certs for RHEL-9

2022-05-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1774#note_940375865 @hertonrk-rh @jmflinuxtx - is this a respectful way to wrap this buildreq? ___ kernel mailing list -- kernel@lists.fedoraproject.org

Re: [OS-BUILD PATCH 0/8] redhat: Cleanup scripts

2022-10-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2095#note_1139862125 @jwilsonrh - +1. directories should have a 'theme'. I am ok with 'scripts/gating'. or 'scripts/osci' as those scripts are specific to osci

Re: [OS-BUILD PATCH 0/3] [redhat] enable some key Tegra SoC features

2022-10-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2083#note_1138460406 @ahs3 - we don't see any code using this config. Am I missing something (other than the defconfig file)? ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH] redhat/configs: Disable CONFIG_FW_LOADER_COMPRESS

2022-10-24 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2112#note_1146743250 @hertonrk-rh - you had to enable this for rhel-9 it appears. can you provide some background? ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH 0/2] Reduce disk space requirements of the kernel build

2022-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1950#note_1114646949 @jwilsonrh - can you reply here? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

[OS-BUILD PATCH] Remove unused ci scripts

2022-10-11 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unused ci scripts The ARK CI process has changed over time and the following scripts are no longer used. Remove them to avoid confusion for future contributors. * ark-rebase-patches.sh used to rebase patches for ark-patches. ark-patches was replaced by the current

Re: [OS-BUILD PATCHv2] redhat/configs: Disable fbdev drivers and use simpledrm everywhere

2022-10-03 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1533#note_1122556452 @Lyude - do you have time for a quick review? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel

Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-08-15 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_1065198628 Closing in favor of https://gitlab.com/cki-project/kernel- ark/-/merge_requests/1900 ___ kernel mailing list -- kernel@lists.fedoraproject.org

Re: [OS-BUILD PATCH] redhat/Makefile: Clean linux tarballs

2022-08-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1975#note_1056132350 Hmm. long ago, creating the linux tarball took a couple of minutes. So if you were constantly running make dist-srpm and friends that couple of minutes got annoying (much like how dist

Re: [OS-BUILD PATCHv4 0/8] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067725269 Thanks @hertonrk-rh for this series! It looks to be an improvement in the overall workflow (which shows with the later commits removing workarounds). I don't have any objections

Re: [OS-BUILD PATCHv4 0/8] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067842595 Thanks! I think this series is good prep work for the real fix, which is to correctly handle the marker != last_marker case (merge update). This patchset will just say 'rebase', which

Re: [OS-BUILD PATCHv4 0/8] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067930412 Perfect example env! So a couple of things. I believe the 'rebase' word is incorrect and the 'Resolves' line should be removed?? (I thought it was with your sed magic?). Sadly, I

Re: [OS-BUILD PATCHv5 0/9] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067980884 I am happy if Justin is happy. Thanks @hertonrk-rh ! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

[OS-BUILD PATCH] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Revert "redhat: fix elf got hardening for vm tools" This reverts commit eb5aebd77015bc57616c5d05a7b6206c8ff871cb. A recent rework of the tools/vm area relies on a proper CFLAGS definition to include the proper header files. The kernel.spec file purposely overwrites

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1219424373 /block ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora

[OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Revert "redhat: fix elf got hardening for vm tools" This reverts commit eb5aebd77015bc57616c5d05a7b6206c8ff871cb. A recent rework of the tools/vm area relies on a proper CFLAGS definition to include the proper header files. The kernel.spec file purposely overwrites

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1219422925 @hertonrk-rh - the revert is Frantisek's change. Not sure if you are familiar with it? I was wondering if we could replace 'CFLAGS=' with 'EXTRA_CFLAGS=' and have it still help

Re: [OS-BUILD PATCHv2] redhat/configs: Enable CRYPTO_CURVE25519 in ark

2022-12-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2225#note_1216392018 @jmflinuxtx - hmm, did we agree to avoid touching common and only let a bot do that? ___ kernel mailing list -- kernel@lists.fedoraproject.org

Re: [OS-BUILD PATCHv3] makefile: Add -Werror support for RHEL

2023-01-06 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2240#note_1230246925 @prarit - hmm, ok I am torn here. I thank you for respecting the 'do not break ARK approach'. But then I wonder what is the value of this in ARK (besides the obvious don't lose changes

Re: [OS-BUILD PATCHv2] Check for kernel config git-push failures

2022-12-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2165#note_1203471910 ugh how dumb of me. I can create an MR that 'touch'es or 'rm -f'. Any preference @jmflinuxtx @ptalbert ? ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCHv2] Check for kernel config git-push failures

2022-12-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2165#note_1203488108 created !2183 with 'touch' fix ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCH 0/2] redhat: Create a git repo when the srpm code is installed

2022-12-05 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2174#note_119735 Agreed. What problem are we solving here? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCH] Remove recommendation to use 'common' for config changes.

2022-11-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2157#note_1177768175 @jmflinuxtx @scweaver @prarit - here is my attempt to downplay the 'common' directory for configs. ___ kernel mailing list -- kernel

[OS-BUILD PATCH] Remove recommendation to use 'common' for config changes.

2022-11-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove recommendation to use 'common' for config changes. Config changes should be focused on the 'ark' or 'fedora' directory. Update the docs to promote that. Signed-off-by: Don Zickus diff --git a/redhat/docs/submitting-contributions.rst b/redhat/docs/submitting

<    1   2   3   4   5   6   7   8   >