Skip to content

Comments

feat: Initial WSL1 support#1207

Merged
facekapow merged 1 commit intodarlinghq:masterfrom
trungnt2910:dev/trungnt2910/wsl1-support
Aug 22, 2022
Merged

feat: Initial WSL1 support#1207
facekapow merged 1 commit intodarlinghq:masterfrom
trungnt2910:dev/trungnt2910/wsl1-support

Conversation

@trungnt2910
Copy link
Contributor

Part of #1206.
Should not be reviewed/merged before darlinghq/darlingserver#4.

  • mldr: Replaced mmap calls with compatible_mmap, a function
    that removes MAP_GROWSDOWN on incompatible systems, and emulates
    MAP_FIXED_NOREPLACE on older kernels.
  • mldr: Implemented lifetime pipes. These pipes are created during
    process creation and sent to darlingserver. The pipe is preserved
    through exec calls and automatically closes on process termination.
  • kernel/emulation: execve now preserves mldr lifetime pipes through
    the __mldr_lifetime_pipe environment variable.
  • kernel/emulation: Parameters for calls to dserver_rpc_checkin have
    been updated to reflect changes in darlingserver.

Copy link
Member

@facekapow facekapow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Let me do some testing locally to check it doesn't break Darling outside of WSL1.

- mldr: Replaced `mmap` calls with `compatible_mmap`, a function
that removes `MAP_GROWSDOWN` on incompatible systems, and emulates
`MAP_FIXED_NOREPLACE` on older kernels.
- mldr: Implemented lifetime pipes. These pipes are created during
process creation and sent to darlingserver. The pipe is preserved
through exec calls and automatically closes on process termination.
- kernel/emulation: `execve` now preserves mldr lifetime pipes through
the `__mldr_lifetime_pipe` environment variable.
- kernel/emulation: Parameters for calls to `dserver_rpc_checkin` have
been updated to reflect changes in darlingserver.
@trungnt2910 trungnt2910 force-pushed the dev/trungnt2910/wsl1-support branch from be9c27e to 7ca12cb Compare August 22, 2022 06:29
@facekapow facekapow merged commit 9825113 into darlinghq:master Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants