Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <a3d5fb640e1614d20ba29d14f4a4ef5b3aa2b266.camel@kernel.org>
Subject: Re: [PATCH v4 0/2] NFSD: memory shrinker for NFSv4 clients
From:   Jeff Layton <jlayton@kernel.org>
To:     Dai Ngo <dai.ngo@oracle.com>, chuck.lever@oracle.com
Cc:     linux-nfs@vger.kernel.org
Date:   Wed, 31 Aug 2022 06:00:24 -0400
In-Reply-To: <1661896113-8013-1-git-send-email-dai.ngo@oracle.com>
References: <1661896113-8013-1-git-send-email-dai.ngo@oracle.com>
Content-Type: text/plain; charset="ISO-8859-15"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.44.4 (3.44.4-1.fc36) 
MIME-Version: 1.0
Precedence: bulk

On Tue, 2022-08-30 at 14:48 -0700, Dai Ngo wrote:
> This patch series implements the memory shrinker for NFSv4 clients
> to react to system low memory condition.
>=20
> The memory shrinker's count callback is used to trigger the laundromat.
> The actual work of destroying the expired clients is done by the
> laundromat itself. We can not destroying the expired clients on the
> memory shrinler's scan callback context to avoid possible deadlock.
>=20
> By destroying the expired clients, all states associated with these
> clients are also released.
>=20
> v2:
> . fix kernel test robot errors in nfsd.h when CONFIG_NFSD_V4 not defined.
>=20
> v3:
> . add mod_delayed_work in nfsd_courtesy_client_scan to kick start
>   the laundromat.
>=20
> v4:
> . replace the use of xchg() with vanilla '=3D' in patch 1.
>=20
> ---
>=20
> Dai Ngo (2):
>       NFSD: keep track of the number of courtesy clients in the system
>       NFSD: add shrinker to reap courtesy clients on low memory condition
>=20
>  fs/nfsd/netns.h     |  5 ++++
>  fs/nfsd/nfs4state.c | 65 ++++++++++++++++++++++++++++++++++++++++++++---=
-
>  fs/nfsd/nfsctl.c    |  6 +++--
>  fs/nfsd/nfsd.h      |  9 +++++--
>  4 files changed, 76 insertions(+), 9 deletions(-)

Reviewed-by: Jeff Layton <jlayton@kernel.org>