Skip to content

Recognize POWER11 on Linux - #7528

Merged
babsingh merged 1 commit into
eclipse-omr:masterfrom
zl-wang:master
Nov 7, 2024
Merged

Recognize POWER11 on Linux#7528
babsingh merged 1 commit into
eclipse-omr:masterfrom
zl-wang:master

Conversation

@zl-wang

@zl-wang zl-wang commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

"power11" patch for Linux kernel was upstreamed a couple months ago. the
recognition mechanism involves comparing it with AT_PLATFORM.

@zl-wang
zl-wang requested a review from babsingh as a code owner November 6, 2024 13:13
@zl-wang

zl-wang commented Nov 6, 2024

Copy link
Copy Markdown
Contributor Author

@babsingh please review/approve/merge

@babsingh babsingh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • The code LGTM.
  • Is AT_PLATFROM -> AT_PLATFORM a typo in the PR description and commit message?
  • I will also recommend to avoid first person: we. Here is a rewrite: The recognition mechanism involves comparing it with AT_PLATFORM.

"power11" patch for Linux kernel was upstreamed a couple months ago. the
recognition mechanism involves comparing it with AT_PLATFORM.

Signed-off-by: Julian <zlwang@ca.ibm.com>
@babsingh

babsingh commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

jenkins build all

@babsingh

babsingh commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

jenkins build plinux,zlinux

2 similar comments
@AdamBrousseau

Copy link
Copy Markdown
Contributor

jenkins build plinux,zlinux

@babsingh

babsingh commented Nov 7, 2024

Copy link
Copy Markdown
Contributor

jenkins build plinux,zlinux

@babsingh babsingh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The relevant PPC builds have passed. No need to wait for the zLinux build since it is not impacted by these changes. An infra issue is preventing zLinux builds to run.

@babsingh
babsingh merged commit a34c903 into eclipse-omr:master Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants