Skip to content
New issue

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

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

Already on GitHub? # to your account

"lotus state compute-state --html" Broken after fvm upgrade #8636

Closed
8 of 18 tasks
Tracked by #531
geoff-vball opened this issue May 10, 2022 · 4 comments
Closed
8 of 18 tasks
Tracked by #531

"lotus state compute-state --html" Broken after fvm upgrade #8636

geoff-vball opened this issue May 10, 2022 · 4 comments
Assignees
Labels
area/fvm kind/bug Kind: Bug need/team-input Hint: Needs Team Input

Comments

@geoff-vball
Copy link
Contributor

Checklist

  • This is not a security-related bug/issue. If it is, please follow please follow the security policy.
  • This is not a question or a support request. If you have any lotus related questions, please ask in the lotus forum.
  • This is not a new feature request. If it is, please file a feature request instead.
  • This is not an enhancement request. If it is, please file a improvement suggestion instead.
  • I have searched on the issue tracker and the lotus forum, and there is no existing related issue or discussion.
  • I am running the Latest release, or the most recent RC(release canadiate) for the upcoming release or the dev branch(master), or have an issue updating to any of these.
  • I did not make any code changes to lotus.

Lotus component

  • lotus daemon - chain sync
  • lotus miner - mining and block production
  • lotus miner/worker - sealing
  • lotus miner - proving(WindowPoSt)
  • lotus miner/market - storage deal
  • lotus miner/market - retrieval deal
  • lotus miner/market - data transfer
  • lotus client
  • lotus JSON-RPC API
  • lotus message management (mpool)
  • Other

Lotus Version

Daemon:  1.15.2-dev+interopnet+git.0295d2274.dirty+api1.5.0
Local: lotus version 1.15.2-dev+interopnet+git.0295d2274.dirty

Describe the Bug

When running lotus state compute-state --html, the output contains the error ERROR: template: message:3:24: executing "message" at <.Msg.To>: nil pointer evaluating *types.Message.To.

This error only occurs when looking at the state after the FVM upgrade (3000 on interopnet).

Logging Information

.

Repo Steps

lotus state compute-state --html

@jennijuju
Copy link
Member

jennijuju commented May 23, 2022

the message cannot be interpreted is from system actor to reward actor , seems like the getCode made be a bit off in

var compStateMsg = `

@Stebalien
Copy link
Member

So, the issue is likely that execution tracing wasn't enabled for this message. Due to how execution tracing currently works in the FVM, we'll end up with an "empty" message.

The lotus VM would always include the root message, even when tracing was disabled.

@jennijuju
Copy link
Member

@arajasek given what steb said - should we expect the issue shouldn't exist when setting LOTUS_VM_ENABLE_GAS_TRACING_VERY_SLOW to be 1?

@arajasek
Copy link
Contributor

@jennijuju No, we expect the issue to exist in all cases (before this fix) -- the problem is that we were only trying to set the execution trace if we had an FVM execution trace (which we did when we had LOTUS_VM_ENABLE_GAS_TRACING_VERY_SLOW=1).

So in the LOTUS_VM_ENABLE_GAS_TRACING_VERY_SLOW=0 case, we would basically just set the execution trace to be nil (because et would be nil on this line https://github.com/filecoin-project/lotus/pull/8706/files#diff-9c51c6db9b9f6bea9717d162a7f80db6e0e144032849d83281750e3226d17ccaL359).

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/fvm kind/bug Kind: Bug need/team-input Hint: Needs Team Input
Projects
None yet
Development

No branches or pull requests

6 participants