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 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] redhat/configs: Enable CONFIG_EFI_SECRET module

2022-11-07 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2124#note_1162366072

I am not quite sure the purpose of this MR. The config option is already
turned on in RHEL and in Fedora, though in different places.

```
cat redhat/configs/ark/generic/x86/CONFIG_EFI_SECRET
CONFIG_EFI_SECRET=m
cat redhat/configs/fedora/generic/CONFIG_EFI_SECRET
CONFIG_EFI_SECRET=m
```

This MR is not a clean-up as it does not delete either of these entries.  It
does not change the RHEL config in practice, because the option depends on
X86_64 at this point.
___
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] redhat/configs: Enable CONFIG_EFI_SECRET module

2022-11-07 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2124#note_1162434713

Not until upstream changes (I expect them to, but it hasn't happened yet):

```
config EFI_SECRET
tristate "EFI secret area securityfs support"
depends on EFI && X86_64
```
___
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] fs: fix leaked psi pressure state

2022-11-07 Thread Justin M. Forbes (via Email Bridge)
From: Johannes Weiner 

fs: fix leaked psi pressure state

When psi annotations were added to to btrfs compression reads, the psi
state tracking over add_ra_bio_pages and btrfs_submit_compressed_read was
faulty.  A pressure state, once entered, is never left.  This results in
incorrectly elevated pressure, which triggers OOM kills.

pflags record the *previous* memstall state when we enter a new one.  The
code tried to initialize pflags to 1, and then optimize the leave call
when we either didn't enter a memstall, or were already inside a nested
stall.  However, there can be multiple PageWorkingset pages in the bio, at
which point it's that path itself that enters repeatedly and overwrites
pflags.  This causes us to miss the exit.

Enter the stall only once if needed, then unwind correctly.

erofs has the same problem, fix that up too.  And move the memstall exit
past submit_bio() to restore submit accounting originally added by
b8e24a9300b0 ("block: annotate refault stalls from IO submission").

Link: https://lkml.kernel.org/r/y2uhrqthnuwui...@cmpxchg.org
Fixes: 4088a47e78f9 ("btrfs: add manual PSI accounting for compressed reads")
Fixes: 99486c511f68 ("erofs: add manual PSI accounting for the compressed 
address space")
Fixes: 118f3663fbc6 ("block: remove PSI accounting from the bio layer")
Link: 
https://lore.kernel.org/r/d20a0a85-e415-cf78-27f9-77dd7a94b...@leemhuis.info/
Signed-off-by: Johannes Weiner 
Reported-by: Thorsten Leemhuis 
Tested-by: Thorsten Leemhuis 
Cc: Chao Yu 
Cc: Chris Mason 
Cc: Christoph Hellwig 
Cc: David Sterba 
Cc: Gao Xiang 
Cc: Jens Axboe 
Cc: Josef Bacik 
Cc: Suren Baghdasaryan 
Signed-off-by: Andrew Morton 

diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
index blahblah..blahblah 100644
--- a/fs/btrfs/compression.c
+++ b/fs/btrfs/compression.c
@@ -512,7 +512,7 @@ static u64 bio_end_offset(struct bio *bio)
 static noinline int add_ra_bio_pages(struct inode *inode,
 u64 compressed_end,
 struct compressed_bio *cb,
-unsigned long *pflags)
+int *memstall, unsigned long *pflags)
 {
struct btrfs_fs_info *fs_info = btrfs_sb(inode->i_sb);
unsigned long end_index;
@@ -581,8 +581,10 @@ static noinline int add_ra_bio_pages(struct inode *inode,
continue;
}
 
-   if (PageWorkingset(page))
+   if (!*memstall && PageWorkingset(page)) {
psi_memstall_enter(pflags);
+   *memstall = 1;
+   }
 
ret = set_page_extent_mapped(page);
if (ret < 0) {
@@ -670,8 +672,8 @@ void btrfs_submit_compressed_read(struct inode *inode, 
struct bio *bio,
u64 em_len;
u64 em_start;
struct extent_map *em;
-   /* Initialize to 1 to make skip psi_memstall_leave unless needed */
-   unsigned long pflags = 1;
+   unsigned long pflags;
+   int memstall = 0;
blk_status_t ret;
int ret2;
int i;
@@ -727,7 +729,7 @@ void btrfs_submit_compressed_read(struct inode *inode, 
struct bio *bio,
goto fail;
}
 
-   add_ra_bio_pages(inode, em_start + em_len, cb, &pflags);
+   add_ra_bio_pages(inode, em_start + em_len, cb, &memstall, &pflags);
 
/* include any pages we added in add_ra-bio_pages */
cb->len = bio->bi_iter.bi_size;
@@ -807,7 +809,7 @@ void btrfs_submit_compressed_read(struct inode *inode, 
struct bio *bio,
}
}
 
-   if (!pflags)
+   if (memstall)
psi_memstall_leave(&pflags);
 
if (refcount_dec_and_test(&cb->pending_ios))
diff --git a/fs/erofs/zdata.c b/fs/erofs/zdata.c
index blahblah..blahblah 100644
--- a/fs/erofs/zdata.c
+++ b/fs/erofs/zdata.c
@@ -1412,8 +1412,8 @@ static void z_erofs_submit_queue(struct 
z_erofs_decompress_frontend *f,
struct block_device *last_bdev;
unsigned int nr_bios = 0;
struct bio *bio = NULL;
-   /* initialize to 1 to make skip psi_memstall_leave unless needed */
-   unsigned long pflags = 1;
+   unsigned long pflags;
+   int memstall = 0;
 
bi_private = jobqueueset_init(sb, q, fgq, force_fg);
qtail[JQ_BYPASS] = &q[JQ_BYPASS]->head;
@@ -1463,14 +1463,18 @@ static void z_erofs_submit_queue(struct 
z_erofs_decompress_frontend *f,
if (bio && (cur != last_index + 1 ||
last_bdev != mdev.m_bdev)) {
 submit_bio_retry:
-   if (!pflags)
-   psi_memstall_leave(&pflags);
submit_bio(bio);
+   if (memstall) {
+   psi_memstall_leave(&pflags);
+   memstall = 0;
+   }
 

Re: [OS-BUILD PATCH] fs: fix leaked psi pressure state

2022-11-14 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2137#note_1170782780

This is upstream now.
___
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] 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.
___
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] 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, that script has been fixed to check both common and ark,
leading to fewer false positives. But we do have a problem with a polluted
common directory, which I plan to clean up after rc7 as an effectively no-op
change so that common is only what is truly "common".  And once that is done,
the script can maintain common in such a way without people having to worry
about Fedora or RHEL configs respectively.
___
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 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 -- 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] 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: https://bugzilla.redhat.com/show_bug.cgi?id=2149273

Reported-by: Heiko Carstens 
Signed-off-by: Justin M. Forbes 

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
@@ -449,7 +449,7 @@ Summary: The Linux kernel
 %define hdrarch s390
 %define all_arch_configs kernel-%{version}-s390x.config
 %define kernel_image arch/s390/boot/bzImage
-%define vmlinux_decompressor arch/s390/boot/compressed/vmlinux
+%define vmlinux_decompressor arch/s390/boot/vmlinux
 %endif
 
 %ifarch %{arm}

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2168
___
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 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, moving the contents up a level to just arm/
___
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 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 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 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 utility to it from that perspective, it made sure patches
had at least some headers, etc.  With kernel-ark now, providing more utility,
and our srpm patches being 1 large diff from upstream, what does this 30-40
seconds buy us?
___
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 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 g...@gitlab.com:cki-project/kernel-ark.git
233Everything up-to-date
234Pushing config update branches
235grep: .push-warnings: No such file or directory
236rm: cannot remove '.push-warnings': No such file or directory
237make[1]: *** [Makefile:734: dist-merge-upstream] Error 1
238make[1]: Leaving directory '/builds/53/kernel-ark/redhat'
239make: *** [makefile:12: dist-merge-upstream-push] Error 2
241
Cleaning up project directory and file based variables
00:01
243ERROR: Job failed: exit code 1
```
___
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] 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: /builddir/build/BUILDROOT/kernel-6.1.0-0.rc8.20221209gi
t0d1409e4ff08.62.eln124.aarch64/etc/sysconfig/cpupower
```
___
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] 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 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -13,6 +13,7 @@ drm_dp_mst_helper_test
 drm_format_helper_test
 drm_format_test
 drm_framebuffer_test
+drm_kunit_helpers
 drm_mm_test
 drm_plane_helper_test
 drm_rect_test

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2218
___
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] 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 a/redhat/scripts/mod/mod-internal.list 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -44,10 +44,12 @@ refscale
 resource_kunit
 rocker
 scftorture
