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