Received: by 2002:a05:6a10:9afc:0:0:0:0 with SMTP id t28csp1076669pxm; Wed, 23 Feb 2022 17:30:44 -0800 (PST) X-Google-Smtp-Source: ABdhPJxvJWaBpMnsm5Yt98gPfQKZxBsb6d5nwglBiahKNDUQGnysB2roljmSdcBf4vsJpVar4QmS X-Received: by 2002:a63:4743:0:b0:374:5575:adcf with SMTP id w3-20020a634743000000b003745575adcfmr468977pgk.38.1645666244004; Wed, 23 Feb 2022 17:30:44 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1645666243; cv=none; d=google.com; s=arc-20160816; b=dPg/PnrMLuj3BwOGXTOS7AkiP+j1N9Co5SLhI3OnVb689O7FsYqCcz/a9kyi5ILzLz 6glLYJHyJCUYDjbiRtSUVlBUHn0tvUSYIhSraeSO9hLsgnfkgVwAYKzhqZ6gqgDv3oL/ CAv3YAQ4Nn+2ZbeFdIF5VFNp9Rzcef0Si1sd9DWqb9lxafaxTUStFR6xnD+MNMrUh9F0 9VY2o9oJjcbkxXq8+Nek9c8Sq03F93bjTMbMAmRYmEm97imKtS9HqS5h/G/0e4JW3IKc YlGloi9R7rpS4p6zgqqFj25UirebmYqJ4G2bJiknG80QhHXCdzoQxB8Np1qwRtpG4Gw9 4XCA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:to:from:dkim-signature; bh=AB3ZTKKYiQtQoBZZhq/3BI3Pk4xnev9ZuXPIyUAYWYk=; b=pXgbOfb/CnJO+nQubs6B4pZu8zrcNIPgSKwKfZch65YiIzeIqYla/JCeKQrDiCJowf ro+qqzIPh4wz0aPPK0KFlhswuHF/Yk4Rb/EVcuKrGkIYYTTWo07UbTWupZCvTVstm7k4 y1in1Wjj567frqcSrDkpqbAwKj/k+BpNtuiDzSxOd6uuVTueX3XkJHK1lgO4q1MOA/z6 VIyO36A1m/FsVDTyY0tWup1Bbjb6wAX+6dPc80Mi90tWU0NrZ4GElbtDD1F2mI83vUk8 7YO+CMJRcTToSwpzQKmscY4TrgGWcaWuNe7X2n/H/Zx0+gcjc20dXdZwKIViqHPMkqV1 A6jw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=CGzgCi17; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [2620:137:e000::1:18]) by mx.google.com with ESMTPS id c8si1323462pgq.293.2022.02.23.17.30.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 23 Feb 2022 17:30:43 -0800 (PST) Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=CGzgCi17; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id E73F71275F3; Wed, 23 Feb 2022 17:11:18 -0800 (PST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234557AbiBWVXh (ORCPT + 99 others); Wed, 23 Feb 2022 16:23:37 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51994 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242438AbiBWVUE (ORCPT ); Wed, 23 Feb 2022 16:20:04 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 31DA84EA37 for ; Wed, 23 Feb 2022 13:19:34 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id D6364B8212F for ; Wed, 23 Feb 2022 21:19:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 72121C340E7 for ; Wed, 23 Feb 2022 21:19:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1645651171; bh=l2HMLuGhFUfP8jfvtELNX0yflm2H6zW06bpe+16b83A=; h=From:To:Subject:Date:From; b=CGzgCi176kjK28HUyYaRuxF8IygXOnGPEuHQm8p2RXLA6WwsxYvPgJvxpA+quUVix mbTOnUQiHDfTCXpF+/W15tWR98HltVp8bR0W5Zd4Cv6HWCpuEavx8YPA3b/vC37ppI pXzwJ2//S+gECFuqKgBTRSeLGX9Q2mDp4ZCxJgw7j9EX98wE6ZI0bkindnqoPJq1IL wgibzcGUQtHaYjvd3IF0WyHnKvCwQyBHrMqP0g0e/fo8V4YFT9lDEPHtSAfQJbLuoU secN8dG/FFGBLBhY4O0FQcthuyHAHlcBfm94msMW7RTepi6HytIbpTIkRecifGJdy0 C09t6YC6LjEgQ== From: trondmy@kernel.org To: linux-nfs@vger.kernel.org Subject: [PATCH v7 00/21] Readdir improvements Date: Wed, 23 Feb 2022 16:12:44 -0500 Message-Id: <20220223211305.296816-1-trondmy@kernel.org> X-Mailer: git-send-email 2.35.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org From: Trond Myklebust The current NFS readdir code will always try to maximise the amount of readahead it performs on the assumption that we can cache anything that isn't immediately read by the process. There are several cases where this assumption breaks down, including when the 'ls -l' heuristic kicks in to try to force use of readdirplus as a batch replacement for lookup/getattr. This series also implement Ben's page cache filter to ensure that we can improve the ability to share cached data between processes that are reading the same directory at the same time, and to avoid live-locks when the directory is simultaneously changing. -- v2: Remove reset of dtsize when NFS_INO_FORCE_READDIR is set v3: Avoid excessive window shrinking in uncached_readdir case v4: Track 'ls -l' cache hit/miss statistics Improved algorithm for falling back to uncached readdir Skip readdirplus when files are being written to v5: bugfixes Skip readdirplus when the acdirmax/acregmax values are low Request a full XDR buffer when doing READDIRPLUS v6: Add tracing Don't have lookup request readdirplus when it won't help v7: Implement Ben's page cache filter Reduce the use of uncached readdir Change indexing of the page cache to improve seekdir() performance. Trond Myklebust (21): NFS: constify nfs_server_capable() and nfs_have_writebacks() NFS: Trace lookup revalidation failure NFS: Use kzalloc() to avoid initialising the nfs_open_dir_context NFS: Calculate page offsets algorithmically NFS: Store the change attribute in the directory page cache NFS: If the cookie verifier changes, we must invalidate the page cache NFS: Don't re-read the entire page cache to find the next cookie NFS: Adjust the amount of readahead performed by NFS readdir NFS: Simplify nfs_readdir_xdr_to_array() NFS: Reduce use of uncached readdir NFS: Improve heuristic for readdirplus NFS: Don't ask for readdirplus unless it can help nfs_getattr() NFSv4: Ask for a full XDR buffer of readdir goodness NFS: Readdirplus can't help lookup for case insensitive filesystems NFS: Don't request readdirplus when revalidation was forced NFS: Add basic readdir tracing NFS: Trace effects of readdirplus on the dcache NFS: Trace effects of the readdirplus heuristic NFS: Convert readdir page cache to use a cookie based index NFS: Fix up forced readdirplus NFS: Remove unnecessary cache invalidations for directories fs/nfs/dir.c | 450 ++++++++++++++++++++++++----------------- fs/nfs/inode.c | 46 ++--- fs/nfs/internal.h | 4 +- fs/nfs/nfs3xdr.c | 7 +- fs/nfs/nfs4proc.c | 2 - fs/nfs/nfs4xdr.c | 6 +- fs/nfs/nfstrace.h | 122 ++++++++++- include/linux/nfs_fs.h | 19 +- 8 files changed, 421 insertions(+), 235 deletions(-) -- 2.35.1