Skip to content

TSAN DartAPI_New failure #46126

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

Closed
dcharkes opened this issue May 25, 2021 · 1 comment
Closed

TSAN DartAPI_New failure #46126

dcharkes opened this issue May 25, 2021 · 1 comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. gardening

Comments

@dcharkes
Copy link
Contributor

https://dart-ci.appspot.com/log/vm-kernel-tsan-linux-release-x64/dartk-tsan-linux-release-x64/1665/vm/cc/DartAPI_New

--- Command "run_vm_unittest" (took 10.000242s):
DART_CONFIGURATION=ReleaseTSANX64 out/ReleaseTSANX64/run_vm_tests --dfe=/b/s/w/ir/out/ReleaseTSANX64/gen/kernel-service.dart.snapshot --ignore-unrecognized-flags DartAPI_New

exit code:
255

stdout:
Running test: DartAPI_New
Done: DartAPI_New

stderr:
../../runtime/vm/dart_api_impl_test.cc: 5666: error: expected 'type' to be a valid handle but found an error handle:
    'Compilation failed /test-lib:15:35: Error: The constructor function type 'MyClass Function(dynamic)' isn't a subtype of 'MyExtraHop Function(dynamic)'.
 - 'MyClass' is from '/test-lib'.
 - 'MyExtraHop' is from '/test-lib'.
  factory MyExtraHop.hop(value) = MyClass.named;
                                  ^
/test-lib:19:38: Error: The constructor function type 'MyExtraHop Function(dynamic)' isn't a subtype of 'MyInterface Function(dynamic)'.
 - 'MyExtraHop' is from '/test-lib'.
 - 'MyInterface' is from '/test-lib'.
  factory MyInterface.named(value) = MyExtraHop.hop;
                                     ^
/test-lib:20:41: Error: The constructor function type 'MyClass Function(dynamic)' isn't a subtype of 'MyInterface Function(dynamic)'.
 - 'MyClass' is from '/test-lib'.
 - 'MyInterface' is from '/test-lib'.
  factory MyInterface.multiply(value) = MyClass.multiply;

[...]


--- Re-run this test:
python3 tools/test.py -n dartk-tsan-linux-release-x64 vm/cc/DartAPI_New

Commit range [dart2js] Migrate Setlet to NNBD...Remove a few tests from the bottom of checker_test.

Suspect commit would be 8999635, it is the only VM change (cc @crelier).

However, it does not fail on any of the other bots, only the TSAN bot.

Started happening at the same time as #46121 (comment).

@dcharkes dcharkes added area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. gardening labels May 25, 2021
@a-siva
Copy link
Contributor

a-siva commented May 25, 2021

This test has been failing in all the configurations and we have an issue tracking this already (#33041). It is possible in TSAN mode we were seeing some flaky crashes which got fixed and now we consistently get runtime errors.

@a-siva a-siva closed this as completed May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. gardening
Projects
None yet
Development

No branches or pull requests

2 participants