Skip to content

Allocate string literals on frozen segments #49576

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jkotas merged 69 commits into from
Sep 13, 2022
Merged

Allocate string literals on frozen segments #49576

Changes from 1 commit
Commits
Show all changes
69 commits
Select commit Hold shift + click to select a range
166ccfe
Naive implementation of FrozenObjectHeap
EgorBo Jul 3, 2021
1cb0927
Add RegisterFrozenSegment
EgorBo Jul 3, 2021
d2b9cbf
Clean up
EgorBo Jul 4, 2021
641365a
Add #include "common.h"
EgorBo Jul 4, 2021
33a126b
Fix build error
EgorBo Jul 4, 2021
bb1b835
Fix build
EgorBo Jul 4, 2021
2ef204a
Add lock
EgorBo Jul 4, 2021
a84a6ee
Clean up
EgorBo Jul 4, 2021
142295a
init memory on first allocation
EgorBo Jul 4, 2021
a446dd6
Clean up
EgorBo Jul 4, 2021
41665c3
revert changes in emitxarch.cpp
EgorBo Jul 4, 2021
66d6106
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Jul 28, 2022
67f32a7
Update branch, fix asserts
EgorBo Jul 28, 2022
9ecf7d7
fix release build
EgorBo Jul 28, 2022
45a454a
Ignore collectible assemblies, fix build
EgorBo Jul 29, 2022
1d0a710
Fix failing tests, address feedback
EgorBo Jul 29, 2022
201565e
Rename bAppDomainWontUnload to bIsCollectible
EgorBo Jul 29, 2022
c7dad85
Address feedback
EgorBo Jul 29, 2022
4202d9f
Clean up
EgorBo Jul 29, 2022
69ab9d2
Merge branch 'main' of github.com:dotnet/runtime into poh2
EgorBo Jul 29, 2022
a144453
Fix assert
EgorBo Jul 29, 2022
2571c9c
Fix more asserts not expecting TYP_REF constants being non zero
EgorBo Jul 29, 2022
286f8ab
Merge branch 'main' of github.com:dotnet/runtime into poh2
EgorBo Jul 29, 2022
28e0331
Test gc fix
EgorBo Jul 29, 2022
6625759
move check to IsEphemeral
EgorBo Jul 30, 2022
89df356
Test commit (do I need to update "ibAllocated"?)
EgorBo Jul 30, 2022
3cb98e9
Implement commit-on-demand
EgorBo Jul 30, 2022
c6df9ed
Clean up
EgorBo Jul 30, 2022
f50b4b9
bump FOH_COMMIT_PAGES
EgorBo Jul 30, 2022
2b55835
Merge branch 'main' of github.com:dotnet/runtime into poh2
EgorBo Jul 30, 2022
6ea2a50
Address feedback, clean up
EgorBo Jul 30, 2022
15ca880
Fix build
EgorBo Jul 30, 2022
193676e
Merge branch 'main' of github.com:dotnet/runtime into poh2
EgorBo Jul 30, 2022
9198043
Unix's ClrVirtualAlloc seems to be non-aligned
EgorBo Jul 30, 2022
7c98ef2
Oops, fix red CI
EgorBo Jul 30, 2022
af6e445
Multiple frozen segments
EgorBo Jul 30, 2022
2183c2f
Merge branch 'main' of github.com:dotnet/runtime into poh2
EgorBo Jul 31, 2022
4ef34dd
Clean up
EgorBo Jul 31, 2022
722eb82
fix typo
EgorBo Jul 31, 2022
0ae890a
Fix assert in assertionprop.cpp
EgorBo Jul 31, 2022
e59d27a
Address feedback
EgorBo Aug 18, 2022
90feb96
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Aug 18, 2022
ce3e19e
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Sep 1, 2022
b6fae80
Address feedback
EgorBo Sep 1, 2022
32cb338
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Sep 1, 2022
d20cc47
Address feedback
EgorBo Sep 2, 2022
31b5fdc
Fix potential issues
EgorBo Sep 3, 2022
1b1ab4c
Fix potential issues
EgorBo Sep 3, 2022
ed8ed7b
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Sep 4, 2022
88ad320
Address feedback
EgorBo Sep 4, 2022
943a235
Address feedback
EgorBo Sep 4, 2022
a3bfc8c
Check if this fixes all asserts
EgorBo Sep 5, 2022
1be12c5
Handle string.Empty (it used to be ignored)
EgorBo Sep 5, 2022
5c2a222
Apply suggestions from code review
EgorBo Sep 5, 2022
19d1734
Address feedback
EgorBo Sep 5, 2022
4762bcc
Address feedback
EgorBo Sep 5, 2022
2e03cc2
Update src/coreclr/vm/stringliteralmap.cpp
EgorBo Sep 5, 2022
2f00f71
Update src/coreclr/vm/stringliteralmap.cpp
EgorBo Sep 5, 2022
ddfbf73
fix compilation errors
EgorBo Sep 5, 2022
598b9c8
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Sep 7, 2022
802106e
Address Jan's feedback
EgorBo Sep 7, 2022
bfe96e7
Update src/coreclr/jit/gentree.cpp
EgorBo Sep 7, 2022
2e10c9c
Update gentree.cpp
EgorBo Sep 7, 2022
3415dc3
Merge branch 'main' of github.com:dotnet/runtime into poh-string-lite…
EgorBo Sep 12, 2022
bac0389
Address Maoni's feedback:
EgorBo Sep 13, 2022
01b5731
heap -> segment in comments
EgorBo Sep 13, 2022
8d139c9
Update src/coreclr/vm/frozenobjectheap.h
EgorBo Sep 13, 2022
b6deb3c
Address Jan's feedback around m_Crst creation
EgorBo Sep 13, 2022
1ba9d97
Fix order of initialization
EgorBo Sep 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix compilation errors
  • Loading branch information
@EgorBo
EgorBo committed Sep 5, 2022
commit ddfbf73e3570d8944d0279ba0c9c5c695464e2f1
16 changes: 3 additions & 13 deletions src/coreclr/vm/stringliteralmap.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -652,21 +652,11 @@ void StringLiteralEntry::DeleteEntry (StringLiteralEntry *pEntry)
}
CONTRACTL_END;

if (pEntry->IsStringFrozen())
{
_ASSERTE (pEntry->IsStringFrozen() || pEntry->GetRefCount() == 0);
#ifdef _DEBUG
memset (&pEntry->m_FrozenStringObj, 0xc, sizeof(pEntry->m_FrozenStringObj));
pEntry->m_bDeleted = TRUE;
memset (&pEntry->m_pStringObj, 0xc, sizeof(pEntry->m_pStringObj));
pEntry->m_bDeleted = TRUE;
#endif
}
else
{
_ASSERTE (pEntry->GetRefCount() == 0);
#ifdef _DEBUG
memset (&pEntry->m_pStringObj, 0xc, sizeof(pEntry->m_pStringObj));
pEntry->m_bDeleted = TRUE;
#endif
}

// The free list needs protection from the m_HashTableCrstGlobal
pEntry->m_pNext = s_FreeEntryList;
Expand Down