On Tue, 21 Dec 2010 22:15:33 +0100
Thomas Pfaff <[email protected]> wrote:

> 1) pf(4) says DIOCSETDEBUG has enum { PF_DEBUG_NONE, PF_DEBUG_URGENT, ...
> but these names are not in pfvar.h nor anywhere else in the source tree
> (AFAICT).  What should the legal values (or names) be?

I guess these should now be one of the LOG_ names from syslog(3) so
maybe something like this?

Not all LOG_ levels are used in the code, as far as I can tell, but I
guess that does not matter and might change in the future (so just do
a man page reference rather than list the currently used levels).

Index: pf.4
===================================================================
RCS file: /cvs/src/share/man/man4/pf.4,v
retrieving revision 1.71
diff -u -p -r1.71 pf.4
--- pf.4        31 May 2010 18:33:54 -0000      1.71
+++ pf.4        26 Dec 2010 21:30:29 -0000
@@ -26,7 +26,7 @@
 .\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
 .\" SUCH DAMAGE.
 .\"
-.Dd $Mdocdate: May 31 2010 $
+.Dd $Mdocdate: March 25 2010 $
 .Dt PF 4
 .Os
 .Sh NAME
@@ -316,10 +316,9 @@ struct pfioc_natlook {
 .Ed
 .It Dv DIOCSETDEBUG Fa "u_int32_t *level"
 Set the debug level.
-.Bd -literal
-enum   { PF_DEBUG_NONE, PF_DEBUG_URGENT, PF_DEBUG_MISC,
-         PF_DEBUG_NOISY };
-.Ed
+See the
+.Xr syslog 3
+man page for a list of valid debug levels.
 .It Dv DIOCGETSTATES Fa "struct pfioc_states *ps"
 Get state table entries.
 .Bd -literal

Reply via email to