+siphash_kunit
 slub_kunit
 soc-topology-test
 soc-utils-test
 stackinit_kunit
+strscpy_kunit
 sysctl-test
 test_bits
 test_bpf

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2224
___
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 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 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] 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 to
solve a particular problem from what I can tell.  Given upstream has
just been notified though, we will keep this as "include in releases"
and not merge until upstream has a proper solution.

Signed-off-by: Justin M. Forbes 

diff --git a/scripts/head-object-list.txt b/scripts/head-object-list.txt
index blahblah..blahblah 100644
--- a/scripts/head-object-list.txt
+++ b/scripts/head-object-list.txt
@@ -15,6 +15,7 @@ arch/alpha/kernel/head.o
 arch/arc/kernel/head.o
 arch/arm/kernel/head-nommu.o
 arch/arm/kernel/head.o
+arch/arm64/kernel/head.o
 arch/csky/kernel/head.o
 arch/hexagon/kernel/head.o
 arch/ia64/kernel/head.o

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2232
___
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 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.
___
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 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
___
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 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 does seem that UKIs would need to follow suit in the
spirit of the change.
___
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 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 turn on for ark, and where it is much less of a
problem for RHEL. Once a year, we get a new toolchain in Fedora. It is typical
that we get a massive number of kernel warnings when a new toolchain is
introduced.  Over time (usually a couple of release cycles, so months) those
warnings are fixed.  RHEL never has this problem.   Turning on full Werror in
ark will result in my having to chase down a whole bunch of fixes to keep the
tree building which would just land naturally long before RHEL needs them.
___
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] 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 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] 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 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 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 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] 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 chose the later to keep existing
functionality, though can change this for RHEL if it is preferred to
turn off SND_SOC_NAU8315.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/configs/common/generic/CONFIG_SND_SOC_NAU8315 
b/redhat/configs/common/generic/CONFIG_SND_SOC_NAU8315
index blahblah..blahblah 100644
--- a/redhat/configs/common/generic/CONFIG_SND_SOC_NAU8315
+++ b/redhat/configs/common/generic/CONFIG_SND_SOC_NAU8315
@@ -1 +1 @@
-# CONFIG_SND_SOC_NAU8315 is not set
+CONFIG_SND_SOC_NAU8315=m
diff --git a/redhat/configs/fedora/generic/CONFIG_SND_SOC_NAU8315 
b/redhat/configs/fedora/generic/CONFIG_SND_SOC_NAU8315
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/fedora/generic/CONFIG_SND_SOC_NAU8315
+++ /dev/null
@@ -1 +0,0 @@
-CONFIG_SND_SOC_NAU8315=m
diff --git a/redhat/configs/pending-ark/generic/CONFIG_SND_SOC_NAU8315 
b/redhat/configs/pending-ark/generic/CONFIG_SND_SOC_NAU8315
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/pending-ark/generic/CONFIG_SND_SOC_NAU8315
+++ /dev/null
@@ -1 +0,0 @@
-CONFIG_SND_SOC_NAU8315=m

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2245
___
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] Turn off CONFIG_MTK_T7XX for S390x

2023-01-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Turn off CONFIG_MTK_T7XX for S390x

A build issue crept in over the weekend with the MTK_T7XX driver and
S390x.  As I started digging, I realized this is likely hardware that
would never be used with S390x, and might be better off just disabled
for that arch. It is a driver for the MediaTek PCIe based 5G WWAN modem.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/configs/common/generic/s390x/CONFIG_MTK_T7XX 
b/redhat/configs/common/generic/s390x/CONFIG_MTK_T7XX
new file mode 100644
index blahblah..blahblah 100644
--- /dev/null
+++ b/redhat/configs/common/generic/s390x/CONFIG_MTK_T7XX
@@ -0,0 +1 @@
+# CONFIG_MTK_T7XX is not set

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2247
___
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 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 you will need to change a value, while things like
DIST_BRANCH, UPSTREAM_BRANCH, VERSION_ON_UPSTREAM, and BUMP_RELEASE are
frequently changed.
___
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] Disable frame pointers

2023-01-23 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Disable frame pointers

Fedora is defaulting to add -fno-omit-frame-pointer by default. This is
not necessary for kernel.

Signed-off-by: Justin M. Forbes 

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
@@ -3,6 +3,9 @@
 # environment changes that affect %%install need to go
 # here before the %%install macro is pre-built.
 
+# Disable frame pointers
+%undefine _include_frame_pointers
+
 # Disable LTO in userspace packages.
 %global _lto_cflags %{nil}
 

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2260
___
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] 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 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 

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,14 +4,11 @@
 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
+# As debug kernels have gotten a bit slower over time, the forced debug
+# builds are going unused. We are no longer forcing debug builds only. 
+# Keep the option around for one off scratch builds though.
+# Two kernels are built, one with debug configuration and one without.
+SPECDEBUG_BUILDS_ENABLED=1
 
 if [ -n "$DISTLOCALVERSION" ]; then
SPECBUILDID=$(printf "%%define buildid %s" "$DISTLOCALVERSION")

--
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


