Date: Mon, 13 Nov 2017 20:35:02 -0800 From: Ravi Pokala <rpokala@mac.com> To: Larry McVoy <lm@mcvoy.com>, freebsd-arch@freebsd.org Cc: scottl@netflix.com, kbowling@llnw.com, gallatin@netflix.com Subject: Re: small patch for numactl. Comments? Message-ID: <BD0DB3A0-BBAF-438C-B91E-5F016140F8CD@mac.com> In-Reply-To: <20171114020138.GA18863@mcvoy.com> References: <20171114020138.GA18863@mcvoy.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Larry,
Might I suggest a small wording change? Three "that"s in the same sentence is a little much. :-)
> +so that programs that know that they need to be run under
> +.Nm
> +can check.
+so programs which know that they need to be run under
+.Nm ,
+can check.
Also, I recommend installing textproc/igor, and using it to scan the manpage for common mistakes.
Thanks,
Ravi (rpokala@)
-----Original Message-----
From: <owner-freebsd-arch@freebsd.org> on behalf of Larry McVoy <lm@mcvoy.com>
Date: 2017-11-13, Monday at 18:01
To: <freebsd-arch@freebsd.org>
Cc: <scottl@netflix.com>, <kbowling@llnw.com>, <gallatin@netflix.com>
Subject: small patch for numactl. Comments?
Hi folks, some CDN people are dragging me out of retirement to work
on FreeBSD. Which I have to say is sort of fun since I started my
kernel hacking on SunOS 4.x which was a very very nice version of
BSD. But FreeBSD has mostly caught up, so it's pleasant.
I'm wacking LMbench to be numa aware and this patch would help me make
sure that when you are a numa machine you could insist that people
run the benchmark via numactl (imma gonna blog about numa, it sucks
unless you are numa aware).
I did some docs but I'm new to the FreeBSD man macros, would love
feedback on how to do that right.
--- numactl.1 2017-11-13 17:51:26.243473000 -0800
+++ numactl.1.lm 2017-11-13 17:51:20.494596000 -0800
@@ -107,6 +107,15 @@
.El
.Sh EXIT STATUS
.Ex -std
+.Sh ENVIRONMENT
+.Nm
+sets the environment variable
+\fINUMACTL=YES\fP
+before running
+.Ar cmd ...
+so that programs that know that they need to be run under
+.Nm
+can check.
.Sh EXAMPLES
Create a
.Pa /bin/sh
--- numactl.c 2017-11-13 16:18:36.134359000 -0800
+++ numactl.c.lm 2017-11-13 16:18:28.530953000 -0800
@@ -231,6 +231,7 @@
(void) set_numa_domain_cpuaffinity(cpu_domain,
CPU_WHICH_PID, -1);
+ putenv("NUMACTL=YES");
errno = 0;
execvp(*argv, argv);
err(errno == ENOENT ? 127 : 126, "%s", *argv);
_______________________________________________
freebsd-arch@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-arch
To unsubscribe, send any mail to "freebsd-arch-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?BD0DB3A0-BBAF-438C-B91E-5F016140F8CD>