From: Mike Yuan Date: Mon, 20 Jan 2025 21:26:47 +0000 (+0100) Subject: mountpoint-util: update comment in is_name_to_handle_at_fatal_error() X-Git-Tag: v258-rc1~1536^2~5 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c3b7f0240fbc0739d5405f922ea5d19ed25346da;p=thirdparty%2Fsystemd.git mountpoint-util: update comment in is_name_to_handle_at_fatal_error() We can now assume name_to_handle_at() syscall is available, however the underlying fs can still lack support for it. --- diff --git a/src/basic/mountpoint-util.c b/src/basic/mountpoint-util.c index 15655cc075d..4ae5f5191fe 100644 --- a/src/basic/mountpoint-util.c +++ b/src/basic/mountpoint-util.c @@ -37,10 +37,10 @@ #define ORIGINAL_MAX_HANDLE_SZ 128 bool is_name_to_handle_at_fatal_error(int err) { - /* name_to_handle_at() can return "acceptable" errors that are due to the context. For example the - * kernel does not support name_to_handle_at() at all (ENOSYS), or the syscall was blocked - * (EACCES/EPERM; maybe through seccomp, because we are running inside of a container), or the mount - * point is not triggered yet (EOVERFLOW, think autofs+nfs4), or some general name_to_handle_at() + /* name_to_handle_at() can return "acceptable" errors that are due to the context. For example + * the file system does not support name_to_handle_at() (EOPNOTSUPP), or the syscall was blocked + * (EACCES/EPERM; maybe through seccomp, because we are running inside of a container), or + * the mount point is not triggered yet (EOVERFLOW, think autofs+nfs4), or some general name_to_handle_at() * flakiness (EINVAL). However other errors are not supposed to happen and therefore are considered * fatal ones. */