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

	Unknown Object (File)
	Thu, Mar 5, 6:34 PM

	Unknown Object (File)
	Thu, Mar 5, 9:08 AM

	Unknown Object (File)
	Wed, Mar 4, 1:15 PM

	Unknown Object (File)
	Mon, Mar 2, 9:46 PM

	Unknown Object (File)
	Mon, Mar 2, 6:14 PM

	Unknown Object (File)
	Sun, Mar 1, 8:01 AM

	Unknown Object (File)
	Sat, Feb 28, 2:03 PM

	Unknown Object (File)
	Sat, Feb 28, 8:29 AM

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 Not Applicable

Unit

Tests Not Applicable

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

head/

sys/

cam/

ata/

60 lines

dev/

ata/

100 lines

sys/

3 lines

Diff 13024

head/sys/cam/ata/ata_all.c

Loading...

head/sys/dev/ata/ata-all.c

Loading...

head/sys/sys/ata.h

Loading...