Page MenuHomeFreeBSD

libnvmf: Improve handling of the CATTR field in CONNECT commands
Needs ReviewPublic

Authored by jhb on Thu, Oct 8, 3:38 PM.
Tags
None
Referenced Files
F175808519: D60469.id189060.diff
Tue, Oct 13, 1:04 PM
F175804862: D60469.id189258.diff
Tue, Oct 13, 12:14 PM
F175798245: D60469.id189450.diff
Tue, Oct 13, 10:55 AM
F175743817: D60469.diff
Mon, Oct 12, 11:35 PM
Unknown Object (File)
Mon, Oct 12, 1:53 PM
Unknown Object (File)
Mon, Oct 12, 12:00 PM
Unknown Object (File)
Mon, Oct 12, 5:00 AM
Unknown Object (File)
Mon, Oct 12, 12:04 AM
Subscribers
None

Details

Reviewers
imp
Group Reviewers
cam
Summary
  • Add an association option to permit advertising support for I/O queue deletion in CATTR from the host side.
  • When validating CONNECT commands in nvmf_accept(), validate the CATTR field rejecting non-zero values in reserved fields.

Diff Detail

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

Event Timeline

jhb requested review of this revision.Thu, Oct 8, 3:38 PM
jhb created this revision.

Neither the host nor controller sides in our NVMeoF bits support deleting individual I/O queues, so adding the new parameter is cosmetic (But needed if we ever had a compelling use case. I should probably implement support for it on the controller/target side at some point, but would need to be able to test it). However, validating that reserved fields aren't set is useful. I guess I can actually add some tests based on some recent tests I've started writing for nvmft at least, so I'll probably defer merging this until I can add some tests.

Add tests for invalid connect attributes