envoyproxy / nighthawk

L7 (HTTP/HTTPS/HTTP2/HTTP3) performance characterization tool
Apache License 2.0
361 stars 81 forks source link

Optionize the cpu warmup delay #53

Open oschaaf opened 5 years ago

oschaaf commented 5 years ago

Currently workers in Nighthawk spend two seconds in a spin/yield loop, polling the clock for assessing time-to-start. This should be configurable, and the default can probably be much shorter.

oschaaf commented 5 years ago

Part of this was done in v0.1, by using a more sane 0.5 second default and testing the execution path (and bugfixing an assert there) when a worker would be ready to start after its designated starting time. This reduced the need for being able to configure this explicitly greatly, but I feel there's still something to say for exposing the option later on.