Re: [PATCH bpf 2/2] bpf: sockmap, add af_unix test with both sockets in map

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH bpf 2/2] bpf: sockmap, add af_unix test with both sockets in map
From: Jakub Sitnicki <jakub@xxxxxxxxxxxxxx>
Date: Mon, 06 Nov 2023 15:42:17 +0100
Cc: bpf@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, yangyingliang@xxxxxxxxxx, martin.lau@xxxxxxxxxx
In-reply-to: <878r7bjb1a.fsf@cloudflare.com>
User-agent: mu4e 1.6.10; emacs 28.3

On Mon, Nov 06, 2023 at 01:44 PM +01, Jakub Sitnicki wrote:
> But I'm able to reproduce the bug by running the VSOCK redir test:

I should clarify - the bug reproduces only without the patch #1 applied.

The fix is good. It just has some unneeded bits (unix_dgram), IMO.

References:
- [PATCH bpf 0/2] sockmap fix for KASAN_VMALLOC and af_unix
  - From: John Fastabend
- [PATCH bpf 2/2] bpf: sockmap, add af_unix test with both sockets in map
  - From: John Fastabend
- Re: [PATCH bpf 2/2] bpf: sockmap, add af_unix test with both sockets in map
  - From: Jakub Sitnicki

Prev by Date: Re: [PATCH bpf 2/2] bpf: sockmap, add af_unix test with both sockets in map
Next by Date: Re: [PATCH] bpf: Use E2BIG instead of ENOENT
Previous by thread: Re: [PATCH bpf 2/2] bpf: sockmap, add af_unix test with both sockets in map
Next by thread: pull-request: bpf-next 2023-10-16
Index(es):
- Date
- Thread

[Index of Archives] [Linux Samsung SoC] [Linux Rockchip SoC] [Linux Actions SoC] [Linux for Synopsys ARC Processors] [Linux NFS] [Linux NILFS] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI]

Powered by Linux