Re: [PATCH net-next v2 0/6] Deliver TLS control records to kernel read_sock consumers

"Chuck Lever" <[email protected]> Tue, 28 Jul 2026 21:46:57 -0400
Newsgroups dev.linux.lists.kernel-tls-handshake,org.kernel.vger.linux-kselftest,org.kernel.vger.linux-nfs,org.kernel.vger.netdev
Message-ID <[email protected]>

On Tue, Jul 28, 2026, at 9:43 PM, Jakub Kicinski wrote:
> On Mon, 20 Jul 2026 10:27:54 -0400 Chuck Lever wrote:
>> - The no-data cap (patch 1) is a prerequisite, not a stand-alone fix.
>>   Once control records reach read_sock, a record carrying no payload
>>   stops advancing the caller's read descriptor, so a peer streaming
>>   such records would pin the socket lock and the kernel receive
>>   context for as long as the flood lasts. Bounding consecutive
>>   no-data records supplies the return boundary a system call would
>>   otherwise provide. The cap is scoped to tls_sw_read_sock() alone:
>>   splice and recvmsg run in the caller's own context, reschedule, and
>>   drop the lock on return, so they need nothing.
>
> Is it just me or this is incomprehensible slop? Run on sentences 
> full of terms no kernel developer would use?

It makes sense to me. Which terms do you have trouble with?

Patch 1 is moving to a pre-requisite series, so this can be dropped
from the cover letter.

-- 
Chuck Lever