[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 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 

---
 redhat/scripts/genspec/genspec.sh |  13 +
 redhat/kernel.spec.template   |   4 ++--
 2 files changed, 7 insertions(+), 10 deletions(-)
___
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 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 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 

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,14 +4,11 @@
 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
+# As debug kernels have gotten a bit slower over time, the forced debug
+# builds are going unused. We are no longer forcing debug builds only. 
+# Keep the option around for one off scratch builds though.
+# Two kernels are built, one with debug configuration and one without.
+SPECDEBUG_BUILDS_ENABLED=1
 
 if [ -n "$DISTLOCALVERSION" ]; then
SPECBUILDID=$(printf "%%define buildid %s" "$DISTLOCALVERSION")

--
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


[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 is considerably larger these days.  Replacing i686 here
makes sense as we no longer build for that arch.

Signed-off-by: Justin M. Forbes 

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
@@ -388,8 +388,8 @@ Summary: The Linux kernel
 %endif
 
 %if 0%{?fedora}
-# don't do debug builds on anything but i686 and x86_64
-%ifnarch i686 x86_64
+# don't do debug builds on anything but aarch64 and x86_64
+%ifnarch aarch64 x86_64
 %define with_debug 0
 %endif
 %endif

--
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


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 them to an appropriate
value
CONFIG_STACKPROTECTOR_PER_TASK=y
Found unset config items in arm lpae-kgcov-fedora, please set them to an
appropriate value
CONFIG_STACKPROTECTOR_PER_TASK=y
Found unset config items in arm debug-fedora, please set them to an
appropriate value
CONFIG_STACKPROTECTOR_PER_TASK=y
Found unset config items in arm fedora, please set them to an appropriate
value
CONFIG_STACKPROTECTOR_PER_TASK=y
Found unset config items in arm kgcov-fedora, please set them to an
appropriate value
CONFIG_STACKPROTECTOR_PER_TASK=y

While we don't build armv7 in F37 or newer, it is still built for F36 and
needs config options set appropriately. Running `make dist-configs-check`
should validate that config changes do not cause a conflict.
___
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 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 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 

---
 redhat/scripts/genspec/genspec.sh   |  13 +
 redhat/self-test/data/centos-78e36f3b0dae.el7.spec  |   1 -
 redhat/self-test/data/centos-78e36f3b0dae.fc25.spec |   1 -
 redhat/self-test/data/centos-fce15c45d3fb.el7.spec  |   1 -
 redhat/self-test/data/centos-fce15c45d3fb.fc25.spec |   1 -
 redhat/self-test/data/fedora-78e36f3b0dae.el7.spec  |   1 -
 redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec |   1 -
 redhat/self-test/data/fedora-fce15c45d3fb.el7.spec  |   1 -
 redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec |   1 -
 redhat/self-test/data/rhel-78e36f3b0dae.el7.spec|   1 -
 redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec   |   1 -
 redhat/self-test/data/rhel-fce15c45d3fb.el7.spec|   1 -
 redhat/self-test/data/rhel-fce15c45d3fb.fc25.spec   |   1 -
 redhat/kernel.spec.template |   4 ++--
 14 files changed, 7 insertions(+), 22 deletions(-)
___
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 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 is considerably larger these days.  Replacing i686 here
makes sense as we no longer build for that arch.

Signed-off-by: Justin M. Forbes 

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
@@ -388,8 +388,8 @@ Summary: The Linux kernel
 %endif
 
 %if 0%{?fedora}
-# don't do debug builds on anything but i686 and x86_64
-%ifnarch i686 x86_64
+# don't do debug builds on anything but aarch64 and x86_64
+%ifnarch aarch64 x86_64
 %define with_debug 0
 %endif
 %endif

--
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


[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 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 

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,14 +4,11 @@
 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
+# As debug kernels have gotten a bit slower over time, the forced debug
+# builds are going unused. We are no longer forcing debug builds only. 
+# Keep the option around for one off scratch builds though.
+# Two kernels are built, one with debug configuration and one without.
+SPECDEBUG_BUILDS_ENABLED=1
 
 if [ -n "$DISTLOCALVERSION" ]; then
SPECBUILDID=$(printf "%%define buildid %s" "$DISTLOCALVERSION")

--
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


[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 a/redhat/self-test/data/centos-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/centos-fce15c45d3fb.el7.spec 
b/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
+++ b/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec 
b/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
+++ b/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec 
b/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
+++ b/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec 
b/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
+++ b/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/

[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 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 

---
 redhat/scripts/genspec/genspec.sh |  9 -
 redhat/kernel.spec.template   |  6 +++---
 2 files changed, 3 insertions(+), 12 deletions(-)
___
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 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 is considerably larger these days.  Replacing i686 here
makes sense as we no longer build for that arch.

Signed-off-by: Justin M. Forbes 

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
@@ -388,8 +388,8 @@ Summary: The Linux kernel
 %endif
 
 %if 0%{?fedora}
-# don't do debug builds on anything but i686 and x86_64
-%ifnarch i686 x86_64
+# don't do debug builds on anything but aarch64 and x86_64
+%ifnarch aarch64 x86_64
 %define with_debug 0
 %endif
 %endif

--
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


[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 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 

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

--
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


[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 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 

---
 redhat/scripts/genspec/genspec.sh   |  9 -
 redhat/self-test/data/centos-78e36f3b0dae.el7.spec  |  1 -
 redhat/self-test/data/centos-78e36f3b0dae.fc25.spec |  1 -
 redhat/self-test/data/centos-fce15c45d3fb.el7.spec  |  1 -
 redhat/self-test/data/centos-fce15c45d3fb.fc25.spec |  1 -
 redhat/self-test/data/fedora-78e36f3b0dae.el7.spec  |  1 -
 redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec |  1 -
 redhat/self-test/data/fedora-fce15c45d3fb.el7.spec  |  1 -
 redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec |  1 -
 redhat/self-test/data/rhel-78e36f3b0dae.el7.spec|  1 -
 redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec   |  1 -
 redhat/self-test/data/rhel-fce15c45d3fb.el7.spec|  1 -
 redhat/self-test/data/rhel-fce15c45d3fb.fc25.spec   |  1 -
 redhat/kernel.spec.template |  6 +++---
 14 files changed, 3 insertions(+), 24 deletions(-)
___
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 PATCHv5 1/3] 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 is considerably larger these days.  Replacing i686 here
makes sense as we no longer build for that arch.

Signed-off-by: Justin M. Forbes 

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
@@ -388,8 +388,8 @@ Summary: The Linux kernel
 %endif
 
 %if 0%{?fedora}
-# don't do debug builds on anything but i686 and x86_64
-%ifnarch i686 x86_64
+# don't do debug builds on anything but aarch64 and x86_64
+%ifnarch aarch64 x86_64
 %define with_debug 0
 %endif
 %endif

--
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


[OS-BUILD PATCHv5 2/3] 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 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 

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

--
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


[OS-BUILD PATCHv5 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 a/redhat/self-test/data/centos-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/centos-fce15c45d3fb.el7.spec 
b/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
+++ b/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec 
b/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
+++ b/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec 
b/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
+++ b/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec 
b/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
+++ b/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/

Re: [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
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263#note_1255439788

Good point, and I am happy to delete unnecessary code when I can. I just
updated the MR to do that. I don't think that my comment is necessary in the
spec file as the spec is well documented on this option to begin with.
___
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 PATCHv6 1/3] 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 is considerably larger these days.  Replacing i686 here
makes sense as we no longer build for that arch.

Signed-off-by: Justin M. Forbes 

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
@@ -388,8 +388,8 @@ Summary: The Linux kernel
 %endif
 
 %if 0%{?fedora}
-# don't do debug builds on anything but i686 and x86_64
-%ifnarch i686 x86_64
+# don't do debug builds on anything but aarch64 and x86_64
+%ifnarch aarch64 x86_64
 %define with_debug 0
 %endif
 %endif

--
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


[OS-BUILD PATCHv6 2/3] 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 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 

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


[OS-BUILD PATCHv6 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 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 

---
 redhat/scripts/genspec/genspec.sh   |  10 --
 redhat/self-test/data/centos-78e36f3b0dae.el7.spec  |   1 -
 redhat/self-test/data/centos-78e36f3b0dae.fc25.spec |   1 -
 redhat/self-test/data/centos-fce15c45d3fb.el7.spec  |   1 -
 redhat/self-test/data/centos-fce15c45d3fb.fc25.spec |   1 -
 redhat/self-test/data/fedora-78e36f3b0dae.el7.spec  |   1 -
 redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec |   1 -
 redhat/self-test/data/fedora-fce15c45d3fb.el7.spec  |   1 -
 redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec |   1 -
 redhat/self-test/data/rhel-78e36f3b0dae.el7.spec|   1 -
 redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec   |   1 -
 redhat/self-test/data/rhel-fce15c45d3fb.el7.spec|   1 -
 redhat/self-test/data/rhel-fce15c45d3fb.fc25.spec   |   1 -
 redhat/kernel.spec.template |   6 +++---
 14 files changed, 3 insertions(+), 25 deletions(-)
___
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 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 a/redhat/self-test/data/centos-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/centos-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/centos-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/centos-fce15c45d3fb.el7.spec 
b/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
+++ b/redhat/self-test/data/centos-fce15c45d3fb.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec 
b/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
+++ b/redhat/self-test/data/centos-fce15c45d3fb.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/fedora-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/fedora-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec 
b/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
+++ b/redhat/self-test/data/fedora-fce15c45d3fb.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec 
b/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
+++ b/redhat/self-test/data/fedora-fce15c45d3fb.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 1
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.16.0
 %define patchversion 5.16
diff --git a/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec 
b/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
+++ b/redhat/self-test/data/rhel-78e36f3b0dae.el7.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec 
b/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
index blahblah..blahblah 100755
--- a/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
+++ b/redhat/self-test/data/rhel-78e36f3b0dae.fc25.spec
@@ -2,7 +2,6 @@
 %global include_rhel 1
 %global patchlist_changelog 0
 %global released_kernel 0
-%define debugbuildsenabled 0
 %define buildid .test
 %define specversion 5.17.0
 %define patchversion 5.17
diff --git a/redhat/self-test/

Re: [OS-BUILD PATCHv6 0/3] Turn off forced debug builds

2023-01-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2263#note_1255858889

That was done with v6, I am guessing this can be resolved now?
___
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] 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 factors in the new options and gives us a built
kernel config similar to what has been supported in previous releases.
We select CONFIG_SQUASHFS_DECOMP_MULTI_PERCPU as the only option. If a
different config is desired, I am happy to change this, but our current
defaults were not working as it forced a config mismatch, and I did not
want the state to get lost.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/configs/ark/generic/CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT 
b/redhat/configs/ark/generic/CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT
new file mode 100644
index blahblah..blahblah 100644
--- /dev/null
+++ b/redhat/configs/ark/generic/CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT
@@ -0,0 +1 @@
+# CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT is not set
diff --git a/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI 
b/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI
new file mode 100644
index blahblah..blahblah 100644
--- /dev/null
+++ b/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI
@@ -0,0 +1 @@
+# CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI is not set
diff --git 
a/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU 
b/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU
new file mode 100644
index blahblah..blahblah 100644
--- /dev/null
+++ b/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU
@@ -0,0 +1 @@
+CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU=y
diff --git a/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_SINGLE 
b/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_SINGLE
new file mode 100644
index blahblah..blahblah 100644
--- /dev/null
+++ b/redhat/configs/ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_SINGLE
@@ -0,0 +1 @@
+# CONFIG_SQUASHFS_COMPILE_DECOMP_SINGLE is not set
diff --git 
a/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT 
b/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT
+++ /dev/null
@@ -1,15 +0,0 @@
-# Symbol: SQUASHFS_CHOICE_DECOMP_BY_MOUNT [=n]
-# Type  : bool
-# Defined at fs/squashfs/Kconfig:69
-#   Prompt: Select the parallel decompression mode during mount
-#   Depends on: MISC_FILESYSTEMS [=y] && SQUASHFS [=m]
-#   Location:
-# -> File systems
-#   -> Miscellaneous filesystems (MISC_FILESYSTEMS [=y])
-# -> SquashFS 4.0 - Squashed file system support (SQUASHFS [=m])
-#   -> Select the parallel decompression mode during mount 
(SQUASHFS_CHOICE_DECOMP_BY_MOUNT [=n])
-# Selects: SQUASHFS_DECOMP_SINGLE [=y] && SQUASHFS_DECOMP_MULTI [=n] && 
SQUASHFS_DECOMP_MULTI_PERCPU [=n] && SQUASHFS_MOUNT_DECOMP_THREADS [=n]
-# 
-# 
-# 
-# CONFIG_SQUASHFS_CHOICE_DECOMP_BY_MOUNT is not set
diff --git 
a/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI 
b/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI
+++ /dev/null
@@ -1,17 +0,0 @@
-# Symbol: SQUASHFS_COMPILE_DECOMP_MULTI [=n]
-# Type  : bool
-# Defined at fs/squashfs/Kconfig:101
-#   Prompt: Use multiple decompressors for parallel I/O
-#   Depends on: 
-#   Location:
-# -> File systems
-#   -> Miscellaneous filesystems (MISC_FILESYSTEMS [=y])
-# -> SquashFS 4.0 - Squashed file system support (SQUASHFS [=m])
-#   -> Select the parallel decompression mode during mount 
(SQUASHFS_CHOICE_DECOMP_BY_MOUNT [=n])
-# -> Select decompression parallel mode at compile time ( 
[=y])
-#   -> Use multiple decompressors for parallel I/O 
(SQUASHFS_COMPILE_DECOMP_MULTI [=n])
-# Selects: SQUASHFS_DECOMP_MULTI [=n]
-# 
-# 
-# 
-# CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI is not set
diff --git 
a/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU
 
b/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU
deleted file mode 100644
index blahblah..blahblah 0
--- 
a/redhat/configs/pending-ark/generic/CONFIG_SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU
+++ /dev/null
@@ -1,17 +0,0 @@
-# Symbol: SQUASHFS_COMPILE_DECOMP_MULTI_PERCPU [=n]
-# Type  : bool
-# Defined at fs/squashfs/Kconfig:116
-#   Prompt: Use percpu multiple decompressors for parallel I/O
-#   Depends on: 
-#   Location:
-# -> File systems
-#   -> Miscellaneous filesystems (MISC_FILESYSTEMS [=y])
-# -> SquashFS 4.0 - Squashed file system support (SQUASHFS [=m])
-#   -> Select the parallel decompression mode during mount 
(SQUASHFS_C

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 changes the config output, however it seems like
logical changes:

```
--- a/kernel-x86_64-rhel.config
+++ b/kernel-x86_64-rhel.config
@@ -868,7 +868,6 @@ CONFIG_CPU_LITTLE_ENDIAN=y
 CONFIG_CPUMASK_KUNIT_TEST=m
 CONFIG_CPUMASK_OFFSTACK=y
 CONFIG_CPUSETS=y
-# CONFIG_CPU_SUP_CENTAUR ignored for predicate CONFIG_EXPERT
 # CONFIG_CPU_THERMAL is not set
 CONFIG_CPU_UNRET_ENTRY=y
 # CONFIG_CRAMFS is not set
@@ -5864,44 +5863,31 @@ CONFIG_SND_SOC_RT715_SDW=m
 # CONFIG_SND_SOC_SMDK_WM8994_PCM is not set
 # CONFIG_SND_SOC_SNOW is not set
 CONFIG_SND_SOC_SOF_ACPI=m
-# CONFIG_SND_SOC_SOF_ALDERLAKE:
 CONFIG_SND_SOC_SOF_ALDERLAKE=m
 # CONFIG_SND_SOC_SOF_AMD_TOPLEVEL is not set
-# CONFIG_SND_SOC_SOF_APOLLOLAKE:
 CONFIG_SND_SOC_SOF_APOLLOLAKE=m
-# CONFIG_SND_SOC_SOF_BAYTRAIL:
 CONFIG_SND_SOC_SOF_BAYTRAIL=m
-# CONFIG_SND_SOC_SOF_BROADWELL:
 CONFIG_SND_SOC_SOF_BROADWELL=m
-# CONFIG_SND_SOC_SOF_CANNONLAKE:
 CONFIG_SND_SOC_SOF_CANNONLAKE=m
-# CONFIG_SND_SOC_SOF_COFFEELAKE:
 CONFIG_SND_SOC_SOF_COFFEELAKE=m
-# CONFIG_SND_SOC_SOF_COMETLAKE:
 CONFIG_SND_SOC_SOF_COMETLAKE=m
-# CONFIG_SND_SOC_SOF_ELKHARTLAKE:
 CONFIG_SND_SOC_SOF_ELKHARTLAKE=m
 CONFIG_SND_SOC_SOF_GEMINILAKE=m
 CONFIG_SND_SOC_SOF_HDA_AUDIO_CODEC=y
 CONFIG_SND_SOC_SOF_HDA_LINK=y
-# CONFIG_SND_SOC_SOF_ICELAKE:
 CONFIG_SND_SOC_SOF_ICELAKE=m
 # CONFIG_SND_SOC_SOF_IMX8M_SUPPORT is not set
 # CONFIG_SND_SOC_SOF_IMX8_SUPPORT is not set
 # CONFIG_SND_SOC_SOF_IMX_TOPLEVEL is not set
-# CONFIG_SND_SOC_SOF_INTEL_SOUNDWIRE:
 CONFIG_SND_SOC_SOF_INTEL_SOUNDWIRE=m
 CONFIG_SND_SOC_SOF_INTEL_TOPLEVEL=y
-# CONFIG_SND_SOC_SOF_JASPERLAKE:
 CONFIG_SND_SOC_SOF_JASPERLAKE=m
 CONFIG_SND_SOC_SOF_KABYLAKE=m
-# CONFIG_SND_SOC_SOF_MERRIFIELD:
 CONFIG_SND_SOC_SOF_MERRIFIELD=m
 CONFIG_SND_SOC_SOF_METEORLAKE=m
 # CONFIG_SND_SOC_SOF_OF is not set
 CONFIG_SND_SOC_SOF_PCI=m
 CONFIG_SND_SOC_SOF_SKYLAKE=m
-# CONFIG_SND_SOC_SOF_TIGERLAKE:
 CONFIG_SND_SOC_SOF_TIGERLAKE=m
 CONFIG_SND_SOC_SOF_TOPLEVEL=y
 # CONFIG_SND_SOC_SPDIF is not set
@@ -7280,12 +7266,8 @@ CONFIG_XILINX_GMII2RGMII=m
 # CONFIG_XILLYBUS is not set
 # CONFIG_XILLYUSB is not set
 CONFIG_XMON_DEFAULT_RO_MODE=y
-# CONFIG_XZ_DEC_ARM ignored for predicate CONFIG_EXPERT
-# CONFIG_XZ_DEC_ARMTHUMB ignored for predicate CONFIG_EXPERT
-# CONFIG_XZ_DEC_IA64 ignored for predicate CONFIG_EXPERT
 # CONFIG_XZ_DEC_MICROLZMA is not set
 CONFIG_XZ_DEC_POWERPC=y
-# CONFIG_XZ_DEC_SPARC ignored for predicate CONFIG_EXPERT
 # CONFIG_XZ_DEC_TEST is not set
 CONFIG_XZ_DEC_X86=y
 CONFIG_XZ_DEC=y
```

Other arches included the same or just a subset as the SND_SOC bits were
lacking deps.   If you look closely, we are losing comments that shouldn't be
in the file, and do not have an impact on the built kernel.  Behavior of the
built kernels should be identical.  For future changes, after merge, things
would work just as before the merge in that anyone making a config change
should verify that the config change they are making has the desired effect.
___
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 PATCHv3 0/4] redhat: update merge.py to handle merge.pl corner cases

2023-01-31 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_1259574917

So this version does make the output match the output from before this MR, but
that means we have our additional comments in the config files.  I do not
consider it a part of the scope of this MR, but it has pointed out that those
comments exist, and should not.  If we are going to accept this version of
merge.py, someone should do another MR to cleanup all of those unnecessary
comments.
___
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 PATCHv7] redhat: Add sub-RPM with a EFI unified kernel image for virtual machines

2023-02-06 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_1267373813

@vkuznets It needs a proper RHEL ack because it changes RHEL as well. I asked
Herton to follow up today. I have been holding my ack because mine would
technically "satisfy the requirements" but I am not RHEL.
___
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 PATCHv2] Enable TDX Guest driver

2023-02-07 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2250#note_1268722798

Done, sorry, this one didn't get Acks::fedora::NeedsReview for some reason.
___
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 PATCHv7] redhat: Add sub-RPM with a EFI unified kernel image for virtual machines

2023-02-08 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_1271181107

While this is working for Fedora it is failing for ELN (RHEL) kernels with:
`dracut: Can't find a uefi stub '/usr/lib/systemd/boot/efi/linuxx64.efi.stub'
to create a UEFI executable`
___
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 PATCHv2] aarch64: enable zboot

2023-02-08 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2283#note_1271283397

While I did originally try to enable this for Fedora, we had to turn it off
quicky as it breaks CKI. It can't be re-enabled until CKI fixes their scripts
to handle it.
___
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 PATCHv7] redhat: Add sub-RPM with a EFI unified kernel image for virtual machines

2023-02-09 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_1272255146

I believe the correct method is just to change the buildreq to systemd-boot-
unsigned for everything, and add a note to redhat/rebase-notes.txt to revert
that new MR for stable fedora until F37 is EOL.  os-build does not build
kernels for anything other than rawhide, and when I do branch for stable
Fedora, that branch will not build kernels for ELN/RHEL, so the current
behavior is correct.
___
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 PATCHv2] aarch64: enable zboot

