Differential D55310

getnetconfig: make nc_error a thread local variable
ClosedPublic
Actions

Authored by brooks on Feb 16 2026, 2:17 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Apr 29, 9:32 AM

	Unknown Object (File)
	Wed, Apr 29, 8:04 AM

	Unknown Object (File)
	Wed, Apr 29, 8:01 AM

	Unknown Object (File)
	Mon, Apr 27, 12:31 PM

	Unknown Object (File)
	Mon, Apr 20, 4:35 AM

	Unknown Object (File)
	Tue, Apr 14, 7:58 AM

	Unknown Object (File)
	Tue, Apr 14, 2:08 AM

	Unknown Object (File)
	Mon, Apr 13, 8:17 PM

View All 35 Files

Subscribers

Details

Reviewers

Commits

rGdb1ecd5854d4: getnetconfig: make nc_error a thread local variable

Summary

Remove a bunch of complexity at the cost of 4 bytes of storage per
thread.

Sponsored by: DARPA, AFRL

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

brooks created this revision.Feb 16 2026, 2:17 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 16 2026, 2:17 PM

brooks requested review of this revision.Feb 16 2026, 2:17 PM

Harbormaster completed remote builds in B70700: Diff 171973.Feb 16 2026, 2:17 PM

I suspect the complexity wasn't to avoid 4 bytes per thread but more because TLS wasn't reliably working on all of our architectures when this was first written (but I could be wrong). :)

This revision is now accepted and ready to land.Feb 25 2026, 1:45 AM

In D55310#1269811, @jhb wrote:

I suspect the complexity wasn't to avoid 4 bytes per thread but more because TLS wasn't reliably working on all of our architectures when this was first written (but I could be wrong). :)

I'm sure that's true, it just turns out to be a small benefit in the common case. There are a bunch more of these that I'm tempted to convert, but they involve large structs.

Closed by commit rGdb1ecd5854d4: getnetconfig: make nc_error a thread local variable (authored by brooks). · Explain WhyMar 5 2026, 10:31 AM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rGdb1ecd5854d4: getnetconfig: make nc_error a thread local variable.

Revision Contents
Changeset List

Path

Size

lib/

libc/

rpc/

39 lines

Diff 173173

lib/libc/rpc/getnetconfig.c

Loading...