Mercurial > hg > truffle
comparison src/share/vm/utilities/quickSort.cpp @ 3779:04760e41b01e
7016112: CMS: crash during promotion testing
Summary: Also reviewed by mikael.gerdin@oracle.com; stdlib:qsort() does byte-by-byte swapping on Windows. This leads to pointer shearing. Fix is to implement a quicksort that does full pointer updates.
Reviewed-by: never, coleenp, ysr
author | brutisso |
---|---|
date | Tue, 28 Jun 2011 14:23:27 +0200 |
parents | |
children | 8ab2f4108d20 |
comparison
equal
deleted
inserted
replaced
3778:5f6f2615433a | 3779:04760e41b01e |
---|---|
1 /* | |
2 * Copyright (c) 2011, Oracle and/or its affiliates. All rights reserved. | |
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. | |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA | |
20 * or visit www.oracle.com if you need additional information or have any | |
21 * questions. | |
22 * | |
23 */ | |
24 | |
25 #include "precompiled.hpp" | |
26 #include "utilities/quickSort.hpp" | |
27 | |
28 #ifndef PRODUCT | |
29 | |
30 // Unit tests | |
31 | |
32 #include "runtime/os.hpp" | |
33 #include <stdlib.h> | |
34 | |
35 static int test_comparator(int a, int b) { | |
36 if (a == b) { | |
37 return 0; | |
38 } | |
39 if (a < b) { | |
40 return -1; | |
41 } | |
42 return 1; | |
43 } | |
44 | |
45 static int test_even_odd_comparator(int a, int b) { | |
46 bool a_is_odd = (a % 2) == 1; | |
47 bool b_is_odd = (b % 2) == 1; | |
48 if (a_is_odd == b_is_odd) { | |
49 return 0; | |
50 } | |
51 if (a_is_odd) { | |
52 return -1; | |
53 } | |
54 return 1; | |
55 } | |
56 | |
57 static int test_stdlib_comparator(const void* a, const void* b) { | |
58 int ai = *(int*)a; | |
59 int bi = *(int*)b; | |
60 if (ai == bi) { | |
61 return 0; | |
62 } | |
63 if (ai < bi) { | |
64 return -1; | |
65 } | |
66 return 1; | |
67 } | |
68 | |
69 void QuickSort::print_array(const char* prefix, int* array, int length) { | |
70 tty->print("%s:", prefix); | |
71 for (int i = 0; i < length; i++) { | |
72 tty->print(" %d", array[i]); | |
73 } | |
74 tty->print_cr(""); | |
75 } | |
76 | |
77 bool QuickSort::compare_arrays(int* actual, int* expected, int length) { | |
78 for (int i = 0; i < length; i++) { | |
79 if (actual[i] != expected[i]) { | |
80 print_array("Sorted array ", actual, length); | |
81 print_array("Expected array", expected, length); | |
82 return false; | |
83 } | |
84 } | |
85 return true; | |
86 } | |
87 | |
88 template <class C> | |
89 bool QuickSort::sort_and_compare(int* arrayToSort, int* expectedResult, int length, C comparator, bool idempotent) { | |
90 sort<int, C>(arrayToSort, length, comparator, idempotent); | |
91 return compare_arrays(arrayToSort, expectedResult, length); | |
92 } | |
93 | |
94 bool QuickSort::test_quick_sort() { | |
95 tty->print_cr("test_quick_sort\n"); | |
96 { | |
97 int* test_array = NULL; | |
98 int* expected_array = NULL; | |
99 assert(sort_and_compare(test_array, expected_array, 0, test_comparator), "Empty array not handled"); | |
100 } | |
101 { | |
102 int test_array[] = {3}; | |
103 int expected_array[] = {3}; | |
104 assert(sort_and_compare(test_array, expected_array, 1, test_comparator), "Single value array not handled"); | |
105 } | |
106 { | |
107 int test_array[] = {3,2}; | |
108 int expected_array[] = {2,3}; | |
109 assert(sort_and_compare(test_array, expected_array, 2, test_comparator), "Array with 2 values not correctly sorted"); | |
110 } | |
111 { | |
112 int test_array[] = {3,2,1}; | |
113 int expected_array[] = {1,2,3}; | |
114 assert(sort_and_compare(test_array, expected_array, 3, test_comparator), "Array with 3 values not correctly sorted"); | |
115 } | |
116 { | |
117 int test_array[] = {4,3,2,1}; | |
118 int expected_array[] = {1,2,3,4}; | |
119 assert(sort_and_compare(test_array, expected_array, 4, test_comparator), "Array with 4 values not correctly sorted"); | |
120 } | |
121 { | |
122 int test_array[] = {7,1,5,3,6,9,8,2,4,0}; | |
123 int expected_array[] = {0,1,2,3,4,5,6,7,8,9}; | |
124 assert(sort_and_compare(test_array, expected_array, 10, test_comparator), "Array with 10 values not correctly sorted"); | |
125 } | |
126 { | |
127 int test_array[] = {4,4,1,4}; | |
128 int expected_array[] = {1,4,4,4}; | |
129 assert(sort_and_compare(test_array, expected_array, 4, test_comparator), "3 duplicates not sorted correctly"); | |
130 } | |
131 { | |
132 int test_array[] = {0,1,2,3,4,5,6,7,8,9}; | |
133 int expected_array[] = {0,1,2,3,4,5,6,7,8,9}; | |
134 assert(sort_and_compare(test_array, expected_array, 10, test_comparator), "Already sorted array not correctly sorted"); | |
135 } | |
136 { | |
137 // one of the random arrays that found an issue in the partion method. | |
138 int test_array[] = {76,46,81,8,64,56,75,11,51,55,11,71,59,27,9,64,69,75,21,25,39,40,44,32,7,8,40,41,24,78,24,74,9,65,28,6,40,31,22,13,27,82}; | |
139 int expected_array[] = {6,7,8,8,9,9,11,11,13,21,22,24,24,25,27,27,28,31,32,39,40,40,40,41,44,46,51,55,56,59,64,64,65,69,71,74,75,75,76,78,81,82}; | |
140 assert(sort_and_compare(test_array, expected_array, 42, test_comparator), "Not correctly sorted"); | |
141 } | |
142 { | |
143 int test_array[] = {2,8,1,4}; | |
144 int expected_array[] = {1,4,2,8}; | |
145 assert(sort_and_compare(test_array, expected_array, 4, test_even_odd_comparator), "Even/odd not sorted correctly"); | |
146 } | |
147 { // Some idempotent tests | |
148 { | |
149 // An array of lenght 3 is only sorted by find_pivot. Make sure that it is idempotent. | |
150 int test_array[] = {1,4,8}; | |
151 int expected_array[] = {1,4,8}; | |
152 assert(sort_and_compare(test_array, expected_array, 3, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
153 } | |
154 { | |
155 int test_array[] = {1,7,9,4,8,2}; | |
156 int expected_array[] = {1,7,9,4,8,2}; | |
157 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
158 } | |
159 { | |
160 int test_array[] = {1,9,7,4,2,8}; | |
161 int expected_array[] = {1,9,7,4,2,8}; | |
162 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
163 } | |
164 { | |
165 int test_array[] = {7,9,1,2,8,4}; | |
166 int expected_array[] = {7,9,1,2,8,4}; | |
167 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
168 } | |
169 { | |
170 int test_array[] = {7,1,9,2,4,8}; | |
171 int expected_array[] = {7,1,9,2,4,8}; | |
172 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
173 } | |
174 { | |
175 int test_array[] = {9,1,7,4,8,2}; | |
176 int expected_array[] = {9,1,7,4,8,2}; | |
177 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
178 } | |
179 { | |
180 int test_array[] = {9,7,1,4,2,8}; | |
181 int expected_array[] = {9,7,1,4,2,8}; | |
182 assert(sort_and_compare(test_array, expected_array, 6, test_even_odd_comparator, true), "Even/odd not idempotent"); | |
183 } | |
184 } | |
185 | |
186 // test sorting random arrays | |
187 for (int i = 0; i < 1000; i++) { | |
188 int length = os::random() % 100; | |
189 int* test_array = new int[length]; | |
190 int* expected_array = new int[length]; | |
191 for (int j = 0; j < length; j++) { | |
192 // Choose random values, but get a chance of getting duplicates | |
193 test_array[j] = os::random() % (length * 2); | |
194 expected_array[j] = test_array[j]; | |
195 } | |
196 | |
197 // Compare sorting to stdlib::qsort() | |
198 qsort(expected_array, length, sizeof(int), test_stdlib_comparator); | |
199 assert(sort_and_compare(test_array, expected_array, length, test_comparator), "Random array not correctly sorted"); | |
200 | |
201 // Make sure sorting is idempotent. | |
202 // Both test_array and expected_array are sorted by the test_comparator. | |
203 // Now sort them once with the test_even_odd_comparator. Then sort the | |
204 // test_array one more time with test_even_odd_comparator and verify that | |
205 // it is idempotent. | |
206 sort(expected_array, length, test_even_odd_comparator, true); | |
207 sort(test_array, length, test_even_odd_comparator, true); | |
208 assert(compare_arrays(test_array, expected_array, length), "Sorting identical arrays rendered different results"); | |
209 sort(test_array, length, test_even_odd_comparator, true); | |
210 assert(compare_arrays(test_array, expected_array, length), "Sorting already sorted array changed order of elements - not idempotent"); | |
211 | |
212 delete[] test_array; | |
213 delete[] expected_array; | |
214 } | |
215 return true; | |
216 } | |
217 | |
218 #endif |