mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-22 14:21:25 +02:00
Code Issues Projects Releases Wiki Activity

MINOR: queue: simplify pendconn_unlink() regarding srv vs px

Browse Source 
Since the code paths became exactly the same except for what log field
to update, let's simplify the code and move further code out of the
lock. The queue position update and the test for server vs proxy do not
need to be inside the lock.
This commit is contained in:
Willy Tarreau 2021-06-23 16:54:16 +02:00
parent 51c63f0f0a
commit d03adce575
1 changed files with 19 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/queue.c
View File

@ -172,43 +172,35 @@ static inline void pendconn_queue_unlock(struct pendconn *p)
* connection is not really dequeued. It will be done during process_stream().
* This function takes all the required locks for the operation. The pendconn
* must be valid, though it doesn't matter if it was already unlinked. Prefer
* pendconn_cond_unlink() to first check <p>. When the locks are already held,
* please use __pendconn_unlink_{srv,prx}() instead.
* pendconn_cond_unlink() to first check <p>.
*/
void pendconn_unlink(struct pendconn *p)
{
int done = 0;
struct queue *q = p->queue;
struct proxy *px = q->px;
struct server *sv = q->sv;
uint oldidx;
int done = 0;
if (sv) {
/* queued in the server */
oldidx = _HA_ATOMIC_LOAD(&p->queue->idx);
HA_SPIN_LOCK(QUEUE_LOCK, &q->lock);
if (p->node.node.leaf_p) {
__pendconn_unlink_srv(p);
eb32_delete(&p->node);
done = 1;
}
HA_SPIN_UNLOCK(QUEUE_LOCK, &q->lock);
if (done) {
oldidx -= p->queue_idx;
if (sv)
p->strm->logs.srv_queue_pos += oldidx;
else
p->strm->logs.prx_queue_pos += oldidx;
_HA_ATOMIC_DEC(&q->length);
_HA_ATOMIC_DEC(&px->totpend);
}
}
else {
/* queued in the proxy */
HA_SPIN_LOCK(QUEUE_LOCK, &q->lock);
if (p->node.node.leaf_p) {
__pendconn_unlink_prx(p);
done = 1;
}
HA_SPIN_UNLOCK(QUEUE_LOCK, &q->lock);
if (done) {
_HA_ATOMIC_DEC(&q->length);
_HA_ATOMIC_DEC(&px->totpend);
}
}
}
/* Retrieve the first pendconn from tree <pendconns>. Classes are always
* considered first, then the time offset. The time does wrap, so the