Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Chuck Lever <chuck.lever@oracle.com>
To:     Steve Dickson <SteveD@RedHat.com>
CC:     Bruce Fields <bfields@fieldses.org>,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH 0/7 V4] The NFSv4 only mounting daemon.
Thread-Topic: [PATCH 0/7 V4] The NFSv4 only mounting daemon.
Thread-Index: AQHXEEhZkdBHYefJxka6Z8w4OsszKqpykGAA
Date:   Wed, 3 Mar 2021 18:10:21 +0000
Message-ID: <06F2DFB7-8EA1-4ACE-BEA9-A68ACB99B361@oracle.com>
References: <20210219200815.792667-1-steved@redhat.com>
 <20210224204944.GG11591@fieldses.org>
 <0dcefec0-1fbf-d43a-b508-cb06edfea866@RedHat.com>
In-Reply-To: <0dcefec0-1fbf-d43a-b508-cb06edfea866@RedHat.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-ID: <955B5FA4DB7E864C9C119DCDF0402B6E@namprd10.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: fc916c72-63f2-4e7d-0426-08d8de6f9c0b
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Mar 2021 18:10:21.2932
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: aF3Fs0fKX0xePZlxM8R5Qk6QuBhZH1nuTKXQ6wvzgmd5pWrHOm5VLGsBU73K5lTUmSJwi7z8AlrVYsLoN0DLFQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR10MB4446
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9912 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 spamscore=0 suspectscore=0
 mlxlogscore=999 bulkscore=0 adultscore=0 phishscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2103030127
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9912 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 impostorscore=0 phishscore=0 mlxscore=0
 suspectscore=0 adultscore=0 malwarescore=0 priorityscore=1501 bulkscore=0
 lowpriorityscore=0 spamscore=0 mlxlogscore=999 clxscore=1015
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2103030128
Precedence: bulk
List-ID: <linux-nfs.vger.kernel.org>
X-Mailing-List: linux-nfs@vger.kernel.org


> On Mar 2, 2021, at 5:39 PM, Steve Dickson <SteveD@RedHat.com> wrote:
>=20
>=20
>=20
> On 2/24/21 3:49 PM, J. Bruce Fields wrote:
>> On Fri, Feb 19, 2021 at 03:08:08PM -0500, Steve Dickson wrote:
>>> nfsv4.exportd is a daemon that will listen for only v4 mount upcalls.
>>> The idea is to allow distros to build a v4 only package
>>> which will have a much smaller footprint than the
>>> entire nfs-utils package.
>>>=20
>>> exportd uses no RPC code, which means none of the=20
>>> code or arguments that deal with v3 was ported,=20
>>> this again, makes the footprint much smaller.=20
>>>=20
>>> The following options were ported:
>>>    * multiple threads
>>>    * state-directory-path option
>>>    * junction support (not tested)
>>>=20
>>> The rest of the mountd options were v3 only options.
>>>=20
>>> V2:
>>>  * Added two systemd services: nfsv4-exportd and nfsv4-server
>>>  * nfsv4-server starts rpc.nfsd -N 3, so nfs.conf mod not needed.
>>=20
>> We really shouldn't make users change how they do things.
> If they only want v4 support...  I'm thinking is a lot easier to
> simple do a nfsv4.server start verse edit config files.
>=20
>>=20
>> Whatever we do, "systemctl start nfs-server" should still be how they
>> start the NFS server.
> Again.. if they install the nfsv4-utils verse the nfs-utils package
> they should expect change... IMHO..

I would prefer having a single nfs-utils package. I don't see
a need for a proliferation of these extra little programs --
let's just make the usual suspects behave better.


> steved.
>>=20
>> --b.
>>=20
>>>=20
>>> V3: Changed the name from exportd to nfsv4.exportd
>>>=20
>>> V4: Added compile flag that will compile in the NFSv4 only server
>>>=20
>>> Steve Dickson (7):
>>>  exportd: the initial shell of the v4 export support
>>>  exportd: Moved cache upcalls routines into libexport.a
>>>  exportd: multiple threads
>>>  exportd/exportfs: Add the state-directory-path option
>>>  exportd: Enabled junction support
>>>  exportd: systemd unit files
>>>  exportd: Added config variable to compile in the NFSv4 only server.
>>>=20
>>> .gitignore                                |   1 +
>>> configure.ac                              |  14 ++
>>> nfs.conf                                  |   4 +
>>> support/export/Makefile.am                |   3 +-
>>> {utils/mountd =3D> support/export}/auth.c   |   4 +-
>>> {utils/mountd =3D> support/export}/cache.c  |  46 +++-
>>> support/export/export.h                   |  34 +++
>>> {utils/mountd =3D> support/export}/fsloc.c  |   0
>>> {utils/mountd =3D> support/export}/v4root.c |   0
>>> {utils/mountd =3D> support/include}/fsloc.h |   0
>>> systemd/Makefile.am                       |   6 +
>>> systemd/nfs.conf.man                      |  10 +
>>> systemd/nfsv4-exportd.service             |  12 +
>>> systemd/nfsv4-server.service              |  31 +++
>>> utils/Makefile.am                         |   4 +
>>> utils/exportd/Makefile.am                 |  65 +++++
>>> utils/exportd/exportd.c                   | 276 ++++++++++++++++++++++
>>> utils/exportd/exportd.man                 |  81 +++++++
>>> utils/exportfs/exportfs.c                 |  21 +-
>>> utils/exportfs/exportfs.man               |   7 +-
>>> utils/mountd/Makefile.am                  |   5 +-
>>> 21 files changed, 606 insertions(+), 18 deletions(-)
>>> rename {utils/mountd =3D> support/export}/auth.c (99%)
>>> rename {utils/mountd =3D> support/export}/cache.c (98%)
>>> create mode 100644 support/export/export.h
>>> rename {utils/mountd =3D> support/export}/fsloc.c (100%)
>>> rename {utils/mountd =3D> support/export}/v4root.c (100%)
>>> rename {utils/mountd =3D> support/include}/fsloc.h (100%)
>>> create mode 100644 systemd/nfsv4-exportd.service
>>> create mode 100644 systemd/nfsv4-server.service
>>> create mode 100644 utils/exportd/Makefile.am
>>> create mode 100644 utils/exportd/exportd.c
>>> create mode 100644 utils/exportd/exportd.man
>>>=20
>>> --=20
>>> 2.29.2

--
Chuck Lever