Details
-
Bug
-
Resolution: Fixed
-
Major
-
None
-
None
-
July 2017
-
74,703
Description
We recently lost a lot of memory after bootup due to the introduction of ntdll_vista.
Memory display in task-manager shows additional allocations of 380KB after bootup.
gcc dbg I18N de-DE
r74707 107296KB 1051Handles 109Threads 17processes
r74702 106916KB 1037Handles 109Threads 17processes
Furthermore we can not longer bootup reactos after renaming or deleting ntdll_vista now.
r74707-DoesNotBootUpWithRenamedNTDLLvista.dll.log
When reading the description of the tasks between r74702-r74707 it seemed their intention was to just move some unused code around out of kernel32.dll and add some more yet unsued code to the new dll (most likely for features to come).
Although that separation makes sense ofc,
Can we reclaim that memory please and ideally make ntdll_vista.dll (and kernel32_vista.dll) only loaded on demand / delay-loaded. Imho those should not be needed initially for system bootup and in case there are already hard-dependencies they eventually should be factored out.
ThFabba noted rpcrt4 imports kernel32_vista and kernel32_vista imports ntdll_vista.
Attachments
Issue Links
- blocks
-
CORE-7546 Implement condition variables
- Resolved
-
CORE-8204 Add nfs client support and pave way for Vagrant
- Resolved
-
CORE-13602 kernel32_vista delay loading
- Untriaged