2023-02-09 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2283#note_1272311260

For the spec changes, we either need to limit those to Fedora only, or we need
to turn on the config for both Fedora and ELN/RHEL.
___
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] Disable frame pointers

2023-02-14 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2260#note_1277926446

Fedora doesn't build userspace tools as part of the kernel package, hasn't for
years.
___
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] Fix underline mark-up after text change

2023-02-15 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Fix underline mark-up after text change

After MR 2278, the docs build was broken complaining that the underline
was too short. Renaming pending-ark to pending-rhel required adding a
character to the underline to match.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/docs/repository-layout.rst 
b/redhat/docs/repository-layout.rst
index blahblah..blahblah 100644
--- a/redhat/docs/repository-layout.rst
+++ b/redhat/docs/repository-layout.rst
@@ -135,7 +135,7 @@ A flavor is defined by:
defined in step 2.
 
 common and pending-rhel
-~~
+~~~
 
 The ``common`` directory contains configuration values that are shared
 across all configuration "flavors". For a configuration to be in

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2290
___
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 PATCHv3] aarch64: enable zboot

2023-02-22 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2283#note_1288515197

So, it turns out this might break chromebook booting.
___
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] Turn on IDLE_INJECT for x86

2023-02-22 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Turn on IDLE_INJECT for x86

