From owner-cvs-all@FreeBSD.ORG  Tue Oct  5 21:27:07 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21])
	by hub.freebsd.org (Postfix) with ESMTP
	id C245216A4CE; Tue,  5 Oct 2004 21:27:06 +0000 (GMT)
Received: from green.homeunix.org (green@localhost [127.0.0.1])
	by green.homeunix.org (8.13.1/8.13.1) with ESMTP id i95LR6oj055668;
	Tue, 5 Oct 2004 17:27:06 -0400 (EDT)
	(envelope-from green@green.homeunix.org)
Received: (from green@localhost)
	by green.homeunix.org (8.13.1/8.13.1/Submit) id i95LR4Ar055667;
	Tue, 5 Oct 2004 17:27:04 -0400 (EDT)
	(envelope-from green)
Date: Tue, 5 Oct 2004 17:27:04 -0400
From: Brian Fundakowski Feldman <green@FreeBSD.org>
To: Max Laier <mlaier@FreeBSD.org>
Message-ID: <20041005212704.GB47017@green.homeunix.org>
References: <200410052044.i95KiOVV072560@repoman.freebsd.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <200410052044.i95KiOVV072560@repoman.freebsd.org>
User-Agent: Mutt/1.5.6i
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/contrib/pf/man pf.4
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 05 Oct 2004 21:27:07 -0000

On Tue, Oct 05, 2004 at 08:44:24PM +0000, Max Laier wrote:
> mlaier      2004-10-05 20:44:24 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:        (Branch: RELENG_5)
>     contrib/pf/man       pf.4 
>   Log:
>   MFC:
>     PFIL_HOOKS in no longer an optional item.
>   
>     Submitted by:   Anders Hanssen

I have a bunch of questions regarding pf documentation...

Do you think we should update pf(4)/pfctl(8) documentation to cross-reference
IPFW at all?

Is it worth explaining in pfctl(8) what the default RED parameters for
ALTQ are and how they relate to qlimit?

Isn't there an altq.4 somewhere?

Shouldn't pfctl(8) document what occurs when there is no memory to add
an ALTQ tag?

P.S. Think we should MFC dc(4) ALTQ support?
P.P.S. Should we look again into changing the pfil locking to not fail-open?

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green@FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\