From owner-freebsd-arch Sun May 7 16: 4:42 2000 Delivered-To: freebsd-arch@freebsd.org Received: from freebie.lemis.com (freebie.lemis.com [192.109.197.137]) by hub.freebsd.org (Postfix) with ESMTP id 6E54E37BB4E for ; Sun, 7 May 2000 16:04:38 -0700 (PDT) (envelope-from grog@freebie.lemis.com) Received: (from grog@localhost) by freebie.lemis.com (8.9.3/8.9.0) id IAA61688; Mon, 8 May 2000 08:33:31 +0930 (CST) Date: Mon, 8 May 2000 08:33:31 +0930 From: Greg Lehey To: Poul-Henning Kamp Cc: FreeBSD-arch@FreeBSD.ORG Subject: Tidiness or kernel bloat? (was: Support for bootable Vinum file systems: please review) Message-ID: <20000508083331.A61488@freebie.lemis.com> References: <20000507202944.I55316@freebie.lemis.com> <12575.957698677@critter.freebsd.dk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 1.0pre2i In-Reply-To: <12575.957698677@critter.freebsd.dk> Organization: LEMIS, PO Box 460, Echunga SA 5153, Australia Phone: +61-8-8388-8286 Fax: +61-8-8388-8725 Mobile: +61-418-838-708 WWW-Home-Page: http://www.lemis.com/~grog X-PGP-Fingerprint: 6B 7B C3 8C 61 CD 54 AF 13 24 52 F8 6D A4 95 EF Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Sunday, 7 May 2000 at 13:24:37 +0200, Poul-Henning Kamp wrote: > In message <20000507202944.I55316@freebie.lemis.com>, Greg Lehey writes: > >>>> >>>> This is pretty much what struct devstat does, except that it's more >>>> global. Why should we maintain two lists, just because one is >>>> primarily intended for statistics? >>>> >>>> Anyway, I assume this isn't so much a criticism of the fix as of the >>>> infrastructure around it. Correct? >>> >>> No, this is an objection to opening up devstat that way. >> >> Can you make an alternative suggestion? > >>>>> The right way to do it is probably to link the "struct disk"s >>>>> registered by the drivers with disk_creat() into a list which can >>>>> be searched. You're not very strong on reasoning here. You don't answer my question: >>>> Why should we maintain two lists, just because one is primarily >>>> intended for statistics? Your only objection appears to be: >>> No, this is an objection to opening up devstat that way. Devstat is already available in userland. What more opening up are you talking about? You approach appears to only add to kernel bloat. Greg -- Finger grog@lemis.com for PGP public key See complete headers for address and phone numbers To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message