DescriptionRevert of Add Reclaim Support to ThreadLocalStorage (patchset #1 id:1 of https://codereview.chromium.org/2363783002/ )
Reason for revert:
Precautionary revert while waiting for https://codereview.chromium.org/2383833004/
Will reland after branch point.
Original issue's description:
> Add Reclaim Support to ThreadLocalStorage
>
> Previously, ThreadLocalStorage::StaticSlot::Free() did not actually
> release the corresponding ThreadLocalStorage slot. It simply cleared
> out the slot and did not reuse it. As a result, each process had a
> finite number of calls to ThreadLocalStorage::StaticSlot::Initialize()
> before running out of slots.
>
> This problem would manifest itself in tests where a single process runs
> many tests that each do their own initialization and uninitialization.
> Tests that involve TLS usage caused the process to run out of TLS slots
> because there was no free support in ThreadLocalStorage.
>
> This change adds in free support by doing what most operating systems
> do, lock and track metadata in an array.
>
> BUG=590907
>
> Committed: https://crrev.com/783d1f9630c86d816669ab23981d0f3722c96808
> Cr-Commit-Position: refs/heads/master@{#421320}
TBR=brettw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590907
Committed: https://crrev.com/75f2f7f6f9bdf3d851b5cc3914d32a3d0836c49d
Cr-Commit-Position: refs/heads/master@{#423575}
Patch Set 1 #
Messages
Total messages: 11 (5 generated)
|