From owner-freebsd-bugs Wed Jul 19 13:21:02 1995 Return-Path: bugs-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.11/8.6.6) id NAA08896 for bugs-outgoing; Wed, 19 Jul 1995 13:21:02 -0700 Received: from cs.weber.edu (cs.weber.edu [137.190.16.16]) by freefall.cdrom.com (8.6.11/8.6.6) with SMTP id NAA08888 for ; Wed, 19 Jul 1995 13:20:58 -0700 Received: by cs.weber.edu (4.1/SMI-4.1.1) id AA28881; Wed, 19 Jul 95 14:13:29 MDT From: terry@cs.weber.edu (Terry Lambert) Message-Id: <9507192013.AA28881@cs.weber.edu> Subject: Re: possible ffs_vget() race condition To: dillon@blob.best.net (Matt Dillon) Date: Wed, 19 Jul 95 14:13:28 MDT Cc: mckusick@McKusick.COM, davidg@root.com, bugs@freebsd.org In-Reply-To: <199507191809.LAA07002@blob.best.net> from "Matt Dillon" at Jul 19, 95 11:09:16 am X-Mailer: ELM [version 2.4dev PL52] Sender: bugs-owner@freebsd.org Precedence: bulk > Oh, there are also some pretty serious problems with the sysctl() > stuff... mainly with all the list traversals the sysctl()'s do > without locking and the copyout's inside the loops (which can > block, causing the list being traverse and most specifically the > 'current' item to be ripped out from under sysctl())... I had > to stop using pstat -T because of the problem (it would crash > the machine every couple of hours). A lot of the system configuration stuff is bogus/unportable/both. The use of manifest constants to do the descriptor structs in sysctl.h is patently bogus. The CTLTYPE_QUAD definition and the size variant CTLTYPE_INT are both rather annoying; these should be sized values, and QUAD should probably not be allowed at all, since 64 bit values are (as yet) intolerably unportable. Not to mention the use of a shift of an embeded constant '63' and a -1 to get the RLIM_INFINITY value in resource.h. Instead of (~(u_quad_t)0). Anything touching on quad code is pretty much half baked. Terry Lambert terry@cs.weber.edu --- Any opinions in this posting are my own and not those of my present or previous employers.