Add initial support for reusing drivers as PE drivers #115

JonasKloseBW · 2024-12-28T05:24:40Z

Add functions to read data from driver .ini files
Copy driver files required into the PE driver folder for system devices, scsi and raid controllers, keyboards and mice

This PR should filter and copy the files for the PE stage. I'll test the code and add fixes as required.

@rbalsleyMSFT
This should minimize the amount of copied files for the PE stage while enabling the usage of all necessary devices.

- Add functions to read data from driver .ini files - Copy driver files required into the PE driver folder for system devices, scsi and raid controllers, keyboards and mice

Fix an error for the DLLImport

- Filter correctly for inf-driver files - Ensure target path is created

- Cast PE driver directory creation to void to avoid unnecessary console output

JonasKloseBW · 2024-12-29T03:17:31Z

First successful test run completed. I'll test the PE drivers on real hardware asap.
The code copied more than 70 drivers to the PE folder.

- Increase the buffer reader size by 8x read longer file name lists - Fix cases in which the driver files are in subdirectories

JonasKloseBW · 2024-12-29T04:24:05Z

Successful test run for Surface Laptop Go 3 (code part only).
The PE driver folder ended to be 200MB though. Which is a bit too large for me.

JonasKloseBW · 2024-12-29T04:37:12Z

The regular Surface Laptop Go 3 driver folder is 2.3GB, so 200MB isn't that bad.
Apparently, Intel says that the 180MB audio driver is required for system stability, so they classified it as System driver.

- Avoid drivers that reference wdmaudio.inf or contain Smart Sound This is necessary since some larger audio drivers that want to be included can be too big for PE. And we don't need audio in the PE stage anyways.

JonasKloseBW · 2024-12-29T05:17:25Z

Avoiding drivers that reference some keywords brings the size of the PE folder for the Surface Laptop Go 3 down to 19.6MB.
I think that's a good size.

I'll test more models later to find out if there are other unexpected sizes.

- Expand the driver exclusion list to: wdmaudio.inf, Sound, Machine Learning, Camera, Firmware - Display the correct manufacturer by using the Provider value (if set correctly)

JonasKloseBW · 2024-12-30T01:37:44Z

The list of tested devices for PE driver copy so far are:

HP Z4 G4 Workstation
Microsoft Surface Book 3
Microsoft Surface Laptop Go 3
Microsoft Surface Pro 6
Microsoft Surface Pro 8
Microsoft Surface Pro 11th
Microsoft Surface Studio 2+
Lenovo ThinkBook 16p G2 ACH

It looks like the current code works exactly as expected so far.
I think now would be the time to test this on real hardware.

- Include HID device drivers (745a17a0-74d3-11d0-b6fe-00a0c90f57da) - These drivers should stay pretty small since they're class drivers but they're necessary for touch support

Copilot

Pull Request Overview

Adds initial support for using existing drivers as PE drivers by filtering and copying only the necessary driver files for Windows PE stage. This implementation parses driver .ini files to identify system devices, SCSI/RAID controllers, keyboards, and mice drivers, then copies only the required files to minimize PE image size while maintaining device compatibility.

Added infrastructure for reading INI files using Windows kernel32.dll APIs
Implemented driver filtering logic based on device class GUIDs and exclusion patterns
Added new parameter UseDriversAsPEDrivers to enable selective driver copying for PE

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-09-10T18:46:21Z

FFUDevelopment/BuildFFUVM.ps1

