From owner-freebsd-arch Sun May 7 4:25:26 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 2229B37B526; Sun, 7 May 2000 04:25:21 -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 NAA12577; Sun, 7 May 2000 13:24:37 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Greg Lehey Cc: "Justin T. Gibbs" , FreeBSD-arch@FreeBSD.ORG Subject: Re: Support for bootable Vinum file systems: please review In-reply-to: Your message of "Sun, 07 May 2000 20:29:44 +0930." <20000507202944.I55316@freebie.lemis.com> Date: Sun, 07 May 2000 13:24:37 +0200 Message-ID: <12575.957698677@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG 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. -- 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