mirror of https://git.kore.io/kore.git
dd2dff2318
The HTTP layer used to make a copy of each incoming header and its value for a request. Stop doing that and make HTTP headers zero-copy all across the board. This change comes with some api function changes, notably the http_request_header() function which now takes a const char ** rather than a char ** out pointer. This commit also constifies several members of http_request, beware. Additional rework how the worker processes deal with the accept lock. Before: if a worker held the accept lock and it accepted a new connection it would release the lock for others and back off for 500ms before attempting to grab the lock again. This approach worked but under high load this starts becoming obvious. Now: - workers not holding the accept lock and not having any connections will wait less long before returning from kore_platform_event_wait(). - workers not holding the accept lock will no longer blindly wait an arbitrary amount in kore_platform_event_wait() but will look at how long until the next lock grab is and base their timeout on that. - if a worker its next_lock timeout is up and failed to grab the lock it will try again in half the time again. - the worker process holding the lock will when releasing the lock double check if it still has space for newer connections, if it does it will keep the lock until it is full. This prevents the lock from bouncing between several non busy worker processes all the time. Additional fixes: - Reduce the number of times we check the timeout list, only do it twice per second rather then every event tick. - Fix solo worker count for TLS (we actually hold two processes, not one). - Make sure we don't accidentally miscalculate the idle time causing new connections under heavy load to instantly drop. - Swap from gettimeofday() to clock_gettime() now that MacOS caught up. |
||
|---|---|---|
| .. | ||
| cookies | ||
| cpp | ||
| generic | ||
| headers | ||
| integers | ||
| json_yajl | ||
| jsonrpc | ||
| ktunnel | ||
| memtag | ||
| messaging | ||
| nohttp | ||
| parameters | ||
| pgsql | ||
| pgsql-sync | ||
| pipe_task | ||
| python | ||
| python-pgsql | ||
| sse | ||
| tasks | ||
| tls-proxy | ||
| upload | ||
| video_stream | ||
| websocket | ||