Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-06-08 Thread Daniel Horak (via Email Bridge)
From: Daniel Horak on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_595305892

The move to z14 is an ARK specific change, so it's safe from the Fedora point
of view. I want to change the Fedora baseline (to z13) in a separate step.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-06-08 Thread Cornelia Huck (via Email Bridge)
From: Cornelia Huck on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_595267298

Unless I'm confused, Fedora stays at the z12 level. If ARK is relying on the
Fedora infrastructure, we have a problem.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-06-08 Thread Florian Weimer (via Email Bridge)
From: Florian Weimer on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_595258853

As far as I know, Fedora infrastructure is not ready for this because it's
still z13. Is this going to a problem for ARK or Fedora? Thanks.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-06-08 Thread Cornelia Huck (via Email Bridge)
From: Cornelia Huck on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_595246522

Acked-by: Cornelia Huck 
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-06-08 Thread Cornelia Huck (via Email Bridge)
From: Cornelia Huck on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_595245657

Looking at the s390-related parts only, it seems good to me. The other changes
also seem reasonable, but it is not my area of expertise.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-06-08 Thread Patrick Talbert (via Email Bridge)
From: Patrick Talbert on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_595204126

What is the status here? Is this good as-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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Cornelia Huck (via Email Bridge)
From: Cornelia Huck on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585732819

FWIW, z13 is currently the highest architecture level that QEMU can emulate,
and I don't think Fedora should target a level higher than that without some
really pressing need. (z14 support for QEMU tcg is in the works.)
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Daniel Horak (via Email Bridge)
From: Daniel Horak on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585721963

Fedora can move to z13 as it's the lowest hardware in the infrastructure,
which can't be easily changed. But for the change from the current zEC12 to
z13 we should follow the processes (a formal Change, touches multiple
packages). Thus I would leave changing the Fedora options for a later time.

For the correct use of the common/ark/fedora dirs I leave it on the kernel
maintainers :-)
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Justin Forbes (via Email Bridge)
From: Justin Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585698139

> If I need to push an other version I can add the Fedora update as well. This
MR already got way bigger than I expected. So I don't see a problem in adding
one more commit.

Either way, if you do it in this request, I am happy to take it. If it goes in
as a separate MR, fedora specific changes do not require the same ack
structure. I review/ack/merge, though with s390x I tend to take whatever you
folks recommend.

> My understanding so far was that common/ should contain configs that are
shared between ark and fedora and not that common/ contains the defaults which
can be overwritten in ark/ and fedora/.

This is indeed the case in theory, if an entry is in common, it should ideally
be the same, with no corresponding entry in fedora or ark directory.  In
practice, the scripts tend to generate a bit differently.  The intention is to
run a clean up script on a regular interval which consolidates entries in
fedora and ark if they are the same, and removes common if they are different.

For this particular case I don't think an entry is necessary in common if they
are going to be different, but I do have to ask if we really need them to be
different. Does Fedora need to support older hardware here?  I am fine with
either answer, I really don't know what hardware people are running Fedora on
for s390x.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Philipp Rudo (via Email Bridge)
From: Philipp Rudo on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585640564

Hi Dan,

1. If I need to push an other version I can add the Fedora update as well.
This MR already got way bigger than I expected. So I don't see a problem in
adding one more commit.

2. I did a quick test and this should indeed work. However I'm not sure if
it's the intended behavior. My understanding so far was that common/ should
contain configs that are shared between ark and fedora and not that common/
contains the defaults which can be overwritten in ark/ and fedora/. Anyway, if
the maintainers agree with your suggestion I'm happy to change 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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585515979

Yes please.  Just indicate those config changes are the result of dummy-tool.
Right now it looks like a copy-n-paste error from the previous commit, so I
can't tell if it was intentional (which in this case it is) or accident.  I
just wanted some extra wording to indicate the copy-n-pasted changelog was on
purpose.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Daniel Horak (via Email Bridge)
From: Daniel Horak on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585355200

@prudo1 - I have couple comments
- the Fedora config has been switched to CONFIG_TUNE_DEFAULT while it should
be CONFIG_TUNE_Z13 (to "match" the CONFIG_MARCH_ZEC12). But looking further
this change happened even before your MR. Lets fix it later (and move to
CONFIG_MARCH_Z13).
- could the `common/generic/s390x` directory contain the "not set" entries for
all MARCH/TUNE options and then let ark and fedora override only their
specific settings? Right now it's missing CONFIG_MARCH_ZEC12 and
CONFIG_MARCH_Z14, adding a slight inconsistency.
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Philipp Rudo (via Email Bridge)
From: Philipp Rudo on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585239426

best choice would be @dhorak1, @cohuck or @thuth
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-26 Thread Philipp Rudo (via Email Bridge)
From: Philipp Rudo on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_585224768

@dzickusrh yes, that's a fallout from using the dummy-tools. These two configs
only work with clang and thus are usually hidden. Plus they are part of a
choice from which CONFIG_INIT_STACK_NONE is set. Do you want me to improve the
wording in the commit message?
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-25 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_584913650

@sfbest - who can review s390x configs changes?
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-25 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_584912176

@prudo1 - nitpick.  the second commit changelog seems off, "configs/common:
disable CONFIG_INIT_STACK_ALL_{PATTERN,ZERO}".  It doesn't really describe the
change.  Is that fallout from using dummy-tools?
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCHv2 0/5] ark: Set Architecture Level Set to Z14 for s390

2021-05-24 Thread Justin Forbes (via Email Bridge)
From: Justin Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1144#note_583699147

Acked-by: Justin Forbes 
(via approve button)
___
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 on the list, report it: 
https://pagure.io/fedora-infrastructure