[PATCH v5 0/6] Clean up common/arch split for debugger.h

Andrew Cooper posted 6 patches 2 years ago
Test gitlab-ci passed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20220420141307.24153-1-andrew.cooper3@citrix.com
xen/arch/arm/include/asm/debugger.h | 15 ------
xen/arch/x86/Makefile               |  2 +-
xen/arch/x86/domain.c               | 14 ------
xen/arch/x86/domctl.c               | 14 +-----
xen/arch/x86/gdbstub.c              |  5 +-
xen/arch/x86/{debug.c => gdbsx.c}   | 37 +++++++++------
xen/arch/x86/hvm/svm/svm.c          |  2 +-
xen/arch/x86/hvm/vmx/realmode.c     |  3 +-
xen/arch/x86/hvm/vmx/vmx.c          |  2 +-
xen/arch/x86/include/asm/debugger.h | 93 +++++--------------------------------
xen/arch/x86/include/asm/gdbsx.h    | 19 ++++++++
xen/arch/x86/nmi.c                  |  1 -
xen/arch/x86/setup.c                |  1 -
xen/arch/x86/traps.c                | 37 +++++++--------
xen/arch/x86/x86_64/gdbstub.c       |  2 +-
xen/common/domain.c                 |  1 -
xen/common/gdbstub.c                |  3 +-
xen/common/keyhandler.c             |  2 +-
xen/common/shutdown.c               |  2 +-
xen/drivers/char/console.c          |  2 +-
xen/include/xen/debugger.h          | 44 ++++++++++++++++++
xen/include/xen/gdbstub.h           |  2 +
22 files changed, 133 insertions(+), 170 deletions(-)
delete mode 100644 xen/arch/arm/include/asm/debugger.h
rename xen/arch/x86/{debug.c => gdbsx.c} (87%)
create mode 100644 xen/arch/x86/include/asm/gdbsx.h
create mode 100644 xen/include/xen/debugger.h
[PATCH v5 0/6] Clean up common/arch split for debugger.h
Posted by Andrew Cooper 2 years ago
This work is primarily to prevent new architectures from needing to implement
a stub debugger.h for something that is in practice only implemented on x86,
and probably bitrotten into oblivion.  It also resolves a lot of technical
debt on the x86 side.

Andrew Cooper (3):
  x86/gdbsx: Move domain_pause_for_debugger() into gdbsx
  x86/gdbstub: Clean up includes
  x86/debugger: Misc cleanup prior to splitting

Bobby Eshleman (3):
  x86/debugger: Remove debugger_trap_entry()
  x86/gdbsx: Rename debug.c to gdbsx.c
  xen: Split x86/debugger.h into common and arch specific parts

 xen/arch/arm/include/asm/debugger.h | 15 ------
 xen/arch/x86/Makefile               |  2 +-
 xen/arch/x86/domain.c               | 14 ------
 xen/arch/x86/domctl.c               | 14 +-----
 xen/arch/x86/gdbstub.c              |  5 +-
 xen/arch/x86/{debug.c => gdbsx.c}   | 37 +++++++++------
 xen/arch/x86/hvm/svm/svm.c          |  2 +-
 xen/arch/x86/hvm/vmx/realmode.c     |  3 +-
 xen/arch/x86/hvm/vmx/vmx.c          |  2 +-
 xen/arch/x86/include/asm/debugger.h | 93 +++++--------------------------------
 xen/arch/x86/include/asm/gdbsx.h    | 19 ++++++++
 xen/arch/x86/nmi.c                  |  1 -
 xen/arch/x86/setup.c                |  1 -
 xen/arch/x86/traps.c                | 37 +++++++--------
 xen/arch/x86/x86_64/gdbstub.c       |  2 +-
 xen/common/domain.c                 |  1 -
 xen/common/gdbstub.c                |  3 +-
 xen/common/keyhandler.c             |  2 +-
 xen/common/shutdown.c               |  2 +-
 xen/drivers/char/console.c          |  2 +-
 xen/include/xen/debugger.h          | 44 ++++++++++++++++++
 xen/include/xen/gdbstub.h           |  2 +
 22 files changed, 133 insertions(+), 170 deletions(-)
 delete mode 100644 xen/arch/arm/include/asm/debugger.h
 rename xen/arch/x86/{debug.c => gdbsx.c} (87%)
 create mode 100644 xen/arch/x86/include/asm/gdbsx.h
 create mode 100644 xen/include/xen/debugger.h

-- 
2.11.0