Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(53)

Issue 59773007: - Add a per-isolate pseudo random number generator to the (Closed)

Created:
7 years, 1 month ago by Ivan Posva
Modified:
7 years, 1 month ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

- Add a per-isolate pseudo random number generator to the VM internals. This PRNG is not used to generate random numbers in Dart code. - Use the PRNG to generate JIT cookies for large constants if needed during assembly. - Add the possibility to set an external entropy source for the PRNG through the C API. R=asiva@google.com Committed: https://code.google.com/p/dart/source/detail?r=29950

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+156 lines, -21 lines) Patch
M runtime/bin/dartutils.h View 1 chunk +2 lines, -0 lines 0 comments Download
M runtime/bin/dartutils.cc View 2 chunks +6 lines, -0 lines 4 comments Download
M runtime/bin/gen_snapshot.cc View 1 chunk +2 lines, -1 line 0 comments Download
M runtime/bin/io_impl_sources.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M runtime/bin/main.cc View 1 chunk +2 lines, -1 line 0 comments Download
M runtime/bin/run_vm_tests.cc View 1 chunk +2 lines, -1 line 0 comments Download
M runtime/include/dart_api.h View 2 chunks +3 lines, -1 line 0 comments Download
M runtime/lib/math_patch.dart View 1 chunk +1 line, -1 line 0 comments Download
M runtime/vm/assembler_ia32.h View 4 chunks +10 lines, -9 lines 0 comments Download
M runtime/vm/assembler_ia32.cc View 2 chunks +12 lines, -3 lines 0 comments Download
M runtime/vm/dart.h View 1 chunk +2 lines, -1 line 0 comments Download
M runtime/vm/dart.cc View 1 chunk +3 lines, -1 line 0 comments Download
M runtime/vm/dart_api_impl.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M runtime/vm/isolate.h View 6 chunks +13 lines, -0 lines 0 comments Download
M runtime/vm/isolate.cc View 2 chunks +2 lines, -0 lines 0 comments Download
A runtime/vm/random.h View 1 chunk +31 lines, -0 lines 0 comments Download
A runtime/vm/random.cc View 1 chunk +59 lines, -0 lines 0 comments Download
M runtime/vm/vm_sources.gypi View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Ivan Posva
7 years, 1 month ago (2013-11-05 23:28:36 UTC) #1
siva
lgtm https://codereview.chromium.org/59773007/diff/1/runtime/bin/dartutils.cc File runtime/bin/dartutils.cc (right): https://codereview.chromium.org/59773007/diff/1/runtime/bin/dartutils.cc#newcode15 runtime/bin/dartutils.cc:15: #include "bin/directory.h" extensions.h should come here in the ...
7 years, 1 month ago (2013-11-05 23:37:46 UTC) #2
Ivan Posva
Thanks! -Ivan https://codereview.chromium.org/59773007/diff/1/runtime/bin/dartutils.cc File runtime/bin/dartutils.cc (right): https://codereview.chromium.org/59773007/diff/1/runtime/bin/dartutils.cc#newcode15 runtime/bin/dartutils.cc:15: #include "bin/directory.h" On 2013/11/05 23:37:46, siva wrote: ...
7 years, 1 month ago (2013-11-05 23:44:55 UTC) #3
jschuh
Looks basically the same as the v8 implementation on the security front. lgtm.
7 years, 1 month ago (2013-11-06 00:27:12 UTC) #4
Ivan Posva
7 years, 1 month ago (2013-11-06 00:59:57 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r29950 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698