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

test-tick-processor failures on PPC #2957

Closed
mhdawson opened this issue Sep 18, 2015 · 6 comments
Closed

test-tick-processor failures on PPC #2957

mhdawson opened this issue Sep 18, 2015 · 6 comments
Labels
v8 engine Issues and PRs related to the V8 dependency.

Comments

@mhdawson
Copy link
Member

The test-tick-process test fails on PPC

https://ci.nodejs.org/job/iojs+pr+ppc/nodes=ppcbe-fedora20/18/console

We have debugged the problem to the point were we believe the issue is due to PPC 64 BE using function descriptors (as does AIX 64 and 32 were we see the same problem). The issue seems to be on the V8 side where it only adds symbols in address ranges marked executable which is not the case for function descriptors. If we remove that check the test passes. We are still looking at how best to address the issue.

@mhdawson
Copy link
Member Author

mhdawson commented Oct 6, 2015

We are still looking at this one and trying out some alternatives for how to fix on the v8 side

@mhdawson mhdawson added v8 engine Issues and PRs related to the V8 dependency. and removed test Issues and PRs related to the tests. labels Oct 6, 2015
@mhdawson
Copy link
Member Author

We are still working to put together change that resolves this

@mhdawson
Copy link
Member Author

FYI, put this in place while we continue to investigate: #3491

@mhdawson
Copy link
Member Author

Latest update:

part of the fix in via https://chromium.googlesource.com/v8/v8.git/+/2bd5914bc3ae8c01c115f0386ae91c9f341c7ccb

Still remaining is a modification to the post processing script to look at the .opd section when gathering symbolic information

@mhdawson
Copy link
Member Author

mhdawson commented Feb 9, 2016

More complete disable for AIX, noting to make sure we remember to revert once fix is in
#5093

@Trott
Copy link
Member

Trott commented Sep 23, 2016

Closing as I think #5093 addressed this more-or-less, test-tick-processor has been broken into three different test files now, and remaining flakiness is being explored in #8725. (Re-open or comment if you think this shouldn't be closed.)

@Trott Trott closed this as completed Sep 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Development

No branches or pull requests

2 participants