Skip to content

core dump in Dart VM #16103

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
DartBot opened this issue Jan 15, 2014 · 11 comments
Closed

core dump in Dart VM #16103

DartBot opened this issue Jan 15, 2014 · 11 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. P1 A high priority bug; for example, a single project is unusable or has many test failures

Comments

@DartBot
Copy link

DartBot commented Jan 15, 2014

This issue was originally filed by @a14n


Running the attached file I get a "Segmentation fault (core dumped)" with:

> dart --version
Dart VM version: 1.1.0-dev.5.10 (Tue Jan 14 03:42:16 2014) on "linux_x64"

The last outputs I can see are:


start 441
  compute lat:-53 lng:-41, 256758, -53
  lat:-53 lng:-41, 256758, -53
end 441
start 442
  compute lat:-21 lng:171, 140546, -79
Segmentation fault (core dumped)


Running this file in the editor I get "exit code=139"


Attachment:
t.dart (1.14 KB)

@lrhn
Copy link
Member

lrhn commented Jan 15, 2014

Added Area-VM, Triaged labels.

@iposva-google
Copy link
Contributor

There seems to be something going wrong with the sin/cos merging optimization.

#­0 0x00235d96 in dart::Instruction::InsertAfter(dart::Instruction*) at /Volumes/Sources/dart-all/dart/runtime/vm/intermediate_language.cc:540
#­1 0x00194262 in dart::FlowGraph::InsertAfter(dart::Instruction*, dart::Instruction*, dart::Environment*, dart::Definition::UseKind) at /Volumes/Sources/dart-all/dart/runtime/vm/flow_graph.cc:123
#­2 0x001e016e in dart::FlowGraphOptimizer::AppendLoadIndexedForMerged(dart::Definition*, long, long) at /Volumes/Sources/dart-all/dart/runtime/vm/flow_graph_optimizer.cc:301
#­3 0x001e081e in dart::FlowGraphOptimizer::TryMergeMathUnary(dart::GrowableArray<dart::MathUnaryInstr*>) at /Volumes/Sources/dart-all/dart/runtime/vm/flow_graph_optimizer.cc:415
#­4 0x001e0cfe in dart::FlowGraphOptimizer::TryOptimizePatterns() at /Volumes/Sources/dart-all/dart/runtime/vm/flow_graph_optimizer.cc:480
#­5 0x0015e9c4 in dart::CompileParsedFunctionHelper(dart::ParsedFunction
, bool, long) at /Volumes/Sources/dart-all/dart/runtime/vm/compiler.cc:349
#­6 0x0015d619 in dart::CompileFunctionHelper(dart::Function const&, bool, long) at /Volumes/Sources/dart-all/dart/runtime/vm/compiler.cc:766
#­7 0x0015dab9 in dart::Compiler::CompileOptimizedFunction(dart::Function const&, long) at /Volumes/Sources/dart-all/dart/runtime/vm/compiler.cc:821
#­8 0x00152a74 in dart::NativeArguments::isolate() const [inlined] at /Volumes/Sources/dart-all/dart/runtime/vm/code_generator.cc:1432
#­9 0x001528e9 in dart::DRT_OptimizeInvokedFunction(dart::NativeArguments) at /Volumes/Sources/dart-all/dart/runtime/vm/code_generator.cc:1425


Set owner to @sgmitrovic.
Removed Priority-Unassigned label.
Added Priority-High, Accepted labels.

@ghost
Copy link

ghost commented Jan 15, 2014

Added Started label.

@ghost
Copy link

ghost commented Jan 15, 2014

Fixed in r31858.


Added Fixed label.

@kasperl
Copy link

kasperl commented Jan 17, 2014

Should the fix be considered for merging to the 1.1 branch?

@DartBot
Copy link
Author

DartBot commented Jan 17, 2014

This comment was originally written by @a14n


I still see this issue happen on dev channel.

@ghost
Copy link

ghost commented Jan 17, 2014

Yes I think this and r31862 (belong together) are small fixes that should be merged to 1.1 branch.

Alexandre, please specify/verify which VM revision number you are running. Thanks.

@DartBot
Copy link
Author

DartBot commented Jan 17, 2014

This comment was originally written by @a14n


I used the Dev channel :
$dart --version
Dart VM version: 1.1.0-dev.5.11 (Wed Jan 15 00:41:27 2014) on "linux_x64"

(it seems to be http://gsdview.appspot.com/dart-archive/channels/dev/raw/31818/)

To grab your fix I have downloaded a raw version on http://gsdview.appspot.com/dart-archive/channels/be/raw/latest/
$dart --version
Dart VM version: 1.2.0-edge.31909 (Fri Jan 17 00:23:59 2014) on "linux_x64"

On this version the sample code works.

@iposva-google
Copy link
Contributor

Issue #16606 has been merged into this issue.


cc @sgmitrovic.
cc @fsc8000.
cc @a-siva.

@kasperl
Copy link

kasperl commented Feb 10, 2014

The fix for this issue is now live on the stable channel in Dart SDK version 1.1.3.

@kasperl
Copy link

kasperl commented Feb 10, 2014

Issue #16606 has been merged into this issue.

@DartBot DartBot added Type-Defect P1 A high priority bug; for example, a single project is unusable or has many test failures area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. labels Feb 10, 2014
@DartBot DartBot assigned ghost Feb 10, 2014
This issue was closed.
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. P1 A high priority bug; for example, a single project is unusable or has many test failures
Projects
None yet
Development

No branches or pull requests

4 participants