Page MenuHomeFreeBSD
Differential D44723

ctl_backend_block: Add support for NVMe
ClosedPublic
Actions

Authored by jhb on Apr 9 2024, 11:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 18, 12:27 PM
Unknown Object (File)
Fri, Jan 17, 11:40 PM
Unknown Object (File)
Fri, Jan 17, 7:15 PM
Unknown Object (File)
Tue, Jan 14, 8:13 AM
Unknown Object (File)
Tue, Jan 14, 8:12 AM
Unknown Object (File)
Tue, Jan 14, 8:12 AM
Unknown Object (File)
Tue, Jan 14, 8:12 AM
Unknown Object (File)
Tue, Jan 14, 8:08 AM
View All Files
Subscribers
None

Details

Reviewers
imp
ken
Group Reviewers
cam
Commits
rG374f12c52f1b: ctl_backend_block: Add support for NVMe
Summary

Sponsored by: Chelsio Communications

Diff Detail

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

Event Timeline

jhb created this revision.Apr 9 2024, 11:04 PM
Herald added a reviewer: cam. · View Herald TranscriptApr 9 2024, 11:04 PM
jhb requested review of this revision.Apr 9 2024, 11:04 PM
Harbormaster completed remote builds in B57014: Diff 136819.Apr 9 2024, 11:04 PM
jhb added a parent revision: D44722: ctl_backend_ramdisk: Add support for NVMe.Apr 9 2024, 11:04 PM
jhb added a child revision: D44724: ctl: Add NVMF port type and ioctls.
jhb added a reviewer: ken.Apr 9 2024, 11:11 PM
imp accepted this revision.Apr 13 2024, 8:34 PM

this again seems a mix of accesor changes and and real changes... likely not a problem, but the former is easier to review when segregated.

This revision is now accepted and ready to land.Apr 13 2024, 8:34 PM
jhb added a comment.Apr 16 2024, 7:25 PM

Hmm, I could split the ramdisk/block backends into one to use the accessors and generic error handlers and a second to add the NVME opcode handling.

jhb updated this revision to Diff 137335.Apr 18 2024, 5:58 PM

Update after splitting into 2 commits

This revision now requires review to proceed.Apr 18 2024, 5:58 PM
Harbormaster completed remote builds in B57196: Diff 137335.Apr 18 2024, 5:58 PM
jhb added a parent revision: D44851: ctl_backend_block: Prepare for NVMe support.Apr 18 2024, 5:58 PM
jhb removed a parent revision: D44722: ctl_backend_ramdisk: Add support for NVMe.
jhb updated this revision to Diff 137973.May 1 2024, 8:30 PM

Update for CTL_IO_ASSERT macro

Harbormaster completed remote builds in B57509: Diff 137973.May 1 2024, 8:30 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 3 2024, 12:17 AM
Closed by commit rG374f12c52f1b: ctl_backend_block: Add support for NVMe (authored by jhb). · Explain Why
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rG374f12c52f1b: ctl_backend_block: Add support for NVMe.

Revision Contents
Changeset List

PathSize
sys/
cam/
ctl/
472 lines

Diff 138079

View Options

sys/cam/ctl/ctl_backend_block.c

Loading...