Page MenuHomeFreeBSD
Differential D41353

nvme: Directly lookup op code
ClosedPublic
Actions

Authored by imp on Aug 7 2023, 10:12 PM.
Tags
None
Referenced Files
F164815631: D41353.diff
Tue, Aug 4, 4:37 AM
Unknown Object (File)
Fri, Jul 31, 4:42 PM
Unknown Object (File)
Wed, Jul 22, 9:54 PM
Unknown Object (File)
Tue, Jul 21, 6:15 AM
Unknown Object (File)
Sun, Jul 19, 4:38 PM
Unknown Object (File)
Sat, Jul 18, 6:43 AM
Unknown Object (File)
Jul 3 2026, 5:07 AM
Unknown Object (File)
Jun 29 2026, 11:38 AM
View All Files
Subscribers
dab

Details

Reviewers
jhb
chuck
mav
Commits
rG2ad9a815fd58: nvme: Directly lookup op code
Summary

Rather than have a table to walk through, use a sparse array.

Suggested by: jhb
Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp created this revision.Aug 7 2023, 10:12 PM
Herald added a subscriber: dab. · View Herald TranscriptAug 7 2023, 10:12 PM
imp requested review of this revision.Aug 7 2023, 10:12 PM
Harbormaster completed remote builds in B53008: Diff 125673.Aug 7 2023, 10:12 PM
imp mentioned this in D41310: nvme: Eliminate redundant code.Aug 7 2023, 10:13 PM
imp added a parent revision: D41311: nvme: Add more NVME Base Spec 2.0 and NVME Command Set Spec 1.0a.
imp added a child revision: D41312: nvme: Convert adjacent bools to a bitfield.
jhb accepted this revision.Aug 7 2023, 10:45 PM
jhb added inline comments.
sys/dev/nvme/nvme_qpair.c
105

I do think you can change the opc type to uint8_t and then you can remove the check against DEFAULT_INDEX.

This revision is now accepted and ready to land.Aug 7 2023, 10:45 PM
Closed by commit rG2ad9a815fd58: nvme: Directly lookup op code (authored by imp). · Explain WhyAug 7 2023, 10:47 PM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG2ad9a815fd58: nvme: Directly lookup op code.

Revision Contents
Changeset List

PathSize
sys/
dev/
nvme/
114 lines

Diff 125682

View Options

sys/dev/nvme/nvme_qpair.c

Loading...