ntb_hw(4): Add sysctls for administrative/test link config, state
ClosedPublic
Actions

Authored by cem on May 17 2016, 10:47 PM.

Details

Reviewers

ngie

Commits

Summary

dev.ntb_hw.0.admin_up=0/1: Like ifconfig UP/DOWN.
dev.ntb_hw.0.active=0/1: Like ifconfig 'status'

Sponsored by: EMC / Isilon Storage Division

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3798
Build 3841: arc lint + arc unit

cem retitled this revision from to ntb_hw(4): Add sysctls for administrative/test link config, state.

cem updated this object.

cem edited the test plan for this revision. (Show Details)

cem added a reviewer: ngie.

Accidentally got an older version of the patch. This is the latest version.

sys/dev/ntb/ntb_hw/ntb_hw.c
2646	Should these be #defines that correspond with admin_up's states?

sys/dev/ntb/ntb_hw/ntb_hw.c
2646	NTB_SPEED/WIDTH are already a set of enums. For now, I don't think there's any need to expose speed/width as administrative configuration options. The hw will just train whatever the best it can sustain. Later, knobs could be added to configure speed/width like ifconfig.