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