Page MenuHomeFreeBSD
Differential D18459

Const poison the command interface
ClosedPublic
Actions

Authored by imp on Dec 6 2018, 6:25 PM.
Tags
None
Referenced Files
F135924334: D18459.diff
Fri, Nov 14, 5:15 AM
F135873334: D18459.id51692.diff
Thu, Nov 13, 6:02 PM
Unknown Object (File)
Wed, Nov 5, 6:25 PM
Unknown Object (File)
Wed, Nov 5, 2:03 PM
Unknown Object (File)
Wed, Nov 5, 11:18 AM
Unknown Object (File)
Wed, Nov 5, 7:14 AM
Unknown Object (File)
Wed, Nov 5, 5:18 AM
Unknown Object (File)
Tue, Nov 4, 11:29 PM
View All 75 Files
Subscribers
None

Details

Reviewers
cem
Commits
rS341658: Const poison the command interface
Summary

Make the pointers we pass into the commands const, also make the
linker set mirrors const.

Suggested by: cem@
Sponsored by: Netflix

Test Plan

made a separate commit because it seemed needlessly churny to include it in the other one.

Diff Detail

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

Event Timeline

imp created this revision.Dec 6 2018, 6:25 PM
Harbormaster completed remote builds in B21404: Diff 51675.Dec 6 2018, 6:25 PM
imp edited the test plan for this revision. (Show Details)Dec 6 2018, 6:26 PM
imp added a reviewer: cem.
cem accepted this revision.Dec 6 2018, 10:43 PM
This revision is now accepted and ready to land.Dec 6 2018, 10:43 PM
Closed by commit rS341658: Const poison the command interface (authored by imp). · Explain WhyDec 6 2018, 10:59 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sbin/
nvmecontrol/
2 lines
2 lines
2 lines
6 lines
6 lines
26 lines
16 lines
8 lines
2 lines
2 lines
2 lines
6 lines

Diff 51692

View Options

head/sbin/nvmecontrol/devlist.c

Loading...
View Options

head/sbin/nvmecontrol/firmware.c

Loading...
View Options

head/sbin/nvmecontrol/format.c

Loading...
View Options

head/sbin/nvmecontrol/identify.c

Loading...
View Options

head/sbin/nvmecontrol/logpage.c

Loading...
View Options

head/sbin/nvmecontrol/ns.c

Loading...
View Options

head/sbin/nvmecontrol/nvmecontrol.h

Loading...
View Options

head/sbin/nvmecontrol/nvmecontrol.c

Loading...
View Options

head/sbin/nvmecontrol/perftest.c

Loading...
View Options

head/sbin/nvmecontrol/power.c

Loading...
View Options

head/sbin/nvmecontrol/reset.c

Loading...
View Options

head/sbin/nvmecontrol/wdc.c

Loading...