+
+    foreach ($keyValue in $keyValues) {
+        if (![string]::IsNullOrEmpty($keyValue)) {
+            $parts = $keyValue -split "="


This code assumes that each key-value pair has exactly one '=' character and will fail if the value contains '=' characters. Use -split with a limit parameter to handle values containing '=' properly: $parts = $keyValue -split '=', 2

Suggested change

$parts = $keyValue -split "="

$parts = $keyValue -split "=", 2

Copilot · 2025-09-10T18:46:21Z

FFUDevelopment/BuildFFUVM.ps1

+    #Find more information about device classes here:
+    #https://learn.microsoft.com/en-us/windows-hardware/drivers/install/system-defined-device-setup-classes-available-to-vendors
+    #For now, included are system devices, scsi and raid controllers, keyboards, mice and HID devices for touch support
+    $filterGUIDs = @("{4D36E97D-E325-11CE-BFC1-08002BE10318}", "{4D36E97B-E325-11CE-BFC1-08002BE10318}", "{4d36e96b-e325-11ce-bfc1-08002be10318}", "{d36e96f-e325-11ce-bfc1-08002be10318}", "{745a17a0-74d3-11d0-b6fe-00a0c90f57da}")


[nitpick] The GUID strings have inconsistent casing (some lowercase 'd' and 'e'). For consistency and to avoid potential matching issues, standardize all GUIDs to uppercase format.

Suggested change

$filterGUIDs = @("{4D36E97D-E325-11CE-BFC1-08002BE10318}", "{4D36E97B-E325-11CE-BFC1-08002BE10318}", "{4d36e96b-e325-11ce-bfc1-08002be10318}", "{d36e96f-e325-11ce-bfc1-08002be10318}", "{745a17a0-74d3-11d0-b6fe-00a0c90f57da}")

$filterGUIDs = @("{4D36E97D-E325-11CE-BFC1-08002BE10318}", "{4D36E97B-E325-11CE-BFC1-08002BE10318}", "{4D36E96B-E325-11CE-BFC1-08002BE10318}", "{0D36E96F-E325-11CE-BFC1-08002BE10318}", "{745A17A0-74D3-11D0-B6FE-00A0C90F57DA}")

Copilot · 2025-09-10T18:46:21Z

FFUDevelopment/BuildFFUVM.ps1

+                    if (!$sourceDiskFiles[$sourceDiskFile].Contains(",")) {
+                        Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force
+                    } else {
+                        $subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]
+                        [void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)
+                        Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force
+                    }
+                }
+
+                #Arch specific files override the files specified in the universal section
+                $sourceDiskFiles = Get-PrivateProfileSection -FileName $infFullName -SectionName "SourceDisksFiles.$WindowsArch"
+                foreach ($sourceDiskFile in $sourceDiskFiles.Keys) {
+                    if (!$sourceDiskFiles[$sourceDiskFile].Contains(",")) {
+                        Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force
+                    } else {
+                        $subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]
+                        [void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)
+                        Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force


This condition checks if the value doesn't contain a comma, but the logic suggests it should check if it does contain a comma to determine if there's a subdirectory specification. The condition should be if ($sourceDiskFiles[$sourceDiskFile].Contains(',')) for the simple case without subdirectory.

Suggested change

if (!$sourceDiskFiles[$sourceDiskFile].Contains(",")) {

Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force

} else {

$subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]

[void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)

Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force

}

}

#Arch specific files override the files specified in the universal section

$sourceDiskFiles = Get-PrivateProfileSection -FileName $infFullName -SectionName "SourceDisksFiles.$WindowsArch"

foreach ($sourceDiskFile in $sourceDiskFiles.Keys) {

if (!$sourceDiskFiles[$sourceDiskFile].Contains(",")) {

Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force

} else {

$subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]

[void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)

Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force

if ($sourceDiskFiles[$sourceDiskFile].Contains(",")) {

$subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]

[void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)

Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force

} else {

Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force

}

}

#Arch specific files override the files specified in the universal section

$sourceDiskFiles = Get-PrivateProfileSection -FileName $infFullName -SectionName "SourceDisksFiles.$WindowsArch"

