Page MenuHomeFreeBSD

PR 135718 enhance qsort(3) to properly handle 32-bit aligned data on 64-bit systems
ClosedPublic

Authored by pfg on Feb 1 2015, 9:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 31, 1:09 PM
Unknown Object (File)
Dec 20 2023, 2:23 AM
Unknown Object (File)
Aug 23 2023, 8:48 AM
Unknown Object (File)
Jul 29 2023, 3:56 PM
Unknown Object (File)
Mar 21 2023, 8:02 PM
Unknown Object (File)
Mar 3 2023, 11:53 PM
Unknown Object (File)
Jan 14 2023, 2:21 PM
Unknown Object (File)
Jan 4 2023, 11:41 AM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

pfg retitled this revision from to PR 135718 enhance qsort(3) to properly handle 32-bit aligned data on 64-bit systems.
pfg updated this object.
pfg edited the test plan for this revision. (Show Details)

This appears to be only a prototype. The code is not ready for public consumption yet. It will be worked through bugzilla first.

Updated version by ache@

Updating D1752: PR 135718 enhance qsort(3) to properly handle 32-bit aligned data on

64-bit systems

Here is some testing I did not long ago:
...
I updated the testing utility with 4096 23 bit datapoints of random data.
...

$ time ./test-heapsort
1..1
ok 1 - heapsort

27.83 real        27.61 user         0.00 sys

$ time ./test-mergesort
1..1
ok 1 - mergesort

27.02 real        26.83 user         0.00 sys

$ time ./test-qsort (old)
1..1
ok 1 - qsort

27.07 real        26.94 user         0.00 sys

...
$ time ./test-qsort (new)
1..1
ok 1 - qsort

26.94 real        26.81 user         0.00 sys

...

IMHO. the performance improvements are measurable, and even when they may not seem huge, they are significant enough to make qsort preferable over heapsort and mergesort (at least for my particular dataset).


I will commit the updated version sometime this week, before I forget about it ;).

pfg added a reviewer: pfg.

Committed as revision 279663.

This revision is now accepted and ready to land.Mar 5 2015, 5:02 PM