Upstream commit 8526eb7fc75ab changed INTEL_POWERCLAMP to select
IDLE_INJECT. As we have INTEL_POWERCLAMP selected  as a module, we must
either turn on IDLE_INJECT or turn off INTEL_POWERCLAMP.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/configs/common/generic/CONFIG_IDLE_INJECT 
b/redhat/configs/common/generic/CONFIG_IDLE_INJECT
index blahblah..blahblah 100644
--- a/redhat/configs/common/generic/CONFIG_IDLE_INJECT
+++ b/redhat/configs/common/generic/CONFIG_IDLE_INJECT
@@ -1 +1 @@
-# CONFIG_IDLE_INJECT is not set
+CONFIG_IDLE_INJECT=y
diff --git a/redhat/configs/pending-fedora/generic/x86/CONFIG_IDLE_INJECT 
b/redhat/configs/pending-fedora/generic/x86/CONFIG_IDLE_INJECT
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/pending-fedora/generic/x86/CONFIG_IDLE_INJECT
+++ /dev/null
@@ -1 +0,0 @@
-CONFIG_IDLE_INJECT=y
diff --git a/redhat/configs/pending-rhel/generic/x86/CONFIG_IDLE_INJECT 
b/redhat/configs/pending-rhel/generic/x86/CONFIG_IDLE_INJECT
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/pending-rhel/generic/x86/CONFIG_IDLE_INJECT
+++ /dev/null
@@ -1 +0,0 @@
-CONFIG_IDLE_INJECT=y

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2302
___
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 PATCHv2] Turn on IDLE_INJECT for x86

2023-02-22 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Turn on IDLE_INJECT for x86

Upstream commit 8526eb7fc75ab changed INTEL_POWERCLAMP to select
IDLE_INJECT. As we have INTEL_POWERCLAMP selected  as a module, we must
either turn on IDLE_INJECT or turn off INTEL_POWERCLAMP.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/configs/pending-fedora/generic/x86/CONFIG_IDLE_INJECT 
b/redhat/configs/common/generic/x86/CONFIG_IDLE_INJECT
rename from redhat/configs/pending-fedora/generic/x86/CONFIG_IDLE_INJECT
rename to redhat/configs/common/generic/x86/CONFIG_IDLE_INJECT
index blahblah..blahblah 100644
--- a/redhat/configs/pending-fedora/generic/x86/CONFIG_IDLE_INJECT
+++ b/redhat/configs/common/generic/x86/CONFIG_IDLE_INJECT
diff --git a/redhat/configs/pending-rhel/generic/x86/CONFIG_IDLE_INJECT 
b/redhat/configs/pending-rhel/generic/x86/CONFIG_IDLE_INJECT
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/pending-rhel/generic/x86/CONFIG_IDLE_INJECT
+++ /dev/null
@@ -1 +0,0 @@
-CONFIG_IDLE_INJECT=y

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2302
___
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 PATCHv2] Turn on IDLE_INJECT for x86

2023-02-22 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2302#note_1288921218

The previous push didn't limit it to x86.
___
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] Add more kunit tests to mod-internal.list for 6.3

2023-02-23 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Add more kunit tests to mod-internal.list for 6.3

I am sure there will be more, but the first batch is in from the drm
merge. Just new kunit tests which need to be in mod-internal for depmod
to succeed.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/scripts/mod/mod-internal.list 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -8,18 +8,23 @@ dev_addr_lists_test
 dmatest
 drm_buddy_test
 drm_cmdline_parser_test
+drm_connector_test
 drm_damage_helper_test
 drm_dp_mst_helper_test
 drm_format_helper_test
 drm_format_test
 drm_framebuffer_test
 drm_kunit_helpers
+drm_managed_test
 drm_mm_test
+drm_modes_test
 drm_plane_helper_test
+drm_probe_helper_test
 drm_rect_test
 ext4-inode-test
 fat_test
 fortify_kunit
+gss_krb5_test
 iio-test-format
 iio-test-rescale
 is_signed_type_kunit

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2313
___
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 PATCHv2 1/2] Add more kunit tests to mod-internal.list for 6.3

2023-02-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Add more kunit tests to mod-internal.list for 6.3

I am sure there will be more, but the first batch is in from the drm
merge. Just new kunit tests which need to be in mod-internal for depmod
to succeed.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/scripts/mod/mod-internal.list 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -8,18 +8,23 @@ dev_addr_lists_test
 dmatest
 drm_buddy_test
 drm_cmdline_parser_test
+drm_connector_test
 drm_damage_helper_test
 drm_dp_mst_helper_test
 drm_format_helper_test
 drm_format_test
 drm_framebuffer_test
 drm_kunit_helpers
+drm_managed_test
 drm_mm_test
+drm_modes_test
 drm_plane_helper_test
+drm_probe_helper_test
 drm_rect_test
 ext4-inode-test
 fat_test
 fortify_kunit
+gss_krb5_test
 iio-test-format
 iio-test-rescale
 is_signed_type_kunit

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2313
___
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 PATCHv2 2/2] Add hashtable_test to mod-internal.list

2023-02-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Add hashtable_test to mod-internal.list

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/scripts/mod/mod-internal.list 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -25,6 +25,7 @@ ext4-inode-test
 fat_test
 fortify_kunit
 gss_krb5_test
+hashtable_test
 iio-test-format
 iio-test-rescale
 is_signed_type_kunit

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2313
___
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 PATCHv2 0/2] Add more kunit tests to mod-internal.list for 6.3

2023-02-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/2313

