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.8        1 Jun 2017 12:26:14 -0000       1.18
> +++ usr.sbin/acpidump/acpidump.8        22 Nov 2020 16:21:01 -0000
> @@ -68,8 +68,9 @@ $ iasl -d <prefix>.<sig>.<id>
>   .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

Reply via email to