Re: [OS-BUILD PATCH] Make RHJOBS container friendly
From: Michael Hofmann on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2375#note_1321311286 iiuc, nproc is coreutils -> gnulib -> https://github.com/coreutils/gnulib/blob/master/lib/nproc.c, which doesn't seem to know anything about these newfangled things like cgroups v1/v2 ___ 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
Re: [OS-BUILD PATCH] Make RHJOBS container friendly
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2375#note_1321278887 Hmm, as an end user it seems odd to have to know what cgroup version and other quirks are on the system to make correctly calculations. I would think that would be part of nproc's calculation which would hide things from the end user. ___ 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
Re: [OS-BUILD PATCH] Make RHJOBS container friendly
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2375#note_1321270325 Yeah, I've known about the cgroups problem but never really had time to address it. I think that this change at least makes things no less better or worse that what we currently have so it's safe to do. IMO, we should look into a better value of RHJOBS taking into account your code above. But that can be done in another MR. ___ 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
Re: [OS-BUILD PATCH] Make RHJOBS container friendly
From: Michael Hofmann on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2375#note_1321162650 just to be clear, that will still not take cgroup limits into account, so any Docker/Kubernetes deployment that just applies "Limits" will still see all cpus on the host machine. For comparison, this is what CKI needs to do to keep parallel jobs from exploding on 96 core k8s nodes: https://gitlab.com/cki- project/pipeline-definition/-/blob/main/pipeline/functions/general.yml#L113 ___ 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
[OS-BUILD PATCH] Make RHJOBS container friendly
From: Don Zickus Make RHJOBS container friendly RHJOBS uses 'nproc --all' which is not friendly to process like containers that use taskset. Instead just use 'nproc'. Folks can override this with rhpkg.mk file if they choose. Signed-off-by: Don Zickus diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah 100644 --- a/redhat/Makefile +++ b/redhat/Makefile @@ -90,7 +90,7 @@ ifndef RHJOBS echo "1"; \ else \ j=$$(echo "$${j}" | tr -d "\-j"); \ - [ -z "$${j}" ] && nproc --all || echo $${j}; \ + [ -z "$${j}" ] && nproc || echo $${j}; \ fi) endif -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2375 ___ 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
[OS-BUILD PATCH] Remove scmversion from kernel.spec.template
From: Don Zickus Remove scmversion from kernel.spec.template The spec.template file was touching scmversion. The use case of scmversion was removed from upstream in 6.3. Let's remove it from our source tree. Signed-off-by: Don Zickus diff --git a/redhat/kernel.spec.template b/redhat/kernel.spec.template index blahblah..blahblah 100644 --- a/redhat/kernel.spec.template +++ b/redhat/kernel.spec.template @@ -1541,9 +1541,6 @@ ApplyOptionalPatch linux-kernel-test.patch chmod +x scripts/checkpatch.pl mv COPYING COPYING-%{version}-%{release} -# This Prevents scripts/setlocalversion from mucking with our version numbers. -touch .scmversion - # Mangle /usr/bin/python shebangs to /usr/bin/python3 # Mangle all Python shebangs to be Python 3 explicitly # -p preserves timestamps -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2374 ___ 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
[OS-BUILD PATCH] arm64/configs: Put some arm64 configs in the right place
From: Mark Salter arm64/configs: Put some arm64 configs in the right place There are some arm64-specific configs in redhat/configs/rhel/generic which should be in an arm64-specfic place, so move them. Except for CONFIG_CRYPTO_SHA512_ARM64_CE which is also in the arm64 directory so delete the one in the generic dir. Signed-off-by: Mark Salter diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA512_ARM64_CE b/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA512_ARM64_CE deleted file mode 100644 index blahblah..blahblah 0 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA512_ARM64_CE +++ /dev/null @@ -1 +0,0 @@ -CONFIG_CRYPTO_SHA512_ARM64_CE=m diff --git a/redhat/configs/rhel/generic/CONFIG_ARM64_ERRATUM_2645198 b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_ARM64_ERRATUM_2645198 rename from redhat/configs/rhel/generic/CONFIG_ARM64_ERRATUM_2645198 rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_ARM64_ERRATUM_2645198 index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_ARM64_ERRATUM_2645198 +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_ARM64_ERRATUM_2645198 diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_CRCT10DIF_ARM64_CE b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_CRCT10DIF_ARM64_CE rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_CRCT10DIF_ARM64_CE rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_CRCT10DIF_ARM64_CE index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_CRCT10DIF_ARM64_CE +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_CRCT10DIF_ARM64_CE diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_POLYVAL_ARM64_CE b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_POLYVAL_ARM64_CE rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_POLYVAL_ARM64_CE rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_POLYVAL_ARM64_CE index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_POLYVAL_ARM64_CE +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_POLYVAL_ARM64_CE diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA256_ARM64 b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA256_ARM64 rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA256_ARM64 rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA256_ARM64 index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA256_ARM64 +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA256_ARM64 diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA3_ARM64 b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA3_ARM64 rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA3_ARM64 rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA3_ARM64 index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA3_ARM64 +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA3_ARM64 diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA512_ARM64 b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA512_ARM64 rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA512_ARM64 rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA512_ARM64 index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SHA512_ARM64 +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SHA512_ARM64 diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SM4_ARM64_CE_CCM b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SM4_ARM64_CE_CCM rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_SM4_ARM64_CE_CCM rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SM4_ARM64_CE_CCM index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SM4_ARM64_CE_CCM +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SM4_ARM64_CE_CCM diff --git a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SM4_ARM64_CE_GCM b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SM4_ARM64_CE_GCM rename from redhat/configs/rhel/generic/CONFIG_CRYPTO_SM4_ARM64_CE_GCM rename to redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SM4_ARM64_CE_GCM index blahblah..blahblah 100644 --- a/redhat/configs/rhel/generic/CONFIG_CRYPTO_SM4_ARM64_CE_GCM +++ b/redhat/configs/rhel/generic/arm/aarch64/CONFIG_CRYPTO_SM4_ARM64_CE_GCM -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2373 ___ 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