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

doc: clarify arch support for power platforms #12679

Closed
wants to merge 2 commits into from

Conversation

@mhdawson
Copy link
Member

commented Apr 26, 2017

Add specifics as to the level of hardware supported for
power platforms. Power 8 is the first processor level to
support ppc little endian. Set the minimum level
for AIX to Power7 for Node version 8 and later.
This will allow the potential to leverage new instructions
and optimizations going forward. We have spoken to the
AIX team and they agree this makes sense. Earlier processor
levels will continue to be supported for version 4.x and 6.x
on AIX.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows [commit guidelines]
Affected core subsystem(s)

doc

doc: clarify arch support for power platforms
Add specifics as to the level of hardware supported for
power platforms.  Power 8 is the first processor level to
support ppc little endian.  Set the minimum level
for AIX to Power7 for Node version 8 and later.
This will allow the potential to leverage new instructions
and optimizations going forward.  We have spoken to the
AIX team and they agree this makes sense. Earlier processor
levels will continue to be supported for version 4.x and 6.x
on AIX.

@mhdawson mhdawson requested a review from gibfahn Apr 26, 2017

BUILDING.md Outdated
| GNU/Linux | Tier 2 | kernel >= 3.13.0, glibc >= 2.19 | ppc64le | |
| AIX | Tier 2 | >= 7.1 TL04 | ppc64be | |
| GNU/Linux | Tier 2 | kernel >= 3.13.0, glibc >= 2.19 | ppc64le, >=power8 | |
| AIX | Tier 2 | >= 7.1 TL04 | ppc64be, >=power7 | |

This comment has been minimized.

Copy link
@gibfahn

gibfahn Apr 27, 2017

Member

Nit: can you remove the commas, elsewhere they seem to be used as or (x64, arm = x64 or arm, ppc64le >=power8 means ppc64le and power8)?

ppc64le, >=power8 -> ppc64le >=power8
ppc64be, >=power7-> ppc64be >=power7

@mhdawson

This comment has been minimized.

Copy link
Member Author

commented Apr 28, 2017

@gibfahn pushed change to address comments.

@gibfahn

This comment has been minimized.

Copy link
Member

commented Apr 28, 2017

Landed in 71911be

@gibfahn gibfahn closed this Apr 28, 2017

gibfahn added a commit that referenced this pull request Apr 28, 2017
doc: clarify arch support for power platforms
Add specifics as to the level of hardware supported for
power platforms.  Power 8 is the first processor level to
support ppc little endian.  Set the minimum level
for AIX to Power7 for Node version 8 and later.
This will allow the potential to leverage new instructions
and optimizations going forward.  We have spoken to the
AIX team and they agree this makes sense. Earlier processor
levels will continue to be supported for version 4.x and 6.x
on AIX.

PR-URL: #12679
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@jasnell jasnell referenced this pull request May 11, 2017
@gibfahn gibfahn referenced this pull request Jun 15, 2017
2 of 3 tasks complete

@mhdawson mhdawson deleted the mhdawson:addAIXarch branch Jun 28, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.