了解有和没有goroutine的渠道选择

时间:2018-04-28 17:23:55

标签: select go closures channel goroutine

Hello stackoverflow社区, 我正在使用github.com/fsnotify/fsnotify将观察者设置为Go中的文件。我的功能看起来像

func SetWatcher(filename string) {
    fmt.Println("Setting watcher to file ", filename)
    Watcher, err = fsnotify.NewWatcher()
    if err != nil {
        fmt.Println("inotify errored. Other methods needs to be implemented.")
        panic(err)
    }
    if err != nil {
        log.Fatal(err)
    }

    done := make(chan bool)
    go func() {
        for {
            select {
            case event := <-Watcher.Events:
                if event.Op == fsnotify.Remove {
                    fmt.Println("File removed, needs to kill the process.")
                } else if event.Op == fsnotify.Rename {
                    fmt.Println("File renamed, need to restart seeking.")
                }

            case err := <-Watcher.Errors:
                log.Println("error:", err)
            }
        }
    }()

    err = Watcher.Add(filename)
    if err != nil {
        log.Fatal(err)
    }
    <-done
}

到目前为止工作,我得到输出

Setting watcher to file  /var/log/syslog
File renamed, need to restart seeking.

但是,如果我尝试删除在goroutine中运行的闭包并将其作为

运行
func SetWatcher(filename string) {
    fmt.Println("Setting watcher to file ", filename)
    Watcher, err = fsnotify.NewWatcher()
    if err != nil {
        fmt.Println("inotify errored. Other methods needs to be implemented.")
        panic(err)
    }
    if err != nil {
        log.Fatal(err)
    }
    //defer Watcher.Close()

    //done := make(chan bool)
    //go func() {
    //  for {
            select {
            case event := <-Watcher.Events:
                if event.Op == fsnotify.Remove {
                    fmt.Println("File removed, needs to kill the process.")
                } else if event.Op == fsnotify.Rename {
                    fmt.Println("File renamed, need to restart seeking.")
                }

            case err := <-Watcher.Errors:
                log.Println("error:", err)
            }
        //}
    //}()

    err = Watcher.Add(filename)
    if err != nil {
        log.Fatal(err)
    }
    //<-done
}

程序从不输出任何东西。我用strace运行它,程序被卡在

[pid  5773] pselect6(0, NULL, NULL, NULL, {tv_sec=0, tv_nsec=20000}, NULL <unfinished ...>
[pid  5772] epoll_wait(4, [], 128, 0)   = 0
[pid  5772] futex(0x598bf8, FUTEX_WAIT, 0, NULL <unfinished ...>
[pid  5773] <... pselect6 resumed> )    = 0 (Timeout)
[pid  5740] <... pselect6 resumed> )    = 0 (Timeout)
[pid  5773] futex(0x598578, FUTEX_WAIT, 0, {tv_sec=60, tv_nsec=0} 
<unfinished ...>
[pid  5740] futex(0xae1f58, FUTEX_WAIT, 0, {tv_sec=60, tv_nsec=0}

并且没有从频道接收并继续等待。

我读过这可能是因为非缓冲频道造成的。因此,为了验证这一点,我修改了库以使用缓冲通道,并且生成新观察者的函数部分看起来像

    w := &Watcher{
    fd:       fd,
    poller:   poller,
    watches:  make(map[string]*watch),
    paths:    make(map[int]string),
    Events:   make(chan Event, 10), // Made it buffered here
    Errors:   make(chan error),
    done:     make(chan struct{}),
    doneResp: make(chan struct{}),
}

但行为是一样的。有人可以帮助我理解为什么通道中的读数在goroutine中有效,并且在没有goroutine的情况下没有工作吗?

谢谢。

1 个答案:

答案 0 :(得分:2)

我对fsnotify并不熟悉,但看起来在调用Watcher.Add()来观看文件之前它不会在频道上发送任何内容,但Watcher.Add()发生在select之后在第二个版本中。在这种情况下,select将永久阻止,因为没有任何信号可以发出信号,因为Watcher.Add()尚未被调用。因此存在僵局。