Date: Mon, 16 Feb 2015 23:17:03 -0800 From: Julian Elischer <julian@freebsd.org> To: mike@karels.net, George Neville-Neil <gnn@neville-neil.com> Cc: "freebsd-net@freebsd.org" <freebsd-net@freebsd.org>, "freebsd-arch@freebsd.org" <freebsd-arch@freebsd.org> Subject: Re: Adding new media types to if_media.h Message-ID: <54E2EAEF.5050201@freebsd.org> In-Reply-To: <201502170150.t1H1ouxM020621@mail.karels.net> References: <201502170150.t1H1ouxM020621@mail.karels.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2/16/15 5:50 PM, Mike Karels wrote: > Well, I developed the prototype as I had planned, using a 64-bit > media word, and found that I got about 100 files in GENERIC that > didn't compile; [...] > so I went to Plan B. Plan B is to steal an unused bit (RFU) to > indicate an "extended" media type. I then used the variant/subtype > field to store the extended type. [...] > I modified one driver (vtnet) to use an extended type, and the rest > of GENERIC is happy. The changes to ifconfig are also fairly small. > The patch is appended, where email programs will screw it up, or at > ftp://ftp.karels.net/outgoing/if_media.patch. The VFAST subtype is a > throw-away for testing. This seems like a reasonably pragmatic > change to support the new 40 Gb/s media types until someone wants to > design an improved but non-backward- compatible interface. I think > it meets the goal of suitability for back-porting; it could be > MFCed. Mike Index: sys/net/if_media.h I like it.. The patch seems appropriately manageable.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?54E2EAEF.5050201>