Skip to content

Conversation

@anmaxvl
Copy link
Contributor

@anmaxvl anmaxvl commented Dec 19, 2025

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
Copy link
Contributor

@helsaawy helsaawy left a comment

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