Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
From:   Chuck Lever III <chuck.lever@oracle.com>
To:     Linus Torvalds <torvalds@linux-foundation.org>
CC:     Neil Brown <neilb@suse.de>,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Jeff Layton <jlayton@kernel.org>
Subject: Re: [GIT PULL] nfsd changes for v6.7 (early)
Thread-Topic: [GIT PULL] nfsd changes for v6.7 (early)
Thread-Index: AQHaB07bp3SsPgtJx0O5OMKo6TKfuLBi4k0AgAAei4A=
Date:   Mon, 30 Oct 2023 23:21:06 +0000
Message-ID: <96800661-0F30-4F9E-89E4-C0B032EFDEB9@oracle.com>
References: <34E014FF-351E-4977-B694-060A5DADD35A@oracle.com>
 <CAHk-=wifhiJ-QbcwrH0RzPaKeZv93GKDQuBUth18ay=sLu5CVA@mail.gmail.com>
In-Reply-To: <CAHk-=wifhiJ-QbcwrH0RzPaKeZv93GKDQuBUth18ay=sLu5CVA@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?qJqp7NSrQMYPJQVD+vvl0k+gpgH3HAPJoHC1odL7PkN7s78RS+qql6wG0peR?=
 =?us-ascii?Q?trcx4PaAtlpZ+CmvgvzT/wsMvJlatXANjLvhxR23O/a8bnvB97P05PLQhguG?=
 =?us-ascii?Q?ceae5MKrLhsDG7wteTsrflvC9+5P/s10dZukyUdhwSG/LqsFL2KkJEqr0x2h?=
 =?us-ascii?Q?OneNKkIa0Pn15iIrrkHn86Qw+D5x8G7+TNHgERWC5CdWy70gT4qck+wGbe3p?=
 =?us-ascii?Q?GH2zOgGDcurMyByjBNqgIxidi74FdSdBnutVTep8ZwlZrlEccChfPVeHzpWq?=
 =?us-ascii?Q?qNvz4xphltXg86tEWJymMNhM72oXj06/sYU5tI/UQIhFt3redWFqotrFx7Aa?=
 =?us-ascii?Q?McRASPmvnzGXKkQ3bSw/jHVn9vkRtZgEtmbZKp9/rArOIxJDsFMmhoOu2s4g?=
 =?us-ascii?Q?JPzfQGZ59wRvruxVvgCp7h4pYCL7/iZyTEnjnY9UlvXuQgClJ3fVsEFmD1G8?=
 =?us-ascii?Q?0fn2ovV2Grl+mxc0XbY3qqL2xk3zQrpmhdDEi5GQM3vnRYXYjTFwa8m8VCsN?=
 =?us-ascii?Q?O2XEkgWr6sLZAPSAuB9QcemKRs/YG1DVUhIByhep2gVo+Cw/I8ftb1fQ40AW?=
 =?us-ascii?Q?M528jo36fYHCsuMkwPxDmL+jqAUGOBU5EoG7nl6LXkTchfyRUE1CKe85SguX?=
 =?us-ascii?Q?7Uv3P4+mcMjgShq2E/vkYPxJ1GFd8ZFs1VsKc4rg1jxJx5Pl0vhc0eZGpHyC?=
 =?us-ascii?Q?LFRe3a8sk51TFv/QksVAwE0ZmhDFD6jdNiuGLGyhU5cREuUJmQ0Oe3xeQD3E?=
 =?us-ascii?Q?p8gq//Yh/kooC4OrLPcI2P8vt4Vcybl16c+U4kBqzhNZpPFgpHs19cWiA1fj?=
 =?us-ascii?Q?xR17ZnAtGRL4mxUehfkGrI4m0vt7ChXZQF9XZ1QZMxYE8wZfwZKCbhP8BrZh?=
 =?us-ascii?Q?1aPsyKxZ/pzRhOZe5h3kDpinHfeQJYReYrLdGgvzYZIdjCcM+DA8tOEQhzub?=
 =?us-ascii?Q?ls/9o7NO6o92VG/ChfeSu0sOGspydZ4uWZ1lSYWqRwCD3xT9/mWR8VMnSNt+?=
 =?us-ascii?Q?cdu9zAe3dMNtLeidv3kV18HBSPyR5rDXZWBFLWiqnmJbB0JkrjIQLOTfuV8c?=
 =?us-ascii?Q?gy9fmBJ2Z5BcAd4e6Jn5hIuTRkAzN3vsKY36eEp6Ntvmeq9Clq92Ch6bML8I?=
 =?us-ascii?Q?HB9FIb5rSd3Lavx0ops6yNU1OstSXyBl/5CE2wwIBReyC5j3oURcpLO9S6Ji?=
 =?us-ascii?Q?AOy5cUpdRlu2yA9KyV/g7/PDTRFstuIa/zC/TORxMC57TEIcJ3b5Z3fnL7+d?=
 =?us-ascii?Q?FE7KYt21YajXGx3vEP2TRz5sBoOganKciBwyeMkfXNZPK+S8nzOMVfkM3amm?=
 =?us-ascii?Q?o7AqUejYvFmmjBaHnLz1iEDAW4m1jWKBGEJ5GrP57KOHklxEOOE6RDMOo6ua?=
 =?us-ascii?Q?4HeY6gnHFsnI/JE/qFMnu3haAojfSkk7UkZamwvj4G1x7/H6cwDbqr8BSfXJ?=
 =?us-ascii?Q?i9lBtNKGozYj0+itGHLrbYlzYnmzQXxmZsF2lsbhHpt8MhGAjNf8QIbxKhs3?=
 =?us-ascii?Q?D2J8OE1JDVia/A8B9QAJrvdtIJHD9IIZ6er6Q9CJA1pJBTKO65zO1gu9wZf4?=
 =?us-ascii?Q?lwf5sD5BGETCibdWGcXIBonsfUy3+4t5LTFwbYO/?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <D5D70A32ED9B6C4CBA0BBBF5FA7FE599@namprd10.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 9PIUqcAQ/8wT62X0exxreU93xclTXEXeJA3xq2fCndeu/C4Z59igr+9PeaMbvMArFSG6H8S5oIqlbDdaimKnP/HTCql+HcTcQlfT28iP94LHq60tIlL+uon0GEFcxgXIQJTSBpvci3bqJihbmxSwF+i9baO84JItLiW964SKqt3at/n/1QAZeWpdmrlnyWdIOXhwbikA7gE8bEQ+OLCg0YALWpus1/TMtT89TqovBpyp15cH9RJpPCjgquP1J1r6x5yBSq0fT1VTCVtym9T3YQP/csRQdVp311oQNWuUp19Jvvtu9/Qm5LVNkdd+c7TXGNiIqvszRD4g5rXoqH6fS+C0zy3NeSuqrbau2darrOQbbMO1r3m+q24N6XlNsOCT3eErkQwQbOymsCYn0SR2vH/j/BzJMUm59tDPvzzlNoZREA+XKoCxJHniivg1llkZ/wUlJRpRbWj6GmGYHGhv1xUtznisO21XWKeqhrGnlCq3kFnrIT7GzOU2HVsFwaSb3gJz8utHGl/vNVuIETBKiXOik9p2ZFgkYlBXbPk5AC7ScLqOGJ0br+0zlZMApYDwX+W/UcGIG4B1fAPXkIQXFAK4DdQTaPNSutmLt35dDwxkPJci3sttDQFDOOA2Ky92AkXdzJ2E+7sRI57OJ0yy5o8mITdEkveOPahvhNyo1sMUsERl9FNMrRJxLpBZlq30yCDmNAbirlkJq9OYz0gi+rPVw7exjXUl1s4eEE5BLkEXQLBLsd1vLeOwt5ep6HklaoROvey0Pq4qZNeI6L4jgmLdPWLbd5ClITasvhueuFTGpY3b8MLmiJ6C+KiNnx1cd1/X12pjiJ2POVsQRL53Nw0CG3WFlWmXr8Z6JGW2NW6oPWhpyYAJyp4ixXwu5DRR
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DS7PR10MB5134.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 73d8eee8-5e26-444c-ef77-08dbd99ee445
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Oct 2023 23:21:06.0326
 (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: TtKtROoG/6owFIKkRtS/ZI8pvnxC8RLiyuaYtwhfjMTCcfm7hLTrWySEQxNjZS5T/Kb736Rbccz+JYHITSGYeg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1PR10MB4500
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.987,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-10-30_13,2023-10-27_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0
 malwarescore=0 mlxscore=0 mlxlogscore=999 adultscore=0 phishscore=0
 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2310240000 definitions=main-2310300182
X-Proofpoint-GUID: 5CfScFAsXm1X8hqwlWL86tvbuxnHl9Xw
X-Proofpoint-ORIG-GUID: 5CfScFAsXm1X8hqwlWL86tvbuxnHl9Xw
X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-nfs.vger.kernel.org>
X-Mailing-List: linux-nfs@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Mon, 30 Oct 2023 16:21:21 -0700 (PDT)


> On Oct 30, 2023, at 2:31 PM, Linus Torvalds <torvalds@linux-foundation.or=
g> wrote:
>=20
> On Wed, 25 Oct 2023 at 04:24, Chuck Lever III <chuck.lever@oracle.com> wr=
ote:
>>=20
>> This release completes the SunRPC thread scheduler work that was
>> begun in v6.6. The scheduler can now find an svc thread to wake in
>> constant time and without a list walk. Thanks again to Neil Brown
>> for this overhaul.
>=20
> Btw, the "help" text for the new Kconfig option that this introduces
> is just ridiculously bad.
>=20
> I react to these things, because I keep telling people that our
> Kconfig is one of the nastier parts to people just building and
> testing their own kernels. Yes, you can start with whatever distro
> default config, and build your own, and install it, but when people
> then introduce new options and ask insane and unhelpful questions,
> that scares off any sane person.
>=20
> So Kconfig questions really need to make sense, and they need to have
> help messages that are useful..
>=20
> Honestly, that LWQ_TEST option probably fails both cases.  The
> "testing" is a toy, and the Kconfig option is horrific. I literally
> think that we would be better off removing that code. Any bug found by
> that testv would be so fundamental as to not be worth testing for.

I have to admit I didn't look too closely at that part
of the series, except to note that there's no maintainer
of record for those files. That's probably why there was
little initial pushback on the scant help text.

Do you need a refreshed PR with the testing bit removed,
or can you live with Neil or me sending a subsequent
fix-up later in the merge window?


--
Chuck Lever