On Sun, Nov 22, 2020 at 05:24:56PM +0100, Emil Engler wrote:
> The man page of acpidump(8) says that "kern.allowkmem" must be set to
> some value in order for the program to work properly. However it does
> not define to what value it must be set. I am not certainly sure if
> there are other options between 0 and 1 but if so the man page should be
> fixed anyway with these options IMO.
>
> Cheers,
> Emil
>
> Index: usr.sbin/acpidump/acpidump.8
> ===
> RCS file: /cvs/src/usr.sbin/acpidump/acpidump.8,v
> retrieving revision 1.18
> diff -u -p -u -p -r1.18 acpidump.8
> --- usr.sbin/acpidump/acpidump.81 Jun 2017 12:26:14 - 1.18
> +++ usr.sbin/acpidump/acpidump.822 Nov 2020 16:21:01 -
> @@ -68,8 +68,9 @@ $ iasl -d ..
> .Nm
> requires the ability to open
> .Pa /dev/kmem
> -which may be restricted based upon the value of the
> +which requires
> .Ar kern.allowkmem
> +to be set to 1 by
> .Xr sysctl 8 .
> .Pp
> .Nm
>
hi.
there should definitely be a comma before "which". i'll fix that.
except for that, i think i prefer the text as it is now. if you play
about with sysctl, it should be fairly obvious how to set values.
nowhere does it say that
> "kern.allowkmem" must be set to
> some value in order for the program to work properly.
it says that it requires access to /dev/mem. and then refers to the
place where you can check what the current settings are.
i think it is clear enough.
jmc
