Skip to content

Commit

Permalink
Merge pull request #4181 from heywji/fix_fwcfg_windbg
Browse files Browse the repository at this point in the history
fwcfg: fix wmic query to ensure windbg installation for fwcfg cases
  • Loading branch information
vivianQizhu authored Dec 6, 2024
2 parents b4d840c + 5bc1af7 commit caf2a37
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 20 deletions.
4 changes: 2 additions & 2 deletions provider/win_dump_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,8 @@ def check_windbg_installed(params, session):
:param params: the dict used for parameters.
:param session: The guest session object.
"""
check_cmd = params["chk_windbg_cmd"] % params["windbg_path"]
status, _ = session.cmd_status_output(check_cmd)
chk_windbg_cmd = params["chk_windbg_cmd"]
status, _ = session.cmd_status_output(chk_windbg_cmd)
return False if status else True


Expand Down
6 changes: 1 addition & 5 deletions qemu/tests/cfg/fwcfg.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -19,21 +19,17 @@
image_name_stg = "images/storage"
remove_image_stg = yes
no ppc64 ppc64le aarch64
chk_windbg_cmd = 'powershell -Command "Get-WmiObject Win32_Product | Where-Object Name -like '*SDK Debuggers*' | Select Name, Version"'
i386:
windbg_path = "x86\windbg.exe"
devcon_dirname += "x86"
chk_windbg_cmd = 'dir "C:\Program Files\Windows Kits\10\Debuggers\%s"'
x86_64:
windbg_path = "x64\windbg.exe"
devcon_dirname += "amd64"
chk_windbg_cmd = 'dir "C:\Program Files (x86)\Windows Kits\10\Debuggers\%s"'
devcon_path = "WIN_UTILS:\devcon\${devcon_dirname}\devcon.exe"
feature = "OptionId.WindowsDesktopDebuggers"
sdk_setup = winsdksetup.exe
Win8..1, Win2012..r1, Win2012..r2:
sdk_setup = sdksetup_81.exe
unzip_cmd = powershell.exe -nologo -noprofile -command "& { Add-Type -A 'System.IO.Compression.FileSystem'; [IO.Compression.ZipFile]::ExtractToDirectory('%s:\Memory.dmp.zip', '%s:'); }"
chk_windbg_cmd = 'dir "C:\Program Files (x86)\Windows Kits\8.1\Debuggers\%s"'
Win2016:
sdk_setup = sdksetup_1607.exe
Win2019:
Expand Down
6 changes: 1 addition & 5 deletions qemu/tests/cfg/fwcfg_enable.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -23,21 +23,17 @@
remove_image_stg = yes
memory_leak_check = yes
no ppc64 ppc64le aarch64
chk_windbg_cmd = 'powershell -Command "Get-WmiObject Win32_Product | Where-Object Name -like '*SDK Debuggers*' | Select Name, Version"'
i386:
windbg_path = "x86\windbg.exe"
devcon_dirname += "x86"
chk_windbg_cmd = 'dir "C:\Program Files\Windows Kits\10\Debuggers\%s"'
x86_64:
windbg_path = "x64\windbg.exe"
devcon_dirname += "amd64"
chk_windbg_cmd = 'dir "C:\Program Files (x86)\Windows Kits\10\Debuggers\%s"'
devcon_path = "WIN_UTILS:\devcon\${devcon_dirname}\devcon.exe"
feature = "OptionId.WindowsDesktopDebuggers"
sdk_setup = winsdksetup.exe
Win8..1, Win2012..r1, Win2012..r2:
sdk_setup = sdksetup_81.exe
unzip_cmd = powershell.exe -nologo -noprofile -command "& { Add-Type -A 'System.IO.Compression.FileSystem'; [IO.Compression.ZipFile]::ExtractToDirectory('%s:\Memory.dmp.zip', '%s:'); }"
chk_windbg_cmd = 'dir "C:\Program Files (x86)\Windows Kits\8.1\Debuggers\%s"'
Win2016:
sdk_setup = sdksetup_1607.exe
Win2019:
Expand Down
10 changes: 2 additions & 8 deletions qemu/tests/cfg/win_virtio_driver_update_test.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -278,6 +278,7 @@
fio_options += '--bs=%s --size=1G --iodepth=16 --numjobs=8 --runtime=1800 --thread'
io_timeout = 2000
- with_fwcfg:
no ppc64 ppc64le aarch64
driver_name = "fwcfg"
required_qemu = [6.0.0, )
vmcoreinfo = yes
Expand All @@ -298,19 +299,12 @@
images += " stg"
image_name_stg = "images/storage"
remove_image_stg = yes
no ppc64 ppc64le aarch64
i386:
windbg_path = "x86\windbg.exe"
chk_windbg_cmd = 'dir "C:\Program Files\Windows Kits\10\Debuggers\%s"'
x86_64:
windbg_path = "x64\windbg.exe"
chk_windbg_cmd = 'dir "C:\Program Files (x86)\Windows Kits\10\Debuggers\%s"'
chk_windbg_cmd = 'powershell -Command "Get-WmiObject Win32_Product | Where-Object Name -like '*SDK Debuggers*' | Select Name, Version"'
feature = "OptionId.WindowsDesktopDebuggers"
sdk_setup = winsdksetup.exe
Win8..1, Win2012..r1, Win2012..r2:
sdk_setup = sdksetup_81.exe
unzip_cmd = powershell.exe -nologo -noprofile -command "& { Add-Type -A 'System.IO.Compression.FileSystem'; [IO.Compression.ZipFile]::ExtractToDirectory('%s:\Memory.dmp.zip', '%s:'); }"
chk_windbg_cmd = 'dir "C:\Program Files (x86)\Windows Kits\8.1\Debuggers\%s"'
Win2016:
sdk_setup = sdksetup_1607.exe
Win2019:
Expand Down

0 comments on commit caf2a37

Please sign in to comment.