Differential D5181

Add defines for WRITE_UNCORRECTABLE ATA command, and improve command logging.
ClosedPublic
Actions

Authored by rpokala on Feb 4 2016, 1:58 AM.

Tags

None

Referenced Files

	F102147768: D5181.id13024.diff
	Fri, Nov 8, 5:18 AM

	Unknown Object (File)
	Sep 28 2024, 5:15 AM

	Unknown Object (File)
	Sep 23 2024, 2:16 PM

	Unknown Object (File)
	Sep 18 2024, 11:04 PM

	Unknown Object (File)
	Sep 16 2024, 9:14 AM

	Unknown Object (File)
	Sep 12 2024, 2:45 AM

	Unknown Object (File)
	Sep 8 2024, 10:45 AM

	Unknown Object (File)
	Aug 30 2024, 5:13 PM

Subscribers

Details

Reviewers

imp
mav

Commits

rS295276: Add defines for WRITE_UNCORRECTABLE ATA command, and improve command logging

Summary

Add #defines for ATA_WRITE_UNCORRECTABLE48 and its features. Update the
decoding in ATACAM to recognize the new values. Also improve command
decoding for a few other commands (SMART, NOP, SET_FEATURES). Bring the
decoding in ata(4) up to parity with ATACAM.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2381
Build 2397: arc lint + arc unit

Event Timeline

rpokala updated this revision to Diff 12988.Feb 4 2016, 1:58 AM

rpokala retitled this revision from to Add defines for WRITE_UNCORRECTABLE ATA command, and improve command logging..

rpokala updated this object.

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

rpokala added a reviewer: mav.

Herald added a subscriber: imp. · View Herald TranscriptFeb 4 2016, 1:58 AM

Remove if_lagg.{c,h} from this review; they're from something else.

Looks good to me. I have no objections. Some commands mentioned there are obsolete now, but that is not really a problem.

This revision is now accepted and ready to land.Feb 4 2016, 12:04 PM

imp accepted this revision.Feb 4 2016, 6:33 PM

imp added a reviewer: imp.

Closed by commit rS295276: Add defines for WRITE_UNCORRECTABLE ATA command, and improve command logging (authored by rpokala). · Explain WhyFeb 4 2016, 7:54 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

cam/

ata/

60 lines

dev/

ata/

100 lines

sys/

3 lines

Diff 12989

sys/cam/ata/ata_all.c

Loading...

sys/dev/ata/ata-all.c

Loading...

sys/sys/ata.h

Loading...