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

	F164845761: D55310.id173173.diff
	Tue, Aug 4, 7:29 AM

	Unknown Object (File)
	Mon, Aug 3, 6:06 AM

	Unknown Object (File)
	Thu, Jul 30, 6:34 PM

	Unknown Object (File)
	Sat, Jul 25, 4:24 PM

	Unknown Object (File)
	Thu, Jul 23, 2:38 AM

	Unknown Object (File)
	Tue, Jul 21, 2:46 PM

	Unknown Object (File)
	Wed, Jul 15, 7:08 AM

	Unknown Object (File)
	Wed, Jul 15, 7:08 AM

View All 56 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...