From owner-freebsd-arch Mon May 8 0:58:56 2000 Delivered-To: freebsd-arch@freebsd.org Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.40.131]) by hub.freebsd.org (Postfix) with ESMTP id 2E21E37B89B for ; Mon, 8 May 2000 00:58:46 -0700 (PDT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost.freebsd.dk [127.0.0.1]) by critter.freebsd.dk (8.9.3/8.9.3) with ESMTP id JAA16524; Mon, 8 May 2000 09:57:09 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Greg Lehey Cc: FreeBSD-arch@FreeBSD.ORG Subject: Re: Tidiness or kernel bloat? (was: Support for bootable Vinum file systems: please review) In-reply-to: Your message of "Mon, 08 May 2000 08:33:31 +0930." <20000508083331.A61488@freebie.lemis.com> Date: Mon, 08 May 2000 09:57:08 +0200 Message-ID: <16522.957772628@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG In message <20000508083331.A61488@freebie.lemis.com>, Greg Lehey writes: >>> 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? Because one of them is *only* intended to care about statistics. Pretty soon you might see devstat track statistics for all logical disk, ie: ad0s1a, ad0s1b, ad0s1, ad0, and you may see it integrated more into the disk mini-layer. None of these changes would affect other users of devstat (except provide more detail) but all of sudden vinum would need magic fixes to only look for whatever types of devices vinum wants to look for. >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. 1. You are not trying to access it from userland. 2. The userland access is a well-defined API. 3. You want to open the implementation to your can grovel around in the entrails of devstats implementation. If you insist on this gross hack, the very least you could do was to add a couple of functions to the devstat code rather than pull all the internals into daylight. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD coreteam member | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message