aboutsummaryrefslogtreecommitdiffhomepage
path: root/internal/file.go
diff options
context:
space:
mode:
authorOphestra <cat@gensokyo.uk>2025-03-13 00:42:38 +0900
committerOphestra <cat@gensokyo.uk>2025-03-13 00:42:38 +0900
commite048f31baad20e16d5b1f92e5a04091a82b5ee6d (patch)
treed921d93cfdc0b20ad4e2e5de6e4d26ea69483782 /internal/file.go
parent6af8b8859fde3a2b63afee20f004e184896cec3a (diff)
internal: pull EINTR loop from stdlib
Signed-off-by: Ophestra <cat@gensokyo.uk>
Diffstat (limited to 'internal/file.go')
-rw-r--r--internal/file.go19
1 files changed, 19 insertions, 0 deletions
diff --git a/internal/file.go b/internal/file.go
new file mode 100644
index 00000000..153f0f99
--- /dev/null
+++ b/internal/file.go
@@ -0,0 +1,19 @@
+package internal
+
+import "syscall"
+
+// IgnoringEINTR makes a function call and repeats it if it returns an
+// EINTR error. This appears to be required even though we install all
+// signal handlers with SA_RESTART: see #22838, #38033, #38836, #40846.
+// Also #20400 and #36644 are issues in which a signal handler is
+// installed without setting SA_RESTART. None of these are the common case,
+// but there are enough of them that it seems that we can't avoid
+// an EINTR loop.
+func IgnoringEINTR(fn func() error) error {
+ for {
+ err := fn()
+ if err != syscall.EINTR {
+ return err
+ }
+ }
+}