From e048f31baad20e16d5b1f92e5a04091a82b5ee6d Mon Sep 17 00:00:00 2001 From: Ophestra Date: Thu, 13 Mar 2025 00:42:38 +0900 Subject: [PATCH] internal: pull EINTR loop from stdlib Signed-off-by: Ophestra --- internal/file.go | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) create mode 100644 internal/file.go diff --git a/internal/file.go b/internal/file.go new file mode 100644 index 0000000..153f0f9 --- /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 + } + } +}