-
Notifications
You must be signed in to change notification settings - Fork 527
[For Testing] Go faster stripes MkII - Collated changes #309
Conversation
32e7d7d
to
20441b4
Compare
3ae6670
to
4241377
Compare
Timings Lower alloc MemoryPool removal, MemoryPool2 rename Memory pool SocketOutput Output buffer packing Release all the locks Ascii headers; MemoryTextWriter removal Pool write requests
Baseline coreclr x64, third run v1.0.0-rc2-15944 77% CPU ~/wrk$ wrk -c 256 -d 30 -t 32 ... -s ./scripts/pipeline.lua -- 16 |
Closing for now until have some numbers; still needs work |
Broke 1M but still higher latency means its worse at 256 ~/wrk$ wrk -c 1024 -d 30 -t 32 ... -s ./scripts/pipeline.lua -- 16 |
Updated #305
Have now split into 10 PRs:
#310 MemoryPool2 Stack->Queue, Allocate returns newest
#311 Task.Run -> ThreadPool.QueueUserWorkItem
#312 Reduce GetString allocs and conversions
#315 Lockless socket output, lower allocs (4 inter dependent) commits
#316 Remove KestrelThread locks
#317 Connection - remove closure allocation
#318 Listener - remove closure allocation
#319 ListenerPrimary - remove closure allocation
#320 ListenerSecondary - reduce closure allocation
#321 Reduce delegate allocation
Can't currently performance test as the feeds aren't matching source