Page MenuHomeFreeBSD

Add the Thunderbolt Apple interfaces to the bge(4) supported list
ClosedPublic

Authored by sevan on Jul 28 2017, 2:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 3:57 PM
Unknown Object (File)
Sat, Nov 23, 7:20 PM
Unknown Object (File)
Sat, Nov 23, 11:27 AM
Unknown Object (File)
Nov 21 2024, 6:45 PM
Unknown Object (File)
Nov 21 2024, 4:35 PM
Unknown Object (File)
Nov 15 2024, 2:28 PM
Unknown Object (File)
Nov 14 2024, 3:07 PM
Unknown Object (File)
Nov 14 2024, 4:16 AM
Subscribers

Details

Summary

Tested both entries (adapter & ethernet interface on monitor)

dmesg from thunderbolt display
http://dmesgd.nycbug.org/index.cgi?do=view&id=3185

dmesg snippet from gigabit adapter (mac address trimmed)
bge0: <Thunderbolt Ethernet, ASIC rev. 0x57766000> mem 0xacb00000-0xacb0ffff,0xacb10000-0xacb1ffff at device 0.0 on pci8
bge0: CHIP ID 0x57766000; ASIC REV 0x57766; CHIP REV 0x577660; PCI-E
miibus0: <MII bus> on bge0
bge0: Using defaults for TSO: 65518/35/2048
bge0: Ethernet address: 40:6c:8f:

Not sure if I should extend the man page to state that hot plug isn't supported on FreeBSD (like on OS X) hence device should be connected prior to system power up and removal should be avoided when system is live (system stays up if I recall correctly, but kernel freaks out that a bge interface just disappeared).

Diff Detail

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

Event Timeline

Not sure if I should extend the man page to state that hot plug isn't supported on FreeBSD (like on OS X) hence device should be connected prior to system power up and removal should be avoided when system is live (system stays up if I recall correctly, but kernel freaks out that a bge interface just disappeared).

That seems like and important limitation, so yeah, I'd include that info.

Also: is that the right URL? The snippet doesn't appear there.

That seems like and important limitation, so yeah, I'd include that info.

Sure, add a BUGS section?

Also: is that the right URL? The snippet doesn't appear there.

The URL is the dmesg output taken from my system with a display connected (search for bge0)

The snippet is from my system, live, with a thunderbolt to gigabit adapter connected.
They're the two separate devices I'm adding to the manual.

That seems like and important limitation, so yeah, I'd include that info.

Sure, add a BUGS section?

If it's something that we plan on fixing soon, I'd say BUGS (i.e. this supposed to work, but we know it doesn't right now); if not, I'd say CAVEATS (i.e. this is a known limitation).

If it's something that we plan on fixing soon, I'd say BUGS (i.e. this supposed to work, but we know it doesn't right now); if not, I'd say CAVEATS (i.e. this is a known limitation).

http://mdocml.bsd.lv/man/mdoc.7.html
mandoc manual states the CAVEATS and BUGS section are for
CAVEATS

Common misuses and misunderstandings should be explained in this section.

BUGS

Known bugs, limitations, and work-arounds should be described in this section.

If you can't find a fitting section, then just put it into the normal text.

Add BUGS section and an entry to the DIAGNOSTICS section for the message reported when the interface is physically removed.

One sentence needs some work, the rest is fine and I like how you solved your "dilemma". ;-)

share/man/man4/bge.4
277 ↗(On Diff #32211)

There is either a "the" missing before Thunderbolt here or it must be s/interface/interfaces/ to make the sentence sound better. Also, I think it must be "Thunderbolt-based".

This revision is now accepted and ready to land.Aug 18 2017, 8:13 PM
This revision was automatically updated to reflect the committed changes.
head/share/man/man4/bge.4
245

"it is attached to" is not needed.

275

s/need to be/must be/

s/hence,/so/

s/prior to system power up on/before applying power on/

276

s/in order//

277
The lack of hotplug support means that Thunderbolt-based interfaces must not be removed
278

s/up as/up, as/