Whamcloud - gitweb
LU-17197 obdclass: preserve fairness when waiting for rpc slot 38/52738/5
authorShaun Tancheff <shaun.tancheff@hpe.com>
Wed, 18 Oct 2023 03:54:59 +0000 (22:54 -0500)
committerAndreas Dilger <adilger@whamcloud.com>
Tue, 7 Nov 2023 20:23:21 +0000 (20:23 +0000)
commit7e28964085a4d98111b926fe125abc7f815e70be
tree0179035462ec7403202cd089b38a93c1d21b5687
parentd7d16447a00be217246f1d73ecaf43a05a832442
LU-17197 obdclass: preserve fairness when waiting for rpc slot

When obd_get_mod_rpc_slot() waits for an available slot it places the
waiting thread at the HEAD of the queue, so it will be woken before
anything else that is already queued.  This is clearly unfair and can
hurt performance.

So change to always add to the tail to ensure a FIFO ordering (except
that CLOSE might sometimes be woken a bit early).

This regression was introduced in a rewrite that was supposed to make
waiting more fair - by avoiding a broadcast wakeup for "close"
requests.

Also fix some stale comments and expose __add_wait_queue_entry_tail

Running mdtest with the patch applied shows about a 3% improvement:

                             master            patched
  mdtest-easy-write      350.585906 kIOPS   353.783545 kIOPS
   mdtest-easy-stat     1320.329353 kIOPS  1408.320419 kIOPS
 mdtest-easy-delete      285.084103 kIOPS   289.625900 kIOPS
            [SCORE]      509.115803 kiops   524.516113 kiops

Fixes: 5243630b09d2 ("LU-15947 obdclass: improve precision of wakeups for mod_rpcs")
Signed-off-by: Mr NeilBrown <neilb@suse.de>
Signed-off-by: Shaun Tancheff <shaun.tancheff@hpe.com>
Change-Id: If767c4299bcbab71589b0f3c01e85bf461686ca5
libcfs/include/libcfs/linux/linux-wait.h
libcfs/libcfs/linux/linux-wait.c
lustre/obdclass/genops.c