I am sure there will be more, but the first batch is in from the drm
merge. Just new kunit tests which need to be in mod-internal for depmod
to succeed.

Signed-off-by: Justin M. Forbes 

---
 redhat/scripts/mod/mod-internal.list |  6 ++
 1 files changed, 6 insertions(+), 0 deletions(-)
___
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] redhat: Add hashtable_test to mod-internal.list

2023-02-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2335#note_1293931289

This was already added to MR 2313
___
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 PATCHv2] redhat/kernel.spec.template: Fix RHEL systemd-boot-unsigned dependency

2023-02-28 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2333#note_1295379598

Not sure how this went from closed to reopened and merged within hours (I
thought we were waiting for Clark's change). This MR is wrong, and breaks ELN
builds which need systemd-boot-unsigned as a buildreq.

`dracut: Can't find a uefi stub '/usr/lib/systemd/boot/efi/linuxx64.efi.stub'
to create a UEFI executable`
___
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] fix tools build after vm to mm rename

2023-02-28 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

fix tools build after vm to mm rename

Upstream commit 799fb82aa132f renamed tools/vm to tools/mm. As a result,
our tools build is tring to build in the wrong directory.

Signed-off-by: Justin M. Forbes 

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
@@ -2464,7 +2464,7 @@ pushd tools/gpio/
 %{tools_make}
 popd
 # build VM tools
-pushd tools/vm/
+pushd tools/mm/
 %{tools_make} slabinfo page_owner_sort
 popd
 pushd tools/tracing/rtla
@@ -2748,7 +2748,7 @@ pushd tools/kvm/kvm_stat
 install -m644 -D kvm_stat.service %{buildroot}%{_unitdir}/kvm_stat.service
 popd
 # install VM tools
-pushd tools/vm/
+pushd tools/mm/
 install -m755 slabinfo %{buildroot}%{_bindir}/slabinfo
 install -m755 page_owner_sort %{buildroot}%{_bindir}/page_owner_sort
 popd

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2340
___
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 PATCHv2] redhat/kernel.spec.template: Fix RHEL systemd-boot-unsigned dependency

2023-02-28 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2333#note_1295548471

It should. And that is the conditional that will eventually be required anyway
once we get to the point that RHEL 10 is a thing. Though in our call
yesterday, it was mentioned that Clark was changing the efiuki variant
definition, and I thought this change was supposed to be a part of that, which
was why Prarit closed it initially. Perhaps I misunderstood.
___
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] redhat/kernel.spec.template: Fix RHEL systemd-boot-unsigned dependency for

2023-02-28 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2341#note_1295944152

Wonder if they plan to change it in Fedora 37 too? Not too concerned with 36,
but if they change 37, we can drop the conditional all together.  Sure, RHEL 8
won't build it, but I don't think we support RHEL 8 building this kernel as it
is.
___
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] Add rtla-hwnoise files

2023-03-01 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Add rtla-hwnoise files

Upstream added the hwnoise tool to rtla. This needs to be added to
files, but as it is also a linked binary, the linking needst to be
manually fixed up in the rpmbuild environment similar to osnoise.

Signed-off-by: Justin M. Forbes 

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
@@ -2754,10 +2754,12 @@ install -m755 page_owner_sort 
%{buildroot}%{_bindir}/page_owner_sort
 popd
 pushd tools/tracing/rtla/
 %{tools_make} DESTDIR=%{buildroot} install
+rm -f %{buildroot}%{_bindir}/hwnoise
 rm -f %{buildroot}%{_bindir}/osnoise
 rm -f %{buildroot}%{_bindir}/timerlat
 (cd %{buildroot}
 
+ln -sf rtla ./%{_bindir}/hwnoise
 ln -sf rtla ./%{_bindir}/osnoise
 ln -sf rtla ./%{_bindir}/timerlat
 )
@@ -3189,8 +3191,10 @@ fi
 
 %files -n rtla
 %{_bindir}/rtla
+%{_bindir}/hwnoise
 %{_bindir}/osnoise
 %{_bindir}/timerlat
+%{_mandir}/man1/rtla-hwnoise.1.gz
 %{_mandir}/man1/rtla-osnoise-hist.1.gz
 %{_mandir}/man1/rtla-osnoise-top.1.gz
 %{_mandir}/man1/rtla-osnoise.1.gz

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2343
___
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] redhat/kernel.spec.template: Fix RHEL systemd-boot-unsigned dependency for

2023-03-01 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2341#note_1297690928

I pinged zbyszek on IRC, the split out can happen for F37 too, so my
recommendation for kernel-ark is just to drop the entire conditional. There
might be a brief period that F36 is a supported release and can't build a
rawhide kernel, but that is short lived.
___
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] redhat/kernel.spec.template: Fix RHEL systemd-boot-unsigned dependency for

2023-03-02 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2341#note_1299058464

Adding this MR as it is to include in release as a temporary work around so
that ELN builds from ark-latest until we determine the proper solution.
___
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] redhat/kernel.spec.template: Fix RHEL systemd-boot-unsigned dependency for

2023-03-02 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2341#note_1299462975

So it seems that RHEL will be keeping this new package too. Once @zbyszek gets
the F37 split pushed, we can drop this and revert the previous conditional.
___
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] kasan, powerpc: Don't rename memintrinsics if compiler adds prefixes

2023-03-13 Thread Justin M. Forbes (via Email Bridge)
From: Marco Elver 

kasan, powerpc: Don't rename memintrinsics if compiler adds prefixes

With appropriate compiler support [1], KASAN builds use __asan prefixed
meminstrinsics, and KASAN no longer overrides memcpy/memset/memmove.

If compiler support is detected (CC_HAS_KASAN_MEMINTRINSIC_PREFIX),
define memintrinsics normally (do not prefix '__').

On powerpc, KASAN is the only user of __mem functions, which are used to
define instrumented memintrinsics. Alias the normal versions for KASAN
to use in its implementation.

Link: https://lore.kernel.org/all/20230224085942.1791837-1-el...@google.com/ [1]
Link: https://lore.kernel.org/oe-kbuild-all/202302271348.u5lvmo0s-...@intel.com/
Reported-by: kernel test robot 
Signed-off-by: Marco Elver 
Acked-by: Michael Ellerman  (powerpc)

diff --git a/arch/powerpc/include/asm/kasan.h b/arch/powerpc/include/asm/kasan.h
index blahblah..blahblah 100644
--- a/arch/powerpc/include/asm/kasan.h
+++ b/arch/powerpc/include/asm/kasan.h
@@ -2,7 +2,7 @@
 #ifndef __ASM_KASAN_H
 #define __ASM_KASAN_H
 
-#ifdef CONFIG_KASAN
+#if defined(CONFIG_KASAN) && !defined(CONFIG_CC_HAS_KASAN_MEMINTRINSIC_PREFIX)
 #define _GLOBAL_KASAN(fn)  _GLOBAL(__##fn)
 #define _GLOBAL_TOC_KASAN(fn)  _GLOBAL_TOC(__##fn)
 #define EXPORT_SYMBOL_KASAN(fn)EXPORT_SYMBOL(__##fn)
diff --git a/arch/powerpc/include/asm/string.h 
b/arch/powerpc/include/asm/string.h
index blahblah..blahblah 100644
--- a/arch/powerpc/include/asm/string.h
+++ b/arch/powerpc/include/asm/string.h
@@ -30,11 +30,17 @@ extern int memcmp(const void *,const void 
*,__kernel_size_t);
 extern void * memchr(const void *,int,__kernel_size_t);
 void memcpy_flushcache(void *dest, const void *src, size_t size);
 
+#ifdef CONFIG_KASAN
+/* __mem variants are used by KASAN to implement instrumented meminstrinsics. 
*/
+#ifdef CONFIG_CC_HAS_KASAN_MEMINTRINSIC_PREFIX
+#define __memset memset
+#define __memcpy memcpy
+#define __memmove memmove
+#else /* CONFIG_CC_HAS_KASAN_MEMINTRINSIC_PREFIX */
 void *__memset(void *s, int c, __kernel_size_t count);
 void *__memcpy(void *to, const void *from, __kernel_size_t n);
 void *__memmove(void *to, const void *from, __kernel_size_t n);
-
-#if defined(CONFIG_KASAN) && !defined(__SANITIZE_ADDRESS__)
+#ifndef __SANITIZE_ADDRESS__
 /*
  * For files that are not instrumented (e.g. mm/slub.c) we
  * should use not instrumented version of mem* functions.
@@ -46,8 +52,9 @@ void *__memmove(void *to, const void *from, __kernel_size_t 
n);
 #ifndef __NO_FORTIFY
 #define __NO_FORTIFY /* FORTIFY_SOURCE uses __builtin_memcpy, etc. */
 #endif
