Re: [PATCH net v3 1/1] rxrpc: fix encap_rcv skb accounting exhaustion

[David Howells]

Zihan Xi <zihanx@xxxxxxxxxx> wrote:

> Reaccount encapsulated packets against the UDP socket before queueing
> them on the RxRPC local queue and drop packets once the socket rcvbuf
> limit is reached.

This seems reasonable.  I do really need to add separate accounting for
buffered incoming DATA packets and for all other incoming packets (especially
ACKs), but I think that can be done after the skb_orphan() point you add.

> Clear sk_user_data under RCU protection and release the socket only after
> the local queues are purged.

Is this actually a fix for a separate issue?

Anyway:

Acked-by: David Howells <dhowells@xxxxxxxxxx>