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)
	Thu, May 14, 2:51 PM

	Unknown Object (File)
	Thu, May 14, 9:48 AM

	Unknown Object (File)
	Wed, May 13, 4:34 PM

	Unknown Object (File)
	Mon, May 11, 11:42 AM

	Unknown Object (File)
	Sun, May 10, 9:12 AM

	Unknown Object (File)
	Sun, May 10, 9:08 AM

	Unknown Object (File)
	Sat, May 9, 1:42 AM

	Unknown Object (File)
	Sat, May 9, 1:36 AM

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