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

Jump to bottom

IsolateMirrorBusyTest: error: expected: function.HasOptimizedCode() #2129

Closed
turnidge opened this issue Mar 13, 2012 · 4 comments
Closed

IsolateMirrorBusyTest: error: expected: function.HasOptimizedCode() #2129

turnidge opened this issue Mar 13, 2012 · 4 comments
Assignees
@turnidge
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-obsolete Closed as the reported issue is no longer relevant

Comments

@turnidge
Copy link
Contributor

My new test seems to be causing problems on the x64 build. I will disable it in runtime/tests/vm_dart.status.

FAILED: vm debug_x64 vm/IsolateMirrorBusyTest
Expected: pass
Actual: crash

stderr:
../runtime/vm/compiler.cc:219: error: expected: function.HasOptimizedCode()
@madsager
Copy link
Contributor

Added Triaged label.

@ghost
Copy link

ghost commented Mar 13, 2012

Crashing now like:

'bootstrap_impl': Error: line 3399 pos 66: circular dependency while initializing static field '_DELETED_KEY@367218ca'
    if ((_keys[index] === null) || (_keys[index] === _DELETED_KEY)) {

cc @iposva-google.
Set owner to @turnidge.
Added Accepted label.

@iposva-google
Copy link
Contributor

Todd, the problem Srdjan ran into is likely due to the fact that the spawned isolate is interrupted during initialization of a static field (in this case DELETED_KEY). The isolate handling code then tries to use that field as well and fails because it reads the "being initialized" sentinel value.

@turnidge
Copy link
Contributor Author

This test has been deleted. Once we are ready to implement remote isolatemirrors we will need to figure out how to deal with this.

Added AssumedStale label.

@turnidge turnidge added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-obsolete Closed as the reported issue is no longer relevant labels May 24, 2012
@turnidge turnidge self-assigned this May 24, 2012
copybara-service bot pushed a commit that referenced this issue Jun 5, 2023
@devoncarew
[deps] rev async, dartdoc, ffi, http, leak_tracker, lints, mockito, t…
03b2abf 
…est, tools, webdev

Revisions updated by `dart tools/rev_sdk_deps.dart`.

async (https://github.com/dart-lang/async/compare/96c29d0..a506993):
  a506993  2023-06-01  Kevin Moore  Update testing SDK in CI (#245)
  9be3fc0  2023-06-01  Kevin Moore  Require Dart 2.19, use latest team lints (#244)

dartdoc (https://github.com/dart-lang/dartdoc/compare/39fe1a8..0c8feac):
  0c8feac6  2023-06-04  Sam Rawlins  Sort enum_test.dart (#3428)

ffi (https://github.com/dart-lang/ffi/compare/7f4acbd..f582ca0):
  f582ca0  2023-05-17  Daco Harkes  Rename `master` branch to `main` (#197)
  604451d  2023-05-16  Devon Carew  blast_repo fixes (#195)

http (https://github.com/dart-lang/http/compare/8834aec..5312366):
  5312366  2023-06-02  Brian Quinlan  Reland "support the nsurl session web socket api" (#950)

leak_tracker (https://github.com/dart-lang/leak_tracker/compare/cbbdeca..f17da61):
  f17da61  2023-06-02  Polina Cherkasova  Add constructor for retaining path. (#72)
  b70e538  2023-06-01  Polina Cherkasova  Fix connection issue. (#70)
  a80f253  2023-06-01  dependabot[bot]  Bump actions/checkout from 3.3.0 to 3.5.2 (#71)

lints (https://github.com/dart-lang/lints/compare/edc28ed..fc74ce0):
  fc74ce0  2023-06-02  Devon Carew  update the readme to clarify the package's goals (#130)
  4e4c18f  2023-06-02  Devon Carew  mv analysis_options file (#129)

mockito (https://github.com/dart-lang/mockito/compare/924f65c..cca4858):
  cca4858  2023-06-02  Ilya Yanok  Fix the unresolved types if used as ignored type-alias arguments
  8652886  2023-06-01  Googler  Prepare for NamedType breaking change in the analyzer.

test (https://github.com/dart-lang/test/compare/3276921..7a6c98d):
  7a6c98d0  2023-06-01  Konstantin Scheglov  Require analyzer 5.12.0, refactor InstanceCreationExpression type name extraction. (#2015)
  23bd4159  2023-06-01  Nate Bosch  Use switch expressions for switch/return pattern (#2027)
  06bdbb65  2023-06-01  Nate Bosch  Make State and Result enums (#2028)
  11805dc5  2023-06-01  Nate Bosch  Migrate to Dart 3 (#2024)

tools (https://github.com/dart-lang/tools/compare/389925f..8d6e8b8):
  8d6e8b8  2023-06-01  Kevin Moore  unified_analytics and graphs: cleanup lints, bump pkg deps (#108)

webdev (https://github.com/dart-lang/webdev/compare/f565d7f..b10d62b):
  b10d62b8  2023-06-02  Anna Gringauze  Support using scope in evaluateInFrame (#2122)
  c0300ce6  2023-06-02  Elliott Brooks  Included requested-by header in `ProxyServerAssetReader` (#2129)
  57699563  2023-06-01  dependabot[bot]  Bump actions/labeler from 4.0.3 to 4.0.4 (#2128)

Change-Id: Ia6114a0cbbe789f13e2c42399f53ff0e4d29d74a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/307502
Auto-Submit: Devon Carew <devoncarew@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
This issue was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@turnidge turnidge

Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-obsolete Closed as the reported issue is no longer relevant
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@madsager @turnidge @iposva-google