Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-05-27 Thread Jakub Sitnicki
On Fri, May 24, 2019 at 05:51 PM CEST, John Fastabend wrote: > Jakub Sitnicki wrote: >> >> Now that those pesky crashes are gone, we plan to look into drops when >> doing echo with sockmap. Marek tried running echo-sockmap [1] with >> latest bpf-next (plus mentioned crash fixes) and reports that no

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-05-24 Thread John Fastabend
Jakub Sitnicki wrote: > On Thu, May 23, 2019 at 05:58 PM CEST, John Fastabend wrote: > > [...] > > > >> > >> Thanks for taking a look at it. Setting MSG_DONTWAIT works great for > >> me. No more crashes in sk_stream_wait_memory. I've tested it on top of > >> current bpf-next (f49aa1de9836). Here's

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-05-24 Thread Jakub Sitnicki
On Thu, May 23, 2019 at 05:58 PM CEST, John Fastabend wrote: > [...] > >> >> Thanks for taking a look at it. Setting MSG_DONTWAIT works great for >> me. No more crashes in sk_stream_wait_memory. I've tested it on top of >> current bpf-next (f49aa1de9836). Here's my: >> >> Tested-by: Jakub Sitnick

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-05-23 Thread John Fastabend
[...] > > Thanks for taking a look at it. Setting MSG_DONTWAIT works great for > me. No more crashes in sk_stream_wait_memory. I've tested it on top of > current bpf-next (f49aa1de9836). Here's my: > > Tested-by: Jakub Sitnicki > > The actual I've tested is below, for completeness. > > BTW.

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-05-22 Thread Jakub Sitnicki
Hi John, On Tue, May 21, 2019 at 10:07 PM CEST, John Fastabend wrote: > Jakub Sitnicki wrote: >> Hi Daniel, >> >> On Tue, Feb 19, 2019 at 05:00 PM CET, Daniel Borkmann wrote: >> > On 02/11/2019 10:09 AM, Jakub Sitnicki wrote: >> >> Backlog work for psock (sk_psock_backlog) might sleep while waitin

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-05-21 Thread John Fastabend
Jakub Sitnicki wrote: > Hi Daniel, > > On Tue, Feb 19, 2019 at 05:00 PM CET, Daniel Borkmann wrote: > > On 02/11/2019 10:09 AM, Jakub Sitnicki wrote: > >> Backlog work for psock (sk_psock_backlog) might sleep while waiting for > >> memory to free up when sending packets. While sleeping, socket can

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-02-20 Thread Jakub Sitnicki
Hi Daniel, On Tue, Feb 19, 2019 at 05:00 PM CET, Daniel Borkmann wrote: > On 02/11/2019 10:09 AM, Jakub Sitnicki wrote: >> Backlog work for psock (sk_psock_backlog) might sleep while waiting for >> memory to free up when sending packets. While sleeping, socket can >> disappear from under our feet

Re: [PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-02-19 Thread Daniel Borkmann
Hi Jakub, On 02/11/2019 10:09 AM, Jakub Sitnicki wrote: > Backlog work for psock (sk_psock_backlog) might sleep while waiting for > memory to free up when sending packets. While sleeping, socket can > disappear from under our feet together with its wait queue because the > userspace has closed it.

[PATCH net] sk_msg: Keep reference on socket file while psock lives

2019-02-11 Thread Jakub Sitnicki
Backlog work for psock (sk_psock_backlog) might sleep while waiting for memory to free up when sending packets. While sleeping, socket can disappear from under our feet together with its wait queue because the userspace has closed it. This breaks an assumption in sk_stream_wait_memory, which expec