[v3] xen: make more debugger support code conditional

[Xen-devel] [PATCH v3 0/2] xen: make more debugger support code conditional

Juergen Gross posted 2 patches 4 years, 4 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/xen tags/patchew/20191219074209.17277-1-jgross@suse.com

xen/Kconfig.debug               |  8 +++++
xen/arch/x86/Kconfig            |  1 -
xen/arch/x86/Makefile           |  2 +-
xen/arch/x86/debug.c            | 78 +++++------------------------------------
xen/arch/x86/domain.c           | 13 +++++++
xen/arch/x86/domctl.c           |  4 +++
xen/arch/x86/hvm/vmx/realmode.c |  1 +
xen/common/Kconfig              |  3 --
xen/common/domain.c             | 14 --------
xen/include/asm-x86/debugger.h  | 34 +++++++++++-------
xen/include/xen/sched.h         |  1 -
11 files changed, 58 insertions(+), 101 deletions(-)

Expand all Fold all

[Xen-devel] [PATCH v3 0/2] xen: make more debugger support code conditional

Posted by Juergen Gross 4 years, 4 months ago

Support for debugging the hypervisor of guests via gdb/gdbsx should be
configurable.

Changes in V3:
- remove possibility to access hypervisor memory via gdbsx domctl
- default gdbsx support to on
- some code moving

Changes in V2:
- split support for gdbstub and gdbsx (Andrew Cooper)

Juergen Gross (2):
  xen: put more code under CONFIG_CRASH_DEBUG
  xen: make gdbsx support configurable

 xen/Kconfig.debug               |  8 +++++
 xen/arch/x86/Kconfig            |  1 -
 xen/arch/x86/Makefile           |  2 +-
 xen/arch/x86/debug.c            | 78 +++++------------------------------------
 xen/arch/x86/domain.c           | 13 +++++++
 xen/arch/x86/domctl.c           |  4 +++
 xen/arch/x86/hvm/vmx/realmode.c |  1 +
 xen/common/Kconfig              |  3 --
 xen/common/domain.c             | 14 --------
 xen/include/asm-x86/debugger.h  | 34 +++++++++++-------
 xen/include/xen/sched.h         |  1 -
 11 files changed, 58 insertions(+), 101 deletions(-)

-- 
2.16.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH v3 0/2] xen: make more debugger support code conditional

Posted by Andrew Cooper 4 years, 4 months ago

On 19/12/2019 07:42, Juergen Gross wrote:
> Support for debugging the hypervisor of guests via gdb/gdbsx should be
> configurable.
>
> Changes in V3:
> - remove possibility to access hypervisor memory via gdbsx domctl
> - default gdbsx support to on
> - some code moving
>
> Changes in V2:
> - split support for gdbstub and gdbsx (Andrew Cooper)
>
> Juergen Gross (2):
>   xen: put more code under CONFIG_CRASH_DEBUG
>   xen: make gdbsx support configurable

LGTM.  Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel