Re: [OS-BUILD PATCH 0/0] fedora: some minor fixups and cleanups from my backlog of review

2022-06-06 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1853#note_972579252 This isn't a RHEL change, the only RHEL movement at all is moving a file from ark to common so that Fedora inherits the RHEL config. The average commit message for fedora configs

Re: [OS-BUILD PATCH 0/0] [redhat] New configs in drivers/ata

2022-04-18 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1675#note_915606655 Commit 55b014159ee7af63770cd7f2b6fe926f6dd99335 upstream reverted this change, there is no need to push this now. ___ kernel mailing list

Re: [OS-BUILD PATCH] redhat/configs: enable Maxlinear GPY115/21x/24x driver

2022-05-10 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1738#note_941638994 `Why wouldn't it be needed for Fedora?` It is already enabled for Fedora, and I have no plan to turn it off. ___ kernel mailing list --

[OS-BUILD PATCHv2 0/4] efi: x86: Set the NX-compatibility flag in the PE header

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1771 Following Baskov Evgeniy's "Handle UEFI NX-restricted page tables" patches, it's safe to set this compatibility flag to let loaders know they don't need to make special

[OS-BUILD PATCHv2 1/4] efi: libstub: declare DXE services table

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Baskov Evgeniy efi: libstub: declare DXE services table UEFI DXE services are not yet used in kernel code but are required to manipulate page table memory protection flags. Add required declarations to use DXE services functions. Signed-off-by: Baskov Evgeniy Link:

[OS-BUILD PATCHv2 2/4] efi: libstub: ensure allocated memory to be executable

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Baskov Evgeniy efi: libstub: ensure allocated memory to be executable There are UEFI versions that restrict execution of memory regions, preventing the kernel from booting. Parts that needs to be executable are: * Area used for trampoline placement. * All memory regions that the kernel

[OS-BUILD PATCHv2 3/4] efi: x86: Set the NX-compatibility flag in the PE header

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Peter Jones efi: x86: Set the NX-compatibility flag in the PE header Following Baskov Evgeniy's "Handle UEFI NX-restricted page tables" patches, it's safe to set this compatibility flag to let loaders know they don't need to make special accommodations for kernel to load if pre-boot NX is

[OS-BUILD PATCHv2 4/4] Add CONFIG_EFI_DXE_MEM_ATTRIBUTES

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add CONFIG_EFI_DXE_MEM_ATTRIBUTES UEFI specification does not guarantee all memory to be accessible for both write and execute as the kernel expects it to be. Use DXE services to check and alter memory protection attributes during boot via EFISTUB to ensure that memory

[OS-BUILD PATCH 0/3] efi: x86: Set the NX-compatibility flag in the PE header

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1771 Following Baskov Evgeniy's "Handle UEFI NX-restricted page tables" patches, it's safe to set this compatibility flag to let loaders know they don't need to make special

[OS-BUILD PATCH 3/3] efi: x86: Set the NX-compatibility flag in the PE header

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Peter Jones efi: x86: Set the NX-compatibility flag in the PE header Following Baskov Evgeniy's "Handle UEFI NX-restricted page tables" patches, it's safe to set this compatibility flag to let loaders know they don't need to make special accommodations for kernel to load if pre-boot NX is

[OS-BUILD PATCH 1/3] efi: libstub: declare DXE services table

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Baskov Evgeniy efi: libstub: declare DXE services table UEFI DXE services are not yet used in kernel code but are required to manipulate page table memory protection flags. Add required declarations to use DXE services functions. Signed-off-by: Baskov Evgeniy Link:

[OS-BUILD PATCH 2/3] efi: libstub: ensure allocated memory to be executable

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Baskov Evgeniy efi: libstub: ensure allocated memory to be executable There are UEFI versions that restrict execution of memory regions, preventing the kernel from booting. Parts that needs to be executable are: * Area used for trampoline placement. * All memory regions that the kernel

Re: [OS-BUILD PATCH] enable DAMON configs

2022-05-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1766#note_940224770 That is correct, there is no redhat/configs/generic used in generating config files, these are just sitting in the tree. redhat/ark/configs/generic would be for RHEL only and

Re: [OS-BUILD PATCH] perf tools: Fix compile error for x86

2022-08-23 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1992#note_1074292303 This is upstream now. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

[OS-BUILD PATCH] Add cpumask_kunit to mod-internal.list

2022-08-29 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add cpumask_kunit to mod-internal.list Another new kunit module that came in with 6.0. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list b/redhat/mod-internal.list index blahblah..blahblah 100644 --- a/redhat/mod-internal.list +++

[OS-BUILD PATCH] perf tools: Fix compile error for x86

2022-08-22 Thread Justin M. Forbes (via Email Bridge)
From: Yang Jihong perf tools: Fix compile error for x86 Commit a0a12c3ed057 ("asm goto: eradicate CC_HAS_ASM_GOTO") eradicates CC_HAS_ASM_GOTO, perf on x86 call asm_volatile_goto when compiling __GEN_RMWcc. However, asm_volatile_goto is not declared, which causes compilation error: In file

Re: [OS-BUILD PATCHv5 0/0] redhat/configs: Enable INIT_STACK_ALL_ZERO for Fedora

2022-09-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1923#note_1116349115 There is a negligible performance impact, rhbz 2115997 has some testing results. ___ kernel mailing list --

Re: [OS-BUILD PATCHv2 0/8] redhat/configs: Remove x86_64 directories

2022-09-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2030#note_1116538902 I reset the approval rules, this is a no-op re-org. It does not change the most config files at all. The only change for RHEL is debug x86_64 kernels now set

Re: [OS-BUILD PATCH 0/0] Revert "redhat: properly handle binary files in patches"

2022-10-16 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2105#note_1137086820 Thanks for finding this, I was just trying to figure out why an issue was popping up in the 6.0 test week. ___ kernel mailing list --

Re: [OS-BUILD PATCH 0/0] Revert "redhat: properly handle binary files in patches"

2022-10-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2105#note_1137946990 The issue I observed is that after a "fedpkg prep" there is no error around git-apply, but the patch itself is not applied. This is actually quite visible in the fact that we have

Re: [OS-BUILD PATCH 0/0] Revert "redhat: properly handle binary files in patches"

2022-10-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2105#note_1137974600 Interestingly, if I switch into that directory, and try to git apply manually, it doesn't complain at all, but doesn't apply. git apply --verbose shows that it skips every patch.

[OS-BUILD PATCH] perf tools: Fix man page build wrt perf-arm-coresight.txt

2022-10-17 Thread Justin M. Forbes (via Email Bridge)
From: Adrian Hunter perf tools: Fix man page build wrt perf-arm-coresight.txt perf build assumes documentation files starting with "perf-" are man pages but perf-arm-coresight.txt is not a man page: asciidoc: ERROR: perf-arm-coresight.txt: line 2: malformed manpage title asciidoc: ERROR:

Re: [OS-BUILD PATCH] redhat/configs: move CONFIG_ACPI_VIDEO to common/generic

2022-10-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2111#note_1142106478 If you are going to do that, you should probably delete redhat/configs/common/generic/arm/CONFIG_ACPI_VIDEO as well. The current configs leave CONFIG_ACPI_VIDEO on for arm and x86.

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

2022-10-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2112#note_1142154016 Why? Not sure about RHEL, but Fedora and ELN are shipping linux-firmware with xz compressed firmware files. ___ kernel mailing list --

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

2022-10-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2112#note_1142157654 /block ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org

Re: [OS-BUILD PATCH] configs: set CONFIG_PTE_MARKER for ARK

2022-10-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2110#note_1142110376 Correct, this MR will do nothing. We already have: ``` cat redhat/configs/common/generic/CONFIG_PTE_MARKER_UFFD_WP CONFIG_PTE_MARKER_UFFD_WP=y ``` And: ``` config

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

2022-10-24 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1533#note_1146665556 The problem there, is you need =m for s390x and =y for s390x zfcpdump. I am not sure how much testing goes into each build with s390x vs the zfcpdump variant, but it seems for

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

2022-10-24 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2112#note_1146671572 https://bugzilla.redhat.com/show_bug.cgi?id=1939095 Was the bug forcing this to be turned on in RHEL, so it seems that this MR is wrong for both Fedora and ELN/RHEL. Merging it

Re: [OS-BUILD PATCH] perf tools: Fix man page build wrt perf-arm-coresight.txt

2022-10-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2108#note_1151085979 This is upstream now. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

Re: [OS-BUILD PATCHv2 0/6] redhat: python replacement for merge.pl

2022-10-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2117#note_1151622605 `BuildRequires: python3-devel` This is already present. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send

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

2022-10-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2083#note_1138719400 I think the real concern is that the actual driver (and Kconfig entry) for CONFIG_DWMAC_TEGRA is not upstream yet, it is not even in linux-next just yet. There was discussion on

Re: [OS-BUILD PATCH] redhat/configs: Restructure and consolidate AArch64 16K page size config options

2022-09-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2022#note_1105911452 Do you have a need to flip CONFIG_ARM64_VA_BITS_47 for some reason? We build with 48, and there is no dep on this config item with current kernels in this repository.

Re: [OS-BUILD PATCH] redhat: sort config options based on their name only

2022-09-14 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2019#note_1100691692 This still seems much more complex than simply adding a buildrequres on glibc- langpack-en and forcing LC_ALL=en_US.UTF-8. ___ kernel

Re: [OS-BUILD PATCH] configs: enable CONFIG_HP_ILO for aarch64

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2009#note_1098347268 The `redhat/configs/generic/aarch64/` does not exist in this tree or in RHEL 9, and putting a config entry there is ignored. Please move this to

Re: [OS-BUILD PATCH] configs: enable CONFIG_HP_ILO for aarch64

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2009#note_1098499621 That will work as well. Prarit and I were just discussing getting rid of the x86_64 subdirectory all together, and then using the clean up script to catch all of the duplicates.

Re: [OS-BUILD PATCH] redhat: set LC_ALL=C before sorting config content

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2011#note_1098582094 The consequence of this is it changed the sorting of generated config files, and now every option turned off is listed first in alphabetical order ( because it starts with '#' )

Re: [OS-BUILD PATCH] redhat: cleaner kunit tests

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2018#note_1098363402 This seems like unnecessary complexity to me. In the end, we are adding spec complexity and making packagers try to divine the correct place to put a new module, when it doesn't

Re: [OS-BUILD PATCH] redhat: set LC_ALL=C before sorting config content

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2011#note_1098637550 I am reverting this, we can perhaps find another solution that will work, but I am not willing to commit the config files that this generated to dist-git for a build today.

Re: [OS-BUILD PATCH] redhat: set LC_ALL=C before sorting config content

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2011#note_1098634600 For example, before the change this is all one chunk beginning at line 1138: ``` CONFIG_CLK_RV110X=y CONFIG_CLK_SP810=y # CONFIG_CLKSRC_STM32_LP is not set

Re: [OS-BUILD PATCH] redhat: set LC_ALL=C before sorting config content

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2011#note_1098747753 I do understand the problem that you are trying to solve, and I also understand that the generated config files do build the kernels in the same way. The problem is in how the

Re: [OS-BUILD PATCH] redhat: set LC_ALL=C before sorting config content

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2011#note_1098852942 If you remove the sort, it is again in a rather illogical order. ___ kernel mailing list -- kernel@lists.fedoraproject.org To

Re: [OS-BUILD PATCH] redhat: set LC_ALL=C before sorting config content

2022-09-13 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2011#note_1098884245 Why would it be a problem to set LC_ALL=en_US.utf8 instead of LC_ALL=C? That still sets a specific locale, and leaves the result in the order that people expect.

Re: [OS-BUILD PATCH] redhat: fix kunit compiling error

2022-09-12 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2016#note_1096312041 I am not sure why you are failing, or seeing what you are, the cpumask_kunit entry was added with MR https://gitlab.com/cki-project/kernel- ark/-/merge_requests/2000 the day that

Re: [OS-BUILD PATCHv2] redhat/configs: Move CONFIG_ARM_SMMU_QCOM_DEBUG to common

2022-09-29 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2035#note_1119734952 Yes, there are a lot of them, and it is an issue with how the scripts create an MR for a new config item. They just aren't smart enough to know if something is arch specific, so

[OS-BUILD PATCH] Add fortify_kunit to mod-internal.list

2022-10-05 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add fortify_kunit to mod-internal.list Another kunit test which has been added and needs to end up in mod-internal so that depmod succeeds. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list b/redhat/mod-internal.list index blahblah..blahblah 100644

[OS-BUILD PATCH] Add new drm kunit tests to mod-internal.list

2022-10-06 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add new drm kunit tests to mod-internal.list The drm subsystem added a number of new kunit tests in the 6.1 cycle. These need to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list

Re: [OS-BUILD PATCHv2] redhat/configs/common: Turn on CONFIG_IKCONFIG_PROC

2022-10-07 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2041#note_1128275734 I might argue when working a bug report, the uname tells you exactly which config it is, and you don't need their /boot to get it. It is in our rpm, if not, they wouldn't have a

[OS-BUILD PATCH] Change acpi_bus_get_acpi_device to acpi_get_acpi_dev

2022-10-06 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Change acpi_bus_get_acpi_device to acpi_get_acpi_dev Upstream commit 45e9aa1fdbb2e renamed acpi_bus_get_acpi_device to acpi_get_acpi_dev. As we are carrying an out of tree patch [1] which calls acpi_bus_get_acpi_device, we need to make the corresponding change ourselves

[OS-BUILD PATCH] Rename rename FORCE_MAX_ZONEORDER to ARCH_FORCE_MAX_ORDER in configs

2022-10-11 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Rename rename FORCE_MAX_ZONEORDER to ARCH_FORCE_MAX_ORDER in configs Upstream commit 0192445cb2f7e renamed FORCE_MAX_ZONEORDER to ARCH_FORCE_MAX_ORDER which made it appear as a new config item for us in pending. While I have fixed up pending, this MR just fixes up the

[OS-BUILD PATCH] Add kasan_test to mod-internal.list

2022-10-11 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add kasan_test to mod-internal.list Another new kunit test that fails depmod until it is added to mod-internal.list. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list b/redhat/mod-internal.list index blahblah..blahblah 100644 ---

[OS-BUILD PATCH] Add acpi video to the filter_modules.sh for rhel

2022-10-07 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add acpi video to the filter_modules.sh for rhel The ACPI_VIDEO module now has a dep on the symbol wmi_evaluate_method and as a result it needs to be added to the module filter or depmod will fail. Signed-off-by: Justin M. Forbes diff --git

[OS-BUILD PATCHv2] Add new drm kunit tests to mod-internal.list

2022-10-07 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add new drm kunit tests to mod-internal.list The drm subsystem added a number of new kunit tests in the 6.1 cycle. These need to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list

Re: [OS-BUILD PATCHv5 0/0] redhat/configs: Enable INIT_STACK_ALL_ZERO for Fedora

2022-08-03 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1923#note_1049409477 That's the rebase, we do it every .0 release. Sorry I have not commented on this yet, as I was on vacation last week, and we are now going into the merge window/test week madness.

[OS-BUILD PATCH] Change CRYPTO_BLAKE2S_X86 from m to y

2022-08-03 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Change CRYPTO_BLAKE2S_X86 from m to y Upstream commit 2d16803c562ec changed the Kconfig entry for CRYPTO_BLAKE2S_X86 from tristate to bool, meaning if we wish to keep it enabled, it must be moved to built-in. Signed-off-by: Justin M. Forbes diff --git

[OS-BUILD PATCH] Add python3-setuptools buildreq for perf

2022-08-10 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add python3-setuptools buildreq for perf With the 6.0 updates for perf, the build fails with a missing ModuleNotFoundError: No module named 'setuptools' if we don't have python3-setuptools as a buildreq. Signed-off-by: Justin M. Forbes diff --git

Re: [OS-BUILD PATCHv3] Enable KUNIT tests for redhat kernel-modules-internal

2022-08-04 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1422#note_1051481242 While this started out as a good idea, it may be coming back to bite us. So far during the 6.0 merge window, 2 new commits have come in which cause major issues with this setup:

[OS-BUILD PATCH] Add new tests to mod-internal.list

2022-08-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add new tests to mod-internal.list New test modules added during the 6.0 merge window need to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list b/redhat/mod-internal.list index blahblah..blahblah

[OS-BUILD PATCHv2 0/2] Add new tests to mod-internal.list

2022-08-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1976 New test modules added during the 6.0 merge window need to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes --- redhat/mod-internal.list | 4

[OS-BUILD PATCHv2 1/2] Add new tests to mod-internal.list

2022-08-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add new tests to mod-internal.list New test modules added during the 6.0 merge window need to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list b/redhat/mod-internal.list index blahblah..blahblah

[OS-BUILD PATCHv2 2/2] Add cros_kunit to mod-internal.list

2022-08-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add cros_kunit to mod-internal.list This one is Fedora only at the moment, and aarch64 Signed-off-by: Justin M. Forbes diff --git a/redhat/mod-internal.list b/redhat/mod-internal.list index blahblah..blahblah 100644 --- a/redhat/mod-internal.list +++

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

2022-08-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067939935 I actually prefer something along the lines of: `- Linux v6.0.0-0.rc1.3cc40a443a04` It technically isn't a rebase, it is a merge, but for as long as I can recall, prior to ark,

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

2022-08-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067979814 That's perfect. The change is in the upstream versioning and now it is reflected clearly. I am happy with that. ___ kernel mailing list

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

2022-12-20 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2225#note_1216465167 That is the general guidance so that we do not end up with uncommon items in common, though this change is correct use of common, and where it would end up anyway.

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

2023-01-06 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2240#note_1230424301 We are already building with `-Werror=format-security` Though we typically do not run across many errors until a new toolchain is introduced. That is where this gets problematic to

Re: [OS-BUILD PATCH] Revert "arm64: remove special treatment for the link order of head.o"

2023-01-10 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2232#note_1234616148 Taking include in release off of this one as it is supposed to be fixed by commit 99cb0d917ffa1ab628bb67364ca9b162c07699b1 ___ kernel

Re: [OS-BUILD PATCH 0/3] redhat: Add sub-RPM with a EFI unified kernel image for virtual machines

2023-01-03 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2175#note_1226168874 This shouldn't come out of draft until it has been approved from FESCo. I still do not approve of renaming kernel-modules to kernel-modules-standard for no actual gain or reason

Re: [OS-BUILD PATCH 0/3] redhat: Add sub-RPM with a EFI unified kernel image for virtual machines

2023-01-03 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2175#note_1226265381 https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org/t hread/HDANRFTC3JIHMSSJYL5Z5H4RM2ULZYZY/#E42HQLKEODQBZZP5LDNVBCDH5N32XHX5 That was the reason, and it

Re: [OS-BUILD PATCHv4 0/0] split sub-rpm kernel-modules-core from kernel-core

2023-01-11 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2179#note_1236077821 Happy to ack this once the unrelated commits are dropped. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send

Re: [OS-BUILD PATCH] Revert "arm64: remove special treatment for the link order of head.o"

2023-01-11 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2232#note_1236075645 Closing this MR as the upstream commit does indeed fix the issue. ___ kernel mailing list -- kernel@lists.fedoraproject.org To

Re: [OS-BUILD PATCHv2] Clean up the config for the Tegra186 timer

2022-12-02 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2166#note_1194864927 Fedora drops 32 bit arm when F36 goes EOL in roughly 7 months. At that point, I expect we will do the same consolidation done for x86, and the aarch64 directory will go away,

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

2022-12-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2165#note_1203309926 So it seems this will now fail if we have no warnings. ``` No new configuration values exposed from merging master into os-build 232Pushing branch os-build to

Re: [OS-BUILD PATCH] redhat: Remove cpupower files

2022-12-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2125#note_1203675945 ``` error: File not found: /builddir/build/BUILDROOT/kernel-6.1.0-0.rc8.20221209gi t0d1409e4ff08.62.eln124.aarch64/usr/lib/systemd/system/cpupower.service error: File not found:

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

2022-12-05 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2174#note_1196429775 Curious as to why. We did have this in Fedora for years before we had kernel- ark. But we also had every single patch broken out, and applied each with git am. There was some

[OS-BUILD PATCH] Add drm_kunit_helpers to mod-internal.list

2022-12-14 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add drm_kunit_helpers to mod-internal.list Another kunit mod that came in during the 6.2 merge window, and needs to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes diff --git a/redhat/scripts/mod/mod-internal.list

Re: [OS-BUILD PATCHv3 0/3] redhat/kernel.spec.template: Add global compression variables

2022-12-16 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2173#note_1212551213 But is it in Fedora 36 - Rawhide built binaries? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

[OS-BUILD PATCH] Revert "arm64: remove special treatment for the link order of head.o"

2022-12-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Revert "arm64: remove special treatment for the link order of head.o" This reverts commit 994b7ac1697b4581b7726d2ac64321e3c840229b. This commit caused a missing build-id for vmlinux on aarch64. I expect it will be a simple revert upstream as this commit was not trying

[OS-BUILD PATCH] Add siphash_kunit and strscpy_kunit to mod-internal.list

2022-12-15 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Add siphash_kunit and strscpy_kunit to mod-internal.list These are 2 more kunit tests that came in during the 6.2 merge window and need to be added to mod-internal.list for depmod to succeed. Signed-off-by: Justin M. Forbes diff --git

Re: [OS-BUILD PATCHv2] Clean up the config for the Tegra186 timer

2022-12-03 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2166#note_1195335799 I was saying it is fine as 32bit will be going away. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an

Re: [OS-BUILD PATCHv2] redhat/configs: consolidate CONFIG_TEST_LIVEPATCH=m

2022-11-22 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2161#note_1181584664 Yes, this makes sense. Fedora was using the common config before, but since it is moved, we needed a new one. ___ kernel mailing list --

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

2022-11-18 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2157#note_1177932177 Not so much, the bigger concern was your script in CS9 claiming mismatch with ark and people dropping configs into common without respect to RHEL or Fedora. As I understand it,

[OS-BUILD PATCH] Adjust path to compressed vmlinux kernel image for s390x

2022-11-29 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Adjust path to compressed vmlinux kernel image for s390x commit edd4a8667355 moved the location of the compressed vmlinux file from arch/s390/boot/compressed/vmlinux to arch/s390/boot/vmlinux and packaging needs to be updated accordingly. Bugzilla:

Re: [OS-BUILD PATCH] redhat/configs: enable ChromeOS ACPI driver

2022-11-16 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2153#note_1175190946 I am not so worried about the common bit, we have a script that consolidates matching ark and fedora entries into common which is run periodically.

[OS-BUILD PATCH] Fix up configs with SND_SOC_NAU8315 mismatch

2023-01-16 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Fix up configs with SND_SOC_NAU8315 mismatch Upstream commit ba7523bb0f494fc440d3a9bb0b665cfcaa192d0c now selects SND_SOC_NAU8315 if SND_SOC_INTEL_SOF_NAU8825_MACH is set. This forces us to either turn on SND_SOC_NAU8315 or turn off SND_SOC_INTEL_SOF_NAU8825_MACH. I

