Software watchdog patch
Hiten Pandya
hmp at FreeBSD.ORG
Sat Jun 21 06:33:42 PDT 2003
On Fri, Jun 20, 2003 at 09:47:07PM -0500, Sean Kelly wrote:
> Greetings.
> I look forward to any feedback, whether positive or negative.
Hello Sean, this stuff looks really promising. Anyways, I
can't comment about the code as I have had no time to read it,
but here are a few mdoc nitpicks I did:
Index: share/man/man4/watchdog.4
===================================================================
+is implemented via a trio of sysctl OIDs. When
+.Va debug.watchdog.enabled
Hmm, the .Va should be replaced with .Li. Also, it is necessary
to remove all hard sentence breaks, ie. instead of:
"via a trio of sysctl OIDs. When..."
It should be:
"via a trio of sysctl OIDs.
When..."
Sysctls should be documented using .Li.
+.Sh SEE ALSO
+.Xr watchdogd 8 ,
+.Xr sysctl 8
+.Sh HISTORY
Manual pages are first ordered by their section numbers, and
then alphabetically, thus watchdogd(8)'s .Xr should be after
sysctl(8).
+.Sh AUTHORS
+.An -nosplit
+The
+.Nm
+code and man page were written by
+.An Sean Kelly Aq smkelly at zombie.org .
Use `manual page' instead of `man page'.
Same sort of comments apply to the watchdogd(8) manual page.
Hope that helps.
Cheers.
-- Hiten (hmp at FreeBSD.ORG)
More information about the freebsd-current
mailing list