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;
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
From:   "NeilBrown" <neilb@suse.de>
To:     "Chuck Lever" <chuck.lever@oracle.com>
Cc:     "Jeff Layton" <jlayton@kernel.org>, linux-nfs@vger.kernel.org
Subject: Re: [PATCH 06/14] SUNRPC: change various server-side #defines to enum
In-reply-to: <169075508761.32308.11692865993389725826@noble.neil.brown.name>
References: <168966227838.11075.2974227708495338626.stgit@noble.brown>,
 <168966228863.11075.8173252015139876869.stgit@noble.brown>,
 <ZLaVtpLf1k2Ig5Kz@bazille.1015granger.net>,
 <ZMaIWPyEm83GaS0q@tissot.1015granger.net>,
 <169075508761.32308.11692865993389725826@noble.neil.brown.name>
Date:   Mon, 31 Jul 2023 08:14:21 +1000
Message-id: <169075526174.32308.13675913011176702739@noble.neil.brown.name>
Precedence: bulk

On Mon, 31 Jul 2023, NeilBrown wrote:
> On Mon, 31 Jul 2023, Chuck Lever wrote:
> > On Tue, Jul 18, 2023 at 09:37:58AM -0400, Chuck Lever wrote:
> > > On Tue, Jul 18, 2023 at 04:38:08PM +1000, NeilBrown wrote:
> > > > When a sequence of numbers are needed for internal-use only, an enum =
is
> > > > typically best.  The sequence will inevitably need to be changed one
> > > > day, and having an enum means the developer doesn't need to think abo=
ut
> > > > renumbering after insertion or deletion.  The patch will be easier to
> > > > review.
> > >=20
> > > Last sentence needs to define the antecedant of "The patch".
> >=20
> > I've changed the last sentence in the description to "Such patches
> > will be easier ..."
> >=20
> > I've applied 1/5 through 5/5, with a few cosmetic changes, to the
> > SUNRPC threads topic branch. 6/6 needed more work:
>=20
> Ah - ok.  I was all set to resubmit with various changes and
> re-ordering.  I guess I waited too long.
>=20
> > All this will appear in the nfsd repo later today.
>=20
> Not yet....

No, they are there - the top date looked rather old so I thought there
was nothing new yet.  Sorry.

NeilBrown