Re: [OS-BUILD PATCHv8 0/4] makefile: Add -Werror support for RHEL

2023-01-19 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2240#note_1245811704 I will approve it, though honestly Makefile.variables was currently arranged from most likely to be changed, meaning the further you go down the file, the less likely it is that

[OS-BUILD PATCH] Turn off forced debug builds

2023-01-23 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 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

[OS-BUILD PATCHv2 0/2] Turn off forced debug builds

2023-01-24 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263 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

[OS-BUILD PATCHv2 1/2] Turn off forced debug builds

2023-01-24 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 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

[OS-BUILD PATCHv2 2/2] Turn on debug builds for aarch64 Fedora

2023-01-24 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Turn on debug builds for aarch64 Fedora Fedora only did debug builds for x86 in the past. This was largely due to slow build times on other arches in koji, coupled with much smaller userbases. Let's turn on aarch64 now as those builders are much faster and the userbase

Re: [OS-BUILD PATCH 0/0] fedora: enable STACKPROTECTOR on arm platforms

2023-01-25 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2268#note_1253088643 Found unset config items in arm lpae-debug-fedora, please set them to an appropriate value CONFIG_STACKPROTECTOR_PER_TASK=y Found unset config items in arm lpae-fedora, please set

[OS-BUILD PATCHv3 3/3] Update self-test data to not expect debugbuildsenabled 0

2023-01-26 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Update self-test data to not expect debugbuildsenabled 0 As we are no longer automatically passing this based on snapshot vs tag status, the self test data must be updated to accomodate the new behavior. Signed-off-by: Justin M. Forbes diff --git

[OS-BUILD PATCHv3 2/3] Turn on debug builds for aarch64 Fedora

2023-01-26 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Turn on debug builds for aarch64 Fedora Fedora only did debug builds for x86 in the past. This was largely due to slow build times on other arches in koji, coupled with much smaller userbases. Let's turn on aarch64 now as those builders are much faster and the userbase

[OS-BUILD PATCHv3 1/3] Turn off forced debug builds

2023-01-26 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 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

[OS-BUILD PATCHv3 0/3] Turn off forced debug builds

2023-01-26 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263 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

[OS-BUILD PATCH] Fix up SQUASHFS decompression configs

2023-01-30 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Fix up SQUASHFS decompression configs The 6.2 merge window changed the way that squashfs decompression threading worked. It now allows the number of threads to be set as mount options if all options are compiled. Until a decicion is made on this functionality, this MR

Re: [OS-BUILD PATCHv2 0/3] redhat: update merge.py to handle merge.pl corner cases

2023-01-30 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2262#note_1258211133 This actually seems like the more desirable behavior, but in either case, having the same config item listed multiple times in the file is a bug. It does appear that this patch

[OS-BUILD PATCHv4 0/2] Turn off forced debug builds

2023-01-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263 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

[OS-BUILD PATCHv4 1/2] Turn on debug builds for aarch64 Fedora

2023-01-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Turn on debug builds for aarch64 Fedora Fedora only did debug builds for x86 in the past. This was largely due to slow build times on other arches in koji, coupled with much smaller userbases. Let's turn on aarch64 now as those builders are much faster and the userbase

[OS-BUILD PATCHv4 2/2] Turn off forced debug builds

2023-01-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 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

[OS-BUILD PATCHv6 3/3] Update self-test data to not expect debugbuildsenabled 0

2023-01-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes Update self-test data to not expect debugbuildsenabled 0 As we are no longer automatically passing this based on snapshot vs tag status, the self test data must be updated to accomodate the new behavior. Signed-off-by: Justin M. Forbes diff --git

[OS-BUILD PATCHv5 0/3] Turn off forced debug builds

2023-01-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263 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

<    1   2   3   4   5   6   >