Merge pull request #16569 from brahmaroutu/ioutil_hang_gccgo

goroutine fairness is not guaranteed causing the hang with GCCGO (x86 GCCGO CI)
This commit is contained in:
Jess Frazelle 2015-09-25 10:24:13 -07:00
commit fdd5ab2fc3

View file

@ -84,6 +84,9 @@ func NewBufReaderWithDrainbufAndBuffer(r io.Reader, drainBuffer []byte, buffer i
func (r *bufReader) drain() {
for {
//Call to scheduler is made to yield from this goroutine.
//This avoids goroutine looping here when n=0,err=nil, fixes code hangs when run with GCC Go.
callSchedulerIfNecessary()
n, err := r.reader.Read(r.drainBuf)
r.Lock()
if err != nil {
@ -98,7 +101,6 @@ func (r *bufReader) drain() {
}
r.wait.Signal()
r.Unlock()
callSchedulerIfNecessary()
if err != nil {
break
}