Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add extension for 64k pages on aarch64 #1351

Merged
merged 2 commits into from
Sep 11, 2023

Conversation

jbtrystram
Copy link
Contributor

@jbtrystram jbtrystram commented Sep 1, 2023

@openshift-ci openshift-ci bot requested review from jlebon and yasminvalim September 1, 2023 13:29
@openshift-ci openshift-ci bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Sep 1, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 1, 2023

Hi @jbtrystram. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

extensions-c9s.yaml Outdated Show resolved Hide resolved
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 1, 2023
@travier
Copy link
Member

travier commented Sep 4, 2023

Let's link to the COS epic (https://issues.redhat.com/browse/COS-2402) in the title and commit instead of the OCPSTRAT card.

@travier
Copy link
Member

travier commented Sep 4, 2023

Can you update the first comment here as well? It's not updated when you push new commits unfortunately :/.

@travier
Copy link
Member

travier commented Sep 4, 2023

For kernel-rt we have the following subpackages:

      - kernel-rt-core
      - kernel-rt-kvm
      - kernel-rt-modules
      - kernel-rt-modules-extra
      - kernel-rt-devel

Don't we have them as well for kernel-64k?

@travier
Copy link
Member

travier commented Sep 4, 2023

Looks like we do:

[   ] [kernel-64k-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-5.14.0-362.el9.aarch64.rpm)                               2023-08-29 09:37  5.1M  
[   ] [kernel-64k-core-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-core-5.14.0-362.el9.aarch64.rpm)                          2023-08-29 09:37   18M  
[   ] [kernel-64k-debug-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-debug-5.14.0-362.el9.aarch64.rpm)                         2023-08-29 09:37  5.1M  
[   ] [kernel-64k-debug-core-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-debug-core-5.14.0-362.el9.aarch64.rpm)                    2023-08-29 09:37   30M  
[   ] [kernel-64k-debug-modules-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-debug-modules-5.14.0-362.el9.aarch64.rpm)                 2023-08-29 09:37   38M  
[   ] [kernel-64k-debug-modules-core-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-debug-modules-core-5.14.0-362.el9.aarch64.rpm)            2023-08-29 09:37   39M  
[   ] [kernel-64k-debug-modules-extra-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-debug-modules-extra-5.14.0-362.el9.aarch64.rpm)           2023-08-29 09:37  5.7M  
[   ] [kernel-64k-modules-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-modules-5.14.0-362.el9.aarch64.rpm)                       2023-08-29 09:37   23M  
[   ] [kernel-64k-modules-core-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-modules-core-5.14.0-362.el9.aarch64.rpm)                  2023-08-29 09:37   26M  
[   ] [kernel-64k-modules-extra-5.14.0-362.el9.aarch64.rpm](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/kernel-64k-modules-extra-5.14.0-362.el9.aarch64.rpm)                 2023-08-29 09:37  5.4M

@travier
Copy link
Member

travier commented Sep 4, 2023

We'll need something like coreos/rpm-ostree#4386 as well for rpm-ostree

@jbtrystram
Copy link
Contributor Author

jbtrystram commented Sep 4, 2023

Looks like we do:

* [gitlab.com/redhat/centos-stream/rpms/kernel/-/blob/c9s/kernel.spec?ref_type=heads#L1560](https://gitlab.com/redhat/centos-stream/rpms/kernel/-/blob/c9s/kernel.spec?ref_type=heads#L1560)

* [composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages](https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/BaseOS/aarch64/os/Packages/)

Thanks for the pointers ! I am updating this.

- kernel-64k-core
- kernel-64k-modules
- kernel-64k-modules-core
- kernel-64k-modules-extra
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: You need an empty line at the end of a file. Make sure to configure your editor to do it for you. In some editors, this empty line is not shown, in some it is.

@jbtrystram jbtrystram force-pushed the arm-64pages branch 2 times, most recently from 52e8eaf to e82bf76 Compare September 5, 2023 11:42
@cgwalters
Copy link
Member

/ok-to-test

@openshift-ci openshift-ci bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Sep 6, 2023
@cgwalters
Copy link
Member

We'll need something like coreos/rpm-ostree#4386 as well for rpm-ostree

Ugh. Yes. And we'll need to get that into 4.14.

cgwalters added a commit to cgwalters/rpm-ostree that referenced this pull request Sep 6, 2023
@cgwalters
Copy link
Member

coreos/rpm-ostree#4583

@jbtrystram
Copy link
Contributor Author

/retest

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sane to me!

@cgwalters
Copy link
Member

/lgtm
But this will still need a new rpm-ostree release in practice it sounds like; hopefully soon

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 11, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 11, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, jbtrystram, jlebon

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [cgwalters,jbtrystram,jlebon]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 11, 2023

@jbtrystram: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot merged commit 85a77ed into openshift:master Sep 11, 2023
jbtrystram added a commit to jbtrystram/openshift-machine-config-operator that referenced this pull request Sep 14, 2023
Since RHEL 9 reverted to 4K memory pages for aarch64, add a way to
switch to a hugepage kernel.
The MachineConfig should contain the following to trigger the kernel
switch:
spec:
  kernelType: 64k-pages

This is exclusive with the `realtime` kernel option.

xref https://issues.redhat.com/browse/COS-2402
This requires openshift/os#1351

Signed-off-by: jbtrystram <[email protected]>
jbtrystram added a commit to jbtrystram/openshift-machine-config-operator that referenced this pull request Oct 20, 2023
Since RHEL 9 reverted to 4K memory pages for aarch64, add a way to
switch to a hugepage kernel.
The MachineConfig should contain the following to trigger the kernel
switch:
spec:
  kernelType: 64k-pages

This is exclusive with the `realtime` kernel option.

xref https://issues.redhat.com/browse/COS-2402
This requires openshift/os#1351

Signed-off-by: jbtrystram <[email protected]>
inesqyx pushed a commit to inesqyx/machine-config-operator that referenced this pull request Oct 23, 2023
Since RHEL 9 reverted to 4K memory pages for aarch64, add a way to
switch to a hugepage kernel.
The MachineConfig should contain the following to trigger the kernel
switch:
spec:
  kernelType: 64k-pages

This is exclusive with the `realtime` kernel option.

xref https://issues.redhat.com/browse/COS-2402
This requires openshift/os#1351

Signed-off-by: jbtrystram <[email protected]>
@jbtrystram jbtrystram deleted the arm-64pages branch February 27, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants