google / gvisor

Application Kernel for Containers

Home Page:https://gvisor.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Enable leak checking for packet buffers

ghananigans opened this issue · comments

Description

Leak checking is only available to things that enable leak checking via

func (r *AtomicRefCount) EnableLeakCheck(name string) {
.

We should look into using refs.AtomicRefCounter for reference counting packet buffers so that we can have leak checking

I'm not sure I understand this issue. Leak checking is already possible in the current implementation with refsvfs2.RefCounter and refs.AtomicRefCounter is deprecated.

Leak checking is being added to all networking tests. Transport/link tests already enable leak checker and we will be enabling network and other misc packages shortly. I will update this bug with the changes.

I believe this is done? Is there something else left to do here?