Page MenuHomeFreeBSD
Differential D44654

nvmecontrol: Move self test status page printing to little endian orderinng
ClosedPublic
Actions

Authored by imp on Apr 6 2024, 6:56 PM.
Tags
None
Referenced Files
F166918259: D44654.id137219.diff
Mon, Aug 17, 6:29 PM
F166885717: D44654.id136617.diff
Mon, Aug 17, 12:56 PM
F166882701: D44654.id136686.diff
Mon, Aug 17, 12:24 PM
F166825105: D44654.id137219.diff
Sun, Aug 16, 9:20 PM
Unknown Object (File)
Sun, Aug 16, 8:31 PM
Unknown Object (File)
Fri, Aug 14, 3:47 PM
Unknown Object (File)
Thu, Aug 13, 6:26 PM
Unknown Object (File)
Thu, Aug 13, 10:41 AM
View All Files
Subscribers
dab

Details

Reviewers
jhb
chuck
mav
Commits
rG067abd7d0b06: nvmecontrol: Move self test status page printing to little endian orderinng
rG98f841efd8ce: nvmecontrol: Move self test status page printing to little endian orderinng
Summary

Also, add printing vnedor_specific field, which doesn't have a valid
bit, so is always valid.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56938
Build 53826: arc lint + arc unit

Event Timeline

imp created this revision.Apr 6 2024, 6:56 PM
Herald added a subscriber: dab. · View Herald TranscriptApr 6 2024, 6:56 PM
imp requested review of this revision.Apr 6 2024, 6:56 PM
Harbormaster completed remote builds in B56938: Diff 136617.Apr 6 2024, 6:56 PM
imp added a parent revision: D44653: nvmecontrol: Move namespace change page printing to little endian orderinng.Apr 6 2024, 6:56 PM
imp added a child revision: D44655: nvmecontrol: Move command effeccts page printing to little endian orderinng.
chuck accepted this revision.Apr 8 2024, 3:10 PM
chuck added inline comments.
sbin/nvmecontrol/logpage.c
593

Probably wrap this with LE2H() for consistency

635

Shouldn't the if expressions wrap valid_diag_info to be consistent? Here and below.

This revision is now accepted and ready to land.Apr 8 2024, 3:10 PM
imp updated this revision to Diff 136686.Apr 8 2024, 4:57 PM

Fix based on review feedback

This revision now requires review to proceed.Apr 8 2024, 4:57 PM
Harbormaster completed remote builds in B56961: Diff 136686.Apr 8 2024, 4:57 PM
chuck accepted this revision.Apr 8 2024, 5:06 PM
This revision is now accepted and ready to land.Apr 8 2024, 5:06 PM
Closed by commit rG98f841efd8ce: nvmecontrol: Move self test status page printing to little endian orderinng (authored by imp). · Explain WhyApr 17 2024, 3:34 AM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG98f841efd8ce: nvmecontrol: Move self test status page printing to little endian orderinng.
jhb added a commit: rG067abd7d0b06: nvmecontrol: Move self test status page printing to little endian orderinng.Aug 27 2024, 1:07 AM

Revision Contents
Changeset List

PathSize
sbin/
nvmecontrol/
31 lines

Diff 136617

View Options

sbin/nvmecontrol/logpage.c

Loading...