1

我做了一个小程序来测试 go 通道的吞吐量,但是它总是死锁,我很努力但不明白为什么:

package main

import (
    "fmt"
    "runtime"
)

const CONCURRENCY = 32
const WORK_PER_WORKER = 100
const TOTAL_WORK = CONCURRENCY * WORK_PER_WORKER

func work() {
    sum := 0
    for i := 0; i < 10000000; i++ {
        sum *= i
    }
}

type WorkItem struct {
    Done chan int
}

func main() {
    runtime.GOMAXPROCS(CONCURRENCY)
    var workQueue [CONCURRENCY]chan *WorkItem
    // initialize workers
    for i := 0; i < CONCURRENCY; i++ {
        workQueue[i] = make(chan *WorkItem)
    }
    // start workers
    for i := 0; i < CONCURRENCY; i++ {
        go func(i int) {
            anItem := <-workQueue[i]
            work()
            anItem.Done <- 1
        }(i)
    }
    completed := make(chan bool, TOTAL_WORK)
    for i := 0; i < TOTAL_WORK; i++ {
        go func(i int) {
            // send work to queues
            workToDo := &WorkItem{Done: make(chan int)}
            workQueue[i/WORK_PER_WORKER] <- workToDo // !! DEADLOCK
            // wait until the work is done
            <-workToDo.Done
            completed <- true
        }(i)
    }
    fmt.Println("Waiting")
    for i := 0; i < TOTAL_WORK; i++ {
        <-completed
    }
}
4

2 回答 2

2

您的代码go func(i int) { anItem := <-workQueue[i]; ... }从中删除了 juste 1 项,workQueue[i]但您正试图将 WORK_PER_WORKER 项填充到其中。您将处理 CONCURRENCY 的许多项目,然后所有读取 goroutine 都已终止,并且您遇到了死锁。

在工作 goroutine 中循环“解决”你的僵局:http ://play.golang.org/p/j2pavqnBDv 只是“解决”,因为这些工作 goroutine 永远不会终止。也许您可以尝试使用close您的通道来通知工作人员 goroutine 什么时候不会发送任何内容。

于 2014-01-08T23:22:14.843 回答
1

因为您的工作人员只处理一项任务然后退出。因此,只有第一个CONCURRENCY项目继续进行,然后workQueue[i/WORK_PER_WORKER] <- workToDo无限期地阻塞。因此,completedchan 永远不会收到足够的值,并且main也永远阻塞。

您的工作人员应该循环工作,如下所示:

for i := 0; i < CONCURRENCY; i++ {
    go func(i int) {
        for anItem := range workQueue[i] {
            work()
            anItem.Done <- 1
        }
    }(i)
}
于 2014-01-08T23:22:04.573 回答