From owner-freebsd-hackers Tue Jul 24 16: 0:12 2001 Delivered-To: freebsd-hackers@freebsd.org Received: from InterJet.elischer.org (c421509-a.pinol1.sfba.home.com [24.7.86.9]) by hub.freebsd.org (Postfix) with ESMTP id 4421A37B407; Tue, 24 Jul 2001 15:59:59 -0700 (PDT) (envelope-from julian@elischer.org) Received: from InterJet.elischer.org (InterJet.elischer.org [192.168.1.1]) by InterJet.elischer.org (8.9.1a/8.9.1) with ESMTP id RAA21029; Tue, 24 Jul 2001 17:04:55 -0700 (PDT) Date: Tue, 24 Jul 2001 17:04:53 -0700 (PDT) From: Julian Elischer To: Brooks Davis Cc: net@freebsd.org, hackers@freebsd.org Subject: Re: review request: ng_split cleanup In-Reply-To: <20010724143734.A1412@Odin.AC.HMC.Edu> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-hackers@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Tue, 24 Jul 2001, Brooks Davis wrote: > Please review the following diff for the ng_split netgraph node. It > cleans up a number of style issues, removes some functions that just did > that the default functions did, and renames the node to split from > ng_split to follow the normal convention. In addition to this diff, I > plan to commit a Makefile update to make this part of the modules build > and a sys/conf/options entry to allow static compilation. > > Thanks, > Brooks > > Index: ng_split.c > =================================================================== > RCS file: /home/ncvs/src/sys/netgraph/ng_split.c,v > retrieving revision 1.1 [...] > diff -u -r1.1 ng_split.c > - /* > - * XXX Really here we should just remove metadata we understand. > - */ > - NGI_GET_META(item, meta); > - NG_FREE_META(meta); this one is tricky.. it was written as part of a set of nodes that pass around their own metadata. The original idea was (so the author suggested) to stop those metadata structures from propogating out of the limited part of the graph that knew about them. However I don't see any harm in letting them go, since any node that doesn't understand a particular metadata type should ignore it.. otherwise your patch seems functionally the same.. please feel free to commit. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message