Comment 1:

Issue #5314 has been merged into this issue.

Comment 2:

So weird. If I do a plain read from the reader, no problem, if I do an ioutil.ReadAll,
it brakes.
reader := bufio.NewReader(conn)
followed by
_, err = reader.Read(make([]byte, 1))
vs.
_, err = ioutil.ReadAll(reader)

Comment 3:

attached output and heap profile
code is:
http://play.golang.org/p/bJF0etsZaE

Attachments:

1. mem.svg (10218 bytes)
2. gc.txt (12161 bytes)

Comment 4:

Two pmaps about a minute apart:

Attachments:

1. first.pmap (1910 bytes)
2. second.pmap (1953 bytes)

Comment 5:

This combination leaks:
reader := bufio.NewReader(conn)
_, err = ioutil.ReadAll(reader)
However, using only ioutil.ReadAll straight off the conn leads to no leak.
            
_, err = ioutil.ReadAll(conn)
Similarly, using only the bufio Read leads to no leak either.
reader := bufio.NewReader(conn)
reader.Read(make([]byte, 1))
As you can see from the gc.txt attachment, it doesn't leak on every loop, only every Nth
loop.

Comment 6:

> This combination leaks:
> 
> reader := bufio.NewReader(conn)
> _, err = ioutil.ReadAll(reader)
Does ioutil.ReadAll try to sense for the presence of ReadFrom/WriteTo ?
What happens if you wrap reader in your own type that masks off the ReadFrom/WriteTo
methods of bufio.Reader ?

Status changed to WaitingForReply.

Comment 7:

Yay! Reduced the example to use os.Pipe to create a blocking fd read instead of the net
package. Its a pretty simple function now, how someone can have a look.

Attachments:

1. ram_test.go (648 bytes)

Comment 8:

I don't see any bug here.
The writer is never closed, so the reads will block forever, and the goroutines will
leak and waste memory. Working as intended.
Yes, the memory from Handle might not be collected as aggressively as possible, but
that's the stack-precise GC that Carl is working on in issue #5134.
But this program in comment #7 still has a leak.

Status changed to Invalid.

Comment 9:

Thanks Brad,
I didn't know we didn't do stack-precise GC - all this makes way more sense now.
I look forward to Carl's patches.