Page MenuHomeFreeBSD

Implement a devtype command.

Authored by imp on Jul 15 2019, 6:26 AM.


Implement a devtype command.

List the device's protocol. The returned value is one of the following:
        ata     direct attach ATA or SATA device
        satl    a SATA device attached via SAS
        scsi    A parallel SCSI or SAS
        nvme    A direct attached NVMe device
        mmcsd   A MMC or SD attached device

Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline added inline comments.
2534 ↗(On Diff #59749)

The wording is a bit funny, device attached via device…

Maybe "attached via an SD/MMC controller"?

2527 ↗(On Diff #59749)

The term sat or satl is a more mainstream term here. I know that the CC_DT_ATA_BEHIND_SCSI name already exists, but for user-facing functionality, let's do it differently (and I wouldn't mind if the CC_DT name changed also).

2538 ↗(On Diff #59749)

Something to consider would be a raid devtype, since there are a fair number of drivers that have devices of this type.

271 ↗(On Diff #59749)

Use satl here.

2527 ↗(On Diff #59749)

I'll go with satl and move these changes through.

2534 ↗(On Diff #59749)

It's from the department of redundancy department. I'll fix their incursion into my work.

2538 ↗(On Diff #59749)

What distinguishes this? all the other things are basically a protocol level thing, while this I'm unsure how to implement.

271 ↗(On Diff #59749)

will do

updates from scott's comments

imp marked 4 inline comments as done.
imp added inline comments.
2538 ↗(On Diff #59749)

I'll add 'raid' in the future once we get sorted out what that means.

rpokala added inline comments.
2523 ↗(On Diff #59781)

I haven't looked at the larger context of this manpage, but shouldn't cam be at least capitalized, if not flagged as an acronym?

2528 ↗(On Diff #59781)

Perhaps spell out SCSI-ATA Translation Layer here?

2540 ↗(On Diff #59781)

s/Some crazy/An unexpected/

#professional ;-)

imp edited the summary of this revision. (Show Details)

Make it #professional, per rpokala

Looks reasonable to my not-very-CAM-savvy eyes.

This revision is now accepted and ready to land.Jul 15 2019, 9:41 PM
This revision was automatically updated to reflect the committed changes.

As this looks like a minor typo for me, I am trying to submit without any "add action".


I am sorry, but didn't you mean "attached"?