Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-13361

regression: we can not longer bootup reactos without ntdll_vista (& kernel32_vista) and waste 380Kb RAM additionally

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Fix Version/s: 0.4.6
    • Component/s: None
    • Labels:
      None

      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

            Activity

              People

              • Assignee:
                Heis Spiter Pierre Schweitzer
                Reporter:
                reactosfanboy reactosfanboy
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: