Whamcloud - gitweb
LU-10826 ptlrpc: fix test_req_buffer_pressure behavior 90/31690/2
authorBruno Faccini <bruno.faccini@intel.com>
Tue, 20 Mar 2018 10:06:47 +0000 (11:06 +0100)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 9 Apr 2018 19:51:08 +0000 (19:51 +0000)
In 2nd patch for LU-9372, to allow limiting number of rqbd-buffers,
a wrong and unnecessary test had been added to enhance
test_req_buffer_pressure feature.
This patch fixes this issue by removing such test.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: I8f1298fabbbc4e0d92078dcf49192ff4f0fbc907
Reviewed-on: https://review.whamcloud.com/31690
Tested-by: Jenkins
Reviewed-by: Wang Shilong <wshilong@ddn.com>
Reviewed-by: Li Dongyang <dongyangli@ddn.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ptlrpc/service.c

index 4cbab10..2930f85 100644 (file)
@@ -140,8 +140,7 @@ ptlrpc_grow_req_bufs(struct ptlrpc_service_part *svcpt, int post)
         for (i = 0; i < svc->srv_nbuf_per_group; i++) {
                 /* NB: another thread might have recycled enough rqbds, we
                 * need to make sure it wouldn't over-allocate, see LU-1212. */
-               if (test_req_buffer_pressure ||
-                   svcpt->scp_nrqbds_posted >= svc->srv_nbuf_per_group ||
+               if (svcpt->scp_nrqbds_posted >= svc->srv_nbuf_per_group ||
                    (svc->srv_nrqbds_max != 0 &&
                     svcpt->scp_nrqbds_total > svc->srv_nrqbds_max))
                        break;