Re: [PATCH v3] bpf, sockmap: Fix cork ownership in tcp_bpf_sendmsg()

From: Chengfeng Ye

Date: Fri Jul 24 2026 - 06:11:24 EST


On Fri, Jul 24, 2026 at 5:40 PM Jakub Sitnicki <jakub@xxxxxxxxxxxxxx> wrote:
>
> On Thu, Jul 23, 2026 at 10:00 AM -07, Jakub Kicinski wrote:
> > On Fri, 24 Jul 2026 00:26:02 +0800 Chengfeng Ye wrote:
> >> Subject: [PATCH v3] bpf, sockmap: Fix cork ownership in tcp_bpf_sendmsg()
> >
> > Since you are spamming the list with the reposts of this could you
> > please add [PATCH bpf] to the subject so that netdev CI is not confused
> > into caring?
>
> +1 because we need bpf CI to run sockmap selftests on this patch.
> Please point your agent at Documentation/bpf/bpf_devel_QA.rst.
>
> Contrary to Emil's ask, I'd recommend fixing one issue at a time.
> Reviewing multiple fixes crammed into one patch is harder, IMO.

Sorry for the disturbance. I was trying to fix the other issues
reported by sashiko,
but it seems it is harder to fix multiple bugs in one patch despite
being the same
function, as more issues are flagged by sashiko when a new patch is sent...

Then please just take the original v1 patch for reference and ignore v2 and v3,
and I will stop sending new patches for this report until there is
further patch review.
If there is a new patch, [PATCH bpf] will be used properly.

Best, Chengfeng