From 88befa6021d136257939a2caba14e69ad9c43dd5 Mon Sep 17 00:00:00 2001 From: isaacs Date: Mon, 25 Feb 2013 08:30:05 -0800 Subject: [PATCH] bench: Make http easier to profile Do not run the http/simple.js server in a child process. Fix #4831 --- benchmark/http/simple.js | 5 ++--- benchmark/http_simple.js | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/benchmark/http/simple.js b/benchmark/http/simple.js index 04a2a2911f2..8a1be980520 100644 --- a/benchmark/http/simple.js +++ b/benchmark/http/simple.js @@ -11,14 +11,13 @@ var bench = common.createBenchmark(main, { function main(conf) { process.env.PORT = PORT; var spawn = require('child_process').spawn; - var simple = require('path').resolve(__dirname, '../http_simple.js'); - var server = spawn(process.execPath, [simple]); + var server = require('../http_simple.js'); setTimeout(function() { var path = '/' + conf.type + '/' + conf.length; //+ '/' + conf.chunks; var args = ['-r', 5000, '-t', 8, '-c', conf.c]; bench.http(path, args, function() { - server.kill(); + server.close(); }); }, 2000); } diff --git a/benchmark/http_simple.js b/benchmark/http_simple.js index 54500b49b50..36800f2c9db 100644 --- a/benchmark/http_simple.js +++ b/benchmark/http_simple.js @@ -22,7 +22,7 @@ if (useDomains) { gdom.enter(); } -var server = http.createServer(function (req, res) { +var server = module.exports = http.createServer(function (req, res) { if (useDomains) { var dom = domain.create(); dom.add(req);