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.

Sign up for GitHub

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

Jump to bottom

FEVM | Eth Compliance Test | Hit with stack overflow error #1428

Closed
shamb0 opened this issue Jan 11, 2023 · 3 comments
Closed

FEVM | Eth Compliance Test | Hit with stack overflow error #1428

shamb0 opened this issue Jan 11, 2023 · 3 comments
Labels
Ethereum Compliance Kind: Bug Something isn't working

Comments

@shamb0
Copy link

shamb0 commented Jan 11, 2023

Hit with error "thread '' has overflowed its stack".,
when executing below test suites. Root cause needs to be analysed.

Skip ID Test Suites
ID-0014 stCallCodes/callcallcall_ABCB_RECURSIVE.json
ID-0019 stCallCreateCallCodeTest/Call1024OOG.json
ID-0020 stCallCreateCallCodeTest/Call1024PreCalls.json
ID-0021 stCallCreateCallCodeTest/CallLoseGasOOG.json
ID-0037 stCallDelegateCodesCallCodeHomestead/callcodecallcodecallcode_ABCB_RECURSIVE.json
ID-0038 stCallDelegateCodesHomestead/callcallcallcode_ABCB_RECURSIVE.json
ID-0041 stCallDelegateCodesHomestead/callcallcodecall_ABCB_RECURSIVE.json
ID-0044 stCallDelegateCodesHomestead/callcallcodecallcode_ABCB_RECURSIVE.json
ID-0048 stCallDelegateCodesHomestead/callcodecallcall_100_SuicideEnd.json
ID-0049 stCallDelegateCodesHomestead/callcodecallcallcode_101_SuicideEnd.json
ID-0051 stCallDelegateCodesHomestead/callcodecallcallcode_ABCB_RECURSIVE.json
ID-0052 stCallDelegateCodesHomestead/callcodecallcodecall_ABCB_RECURSIVE.json
@shamb0
Copy link
Author

shamb0 commented Jan 11, 2023

CC @maciejwitowski , @mriise

@shamb0 shamb0 mentioned this issue Jan 11, 2023
Closed
@maciejwitowski maciejwitowski transferred this issue from filecoin-project/builtin-actors Jan 11, 2023
@raulk
Copy link
Member

raulk commented Jan 11, 2023

This is similar to #1427. Stack sizes are bounded by the Wasm environment, and because the compliance tests run outside of Wasm, they fail in a seemingly uncontrolled manner.

@shamb0
Copy link
Author

shamb0 commented Jan 12, 2023

Replaced with ref-fvm Issue ID #1437

@shamb0 shamb0 closed this as completed Jan 12, 2023
@shamb0 shamb0 mentioned this issue Jan 12, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Ethereum Compliance Kind: Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@raulk @maciejwitowski @shamb0