Page MenuHomeFreeBSD

net: Add new 10G and 25G media types to if_media.h
ClosedPublic

Authored by erj on Apr 19 2017, 12:18 AM.
Referenced Files
Unknown Object (File)
Tue, Apr 9, 2:20 PM
Unknown Object (File)
Dec 20 2023, 2:48 AM
Unknown Object (File)
Nov 18 2023, 7:49 PM
Unknown Object (File)
Nov 18 2023, 1:43 PM
Unknown Object (File)
Nov 16 2023, 8:05 PM
Unknown Object (File)
Nov 11 2023, 6:42 PM
Unknown Object (File)
Nov 10 2023, 3:38 PM
Unknown Object (File)
Nov 9 2023, 6:48 PM
Subscribers

Details

Summary

A new XXV710 (uses the ixl(4) driver) firmware coming out soon is supposed to add support for new Active Optical / Copper cables, so add them here so that they can be displayed in ifconfig.

Also add 25Gbase-LR because that wasn't in the media type list, and should theoretically be already supported by the current cards.

Test Plan

Plug in each of the above mentioned cables and verify that they're displayed in ifconfig.

(Hasn't been tested yet)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

smh added a subscriber: smh.

LGTM

This revision is now accepted and ready to land.Apr 19 2017, 3:40 AM

I kind of want to bump __FreeBSD_version to make it easier for our drivers to see if there's support for these new media types.

In D10425#216318, @erj wrote:

I kind of want to bump __FreeBSD_version to make it easier for our drivers to see if there's support for these new media types.

__FreeBSD_version numbers aren't that much of a scarce commodity so you could bump it if needed, but what about just #ifdef IFM_10G_AOC etc.?

You could ifdef the things for which version supports it. But that interacts poorly with the ports system, so I'd go for it on bumping the FreeBSD_version. It's cheap enough.

In D10425#216333, @imp wrote:

You could ifdef the things for which version supports it. But that interacts poorly with the ports system, so I'd go for it on bumping the FreeBSD_version. It's cheap enough.

I mean in the driver, just do #ifdef IFM_10G_AOC instead of #if __FreeBSD_version >= xxxxxx.
But agreed __FreeBSD_version is cheap enough so it's fine too.

I've decided against bumping the FreeBSD version number -- there aren't very many defines added here, so adding #ifdef's isn't too big of a deal.

This revision was automatically updated to reflect the committed changes.