Paths

Table of Contentst

Differential D48456

nvmecontrol: Only do name resolution once for Fabrics connections
ClosedPublic
Actions

Authored by jhb on Jan 14 2025, 5:33 PM.

Tags

None

Referenced Files

	F176149917: D48456.id149246.diff
	Fri, Oct 16, 12:33 PM

	Unknown Object (File)
	Wed, Oct 14, 6:57 PM

	Unknown Object (File)
	Wed, Oct 14, 6:55 PM

	Unknown Object (File)
	Wed, Oct 14, 6:50 PM

	Unknown Object (File)
	Wed, Oct 14, 6:47 PM

	Unknown Object (File)
	Wed, Oct 14, 3:57 PM

	Unknown Object (File)
	Mon, Oct 12, 9:34 PM

	Unknown Object (File)
	Mon, Oct 12, 9:33 PM

Subscribers

Details

Reviewers

chuck
imp
dab

Commits

rGec282601f720: nvmecontrol: Only do name resolution once for Fabrics connections

Summary

Reuse the addrinfo structure from the admin queue to establish
connections to I/O queues for the TCP transport.

Sponsored by: Chelsio Communications

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 61950
Build 58834: arc lint + arc unit

Event Timeline

jhb created this revision.Jan 14 2025, 5:33 PM

Herald added a subscriber: dab. · View Herald TranscriptJan 14 2025, 5:33 PM

jhb requested review of this revision.Jan 14 2025, 5:33 PM

Harbormaster completed remote builds in B61717: Diff 149245.Jan 14 2025, 5:33 PM

jhb added a parent revision: D48214: nvmf: Refactor reconnection support.Jan 14 2025, 5:34 PM

Rebase

Harbormaster completed remote builds in B61718: Diff 149246.Jan 14 2025, 5:39 PM

imp accepted this revision.Jan 14 2025, 10:08 PM

This revision is now accepted and ready to land.Jan 14 2025, 10:08 PM

dab requested changes to this revision.Jan 18 2025, 1:46 AM

dab added inline comments.

sbin/nvmecontrol/fabrics.c
504	Isn't a `freeaddrinfo(list);` needed here before the `return`? Perhaps a new error-out goto label to consolidate the error cases where just that is needed.

This revision now requires changes to proceed.Jan 18 2025, 1:46 AM

imp added inline comments.Jan 18 2025, 2:08 AM

sbin/nvmecontrol/fabrics.c
504	oh my! I missed this, but I think you're right.

jhb added inline comments.Jan 23 2025, 10:35 PM

sbin/nvmecontrol/fabrics.c
504	Isn't a `freeaddrinfo(list);` needed here before the `return`? Perhaps a new error-out goto label to consolidate the error cases where just that is needed. Hmmm, if I clear the io[] array earlier, then the existing `out` label can work for most of these.

Add missing free

Harbormaster completed remote builds in B61950: Diff 149839.Jan 23 2025, 11:29 PM

dab accepted this revision.Jan 24 2025, 1:12 AM

This revision is now accepted and ready to land.Jan 24 2025, 1:12 AM

jhb added a parent revision: D48654: nvmecontrol: Consolidate error paths in connect_nvm_queues().Jan 24 2025, 2:42 PM

Closed by commit rGec282601f720: nvmecontrol: Only do name resolution once for Fabrics connections (authored by jhb). · Explain WhyJan 30 2025, 3:52 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGec282601f720: nvmecontrol: Only do name resolution once for Fabrics connections.

Revision Contents
Changeset List

Path

Size

sbin/

nvmecontrol/

43 lines

Diff 149839

sbin/nvmecontrol/fabrics.c

Loading...