From owner-freebsd-virtualization@FreeBSD.ORG Tue Sep 2 14:12:10 2008 Return-Path: Delivered-To: freebsd-virtualization@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 67BFA1065677; Tue, 2 Sep 2008 14:12:10 +0000 (UTC) (envelope-from brooks@lor.one-eyed-alien.net) Received: from lor.one-eyed-alien.net (lor.one-eyed-alien.net [69.66.77.232]) by mx1.freebsd.org (Postfix) with ESMTP id 1AB468FC22; Tue, 2 Sep 2008 14:12:09 +0000 (UTC) (envelope-from brooks@lor.one-eyed-alien.net) Received: from lor.one-eyed-alien.net (localhost [127.0.0.1]) by lor.one-eyed-alien.net (8.14.2/8.14.2) with ESMTP id m82ECj8Z060224; Tue, 2 Sep 2008 09:12:45 -0500 (CDT) (envelope-from brooks@lor.one-eyed-alien.net) Received: (from brooks@localhost) by lor.one-eyed-alien.net (8.14.2/8.14.2/Submit) id m82ECj9a060223; Tue, 2 Sep 2008 09:12:45 -0500 (CDT) (envelope-from brooks) Date: Tue, 2 Sep 2008 09:12:45 -0500 From: Brooks Davis To: Julian Elischer Message-ID: <20080902141245.GB48622@lor.one-eyed-alien.net> References: <20080828185639.P66593@maildrop.int.zabbadoz.net> <20080902000516.GA48622@lor.one-eyed-alien.net> <48BCED8E.5030109@elischer.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="O5XBE6gyVG5Rl6Rj" Content-Disposition: inline In-Reply-To: <48BCED8E.5030109@elischer.org> User-Agent: Mutt/1.5.17 (2007-11-01) Cc: "Bjoern A. Zeeb" , FreeBSD virtualization mailing list Subject: Re: Step 1.5 needs review X-BeenThere: freebsd-virtualization@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion of various virtualization techniques FreeBSD supports." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Sep 2008 14:12:10 -0000 --O5XBE6gyVG5Rl6Rj Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Sep 02, 2008 at 12:38:54AM -0700, Julian Elischer wrote: > Brooks Davis wrote: >> On Thu, Aug 28, 2008 at 07:03:30PM +0000, Bjoern A. Zeeb wrote: >>> Hi, >>>=20 >>> in case you are interested or have volunteered before to review Step 1.5 >>> as described on http://wiki.freebsd.org/Image/Notes200808DevSummit >>> there are few things to do: >>>=20 >>> - review the diff (Julian posted an initial one). >>> - make sure all (relevant) sysctl were caught. >>> - make sure the INIT_VNET_* macro is there whereever it is needed. >>> - do builds according to "HOWTO verify that the pure style changes are >>> all right" on the above mentioned page and verify that it is all >>> style changes. In case there are others we shoudl decide to either >>> commit them either upfront or afterwards if possible. >>> - the 'include headers' one way or the other (as we have discussed at >>> the devsummit and that Julian has told me again) needs resolving. >>> As this has bikeshed potential, I'd prefer that the 'singed up' >>> reviewers decide that. >>> - possibly more... >>>=20 >>> The plan would be to have a final patch by Monday morning UTC to be >>> comitted by a volunteer. >>=20 >> I've gone over the patch and fixed some white space issues. I've also >> found some things I'm not sure what to do with. Comments: >>=20 >> - GENERIC_NODEBUG should not be committed >> - VNET_ITERLOOP_BEGIN/END is evil. It would be really nice to find a >> way to do this that preserves {} pairs and isn't too magic. >=20 > The requirement is to take soem code that doesn something once. > and do it once for each vimage. There are of course many ways to do this.. >=20 > Once we have the code in, I think we should expand this out > and correctly indent the code, but for reasons of "minimum diff size" > teh current way seems ok to me though it doens't look pretty.. >=20 > I suggest that we eventually replace: >=20 > VNET_ITERLOOP_BEGIN > stuff > VNET_ITERLOOP_END >=20 >=20 > with (eventually) >=20 > FOREACH_VNET(vnet) { > stuff > } >=20 > but that would require that the entire contents of "stuff" > would appear in the diff. Thinking about it more, at a minimum, I think we should do: VNET_ITERLOOP_BEGIN stuff VNET_ITERLOOP_END >> - sys/kern/tty.c: >> - There's some #if 0 code that presumably should stay in the vimage >> branch for now and be fixed before the final commit. >> - TIOCDRAIN is being removed. Is this a merge issue or something >> else? >=20 >=20 > not sure myself.. I've been only following the tty mashup from a distance. >=20 >> - sys/net/if.h: >> - shouldn't net/vnet.h be included in if_var.h instead? *_var.h is >> supposed to be the internals and I think this qualifies. If so, >> there will be a number of files that added if.h includes that >> should add if_var.h includes instead. >=20 > I actually looked around to find a good place to icnlude vnet.h from > and decided on if.h because it seemt o be included almist everywhere > that vnet.h needed to be included, but I'm not religious on it. >=20 > teh original code actually includes vnet.h directly in about 50 source=20 > files. >=20 > my attempt to include it from if.h cut that down to 3. >=20 > I'm not sure I want to actually include the contents directly into > if.h or any other place.. I think keeping a separate vnet.h and > vinet.h seems ok to me. The #ifdef _KERNEL is a strong hint that it belongs in if_var.h if it's going to be included in another header (IMO, the vnet/vinet.h files aren't a good idea in the long term). -- Brooks --O5XBE6gyVG5Rl6Rj Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.8 (FreeBSD) iD8DBQFIvUncXY6L6fI4GtQRAjqFAJ4rDoDm0TsK9SkfpYB4Z2WvMe7jOACg13NV ESD2RDjG6tdy5TF0rWAZZlE= =CjHo -----END PGP SIGNATURE----- --O5XBE6gyVG5Rl6Rj--