Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690565580 Okay based on that I'm resolving the thread. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690503573 Acked-by: Herton R. Krzesinski (via approve button) ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an

Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690502948 So the question here is if when we do the kernel-ark sync to centos/rhel dist- git next time, will we set INCLUDE_FEDORA_FILES to 0? I believe not, so from that use case we are

Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690495733 No, because there is a window in the future where we will use again kernel-ark to sync to dist-git (in next centos stream/rhel major version), and then we will still have fedora

Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690443819 Good point @hertonrk-rh. My plan is to remove the fedora_files, configs/fedora, priority.fedora, and update flavors in RHEL & centos-stream. Would that solve your concerns?

Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690416057 I think this need an additional change on top: If INCLUDE_FEDORA_FILES is disabled, you don't need to copy ```fedora_files/*```. The same with INCLUDE_RHEL_FILES. It is harmless

Re: [OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413#note_690382344 Fixed :) ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org

[OS-BUILD PATCH 5/5] redhat/kernel.spec.template: Cleanup source numbering

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/kernel.spec.template: Cleanup source numbering Cleanup the numbering of the source files. Signed-off-by: Prarit Bhargava diff --git a/redhat/kernel.spec.template b/redhat/kernel.spec.template index blahblah..blahblah 100755 --- a/redhat/kernel.spec.template +++

[OS-BUILD PATCH 3/5] redhat/kernel.spec.template: Add include_fedora and include_rhel variables

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/kernel.spec.template: Add include_fedora and include_rhel variables The centos-stream-9 (c9s) [1] repository contains files for both RHEL and Fedora configs. The Fedora configs will never be updated, and will quickly be out-of-date with respect to upstream. Having

[OS-BUILD PATCH 4/5] redhat/kernel.spec.template: Reorganize RHEL and Fedora specific files

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/kernel.spec.template: Reorganize RHEL and Fedora specific files Reorganize the RHEL and Fedora specific files into a single section. Signed-off-by: Prarit Bhargava diff --git a/redhat/kernel.spec.template b/redhat/kernel.spec.template index blahblah..blahblah

[OS-BUILD PATCH 1/5] redhat/Makefile: Use flavors file

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/Makefile: Use flavors file Use the configs/flavors file instead of hardcoding directory copies. Signed-off-by: Prarit Bhargava diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah 100644 --- a/redhat/Makefile +++ b/redhat/Makefile @@ -254,8

[OS-BUILD PATCH 0/5] configs: Add include_rhel and include_fedora to kernel.spec

2021-09-29 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1413 The centos-stream-9 (c9s) [1] repository contains files for both RHEL and Fedora configs. The Fedora configs will never be updated, and will quickly be out-of-date with respect to

Re: [OS-BUILD PATCH] redhat/configs: Remove CONFIG_INFINIBAND_I40IW

2021-09-29 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1354#note_690337197 We supposedly have a script that seeks to confirm settings and merge into common where appropriate. It hasn't been run in a long time, I need to do that soon.

Re: [OS-BUILD PATCH] redhat/configs: Remove CONFIG_INFINIBAND_I40IW

2021-09-29 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1354#note_690318301 Approved and merged by Justin. BTW, redhat/configs/fedora/generic/CONFIG_INFINIBAND_IRDMA and redhat/configs/fedora/generic/CONFIG_INFINIBAND_IRDMA set the same value, so I

Re: [OS-BUILD PATCH] redhat/configs: Remove CONFIG_INFINIBAND_I40IW

2021-09-29 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1354#note_690317171 Acked-by: Herton R. Krzesinski (via approve button) ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an

Re: [OS-BUILD PATCH] redhat/configs: Remove CONFIG_INFINIBAND_I40IW

2021-09-29 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1354#note_690086631 It needs an ack from someone on the RHEL side (my ack is for Fedora) ___ kernel mailing list -- kernel@lists.fedoraproject.org To

Re: [OS-BUILD PATCH] redhat/configs: Remove CONFIG_INFINIBAND_I40IW

2021-09-29 Thread Kamal Heib (via Email Bridge)
From: Kamal Heib on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1354#note_689908371 Hi @hertonrk-rh, Could you please tell what is blocking this MR from getting merged? Thanks, Kamal ___ kernel mailing list --

Re: [OS-BUILD PATCH] redhat/configs: Enable Hyper-V guests on ARM64

2021-09-29 Thread Vitaly Kuznetsov (via Email Bridge)
From: Vitaly Kuznetsov on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1394#note_689896007 Sorry, I didn't actually have any Hyper-V/ARM hardware myself, was mostly relying on Microsoft reporting success from testing upstream kernel internally, not sure about the exact