Re: [PATCH v2] nfsd: Use MD5 library instead of crypto_shash

[Chuck Lever <cel@kernel.org>]

On 10/16/25 2:53 PM, Eric Biggers wrote:
> On Thu, Oct 16, 2025 at 02:31:46PM -0400, Chuck Lever wrote:
>> From: Chuck Lever <chuck.lever@oracle.com>
>>
>> On Thu, 16 Oct 2025 11:15:34 -0700, Eric Biggers wrote:
>>> Update NFSD's support for "legacy client tracking" (which uses MD5) to
>>> use the MD5 library instead of crypto_shash.  This has several benefits:
>>>
>>> - Simpler code.  Notably, much of the error-handling code is no longer
>>>   needed, since the library functions can't fail.
>>>
>>> - Improved performance due to reduced overhead.  A microbenchmark of
>>>   nfs4_make_rec_clidname() shows a speedup from 1455 cycles to 425.
>>>
>>> [...]
>>
>> Applied to nfsd-testing, thanks!
>>
>> Note that the posted version of this patch does not apply cleanly to
>> the nfsd-testing branch here:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/cel/linux.git/log/?h=nfsd-testing
>>
>> Be sure to rebase on that branch when posting subsequence patches
>> that target NFSD.
> 
> Sorry, I had just based it on v6.18-rc1.  It did also apply cleanly to
> nfsd-testing f59a20b8390dd with either 'git am -3' or 'git cherry-pick'.
> 
> I noticed you changed the author to yourself when applying.  Could you
> fix that?

That was unintentional, due to applying the patch by hand. Fixed.


-- 
Chuck Lever