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

"Chuck Lever" <[email protected]> Tue, 28 Jul 2026 22:16:22 -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:55 PM, Jakub Kicinski wrote:
> On Tue, 28 Jul 2026 21:46:57 -0400 Chuck Lever wrote:
>> On Tue, Jul 28, 2026, at 9:43 PM, Jakub Kicinski wrote:
>> > On Mon, 20 Jul 2026 10:27:54 -0400 Chuck Lever wrote: =20
>> >> - The no-data cap (patch 1) is a prerequisite, not a stand-alone f=
ix.
>> >>   Once control records reach read_sock, a record carrying no paylo=
ad
>> >>   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. =20
>> >
>> > Is it just me or this is incomprehensible slop? Run on sentences=20
>> > full of terms no kernel developer would use? =20
>>=20
>> It makes sense to me. Which terms do you have trouble with?
>>=20
>> Patch 1 is moving to a pre-requisite series, so this can be dropped
>> from the cover letter.
>
> Did you write this? If not what model did you use?

I did not write this text. It was written with Opus 5 and reviewed by
gpt-5.6, sashiko (gemini), and myself, repeatedly.

I=E2=80=99m not aware of a requirement to add an AI-related tag to a ser=
ies
cover letter, nor am I aware of any a priori prohibition against
using assistance when composing one.

I agree it needlessly duplicates content from the patch commit messages.
It is bland, but I don=E2=80=99t find it nonsensical, or I wouldn=E2=80=99=
t have posted
it for review.


--=20
Chuck Lever