Differential D34023

libthr: Use TLS_TCB_* in _tcb_[cd]tor.
ClosedPublic
Actions

Authored by jhb on Jan 25 2022, 12:24 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 15, 6:06 AM

	Unknown Object (File)
	Fri, Nov 15, 5:23 AM

	Unknown Object (File)
	Sep 21 2024, 8:06 AM

	Unknown Object (File)
	Sep 18 2024, 4:14 AM

	Unknown Object (File)
	Sep 17 2024, 4:20 PM

	Unknown Object (File)
	Sep 16 2024, 8:46 PM

	Unknown Object (File)
	Sep 7 2024, 10:14 PM

	Unknown Object (File)
	Sep 5 2024, 1:52 PM

View All 41 Files

Subscribers

Details

Reviewers

Commits

rGc4ec1f6efb52: libthr: Use TLS_TCB_* in _tcb_[cd]tor.
rG8de1a8131e42: libthr: Use TLS_TCB_* in _tcb_[cd]tor.

Summary

This matches libc and rtld in using the alignment (TLS_TCB_ALIGN) from
machine/tls.h instead of hardcoding 16.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Jan 25 2022, 12:24 AM

Herald added a subscriber: imp. · View Herald TranscriptJan 25 2022, 12:24 AM

jhb requested review of this revision.Jan 25 2022, 12:24 AM

Harbormaster completed remote builds in B44077: Diff 101866.Jan 25 2022, 12:24 AM

I missed this in my earlier machine/tls.h changes. I only changed the size argument to be consistent with rtld/libc that use TLS_TCB_SIZE.

I believe this is fine, although it indeed changes the alignment.

This revision is now accepted and ready to land.Jan 25 2022, 12:37 AM

Closed by commit rG8de1a8131e42: libthr: Use TLS_TCB_* in _tcb_[cd]tor. (authored by jhb). · Explain WhyJan 25 2022, 7:40 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG8de1a8131e42: libthr: Use TLS_TCB_* in _tcb_[cd]tor..

jhb added a commit: rGc4ec1f6efb52: libthr: Use TLS_TCB_* in _tcb_[cd]tor..Apr 29 2022, 11:13 PM

Revision Contents
Changeset List

Path

Size

lib/

libthr/

thread/

4 lines

Diff 101916

lib/libthr/thread/thr_ctrdtr.c

Loading...