Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Message-ID: <fa8ac10aa7c12f344109be815c6e3da480a33137.camel@kernel.org>
Subject: Re: [PATCHv2 nfsd/master 7/7] dlm: implement EXPORT_OP_ASYNC_LOCK
From:   Jeff Layton <jlayton@kernel.org>
To:     Alexander Aring <aahringo@redhat.com>, linux-nfs@vger.kernel.org
Cc:     gfs2@lists.linux.dev, ocfs2-devel@lists.linux.dev,
        linux-fsdevel@vger.kernel.org, teigland@redhat.com,
        rpeterso@redhat.com, agruenba@redhat.com,
        trond.myklebust@hammerspace.com, anna@kernel.org,
        chuck.lever@oracle.com
Date:   Thu, 14 Sep 2023 06:29:17 -0400
In-Reply-To: <20230912215324.3310111-8-aahringo@redhat.com>
References: <20230912215324.3310111-1-aahringo@redhat.com>
         <20230912215324.3310111-8-aahringo@redhat.com>
Content-Type: text/plain; charset="ISO-8859-15"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

On Tue, 2023-09-12 at 17:53 -0400, Alexander Aring wrote:
> This patch is activating the EXPORT_OP_ASYNC_LOCK export flag to
> signal lockd that both filesystems are able to handle async lock
> requests. The cluster filesystems gfs2 and ocfs2 will redirect their
> lock requests to DLMs plock implementation that can handle async lock
> requests.
>=20
> Signed-off-by: Alexander Aring <aahringo@redhat.com>
> ---
>  fs/gfs2/export.c  | 1 +
>  fs/ocfs2/export.c | 1 +
>  2 files changed, 2 insertions(+)
>=20
> diff --git a/fs/gfs2/export.c b/fs/gfs2/export.c
> index cf40895233f5..ef1013eff936 100644
> --- a/fs/gfs2/export.c
> +++ b/fs/gfs2/export.c
> @@ -192,5 +192,6 @@ const struct export_operations gfs2_export_ops =3D {
>  	.fh_to_parent =3D gfs2_fh_to_parent,
>  	.get_name =3D gfs2_get_name,
>  	.get_parent =3D gfs2_get_parent,
> +	.flags =3D EXPORT_OP_ASYNC_LOCK,
>  };
> =20
> diff --git a/fs/ocfs2/export.c b/fs/ocfs2/export.c
> index eaa8c80ace3c..b8b6a191b5cb 100644
> --- a/fs/ocfs2/export.c
> +++ b/fs/ocfs2/export.c
> @@ -280,4 +280,5 @@ const struct export_operations ocfs2_export_ops =3D {
>  	.fh_to_dentry	=3D ocfs2_fh_to_dentry,
>  	.fh_to_parent	=3D ocfs2_fh_to_parent,
>  	.get_parent	=3D ocfs2_get_parent,
> +	.flags		=3D EXPORT_OP_ASYNC_LOCK,
>  };

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