[PATCH bpf-next 0/2] bpf: Cleanup & optimization

Michal Luczaj posted 2 patches 1 month, 2 weeks ago
net/core/filter.c   | 58 ++++++++++++++++++++++++-----------------------------
net/core/sock_map.c |  8 ++++----
2 files changed, 30 insertions(+), 36 deletions(-)
[PATCH bpf-next 0/2] bpf: Cleanup & optimization
Posted by Michal Luczaj 1 month, 2 weeks ago
Non-functional cleanup and a refcnt bumping simplification.

Series extracted from a fix attempt[1] currently under discussion.

[1]: https://lore.kernel.org/bpf/20260803-sockmap-lookup-tcp-leak-v2-0-306e025bfe66@rbox.co/

Signed-off-by: Michal Luczaj <mhal@rbox.co>
---
Michal Luczaj (2):
      bpf: Extract shared reqsk-to-listener upgrade
      bpf, sockmap: Use sock_hold() instead of refcount_inc_not_zero() in lookup

 net/core/filter.c   | 58 ++++++++++++++++++++++++-----------------------------
 net/core/sock_map.c |  8 ++++----
 2 files changed, 30 insertions(+), 36 deletions(-)
---
base-commit: 6f033615ef8fb2374daa7e50a8ff68616bc850d2
change-id: 20260803-sockmap-lookup-get-ref-3f76e4c1ffe2

Best regards,
--  
Michal Luczaj <mhal@rbox.co>
Re: [PATCH bpf-next 0/2] bpf: Cleanup & optimization
Posted by Jakub Sitnicki 1 month, 2 weeks ago
On Thu, Aug 13, 2026 at 02:41 PM +02, Michal Luczaj wrote:
> Non-functional cleanup and a refcnt bumping simplification.
>
> Series extracted from a fix attempt[1] currently under discussion.
>
> [1]: https://lore.kernel.org/bpf/20260803-sockmap-lookup-tcp-leak-v2-0-306e025bfe66@rbox.co/

Discussion seems to have stalled. Or it's just vacation season.
Anyway, I'm fine with fixing the refcounting your way.
Merely wanted to offer an alternative idea for consideration.
Thanks for working on this.
Re: [PATCH bpf-next 0/2] bpf: Cleanup & optimization
Posted by Michal Luczaj 1 month, 1 week ago
On 8/14/26 12:45, Jakub Sitnicki wrote:
> On Thu, Aug 13, 2026 at 02:41 PM +02, Michal Luczaj wrote:
>> Non-functional cleanup and a refcnt bumping simplification.
>>
>> Series extracted from a fix attempt[1] currently under discussion.
>>
>> [1]: https://lore.kernel.org/bpf/20260803-sockmap-lookup-tcp-leak-v2-0-306e025bfe66@rbox.co/
> 
> Discussion seems to have stalled. Or it's just vacation season.
> Anyway, I'm fine with fixing the refcounting your way.
> Merely wanted to offer an alternative idea for consideration.
> Thanks for working on this.

All right, I'll wait for this series to land in bpf.git (to avoid merge
conflicts) and then repost the fixes.

thanks,
Michal