Skip to content

Disable managed memory on WSL when concurrentManagedAccess is not supported - #3095

Merged
zcbenz merged 2 commits into
ml-explore:mainfrom
jessegross:wsl
Feb 5, 2026
Merged

zcbenz merged 2 commits into
ml-explore:mainfrom
jessegross:wsl

Conversation

@jessegross

Copy link
Copy Markdown
Contributor

Extend the Windows managed memory check from #3075 to also apply to WSL, as the underlying behavior is the same.

Otherwise, on WSL we get a segfault on load

…ported

Extend the Windows managed memory check from #3075 to also apply to WSL,
as the underlying behavior is the same.

@zcbenz zcbenz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me since it is what everyone does: microsoft/WSL#423 (comment).

Thanks for fixing this!

@zcbenz

zcbenz commented Feb 3, 2026

Copy link
Copy Markdown
Member

Can you fix the lint error?

@jessegross

Copy link
Copy Markdown
Contributor Author

Just did :)

@zcbenz
zcbenz merged commit f47729c into ml-explore:main Feb 5, 2026
16 checks passed
@jessegross
jessegross deleted the wsl branch February 5, 2026 19:56
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