From owner-freebsd-current Fri Apr 11 14:42:49 1997 Return-Path: Received: (from root@localhost) by freefall.freebsd.org (8.8.5/8.8.5) id OAA23554 for current-outgoing; Fri, 11 Apr 1997 14:42:49 -0700 (PDT) Received: from phaeton.artisoft.com (phaeton.Artisoft.COM [198.17.250.50]) by freefall.freebsd.org (8.8.5/8.8.5) with SMTP id OAA23538 for ; Fri, 11 Apr 1997 14:42:38 -0700 (PDT) Received: (from terry@localhost) by phaeton.artisoft.com (8.6.11/8.6.9) id OAA12385; Fri, 11 Apr 1997 14:22:45 -0700 From: Terry Lambert Message-Id: <199704112122.OAA12385@phaeton.artisoft.com> Subject: Re: WHY? ...non-use of TAILQ macros... To: phk@dk.tfs.com Date: Fri, 11 Apr 1997 14:22:45 -0700 (MST) Cc: terry@lambert.org, current@freebsd.org In-Reply-To: <1094.860742468@critter> from "phk@dk.tfs.com" at Apr 11, 97 09:07:48 am X-Mailer: ELM [version 2.4 PL24] MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-current@freebsd.org X-Loop: FreeBSD.org Precedence: bulk > >On the theory that the TAILQ_xxx macros were being avoided and the > >queue elemments themselves directly refernce for No Good Reason(tm), > >I just compiled the following: > > You're way behind Terry! :-) > > I argued this long time ago and started to introduce the _FIRST, _LAST, > _NEXT, _PREV and _EMPTY macros as I have got to it. > > A patch for this from you would be most welcome of course. I am maintaining several source trees right now on the machine where I have the post-Lite2 merged source repository. I don't have another 10M for another checked out tree, unless I want to truncate some of my build environment. I assume you would want me to run a kernel with these patches installed to make sure it runs and that there are no obvious memory leaks, etc.? Unfortunately, the post Lite2 merge tree on my machine has already got local modifications to bring the namei/nameifree and EXCLUDE vop flag code up to date with -current (I have uploaded today the diffs as taken against the -current tree; they are on freefall in the file ~terry/current-namei.diff). I could do the patch for the macro references (the mail message I sent in this thread already had most of the replacement lines that would result, already, for all tqh_first reference in /sys/kern), but it touches several of the same files, so I would not be able to give you a clean diff without including the namei/nameifree and EXCLUDE stuff. Too bad I can't just branch and check in my changes and check out an untouched tree... CVS is too dumb to let me unbranch later, and the next CVSup would kill me anyway. 8-(. Regards, Terry Lambert terry@lambert.org --- Any opinions in this posting are my own and not those of my present or previous employers.