foreach ($sourceDiskFile in $sourceDiskFiles.Keys) {

if ($sourceDiskFiles[$sourceDiskFile].Contains(",")) {

$subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]

[void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)

Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force

} else {

Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force

Copilot · 2025-09-10T18:46:22Z

FFUDevelopment/BuildFFUVM.ps1

+                    if (!$sourceDiskFiles[$sourceDiskFile].Contains(",")) {
+                        Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force
+                    } else {
+                        $subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]
+                        [void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)
+                        Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force


Same issue as above - this condition is inverted. The logic should check for the presence of a comma to determine if there's a subdirectory specification.

Suggested change

if (!$sourceDiskFiles[$sourceDiskFile].Contains(",")) {

Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force

} else {

$subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]

[void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)

Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force

if ($sourceDiskFiles[$sourceDiskFile].Contains(",")) {

$subdir = ($sourceDiskFiles[$sourceDiskFile] -split ",")[1]

[void](New-Item -Path "$targetPath\$subdir" -ItemType Directory -Force)

Copy-Item -Path "$infPath\$subdir\$sourceDiskFile" -Destination "$targetPath\$subdir" -Force

} else {

Copy-Item -Path "$infPath\$sourceDiskFile" -Destination $targetPath -Force

JonasKloseBW added 4 commits December 28, 2024 06:20

Add initial support for reusing drivers as PE drivers

ebf06f3

- Add functions to read data from driver .ini files - Copy driver files required into the PE driver folder for system devices, scsi and raid controllers, keyboards and mice

Fix Add-Type DLLImport

43916f7

Fix an error for the DLLImport

Fix issues

c80f542

- Filter correctly for inf-driver files - Ensure target path is created

Silence directory creation

2b4b907

- Cast PE driver directory creation to void to avoid unnecessary console output

Fixes for Surface Laptop Go 3

ab2a3fa

- Increase the buffer reader size by 8x read longer file name lists - Fix cases in which the driver files are in subdirectories

Reduce driver size for Surface Laptop Go3

4716bac

- Avoid drivers that reference wdmaudio.inf or contain Smart Sound This is necessary since some larger audio drivers that want to be included can be too big for PE. And we don't need audio in the PE stage anyways.

Fix: Don't copy unnecessary drivers

bd10f3e

- Expand the driver exclusion list to: wdmaudio.inf, Sound, Machine Learning, Camera, Firmware - Display the correct manufacturer by using the Provider value (if set correctly)

JonasKloseBW added 2 commits December 31, 2024 01:09

Include HID devices

1309020

- Include HID device drivers (745a17a0-74d3-11d0-b6fe-00a0c90f57da) - These drivers should stay pretty small since they're class drivers but they're necessary for touch support

Merge branch 'rbalsleyMSFT:main' into 2412.1-Reuse-drivers

e90a933

rbalsleyMSFT changed the base branch from dev to UI_2508 September 10, 2025 18:43

rbalsleyMSFT requested a review from Copilot September 10, 2025 18:45

Copilot AI reviewed Sep 10, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add initial support for reusing drivers as PE drivers #115

Add initial support for reusing drivers as PE drivers #115

Uh oh!

JonasKloseBW commented Dec 28, 2024

Uh oh!

JonasKloseBW commented Dec 29, 2024

Uh oh!

JonasKloseBW commented Dec 29, 2024 •

edited

Loading

Uh oh!

JonasKloseBW commented Dec 29, 2024

Uh oh!

JonasKloseBW commented Dec 29, 2024

Uh oh!

JonasKloseBW commented Dec 30, 2024

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Sep 10, 2025

Uh oh!

Copilot AI Sep 10, 2025

Uh oh!

Copilot AI Sep 10, 2025

Uh oh!

Copilot AI Sep 10, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

	$parts = $keyValue -split "="
	$parts = $keyValue -split "=", 2

	$filterGUIDs = @("{4D36E97D-E325-11CE-BFC1-08002BE10318}", "{4D36E97B-E325-11CE-BFC1-08002BE10318}", "{4d36e96b-e325-11ce-bfc1-08002be10318}", "{d36e96f-e325-11ce-bfc1-08002be10318}", "{745a17a0-74d3-11d0-b6fe-00a0c90f57da}")
	$filterGUIDs = @("{4D36E97D-E325-11CE-BFC1-08002BE10318}", "{4D36E97B-E325-11CE-BFC1-08002BE10318}", "{4D36E96B-E325-11CE-BFC1-08002BE10318}", "{0D36E96F-E325-11CE-BFC1-08002BE10318}", "{745A17A0-74D3-11D0-B6FE-00A0C90F57DA}")

Add initial support for reusing drivers as PE drivers #115

Are you sure you want to change the base?

Add initial support for reusing drivers as PE drivers #115

Uh oh!

Conversation

JonasKloseBW commented Dec 28, 2024

Uh oh!

JonasKloseBW commented Dec 29, 2024

Uh oh!

JonasKloseBW commented Dec 29, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

JonasKloseBW commented Dec 29, 2024

Uh oh!

JonasKloseBW commented Dec 29, 2024

Uh oh!

JonasKloseBW commented Dec 30, 2024

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Sep 10, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 10, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 10, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 10, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

JonasKloseBW commented Dec 29, 2024 •

edited

Loading