-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
stream: improve Readable.read() performance
read() performance is improved most by switching from an array to a linked list for storing buffered data. However, other changes that also contribute include: making some hot functions inlinable, faster read() argument checking, and misc code rearrangement to avoid unnecessary code execution. PR-URL: #7077 Reviewed-By: Calvin Metcalf <calvin.metcalf@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
- Loading branch information
1 parent
d8fee36
commit 5f35204
Showing
10 changed files
with
420 additions
and
137 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
'use strict'; | ||
|
||
const common = require('../common'); | ||
const v8 = require('v8'); | ||
const Readable = require('stream').Readable; | ||
|
||
const bench = common.createBenchmark(main, { | ||
n: [100e1] | ||
}); | ||
|
||
function main(conf) { | ||
const n = +conf.n; | ||
const b = new Buffer(32); | ||
const s = new Readable(); | ||
function noop() {} | ||
s._read = noop; | ||
|
||
// Force optimization before starting the benchmark | ||
s.push(b); | ||
v8.setFlagsFromString('--allow_natives_syntax'); | ||
eval('%OptimizeFunctionOnNextCall(s.read)'); | ||
s.push(b); | ||
while (s.read(128)); | ||
|
||
bench.start(); | ||
for (var k = 0; k < n; ++k) { | ||
for (var i = 0; i < 1e4; ++i) | ||
s.push(b); | ||
while (s.read(128)); | ||
} | ||
bench.end(n); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
'use strict'; | ||
|
||
const common = require('../common'); | ||
const v8 = require('v8'); | ||
const Readable = require('stream').Readable; | ||
|
||
const bench = common.createBenchmark(main, { | ||
n: [100e1] | ||
}); | ||
|
||
function main(conf) { | ||
const n = +conf.n; | ||
const b = new Buffer(32); | ||
const s = new Readable(); | ||
function noop() {} | ||
s._read = noop; | ||
|
||
// Force optimization before starting the benchmark | ||
s.push(b); | ||
v8.setFlagsFromString('--allow_natives_syntax'); | ||
eval('%OptimizeFunctionOnNextCall(s.read)'); | ||
s.push(b); | ||
while (s.read(106)); | ||
|
||
bench.start(); | ||
for (var k = 0; k < n; ++k) { | ||
for (var i = 0; i < 1e4; ++i) | ||
s.push(b); | ||
while (s.read(106)); | ||
} | ||
bench.end(n); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
'use strict'; | ||
|
||
const common = require('../common'); | ||
const v8 = require('v8'); | ||
const Readable = require('stream').Readable; | ||
|
||
const bench = common.createBenchmark(main, { | ||
n: [200e1] | ||
}); | ||
|
||
function main(conf) { | ||
const n = +conf.n; | ||
const b = new Buffer(32); | ||
const s = new Readable(); | ||
function noop() {} | ||
s._read = noop; | ||
|
||
// Force optimization before starting the benchmark | ||
s.push(b); | ||
v8.setFlagsFromString('--allow_natives_syntax'); | ||
eval('%OptimizeFunctionOnNextCall(s.push)'); | ||
eval('%OptimizeFunctionOnNextCall(s.read)'); | ||
s.push(b); | ||
while (s.read(32)); | ||
|
||
bench.start(); | ||
for (var k = 0; k < n; ++k) { | ||
for (var i = 0; i < 1e4; ++i) | ||
s.push(b); | ||
while (s.read(32)); | ||
} | ||
bench.end(n); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
'use strict'; | ||
|
||
const common = require('../common'); | ||
const v8 = require('v8'); | ||
const Readable = require('stream').Readable; | ||
|
||
const bench = common.createBenchmark(main, { | ||
n: [50e2] | ||
}); | ||
|
||
function main(conf) { | ||
const n = +conf.n; | ||
const b = new Buffer(32); | ||
const s = new Readable(); | ||
function noop() {} | ||
s._read = noop; | ||
|
||
// Force optimization before starting the benchmark | ||
s.push(b); | ||
v8.setFlagsFromString('--allow_natives_syntax'); | ||
eval('%OptimizeFunctionOnNextCall(s.read)'); | ||
s.push(b); | ||
while (s.read()); | ||
|
||
bench.start(); | ||
for (var k = 0; k < n; ++k) { | ||
for (var i = 0; i < 1e4; ++i) | ||
s.push(b); | ||
while (s.read()); | ||
} | ||
bench.end(n); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
'use strict'; | ||
|
||
const common = require('../common'); | ||
const v8 = require('v8'); | ||
const Readable = require('stream').Readable; | ||
|
||
const bench = common.createBenchmark(main, { | ||
n: [100e1] | ||
}); | ||
|
||
function main(conf) { | ||
const n = +conf.n; | ||
const b = new Buffer(32); | ||
const s = new Readable(); | ||
function noop() {} | ||
s._read = noop; | ||
|
||
// Force optimization before starting the benchmark | ||
s.push(b); | ||
v8.setFlagsFromString('--allow_natives_syntax'); | ||
eval('%OptimizeFunctionOnNextCall(s.read)'); | ||
s.push(b); | ||
while (s.read(12)); | ||
|
||
bench.start(); | ||
for (var k = 0; k < n; ++k) { | ||
for (var i = 0; i < 1e4; ++i) | ||
s.push(b); | ||
while (s.read(12)); | ||
} | ||
bench.end(n); | ||
} |
Oops, something went wrong.