]> go.fuhry.dev Git - fsnotify.git/commit
Remove TestInotifyNoBlockingSyscalls (#532)
authorMartin Tournoij <martin@arp242.net>
Tue, 8 Nov 2022 20:43:19 +0000 (21:43 +0100)
committerGitHub <noreply@github.com>
Tue, 8 Nov 2022 20:43:19 +0000 (21:43 +0100)
commit0d1c8342fcec1c0e2ff5ae2e460fdaf908fd2188
treeb6735f1eb76bd5dba51ba4b9c47f61cf864c2ea6
parent7ed195facfcd33fc8a54c731565fbbe1f959edd6
Remove TestInotifyNoBlockingSyscalls (#532)

This test checks if no additional OS threads are created; this was added
after the old epoll-based inotify code was removed (which would create
additional threads). It's always been a bit flaky: it currently already
runs the test twice and "fails" only if both fail.

We don't really control the Go scheduler and when it creates/destroys OS
threads, or have all that much insight in it (the "threadcreate" profile
in pprof has been broken for years). I tried to make this test a bit
more reliable a few months ago, and the run-it-twice "solution" was the
best I could come up.

It still causes some failures though; see the links below. I don't know
why it consistently fails on armle (and not e.g. arm64), but that seems
a Go thing, rather than a fsnotify thing.

I'm not sure if this test adds all that much value; I don't really
foresee a regression where we accidentally create more OS threads than
we should.

Fixes #531
Also see: https://bugs.launchpad.net/ubuntu/+source/golang-fsnotify/+bug/1971967
backend_inotify_test.go