Thread (9 messages) flat view 9 messages, 2 authors, 2018-01-16

Re: [Patch net v3] tun: fix a memory leak for tfile->tx_array

From: Cong Wang <hidden>
Date: 2018-01-16 06:49:45

On Mon, Jan 15, 2018 at 10:37 PM, Jason Wang [off-list ref] wrote:

On 2018年01月16日 14:33, Cong Wang wrote:
quoted
But __tun_detach(true) is not a hot path, a memset() doesn't harm
anything.

Yes, but it looks more more like a workaround or trick to me.
I'd blame skb_array API for this. ;) Ideally, skb_array_cleanup()
should take care of everything I put in tun_cleanup_tx_array().
As I mentioned in changelog, we can always improve it in
-net-next, so I don't want to bother it for -net.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help