@tclem would you might providing a brief explanation on the thought process behind this?
In the usecase where the host has below 48bits of address space (commonly the case on arm64 linux kernels which might use 39bit or even 36bit address space) this results in the following similar lines of text being basically spammed in the log during process creation
```
002c:err:virtual:map_fixed_area out of memory for 0x6fffffcd0000-0x6ffffffef000
002c:err:virtual:try_map_free_area mmap() error Cannot allocate memory, range 0x7fffffcd0000-0x7ffffffef000, unix_prot 0x7.
```
Are 48bits of address space really a requirement for most applications and cause fatal errors without it?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3598#note_75871
--
v2: server: Skip inaccessible threads in (get_next_thread).
server: Do not allow zero access mask when opening some objects.
ntoskrnl.exe/tests: Open directory object with nonzero access in test_permanent().
ntdll/tests: Add tests for opening some objects with zero access.
https://gitlab.winehq.org/wine/wine/-/merge_requests/6047
This is part XIX of cmd engine rewrite.
It's the last MR (for now) for setting success/failure
return code on internal commands.
The remaining todo in this area are due to:
- incompletly implemented builtin (eg: TYPE not handling
multiple files, not wildcards; CHOICE missing a bunch
of features; ...),
- some commands (like START) being implemented as external
executables, hence forcing RETURN CODE = ERRORLEVEL
which is not always wanted,
--
v2: cmd: Cleanup transition bits.
cmd: Improve return code/errorlevel support for external commands.
cmd: Move empty batch command handling to WCMD_batch() callers.
cmd: Set success/failure return code for START command.
cmd: Set success/failure return code for MKLINK command.
cmd: Add more tests for return codes in builtin commands.
cmd: Get rid of CTTY command.
https://gitlab.winehq.org/wine/wine/-/merge_requests/6041