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

Side by Side Diff: runtime/vm/verified_memory_test.cc

Issue 1275353005: VM thread shutdown. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Merge Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « runtime/vm/thread_pool_test.cc ('k') | tests/isolate/nested_spawn2_test.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "platform/assert.h" 5 #include "platform/assert.h"
6 #include "vm/unit_test.h" 6 #include "vm/unit_test.h"
7 #include "vm/verified_memory.h" 7 #include "vm/verified_memory.h"
8 8
9 namespace dart { 9 namespace dart {
10 10
11 void Init() { 11 void Init() {
12 #if defined(DEBUG) 12 #if defined(DEBUG)
13 FLAG_verified_mem = true; 13 FLAG_verified_mem = true;
14 #endif 14 #endif
15 } 15 }
16 16
17 17
18 void Shutdown() {
19 #if defined(DEBUG)
20 // We must reset this to false to avoid checking some assumptions in
21 // VM shutdown that are left violated by these tests.
22 FLAG_verified_mem = false;
23 #endif
24 }
25
26
18 UNIT_TEST_CASE(VerifiedMemoryReserve) { 27 UNIT_TEST_CASE(VerifiedMemoryReserve) {
19 Init(); 28 Init();
20 const intptr_t kReservationSize = 64 * KB; 29 const intptr_t kReservationSize = 64 * KB;
21 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize); 30 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize);
22 EXPECT_EQ(kReservationSize, vm->size()); 31 EXPECT_EQ(kReservationSize, vm->size());
23 delete vm; 32 delete vm;
33 Shutdown();
24 } 34 }
25 35
26 36
27 UNIT_TEST_CASE(VerifiedMemoryCommit) { 37 UNIT_TEST_CASE(VerifiedMemoryCommit) {
28 Init(); 38 Init();
29 const intptr_t kReservationSize = 64 * KB; 39 const intptr_t kReservationSize = 64 * KB;
30 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize); 40 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize);
31 EXPECT_EQ(kReservationSize, vm->size()); 41 EXPECT_EQ(kReservationSize, vm->size());
32 vm->Commit(false); 42 vm->Commit(false);
33 delete vm; 43 delete vm;
44 Shutdown();
34 } 45 }
35 46
36 47
37 UNIT_TEST_CASE(VerifiedMemoryBasic) { 48 UNIT_TEST_CASE(VerifiedMemoryBasic) {
38 Init(); 49 Init();
39 const intptr_t kReservationSize = 64 * KB; 50 const intptr_t kReservationSize = 64 * KB;
40 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize); 51 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize);
41 EXPECT_EQ(kReservationSize, vm->size()); 52 EXPECT_EQ(kReservationSize, vm->size());
42 vm->Commit(false); 53 vm->Commit(false);
43 double* addr = reinterpret_cast<double*>(vm->address()); 54 double* addr = reinterpret_cast<double*>(vm->address());
44 VerifiedMemory::Write(&addr[0], 0.5); 55 VerifiedMemory::Write(&addr[0], 0.5);
45 EXPECT_EQ(0.5, addr[0]); 56 EXPECT_EQ(0.5, addr[0]);
46 VerifiedMemory::Write(&addr[1], 1.5); 57 VerifiedMemory::Write(&addr[1], 1.5);
47 VerifiedMemory::Write(&addr[2], 2.5); 58 VerifiedMemory::Write(&addr[2], 2.5);
48 VerifiedMemory::Write(&addr[0], 0.25); 59 VerifiedMemory::Write(&addr[0], 0.25);
49 static const double kNaN = NAN; 60 static const double kNaN = NAN;
50 VerifiedMemory::Write(&addr[0], kNaN); // Bitwise comparison should be used. 61 VerifiedMemory::Write(&addr[0], kNaN); // Bitwise comparison should be used.
51 VerifiedMemory::Write(&addr[0], 0.5); 62 VerifiedMemory::Write(&addr[0], 0.5);
52 int64_t* unverified = reinterpret_cast<int64_t*>(&addr[3]); 63 int64_t* unverified = reinterpret_cast<int64_t*>(&addr[3]);
53 *unverified = 123; 64 *unverified = 123;
54 VerifiedMemory::Verify(reinterpret_cast<uword>(addr), 3 * sizeof(double)); 65 VerifiedMemory::Verify(reinterpret_cast<uword>(addr), 3 * sizeof(double));
55 delete vm; 66 delete vm;
67 Shutdown();
56 } 68 }
57 69
58 70
59 UNIT_TEST_CASE(VerifiedMemoryAccept) { 71 UNIT_TEST_CASE(VerifiedMemoryAccept) {
60 Init(); 72 Init();
61 const intptr_t kReservationSize = 64 * KB; 73 const intptr_t kReservationSize = 64 * KB;
62 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize); 74 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize);
63 EXPECT_EQ(kReservationSize, vm->size()); 75 EXPECT_EQ(kReservationSize, vm->size());
64 vm->Commit(false); 76 vm->Commit(false);
65 double* addr = reinterpret_cast<double*>(vm->address()); 77 double* addr = reinterpret_cast<double*>(vm->address());
66 VerifiedMemory::Write(&addr[0], 0.5); 78 VerifiedMemory::Write(&addr[0], 0.5);
67 VerifiedMemory::Write(&addr[1], 1.5); 79 VerifiedMemory::Write(&addr[1], 1.5);
68 VerifiedMemory::Write(&addr[2], 2.5); 80 VerifiedMemory::Write(&addr[2], 2.5);
69 VerifiedMemory::Write(&addr[0], 0.25); 81 VerifiedMemory::Write(&addr[0], 0.25);
70 // Unverified write followed by Accept ("I know what I'm doing"). 82 // Unverified write followed by Accept ("I know what I'm doing").
71 memset(addr, 0xf3, 2 * sizeof(double)); 83 memset(addr, 0xf3, 2 * sizeof(double));
72 VerifiedMemory::Accept(reinterpret_cast<uword>(addr), 2 * sizeof(double)); 84 VerifiedMemory::Accept(reinterpret_cast<uword>(addr), 2 * sizeof(double));
73 VerifiedMemory::Verify(reinterpret_cast<uword>(addr), 3 * sizeof(double)); 85 VerifiedMemory::Verify(reinterpret_cast<uword>(addr), 3 * sizeof(double));
74 delete vm; 86 delete vm;
87 Shutdown();
75 } 88 }
76 89
77 90
78 // Negative tests below. 91 // Negative tests below.
79 92
80 UNIT_TEST_CASE(VerifyImplicit_Crash) { 93 UNIT_TEST_CASE(VerifyImplicit_Crash) {
81 Init(); 94 Init();
82 const intptr_t kReservationSize = 64 * KB; 95 const intptr_t kReservationSize = 64 * KB;
83 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize); 96 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize);
84 EXPECT_EQ(kReservationSize, vm->size()); 97 EXPECT_EQ(kReservationSize, vm->size());
85 vm->Commit(false); 98 vm->Commit(false);
86 double* addr = reinterpret_cast<double*>(vm->address()); 99 double* addr = reinterpret_cast<double*>(vm->address());
87 addr[0] = 0.5; // Forget to use Write. 100 addr[0] = 0.5; // Forget to use Write.
88 VerifiedMemory::Write(&addr[0], 1.5); 101 VerifiedMemory::Write(&addr[0], 1.5);
102 Shutdown();
89 } 103 }
90 104
91 105
92 UNIT_TEST_CASE(VerifyExplicit_Crash) { 106 UNIT_TEST_CASE(VerifyExplicit_Crash) {
93 Init(); 107 Init();
94 const intptr_t kReservationSize = 64 * KB; 108 const intptr_t kReservationSize = 64 * KB;
95 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize); 109 VirtualMemory* vm = VerifiedMemory::Reserve(kReservationSize);
96 EXPECT_EQ(kReservationSize, vm->size()); 110 EXPECT_EQ(kReservationSize, vm->size());
97 vm->Commit(false); 111 vm->Commit(false);
98 double* addr = reinterpret_cast<double*>(vm->address()); 112 double* addr = reinterpret_cast<double*>(vm->address());
99 VerifiedMemory::Write(&addr[0], 0.5); 113 VerifiedMemory::Write(&addr[0], 0.5);
100 VerifiedMemory::Write(&addr[1], 1.5); 114 VerifiedMemory::Write(&addr[1], 1.5);
101 addr[1] = 3.5; // Forget to use Write. 115 addr[1] = 3.5; // Forget to use Write.
102 VerifiedMemory::Write(&addr[2], 2.5); 116 VerifiedMemory::Write(&addr[2], 2.5);
103 VerifiedMemory::Verify(reinterpret_cast<uword>(addr), 3 * sizeof(double)); 117 VerifiedMemory::Verify(reinterpret_cast<uword>(addr), 3 * sizeof(double));
118 Shutdown();
104 } 119 }
105 120
106 } // namespace dart 121 } // namespace dart
OLDNEW
« no previous file with comments | « runtime/vm/thread_pool_test.cc ('k') | tests/isolate/nested_spawn2_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698