From 5614305853f0f87a2418a6fb7ca472a8ef966084 Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Sat, 14 May 2011 08:55:50 +0100 Subject: [PATCH] rx: Reverse the consumption order of idle queue Currently, the rx server thread idle queue is used in an LRU manner. This means that we round robin requests between all of the threads configured on a given system, which means that we end up thrashing CPU caches on machines whose workload doesn't require that all of the configured threads be used. Change this so that we always use the most recently idle thread. This isn't as "fair" to all of our waiting threads, but should mean that we scale better on SMP machines, as a thread that is recently idle is likely to have been recently scheduled. Performance numbers to follow ... Change-Id: I6072183b5ca72754755c3ac854e30a064c4187e5 Reviewed-on: http://gerrit.openafs.org/4871 Reviewed-by: Derrick Brashear Tested-by: BuildBot Reviewed-by: Jeffrey Altman Tested-by: Jeffrey Altman --- src/rx/rx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/rx/rx.c b/src/rx/rx.c index 5083aac..5c08736 100644 --- a/src/rx/rx.c +++ b/src/rx/rx.c @@ -4748,7 +4748,7 @@ rxi_AttachServerProc(struct rx_call *call, queue_Append(&rx_incomingCallQueue, call); } } else { - sq = queue_First(&rx_idleServerQueue, rx_serverQueueEntry); + sq = queue_Last(&rx_idleServerQueue, rx_serverQueueEntry); /* If hot threads are enabled, and both newcallp and sq->socketp * are non-null, then this thread will process the call, and the -- 1.9.4