vbatts/pkg
1
0
Fork 0
Code Pull requests Releases Activity

goroutine fairness is not guaranteed causing the hang with GCCGO

Browse source 
Signed-off-by: Srini Brahmaroutu <srbrahma@us.ibm.com>
This commit is contained in:
Srini Brahmaroutu 2015-09-25 00:13:38 +00:00
parent ad3c33e73d
commit 2d0927a08b
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ioutils/readers.go
View file

@ -84,6 +84,9 @@ func NewBufReaderWithDrainbufAndBuffer(r io.Reader, drainBuffer []byte, buffer i
func (r *bufReader) drain() { func (r *bufReader) drain() {
for { 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) n, err := r.reader.Read(r.drainBuf)
r.Lock() r.Lock()
if err != nil { if err != nil {
@ -98,7 +101,6 @@ func (r *bufReader) drain() {
} }
r.wait.Signal() r.wait.Signal()
r.Unlock() r.Unlock()
callSchedulerIfNecessary()
if err != nil { if err != nil {
break break
} }