Date: Fri, 1 Nov 2002 10:54:23 -0800 From: Brooks Davis <brooks@one-eyed-alien.net> To: hackers@freebsd.org Subject: [brooks@one-eyed-alien.net: [PATCH] switching to if_xname] Message-ID: <20021101105423.A9676@Odin.AC.HMC.Edu>
next in thread | raw e-mail | index | archive | help
--tThc/1wpZn/ma/RB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable I'm trying to get some review for the following patch. I realize it's quite large, but most it is is trivial. The ipfw code is the only thing that worries me significantly. I have promised Kris that I will fix ports that break with this change so you don't need to worry about that issue. I'd like to commit this prior to 5.0-R pending RE approval (post DP2 would be fine if that would help get it out the door). Thanks, Brooks ----- Forwarded message from Brooks Davis <brooks@one-eyed-alien.net> ----- From: Brooks Davis <brooks@one-eyed-alien.net> Date: Wed, 23 Oct 2002 20:50:38 -0700 To: net@FreeBSD.ORG Subject: [PATCH] switching to if_xname As previously discussed, I've created a patch to replace if_unit and if_name with if_xname in struct ifnet following the example of NetBSD and OpenBSD. I've tried to place the more intresting parts of the diff near the top. This patch will break ports which use libkvm to read interface statistics. This effects at least wmnet and wmnet2 (though both segfault on current without this patch). The patches to fix this should be trivial since most of those applications probably support NetBSD or OpenBSD and I plan to bump __FreeBSD_version. Other then those issues and a generalization of interface globing in IPFW, this patch should be a giant no-op from the user perspective. Real features will come later, but the API/ABI change needs to happen soon or it's going to be a 6.0 feature. I'm running this patch on my laptop with a GENERIC kernel without any problems so far. For all it's size, most of the changes are log() or printf() calls plus a fairly consistant change to each network driver's attach function so this should be generally low impact. The patch is available at the URL below. I tried to send a copy to the list, but it looks like it got silently tossed as too large. If you want a copy e-mailed to you, feel free to ask me for one. http://people.freebsd.org/~brooks/patches/if_xname.diff Please review. Thanks, Brooks --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 ----- End forwarded message ----- --tThc/1wpZn/ma/RB Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (GNU/Linux) Comment: For info see http://www.gnupg.org iD8DBQE9ws3bXY6L6fI4GtQRAhSAAKC20rlugr41WvZ3kFxjyrehr/Po5wCgzQxG e5UryWvCkKbCFduDjPayARc= =spD5 -----END PGP SIGNATURE----- --tThc/1wpZn/ma/RB-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20021101105423.A9676>