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)
Fri, Nov 15, 9:39 PM
Unknown Object (File)
Nov 14 2024, 1:29 PM
Unknown Object (File)
Nov 14 2024, 10:13 AM
Unknown Object (File)
Nov 13 2024, 3:01 PM
Unknown Object (File)
Nov 13 2024, 9:56 AM
Unknown Object (File)
Nov 12 2024, 10:27 PM
Unknown Object (File)
Nov 12 2024, 5:40 PM
Unknown Object (File)
Dec 20 2023, 2:48 AM
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

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8789
Build 9150: CI src buildJenkins
Build 9149: arc lint + arc unit

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.