Date: Tue, 30 Sep 2003 10:48:19 -0700 From: Brooks Davis <brooks@one-eyed-alien.net> To: John Baldwin <jhb@freebsd.org> Cc: net@freebsd.org Subject: Re: adding if_dev member to struct ifnet Message-ID: <20030930174815.GC31908@Odin.AC.HMC.Edu> In-Reply-To: <XFMail.20030930131439.jhb@FreeBSD.org> References: <11374.1064934514@critter.freebsd.dk> <XFMail.20030930131439.jhb@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--96YOpH+ONegL0A3E Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Sep 30, 2003 at 01:14:39PM -0400, John Baldwin wrote: >=20 > Fair enough. I think that Brooks planned to use a NULL device_t for > interfaces w/o a backing new-bus device. However, that means you > still need if_name for all the non-newbus devices, so this seems > somewhat pointless if if_name is the only reason. Another counterpoint > is that the new-bus namespace and the netif namespace aren't the same > anyway and that seemed to be the point of this linkage. The > dev_t <> softc <> device_t linkages aren't about unifying namespaces. The idea here is that virtually all uses of if_name/if_unit that aren't just there for the users benefit are actually references to the underlying driver not name of the interface. Currently they are the same (i.e. ifname is nearly always device_get_name(dev) or a bug prone manual version there of), but I would like to separate them so we can rename interfaces. Since device_t is as close to a repository of driver/instance information as we've got, I though using it would be a reasonable way to go. As a side benefit, most drivers have a copy of it in their softc already so you'd have a standard place to put it. I suppose a usable alternative would be to revive if_name and if_unit as something like if_drvname and if_drvunit. -- 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 --96YOpH+ONegL0A3E Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQE/ecHeXY6L6fI4GtQRAoM3AKCoaXbVJIwWuCegOL01REpap2jrfwCgiNiO vPGLE0NwLisRNtuK8jp0e2g= =S9HB -----END PGP SIGNATURE----- --96YOpH+ONegL0A3E--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030930174815.GC31908>