[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/39] Windbg supporting
From: |
Changbin Du |
Subject: |
Re: [Qemu-devel] [PATCH 00/39] Windbg supporting |
Date: |
Sat, 1 Dec 2018 21:46:27 +0800 |
User-agent: |
NeoMutt/20180716 |
I applied on top of latest master (tag: v3.1.0-rc3) and encounter below building
failure. CONTEXT_CONTROL should be CPU_CONTEXT_CONTROL?
$./configure --enable-kvm --enable-vhost-net --enable-libssh2
--target-list=x86_64-softmmu --enable-sdl --enable-curses --enable-vnc
--enable-fdt --enable-debug --enable-debug-info --enable-trace-backends=log
$make -j
[...]
CC x86_64-softmmu/trace/generated-helpers.o
/home/changbin/Desktop/work/qemu/windbgstub-utils.c: In function
‘windbg_search_vmaddr’:
/home/changbin/Desktop/work/qemu/windbgstub-utils.c:179:16: error: ‘ret’ may be
used uninitialized in this function [-Werror=maybe-uninitialized]
return ret;
^~~
cc1: all warnings being treated as errors
/home/changbin/Desktop/work/qemu/rules.mak:69: recipe for target
'windbgstub-utils.o' failed
make[1]: *** [windbgstub-utils.o] Error 1
make[1]: *** Waiting for unfinished jobs....
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c: In function
‘windbg_read_context’:
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:569:29: error:
‘CONTEXT_CONTROL’ undeclared (first use in this function); did you mean
‘CPU_CONTEXT_CONTROL’?
if (ctx_flags & CONTEXT_CONTROL) {
\
^
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:391:9: note: in
definition of macro ‘CASE_FIELD’
block;
\
^~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:568:9: note: in
expansion of macro ‘CASE_FIELD_X32_64’
CASE_FIELD_X32_64(CPU_CONTEXT, Esp, Rsp, f_size, {
\
^~~~~~~~~~~~~~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:890:1: note: in
expansion of macro ‘GEN_WINDBG_CONTEXT_RW’
GEN_WINDBG_CONTEXT_RW(windbg_read_context, false)
^~~~~~~~~~~~~~~~~~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:569:29: note: each
undeclared identifier is reported only once for each function it appears in
if (ctx_flags & CONTEXT_CONTROL) {
\
^
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:391:9: note: in
definition of macro ‘CASE_FIELD’
block;
\
^~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:568:9: note: in
expansion of macro ‘CASE_FIELD_X32_64’
CASE_FIELD_X32_64(CPU_CONTEXT, Esp, Rsp, f_size, {
\
^~~~~~~~~~~~~~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:890:1: note: in
expansion of macro ‘GEN_WINDBG_CONTEXT_RW’
GEN_WINDBG_CONTEXT_RW(windbg_read_context, false)
^~~~~~~~~~~~~~~~~~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c: In function
‘windbg_write_context’:
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:569:29: error:
‘CONTEXT_CONTROL’ undeclared (first use in this function); did you mean
‘CPU_CONTEXT_CONTROL’?
if (ctx_flags & CONTEXT_CONTROL) {
\
^
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:391:9: note: in
definition of macro ‘CASE_FIELD’
block;
\
^~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:568:9: note: in
expansion of macro ‘CASE_FIELD_X32_64’
CASE_FIELD_X32_64(CPU_CONTEXT, Esp, Rsp, f_size, {
\
^~~~~~~~~~~~~~~~~
/home/changbin/Desktop/work/qemu/target/i386/windbgstub.c:891:1: note: in
expansion of macro ‘GEN_WINDBG_CONTEXT_RW’
GEN_WINDBG_CONTEXT_RW(windbg_write_context, true)
^~~~~~~~~~~~~~~~~~~~~
/home/changbin/Desktop/work/qemu/rules.mak:69: recipe for target
'target/i386/windbgstub.o' failed
make[1]: *** [target/i386/windbgstub.o] Error 1
Makefile:483: recipe for target 'subdir-x86_64-softmmu' failed
make: *** [subdir-x86_64-softmmu] Error 2
On Thu, Nov 29, 2018 at 05:27:30PM +0300, Mikhail Abakumov wrote:
> We made the debugger module WinDbg (like GDB) for QEMU. This is the
> replacement
> of the remote server in Windows kernel. Used for remote Windows kernel
> debugging
> without debugging mode.
>
> WinDbg is a multipurpose debugger for the Microsoft Windows computer operating
> system, distributed by Microsoft. Recent versions of WinDbg have been
> and are being distributed as part of the free Debugging Tools for Windows
> suite.
>
> How to start debugging QEMU using WinDbg:
> Run QEMU with next option:
> -windbg pipe:<name>
> QEMU will start and pause for waiting WinDbg connection.
> Run WinDbg with next options:
> -b -k com:pipe,baud=115200,port=\\.\pipe\<name>,resets=0
> Wait for debugger connect to kernel.
>
> Note: You can add Symbol Search Path in WinDbg
> such as srv*c:\tmp*http://msdl.microsoft.com/download/symbols.
>
> How it works:
> The WinDbg debugger has the possibility of connecting to a remote debug server
> (Kdsrv.exe) in the Windows kernel. Therefore, it is possible to connect
> to the guest system running in the QEMU emulator. Kernel debugging is possible
> only with the enabled debugging mode, may change at the same time.
> Our module of WinDbg debugger for QEMU is an alternative of the remote
> debugging
> service in the kernel. Thus, the debugger connects to the debugging module,
> not to the kernel of the operating system. The module obtains all the
> necessary
> information answering debugger requests from the QEMU emulator. At the same
> time
> for debugging there is no need to enable debugging mode in the kernel.
> This leads to hidden debugging. Our module supports all features of WinDbg
> regarding remote debugging, besides interception of events and exceptions.
> Supports i386 and x86_64 architectures.
>
> Tested-by: Ladi Prosek <address@hidden>
> ---
>
> Mikhail Abakumov (39):
> windbg: add empty windbgstub files
> windbg: add windbg's KD header file
> windbg: add -windbg option
> windbg: add helper features
> windbg: add WindbgState
> windbg: add chardev
> windbg: hook to wrmsr operation
> windbg: implement windbg_on_load
> windbg: implement find_KPCR
> windbg: implement find_kdVersion
> windbg: add windbg_search_vmaddr
> windbg: implement find_kdDebuggerDataBlock
> windbg: parsing data stream
> windbg: send data and control packets
> windbg: handler of parsing context
> windbg: init DBGKD_ANY_WAIT_STATE_CHANGE
> windbg: generate ExceptionStateChange and LoadSymbolsStateChange
> windbg: implement windbg_process_control_packet
> windbg: implement windbg_process_data_packet
> windbg: implement windbg_process_manipulate_packet
> windbg: implement kd_api_read_virtual_memory and
> kd_api_write_virtual_memory
> windbg: some kernel structures
> windbg: add helper functions
> windbg: [de]serialization cpu context
> windbg: [de]serialization cpu spec registers
> windbg: implement kd_api_get_context and kd_api_set_context
> windbg: implement kd_api_get_context_ex and kd_api_set_context_ex
> windbg: implement kd_api_read_control_space and
> kd_api_write_control_space
> windbg: implement kd_api_write_breakpoint and kd_api_restore_breakpoint
> windbg: debug exception subscribing
> windbg: implement kd_api_continue
> windbg: implement kd_api_read_io_space and kd_api_write_io_space
> windbg: implement kd_api_read_physical_memory and
> kd_api_write_physical_memory
> windbg: implement kd_api_get_version
> windbg: implement kd_api_read_msr and kd_api_write_msr
> windbg: implement kd_api_search_memory
> windbg: implement kd_api_fill_memory
> windbg: implement kd_api_query_memory
> windbg: maintainers
>
>
> MAINTAINERS | 12
> Makefile.target | 3
> cpus.c | 19 +
> default-configs/i386-softmmu.mak | 1
> gdbstub.c | 4
> include/exec/windbgkd.h | 928 ++++++++++++++++++++++++++
> include/exec/windbgstub-utils.h | 104 +++
> include/exec/windbgstub.h | 25 +
> include/sysemu/sysemu.h | 2
> qemu-options.hx | 8
> stubs/Makefile.objs | 1
> stubs/windbgstub.c | 22 +
> target/i386/Makefile.objs | 1
> target/i386/cpu.h | 5
> target/i386/misc_helper.c | 38 +
> target/i386/windbgstub.c | 1368
> ++++++++++++++++++++++++++++++++++++++
> vl.c | 8
> windbgstub-utils.c | 508 ++++++++++++++
> windbgstub.c | 545 +++++++++++++++
> 19 files changed, 3592 insertions(+), 10 deletions(-)
> create mode 100644 include/exec/windbgkd.h
> create mode 100644 include/exec/windbgstub-utils.h
> create mode 100644 include/exec/windbgstub.h
> create mode 100644 stubs/windbgstub.c
> create mode 100644 target/i386/windbgstub.c
> create mode 100644 windbgstub-utils.c
> create mode 100644 windbgstub.c
>
> --
> Mikhail Abakumov
>
--
Cheers,
Changbin Du
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 00/39] Windbg supporting,
Changbin Du <=