-
-#endif
+#endif /* !__SANITIZE_ADDRESS__ */
+#endif /* CONFIG_CC_HAS_KASAN_MEMINTRINSIC_PREFIX */
+#endif /* CONFIG_KASAN */
 
 #ifdef CONFIG_PPC64
 #ifndef CONFIG_KASAN
diff --git a/arch/powerpc/kernel/prom_init_check.sh 
b/arch/powerpc/kernel/prom_init_check.sh
index blahblah..blahblah 100644
--- a/arch/powerpc/kernel/prom_init_check.sh
+++ b/arch/powerpc/kernel/prom_init_check.sh
@@ -13,8 +13,13 @@
 # If you really need to reference something from prom_init.o add
 # it to the list below:
 
-grep "^CONFIG_KASAN=y$" ${KCONFIG_CONFIG} >/dev/null
-if [ $? -eq 0 ]
+has_renamed_memintrinsics()
+{
+   grep -q "^CONFIG_KASAN=y$" ${KCONFIG_CONFIG} && \
+   ! grep -q "^CONFIG_CC_HAS_KASAN_MEMINTRINSIC_PREFIX=y" 
${KCONFIG_CONFIG}
+}
+
+if has_renamed_memintrinsics
 then
MEM_FUNCS="__memcpy __memset"
 else

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2363
___
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 PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1318990719

It has acks, I set to merge when the pipeline succeeeds.
___
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 PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1318995474

CKI can (and probably should) create an .rhpkg.mk file in either $HOME or
$TOPDIR.  RHJOBS is also used for the config processing and makes a large
difference there too.
___
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 PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1319136647

@mh21
```
ifndef RHJOBS
  RHJOBS=$(shell j=$$(echo $(MAKEFLAGS) | grep -Eo "(^|[ ])-j[0-9]*" | xargs
); \
  if [ -z "$${j}" ]; then \
echo "1"; \
  else \
j=$$(echo "$${j}" | tr -d "\-j"); \
[ -z "$${j}" ] && nproc --all || echo $${j}; \
  fi)
endif
```
___
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] redhat/configs: Enable CONFIG_X86_KERNEL_IBT for Fedora and ARK

2023-03-21 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2346#note_1322203317

This seems unrelated
___
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] redhat/configs: Enable CONFIG_X86_KERNEL_IBT for Fedora and ARK

2023-03-21 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2346#note_1322275937

Verified with a local build, flipping this config item doesn't seem to break
things.
___
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] redhat/configs: Enable CONFIG_X86_KERNEL_IBT for Fedora and ARK

2023-03-24 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2346#note_1327284559

Right, so this does seem to fail in rawhide, but not when built on f37. It
seems the version of dwarves (pahole) matters, and the newer pahole chokes.
___
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] kasan, powerpc: Don't rename memintrinsics if compiler adds prefixes

2023-03-25 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2363#note_1328251341

This is upstream now, closing.
___
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 PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341181644

> What are the customer expectations here? Will they think a 1.1.x kernel has
more features than a 0.1.x kernel even though the source-git is the same?

Certainly RHEL and Fedora are different, but I would expect a lot of RHEL
administrators are also Fedora users.  We have been doing this for years due
to upgrade issues.  Right now F36 kernels are 100.fc36 F37 are 200.fc37 and
F38 are 300.f38.  When F36 is EOL and we rebase stable Fedora to 6.3 kernels,
F37 will become 100.fc37.
___
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 PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341223991

I don't know exactly what upgrade methods RHEL supports, but there is a
possible problem in that dnf doesn't consider kernel-redhat-1.1.x.el9 to be a
different version than kernel-redhat-1.1.x.el10, meaning an upgrade using dnf
would likely leave the el9 kernel on the system and not install the el10
version at all.  Of course once 1.2 kernels are available, it would switch the
the el10 version.  This was the reasoning for the versioning split in Fedora
at least.
___
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 PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341240532

This has nothing to do with support, and everything to do with what dnf
automatically does when presented with 2 version strings. At least it used to
be, the dist wasn't part of the comparison, so to dnf kernel-redhat-1.1.x.el9
*is* kernel-redhat-1.1.x.el10.  If that is no longer the case, or if RHEL has
some other mechanism to force package upgrades on dist upgrade, that is fine.
___
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 PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341279929

It only happens when I user is running el9 and decides to upgrade to el10. The
upgrade would skip installing the el10 kernel as that version is already
installed.   When the next kernel upgrade happens, the system would move to an
el10 kernel because it is a newer version.
___
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 PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341370032

It is a problem for Fedora upgrades, which is why we have
kernel-6.2.9-100.fc36, kernel-6.2.9-200.fc37, and kernel-6.2.9-300.fc38
___
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 PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-05 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1342277965

I just did a quick test rebuilding the F37 kernel-test against F38 and dnf
will consider it an upgrade.  This was originally an issue in Fedora a bit
over 10 years ago, so there has been a lot of change in that time, dnf wasn't
even a thing yet, still yum.
___
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 PATCHv2] Revert "redhat: configs: Disable xtables and ipset"

2023-04-12 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2267#note_1350541860

While this MR placed the files in the common directory as they matched the
fedora configs, it did not take into account the fact that the ark directory
had been renamed to rhel.  As a result, it did not delete the files in
redhat/configs/rhel/generic and the files it tried to delete in
redhat/configs/ark/generic were already deleted as they had moved to rhel.  As
a result of all of this, the configs in redhat/configs/rhel/generic are
overriding the configs in redhat/configs/common/generic, and this MR ended up
changing nothing in generated configs.
___
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] Fix up the RHEL configs for xtables and ipset

2023-04-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Fix up the RHEL configs for xtables and ipset

At some point, we had an MR which disabled xtables and ipset.  This put
configs to turn them off in the ark/generic directory.  Quite a bit
later, it was decided to rever that change, and MR2267 was created to do
that. In the meantime, the ark directory was renamed to rhel.  As that
MR was deleting files in ark, and they had already been deleted, this
was not raised as a problem, but we did not get the desired result
because those files do still exist in the rhel directory.  This change
simply deletes the files in the rhel/generic directory making things
work as MR2267 intended.

Link:https://gitlab.com/cki-project/kernel-ark/-/issues/105
Cc: Phil Sutter 
Signed-off-by: Justin M. Forbes 

diff --git a/redhat/configs/rhel/generic/CONFIG_BRIDGE_NETFILTER 
b/redhat/configs/rhel/generic/CONFIG_BRIDGE_NETFILTER
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_BRIDGE_NETFILTER
+++ /dev/null
@@ -1 +0,0 @@
-# CONFIG_BRIDGE_NETFILTER is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_BRIDGE_NF_EBTABLES 
b/redhat/configs/rhel/generic/CONFIG_BRIDGE_NF_EBTABLES
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_BRIDGE_NF_EBTABLES
+++ /dev/null
@@ -1,2 +0,0 @@
-# This CONFIG has been disabled in RHEL by RHEL Engineering. Please contact 
Red Hat Support for further assistance.
-# CONFIG_BRIDGE_NF_EBTABLES is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_IP6_NF_IPTABLES 
b/redhat/configs/rhel/generic/CONFIG_IP6_NF_IPTABLES
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_IP6_NF_IPTABLES
+++ /dev/null
@@ -1,2 +0,0 @@
-# This CONFIG has been disabled in RHEL by RHEL Engineering. Please contact 
Red Hat Support for further assistance.
-# CONFIG_IP6_NF_IPTABLES is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_IP_NF_ARPTABLES 
b/redhat/configs/rhel/generic/CONFIG_IP_NF_ARPTABLES
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_IP_NF_ARPTABLES
+++ /dev/null
@@ -1,2 +0,0 @@
-# This CONFIG has been disabled in RHEL by RHEL Engineering. Please contact 
Red Hat Support for further assistance.
-# CONFIG_IP_NF_ARPTABLES is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_IP_NF_IPTABLES 
b/redhat/configs/rhel/generic/CONFIG_IP_NF_IPTABLES
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_IP_NF_IPTABLES
+++ /dev/null
@@ -1,2 +0,0 @@
-# This CONFIG has been disabled in RHEL by RHEL Engineering. Please contact 
Red Hat Support for further assistance.
-# CONFIG_IP_NF_IPTABLES is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_IP_SET 
b/redhat/configs/rhel/generic/CONFIG_IP_SET
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_IP_SET
+++ /dev/null
@@ -1,2 +0,0 @@
-# This CONFIG has been disabled in RHEL by RHEL Engineering. Please contact 
Red Hat Support for further assistance.
-# CONFIG_IP_SET is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_NETFILTER_XTABLES 
b/redhat/configs/rhel/generic/CONFIG_NETFILTER_XTABLES
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_NETFILTER_XTABLES
+++ /dev/null
@@ -1 +0,0 @@
-# CONFIG_NETFILTER_XTABLES is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_NFT_COMPAT 
b/redhat/configs/rhel/generic/CONFIG_NFT_COMPAT
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_NFT_COMPAT
+++ /dev/null
@@ -1,2 +0,0 @@
-# This CONFIG has been disabled in RHEL by RHEL Engineering. Please contact 
Red Hat Support for further assistance.
-# CONFIG_NFT_COMPAT is not set
diff --git a/redhat/configs/rhel/generic/CONFIG_NF_CONNTRACK_LABELS 
b/redhat/configs/rhel/generic/CONFIG_NF_CONNTRACK_LABELS
deleted file mode 100644
index blahblah..blahblah 0
--- a/redhat/configs/rhel/generic/CONFIG_NF_CONNTRACK_LABELS
+++ /dev/null
@@ -1 +0,0 @@
-CONFIG_NF_CONNTRACK_LABELS=y

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2402
___
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 PATCHv2 0/0] Support querying tags as well as building stable rc and next releases

