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)
	Wed, Oct 15, 1:31 AM

	Unknown Object (File)
	Wed, Oct 15, 1:31 AM

	Unknown Object (File)
	Wed, Oct 15, 1:31 AM

	Unknown Object (File)
	Tue, Oct 14, 6:34 PM

	Unknown Object (File)
	Tue, Oct 14, 4:44 PM

	Unknown Object (File)
	Mon, Oct 13, 8:05 AM

	Unknown Object (File)
	Wed, Sep 24, 7:10 PM

	Unknown Object (File)
	Mon, Sep 22, 11:16 PM

View All 76 Files

Subscribers

imp

Details

Reviewers

kib

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/

thr_ctrdtr.c

4 lines

Diff 101916

View Options

lib/libthr/thread/thr_ctrdtr.c

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