Whamcloud - gitweb
LU-13799 llite: Do not get/put DIO pages 38/39438/27
authorPatrick Farrell <pfarrell@whamcloud.com>
Fri, 30 Jul 2021 16:14:16 +0000 (12:14 -0400)
committerOleg Drokin <green@whamcloud.com>
Wed, 26 Jan 2022 05:14:23 +0000 (05:14 +0000)
commit881b4c722296ff7ac22c6fd7988363f2cdad9f1e
tree2783fe545ae1a7b414675a24315514d5493dd3bc
parent7f9b8465bc1125e51aa29cdc27db9a9d6fdc0b89
LU-13799 llite: Do not get/put DIO pages

We've already told the kernel we're working with these pages
using the get/put_user_pages functions, and userspace must
hold references on them throughout the i/o anyway.

So getting/putting these vmpages is unnecessary.  This
saves around 7% of the time in DIO page submission, netting
about that much of a performance improvement.

This patch reduces i/o time in ms/GiB by:
Write: 22 ms/GiB
Read: 19 ms/GiB

Totals:
Write: 135 ms/GiB
Read: 143 ms/GiB

mpirun -np 1  $IOR -w -r -t 64M -b 64G -o ./iorfile --posix.odirect

With previous patches in series:
write     6470 MiB/s
read      6354 MiB/s

Plus this patch:
write     7531 MiB/s
read      7179 MiB/s

Signed-off-by: Patrick Farrel <pfarrell@whamcloud.com>
Change-Id: Ic457c21ebca9624da2422463da453b535dcfd10e
Reviewed-on: https://review.whamcloud.com/39438
Reviewed-by: Shaun Tancheff <shaun.tancheff@hpe.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Yingjin Qian <qian@ddn.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/llite/vvp_page.c