2023-04-17 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2275#note_1355858376

I wouldn't say "we don't care" they may not have anything to do with the
official builds generated out of this repository, but I do build and test with
stable RC kernels, and I am happy to see that cleaned up a bit here.
___
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] Revert "drm/ttm: Reduce the number of used allocation orders for TTM pages"

2023-04-26 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Revert "drm/ttm: Reduce the number of used allocation orders for TTM pages"

This reverts commit 322458c2bb1a0398c5775333e1e71e1ece8a461f.

diff --git a/drivers/gpu/drm/ttm/ttm_pool.c b/drivers/gpu/drm/ttm/ttm_pool.c
index blahblah..blahblah 100644
--- a/drivers/gpu/drm/ttm/ttm_pool.c
+++ b/drivers/gpu/drm/ttm/ttm_pool.c
@@ -47,11 +47,6 @@
 
 #include "ttm_module.h"
 
-#define TTM_MAX_ORDER (PMD_SHIFT - PAGE_SHIFT)
-#define __TTM_DIM_ORDER (TTM_MAX_ORDER + 1)
-/* Some architectures have a weird PMD_SHIFT */
-#define TTM_DIM_ORDER (__TTM_DIM_ORDER <= MAX_ORDER ? __TTM_DIM_ORDER : 
MAX_ORDER)
-
 /**
  * struct ttm_pool_dma - Helper object for coherent DMA mappings
  *
@@ -70,11 +65,11 @@ module_param(page_pool_size, ulong, 0644);
 
 static atomic_long_t allocated_pages;
 
-static struct ttm_pool_type global_write_combined[TTM_DIM_ORDER];
-static struct ttm_pool_type global_uncached[TTM_DIM_ORDER];
+static struct ttm_pool_type global_write_combined[MAX_ORDER];
+static struct ttm_pool_type global_uncached[MAX_ORDER];
 
-static struct ttm_pool_type global_dma32_write_combined[TTM_DIM_ORDER];
-static struct ttm_pool_type global_dma32_uncached[TTM_DIM_ORDER];
+static struct ttm_pool_type global_dma32_write_combined[MAX_ORDER];
+static struct ttm_pool_type global_dma32_uncached[MAX_ORDER];
 
 static spinlock_t shrinker_lock;
 static struct list_head shrinker_list;
@@ -449,7 +444,7 @@ int ttm_pool_alloc(struct ttm_pool *pool, struct ttm_tt *tt,
else
gfp_flags |= GFP_HIGHUSER;
 
-   for (order = min_t(unsigned int, TTM_MAX_ORDER, __fls(num_pages));
+   for (order = min_t(unsigned int, MAX_ORDER - 1, __fls(num_pages));
 num_pages;
 order = min_t(unsigned int, order, __fls(num_pages))) {
struct ttm_pool_type *pt;
@@ -568,7 +563,7 @@ void ttm_pool_init(struct ttm_pool *pool, struct device 
*dev,
 
if (use_dma_alloc) {
for (i = 0; i < TTM_NUM_CACHING_TYPES; ++i)
-   for (j = 0; j < TTM_DIM_ORDER; ++j)
+   for (j = 0; j < MAX_ORDER; ++j)
ttm_pool_type_init(&pool->caching[i].orders[j],
   pool, i, j);
}
@@ -588,7 +583,7 @@ void ttm_pool_fini(struct ttm_pool *pool)
 
if (pool->use_dma_alloc) {
for (i = 0; i < TTM_NUM_CACHING_TYPES; ++i)
-   for (j = 0; j < TTM_DIM_ORDER; ++j)
+   for (j = 0; j < MAX_ORDER; ++j)
ttm_pool_type_fini(&pool->caching[i].orders[j]);
}
 
@@ -642,7 +637,7 @@ static void ttm_pool_debugfs_header(struct seq_file *m)
unsigned int i;
 
seq_puts(m, "\t ");
-   for (i = 0; i < TTM_DIM_ORDER; ++i)
+   for (i = 0; i < MAX_ORDER; ++i)
seq_printf(m, " ---%2u---", i);
seq_puts(m, "\n");
 }
@@ -653,7 +648,7 @@ static void ttm_pool_debugfs_orders(struct ttm_pool_type 
*pt,
 {
unsigned int i;
 
-   for (i = 0; i < TTM_DIM_ORDER; ++i)
+   for (i = 0; i < MAX_ORDER; ++i)
seq_printf(m, " %8u", ttm_pool_type_count(&pt[i]));
seq_puts(m, "\n");
 }
@@ -756,16 +751,13 @@ int ttm_pool_mgr_init(unsigned long num_pages)
 {
unsigned int i;
 
-   BUILD_BUG_ON(TTM_DIM_ORDER > MAX_ORDER);
-   BUILD_BUG_ON(TTM_DIM_ORDER < 1);
-
if (!page_pool_size)
page_pool_size = num_pages;
 
spin_lock_init(&shrinker_lock);
INIT_LIST_HEAD(&shrinker_list);
 
-   for (i = 0; i < TTM_DIM_ORDER; ++i) {
+   for (i = 0; i < MAX_ORDER; ++i) {
ttm_pool_type_init(&global_write_combined[i], NULL,
   ttm_write_combined, i);
ttm_pool_type_init(&global_uncached[i], NULL, ttm_uncached, i);
@@ -798,7 +790,7 @@ void ttm_pool_mgr_fini(void)
 {
unsigned int i;
 
-   for (i = 0; i < TTM_DIM_ORDER; ++i) {
+   for (i = 0; i < MAX_ORDER; ++i) {
ttm_pool_type_fini(&global_write_combined[i]);
ttm_pool_type_fini(&global_uncached[i]);
 

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2421
___
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 2/2] Add handshake-test to mod-intenal.list

2023-04-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Add handshake-test to mod-intenal.list

Another kunit test for the 6.4 window.

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/scripts/mod/mod-internal.list 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -25,6 +25,7 @@ ext4-inode-test
 fat_test
 fortify_kunit
 gss_krb5_test
+handshake-test
 hashtable_test
 iio-test-format
 iio-test-rescale

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2419
___
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 0/2] Add regmap-kunit to mod-internal.list

2023-04-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/2419

As with any new kunit test, it has to be added to mod-internal.list or we fail
with depmod issues.

Signed-off-by: Justin M. Forbes 

---
 redhat/scripts/mod/mod-internal.list |  2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
___
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 1/2] Add regmap-kunit to mod-internal.list

2023-04-27 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes 

Add regmap-kunit to mod-internal.list

Signed-off-by: Justin M. Forbes 

diff --git a/redhat/scripts/mod/mod-internal.list 
b/redhat/scripts/mod/mod-internal.list
index blahblah..blahblah 100644
--- a/redhat/scripts/mod/mod-internal.list
+++ b/redhat/scripts/mod/mod-internal.list
@@ -48,6 +48,7 @@ rational-test
 rcuscale
 rcutorture
 refscale
+regmap-kunit
 resource_kunit
 rocker
 scftorture

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2419
___
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


<    1   2   3   4   5   6   >