~statik/haproxy/trunk

From Revision 784 to 765

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
784		[TESTS] test-fsm: 22 regression tests for state machines [TESTS] test-fsm: 22 regression tests for state machines 22 regression tests for state machines are managed by the new file tests/test-fsm.cfg. Check it, they are all documented inside. Most of the bugs introduced during the FSM extraction have been found with these tests.	Willy Tarreau	15 years ago
783		[BUG] process_request: HTTP body analysis must return zero i... [BUG] process_request: HTTP body analysis must return zero if missing data This missing return and timeout check caused an infinite loop too.	Willy Tarreau	15 years ago
782		[BUG] process_cli/process_srv: don't call shutdown when alre... [BUG] process_cli/process_srv: don't call shutdown when already done A few missing checks of BF_SHUTR and BF_SHUTW caused busy loops upon some error paths.	Willy Tarreau	15 years ago
781		[MEDIUM] merge inspect_exp and txn->exp into request buffer [MEDIUM] merge inspect_exp and txn->exp into request buffer Since we may have several analysers on a buffer, it's more convenient to have the analyser timeout attached to the buffer itself.	Willy Tarreau	15 years ago
780		[BUILD] fix warning in proto_tcp.c with gcc >= 4 [BUILD] fix warning in proto_tcp.c with gcc >= 4 signedness issues.	Willy Tarreau	15 years ago
779		[BUG] regparm is broken on gcc < 3 [BUG] regparm is broken on gcc < 3 Gcc < 3 does not consider regparm declarations for function pointers. This causes big trouble at least with pollers (and with any function pointer after all). Disable CONFIG_HAP_USE_REGPARM for gcc < 3.	Willy Tarreau	15 years ago
778		[OPTIM] process_cli/process_srv: reduce the number of tests [OPTIM] process_cli/process_srv:  reduce the number of tests We can skip a number of tests by simply checking a few flags, it saves a few CPU cycles in the fast path.	Willy Tarreau	15 years ago
777		[MEDIUM] session: move the analysis bit field to the buffer [MEDIUM] session: move the analysis bit field to the buffer It makes more sense to store the list of analysers in the buffer than in the session since they are precisely plugged onto one buffer.	Willy Tarreau	15 years ago
776		[MINOR] ensure the termination flags are set by process_xxx [MINOR] ensure the termination flags are set by process_xxx When any processing remains on a buffer, it must be up to the processing functions to set the termination flags, because they are the only ones who know about higher levels.	Willy Tarreau	15 years ago
775		[MEDIUM] centralize buffer timeout checks at the top of proc... [MEDIUM] centralize buffer timeout checks at the top of process_session it's more efficient and easier to check all the timeouts at once and always rely on the buffer flags than to check them everywhere.	Willy Tarreau	15 years ago
774		[MEDIUM] use buffer->wex instead of buffer->cex for connect ... [MEDIUM] use buffer->wex instead of buffer->cex for connect timeout It's a shame not to use buffer->wex for connection timeouts since by definition it cannot be used till the connection is not established. Using it instead of ->cex also makes the buffer processing more symmetric.	Willy Tarreau	15 years ago
773		[MAJOR] process_session: rely only on buffer flags [MAJOR] process_session: rely only on buffer flags Instead of calling all functions in a loop, process_session now calls them according to buffer flags changes. This ensures that we almost never call functions for nothing. The flags settings are still quite coarse, but the number of average functions calls per session has dropped from 31 to 18 (the calls to process_srv dropped from 13 to 7 and the calls to process_cli dropped from 13 to 8). This could still be improved by memorizing which flags each function uses, but that would add a level of complexity which is not desirable and maybe even not worth the small gain.	Willy Tarreau	15 years ago
772		[MEDIUM] buffers: add BF_EMPTY and BF_FULL to remove depende... [MEDIUM] buffers: add BF_EMPTY and BF_FULL to remove dependency on req/rep->l It is not always convenient to run checks on req->l in functions to check if a buffer is empty or full. Now the stream_sock functions set flags BF_EMPTY and BF_FULL according to the buffer contents. Of course, functions which touch the buffer contents adjust the flags too.	Willy Tarreau	15 years ago
771		[CLEANUP] get rid of BF_SHUT*_PENDING [CLEANUP] get rid of BF_SHUT*_PENDING BF_SHUTR_PENDING and BF_SHUTW_PENDING were poor ideas because BF_SHUTR is the pending of BF_SHUTW_DONE and BF_SHUTW is the pending of BF_SHUTR_DONE. Remove those two useless and confusing "pending" versions and rename buffer_shut{r,w}_* functions.	Willy Tarreau	15 years ago
770		[BUG] maintain_proxies must not disable backends [BUG] maintain_proxies must not disable backends maintain_proxies could disable backends (p->maxconn == 0) which is wrong (but apparently harmless). Add a check for p->maxconn == 0.	Willy Tarreau	15 years ago
769		[BUG] process_response: do not touch srv_state [BUG] process_response: do not touch srv_state process_response is not allowed to touch srv_state (this is an incident which has survived the code migration). This bug was causing connection exhaustion on frontend due to some closed sockets marked SV_STDATA again.	Willy Tarreau	15 years ago
768		[BUG] buffers: remove BF_MAY_CONNECT and fix forwarding issu... [BUG] buffers: remove BF_MAY_CONNECT and fix forwarding issue It wasn't really wise to separate BF_MAY_CONNECT and BF_MAY_FORWARD, as it caused trouble in TCP mode because the connection was allowed but not the forwarding. Remove BF_MAY_CONNECT.	Willy Tarreau	15 years ago
767		[BUG] process_response must not enable the read FD [BUG] process_response must not enable the read FD Since the separation of TCP and HTTP state machines, the HTTP code must not play anymore with the file descriptor status without checking if they are closed. Remains of such practice have caused busy loops under some circumstances (mainly when client closed during headers response).	Willy Tarreau	15 years ago
766		[BUG] ev_sepoll: closed file descriptors could persist in th... [BUG] ev_sepoll: closed file descriptors could persist in the spec list If __fd_clo() was called on a file descriptor which was previously disabled, it was not removed from the spec list. This apparently could not happen on previous code because the TCP states prevented this, but now it happens regularly. The effects are spec entries stuck populated, leading to busy loops.	Willy Tarreau	15 years ago
765		[MINOR] term_trace: add better instrumentations to trace the... [MINOR] term_trace: add better instrumentations to trace the code A new member has been added to the struct session. It keeps a trace of what block of code performs a close or a shutdown on a socket, and in what sequence. This is extremely convenient for post-mortem analysis where flag combinations and states seem impossible. A new ABORT_NOW() macro has also been added to make the code immediately segfault where called.	Willy Tarreau	15 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1