Skip to content

switch to CimWriter.dll and regenerate syscalls - #2578

Merged
anmaxvl merged 2 commits into
microsoft:mainfrom
anmaxvl:cimwriter
Dec 23, 2025
Merged

switch to CimWriter.dll and regenerate syscalls#2578
anmaxvl merged 2 commits into
microsoft:mainfrom
anmaxvl:cimwriter

Conversation

@anmaxvl

@anmaxvl anmaxvl commented Dec 19, 2025

Copy link
Copy Markdown
Contributor

Update CIMFS image creation syscalls to use CimWriter.dll.

Update CIMFS image creation syscalls to use CimWriter.dll.

Signed-off-by: Maksim An <maksiman@microsoft.com>
@anmaxvl
anmaxvl requested a review from a team as a code owner December 19, 2025 22:16

@helsaawy helsaawy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe not in this PR, but can we here:

func CimFsSupported() bool {
	modcimfs.Load() == nil && modcimwriter.Load() == nil
}

That way we can have in pkg\cimfs\cimfs.go:

func IsCimFSSupported() bool {
	return (build >= osversion.V25H1Server || (build == osversion.V21H2Server && rv >= 2031)) && winapi.CimFsSupported()
}

And avoid the test failures because the DLL isn't available?

Add `CimFsSupported` utility to check for CIM FS dlls.

Update `pkg/cimfs/cimfs.go` to use `CimFsSupported` together
with build number check.

Signed-off-by: Maksim An <maksiman@microsoft.com>
@msscotb msscotb self-assigned this Dec 22, 2025
@anmaxvl
anmaxvl merged commit 9df3a80 into microsoft:main Dec 23, 2025
35 of 41 checks passed
@anmaxvl
anmaxvl deleted the cimwriter branch December 23, 2025 17:32
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.

3 participants