On Sun, Jan 02, 2022 at 10:45:14PM +0100, Stefan Esser wrote:
> Am 02.01.22 um 20:51 schrieb Antoine Brodin:
> > Hi,
> >
> > It seems that the 2 main ports failing are math/py-numpy (503 ports
> > skipped) and sysutils/slurm-wlm (232 ports skipped)
>
> Hi Antoine,
>
> thank you for the information!
>
> > Failure logs:
> > http://beefy18.nyi.freebsd.org/data/main-amd64-default/pe2d17ded99d5_s5169832c96/logs/errors/py38-numpy-1.20.3,1.log
>
> Python 3.8.12 (default, Dec 31 2021, 10:50:47)
> >>> import os
> >>> os.sched_getaffinity(0)
> Traceback (most recent call last):
> File "<stdin>", line 1, in <module>
> OSError: [Errno 34] Result too large
>
> This is a Python interpreter problem: it seems that the wrapper
> for the sched_getaffinity() function that has been introduced by
> kib in <sched.h> is buggy.
>
> As a work-around I have added a patch to comment out the
> os.sched_getaffinity(0) call (which used to cause an Attribute
> error that was caught by try/except, before).
>
> See ports commit 507c189b2876.
Buggy in which way?
Our cpuset_getaffinity(2) syscall returns ERANGE for cpuset size not
equal to CPU_SETSIZE. It seems that python source expects EINVAL in
this case.
I can change the wrapper to translate ERANGE to EINVAL. sched_setaffinity()
probably would require a symmetrical patch, but lets postpone it.
diff --git a/lib/libc/gen/sched_getaffinity.c b/lib/libc/gen/sched_getaffinity.c
index 2ae8c5b763a3..8748d7a60278 100644
--- a/lib/libc/gen/sched_getaffinity.c
+++ b/lib/libc/gen/sched_getaffinity.c
@@ -26,11 +26,29 @@
* SUCH DAMAGE.
*/
+#include <errno.h>
#include <sched.h>
+#include <string.h>
int
sched_getaffinity(pid_t pid, size_t cpusetsz, cpuset_t *cpuset)
{
+ /*
+ * Be more Linux-compatible:
+ * - return EINVAL in passed size is less than size of cpuset_t
+ * in advance, instead of ERANGE from the syscall
+ * - if passed size is larger than the size of cpuset_t, be
+ * permissive by claming it back to sizeof(cpuset_t) and
+ * zeroing the rest.
+ */
+ if (cpusetsz < sizeof(cpuset_t))
+ return (EINVAL);
+ if (cpusetsz > sizeof(cpuset_t)) {
+ memset((char *)cpuset + sizeof(cpuset_t), 0,
+ cpusetsz - sizeof(cpuset_t));
+ cpusetsz = sizeof(cpuset_t);
+ }
+
return (cpuset_getaffinity(CPU_LEVEL_WHICH, CPU_WHICH_PID,
pid == 0 ? -1 : pid, cpusetsz, cpuset));
}
