aboutsummaryrefslogtreecommitdiffstats
path: root/tools/debugger
Commit message (Collapse)AuthorAgeFilesLines
* gdbsx: clear sockaddr before using itMatthew Daley2013-09-211-0/+1
| | | | | | | | ...so that sin_zero is actually zero. Coverity-ID: 1056070 Signed-off-by: Matthew Daley <mattjd@gmail.com> Acked-by: Ian Campbell <ian.campbell@citrix.com>
* kdd: fix free of array-typed valueMatthew Daley2013-09-131-1/+0
| | | | | | | | | | g->id is an array and is allocated as part of g itself; it's not a separate allocation. Coverity-ID: 1054980 Signed-off-by: Matthew Daley <mattjd@gmail.com> Acked-by: Tim Deegan <tim@xen.org> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
* tools/debugger/kdd: Remove dependencies files during make cleanDaniel Kiper2013-05-081-1/+1
| | | | | | | Remove dependencies files during make clean. Signed-off-by: Daniel Kiper <daniel.kiper@oracle.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
* tools: Fix memset(&p,0,sizeof(p)) idiom in several places.Michael Young2013-02-131-1/+1
| | | | | | | | | gcc 4.8 identifies several places where code of the form memset(x, 0, sizeof(x)); is used incorrectly, meaning that less memory is set to zero than required. Signed-off-by: Michael Young <m.a.young@durham.ac.uk> Committed-by: Keir Fraser <keir@xen.org>
* tools/gdbsx: fix build failure with glibc-2.17Olaf Hering2012-12-061-0/+1
| | | | | Signed-off-by: Olaf Hering <olaf@aepfle.de> Committed-by: Keir Fraser <keir@xen.org>
* tools: drop ia64 supportIan Campbell2012-09-1219-19580/+0
| | | | | | | | | | | | | | | | | Removed support from libxc and mini-os. This also took me under xen/include/public via various symlinks. Dropped tools/debugger/xenitp entirely, it was described upon commit as: "Xenitp is a low-level debugger for ia64" and doesn't appear to be linked into the build anywhere. 99 files changed, 14 insertions(+), 32361 deletions(-) Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> Committed-by: Ian Campbell <ian.campbell@citrix.com>
* build: add autoconf to replace custom checks in tools/checkIan Jackson2012-02-221-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added autotools magic to replace custom check scripts. The previous checks have been ported to autoconf, and some additional ones have been added (plus the suggestions from running autoscan). Two files are created as a result from executing configure script, config/Tools.mk and config.h. conf/Tools.mk is included by tools/Rules.mk, and contains most of the options previously defined in .config, that can now be set passing parameters or defining environment variables when executing configure script. config.h is only used by libxl/xl to detect yajl_version.h. [ tools/config.sub and config.guess copied from autotools-dev 20100122.1 from Debian squeeze i386, which is GPLv2. tools/configure generated using the included ./autogen.sh which ran autoconf 2.67-2 from Debian squeeze i386. autoconf is GPLv3+ but has a special exception for the autoconf output; this exception applies to us and exempts us from complying with GPLv3+ for configure, which is good as Xen is GPL2 only. - Ian Jackson ] Signed-off-by: Roger Pau Monne <roger.pau@entel.upc.edu> Tested-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
* tools: remove pattern matched linking rulesIan Campbell2011-03-311-24/+3
| | | | | | | | | Most subdirs only build a single tool to start with and those which build multiple tools often have different linkage requirements. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
* tools: Remove $(CFLAGS) from links lines.Ian Campbell2011-03-311-1/+1
| | | | | | | | The relevant variable in these circumstances is called $(LDFLAGS). Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
* tools: consistently use $(CFLAGS_xeninclude) instead of open coding.Ian Campbell2011-03-211-1/+1
| | | | | | | | Renamed from the slightly ambiguous CFLAGS_include. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
* tools: remove unnecessary uses of -I.Ian Campbell2011-03-211-1/+1
| | | | | | | | | | It's not needed unless you are including headers in the current directory _and_ you can't use "" instead of <> because the headers are to be installed. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
* tools: Drop use of $(INCLUDES)Ian Campbell2011-03-211-1/+1
| | | | | | | | | | Several places include it in CFLAGS even though it is never defined. A few others use it as nothing more than a gathering point for CFLAGS. Get rid of it. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
* build: Make XEN_ROOT an absolute path.Keir Fraser2011-03-175-5/+5
| | | | | | | | Otherwise make can search the path relative to certain standard paths such as /usr/include (e.g., the line '-include $(XEN_ROOT)/.config' in Config.mk suffers from this). Signed-off-by: Keir Fraser <keir@xen.org>
* tools/gdbsx: Update gdbsx READMEMukesh Rathor2011-01-041-15/+2
| | | | | Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
* tools: gdbsx: Check return of write()Ian Jackson2010-11-031-2/+8
| | | | | | | | Not checking leads to warn_unused_result checks triggering in some libraries and compilers. Combined with -Werror this breaks the build. Signed-off-by: Gianni Tedesco <gianni.tedesco@citrix.com> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
* gdbsx: Makefiles: correctly generate dependenciesGianni Tedesco2010-10-283-7/+2
| | | | | | | | | | | gdbsx hacks CFLAGS to remove optimisations for easier debugging, however this breaks dep generation due to an early evaluation of CFLAGS and leaves stale files called "..d" after a make clean. We could possibly fix this by separating dep generation from the rest of CFLAGS but it seems a little invasive. The gdbsx hackers can do their own thing during development and testing anyway... Signed-off-by: Gianni Tedesco <gianni.tedesco@citrix.com>
* "kdd" Windows debugger stub.Tim Deegan2010-10-264-0/+2212
| | | | | | | | | | | | kdd runs in dom0, attaches to a domain and speaks the Windows kd serial protocol over a TCP connection (which should go to kd or windbg, e.g. by having another VM with its virtual COM1 set up as a TCP listener). It doesn't do breakpoints &c yet, and windbg can get quite confused since the kernel debugger's not actually running, but it's good enough to extract backtraces from wedged VMs. Signed-off-by: Tim Deegan <Tim.Deegan@citrix.com>
* tools/debugger/gdbsx: use MTF flag for HVM guests for single stepMukesh Rathor2010-09-071-0/+14
| | | | | | | | | | The attached patch first tries the MTF flag for HVM guest single step, reverting to manually setting the TF flag if MTF fails. Tested on 4.0.1. Compile tested on unstable. Signed-off-by: mukesh.rathor@oracle.com Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
* split LDLIBS from LDFLAGS to fix link errors in recent toolchainsStefano Stabellini2010-08-111-2/+2
| | | | | | | | | | | | | | | | Linker command lines are order-sensitive. Move linker options -Lfoo -lfoo from LDFLAGS to LDLIBS and place this new variable after the objects to link. This resolves build errors in xenpagin and blktap with recent toolchains. rename SHLIB_CFLAGS to SHLIB_LDFLAGS rename LDFLAGS_* to LDLIBS_* move LDFLAGS usage after CFLAGS in CC calls remove stale comments in xenpaging Makefile Signed-off-by: Olaf Hering <olaf@aepfle.de> Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
* gdbsx: Install into correct directory $(SBINDIR)Keir Fraser2010-08-021-4/+2
| | | | Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>
* gdbsx: update README and remove space in q packetIan Jackson2010-07-152-6/+6
| | | | | | | | Newer version of gdb, version 7*, seems to have bug where it is not parsing thread list from gdbsx properly. Getting rid of the space in thread list works around it. It's ok with older gdb also. Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
* tools/debugger/gdb: Remove gdbIan Jackson2010-07-029-6678/+0
| | | | | This code is not maintained, does not work properly, and no-one is using it. Delete it, following discussion on xen-devel.
* libxc: eliminate static variables, use xentoollog; API changeKeir Fraser2010-05-282-6/+6
| | | | | | | | | | | | | | | | | | | | This patch eliminate the global variables in libxenctrl (used for logging and error reporting). Instead the information which was in the global variables is now in a new xc_interface* opaque structure, which xc_interface open returns instead of the raw file descriptor; furthermore, logging is done via xentoollog. There are three new parameters to xc_interface_open to control the logging, but existing callers can just pass "0" for all three to get the old behaviour. All libxc callers have been adjusted accordingly. Also update QEMU_TAG for corresponding qemu change. Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
* gdbsx: malloc extra bye for null charKeir Fraser2009-10-192-2/+3
| | | | Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
* gdbsx: a gdbserver stub for xen.Keir Fraser2009-10-1513-0/+2629
| | | | | | | | It should be run on dom0 on gdbsx enabled hypervisor. For details, please see tools/debugger/gdbsx/README Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com> Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
* Remove redundant semicolonsKeir Fraser2009-07-011-1/+1
| | | | Signed-off-by: Rikiya Ayukawa <ayukawa.rikiya@jp.fujitsu.com>
* x86_64: allow more vCPU-s per guestKeir Fraser2009-06-181-3/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | Since the shared info layout is fixed, guests are required to use VCPUOP_register_vcpu_info prior to booting any vCPU beyond the traditional limit of 32. MAX_VIRT_CPUS, being an implemetation detail of the hypervisor, is no longer being exposed in the public headers. The tools changes are clearly incomplete (and done only so things would build again), and the current state of the tools (using scalar variables all over the place to represent vCPU bitmaps) very likely doesn't permit booting DomU-s with more than the traditional number of vCPU-s. Testing of the extended functionality was done with Dom0 (96 vCPU-s, as well as 128 vCPU-s out of which the kernel elected - by way of a simple kernel side patch - to use only some, resulting in a sparse bitmap). ia64 changes only to make things build, and build-tested only (and the tools part only as far as the build would go without encountering unrelated problems in the blktap code). Signed-off-by: Jan Beulich <jbeulich@novell.com>
* gdbserver: Fix build failure.Keir Fraser2008-12-101-1/+1
| | | | | From: Edwin Zhai <edwin.zhai@intel.com> Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
* ia64: fix make install under tools/debugger/xenitpKeir Fraser2008-09-241-3/+9
| | | | | | | | | | | | | | | | | | | This patch fixes the following error with make install under the directory, tools/debugger/xenitp by checking whether the variable is length zero string. > # make install > ../../../tools/cross-install -d -m0755 -p //usr/bin > [ -z " xenitp" ] || ../../../tools/cross-install -d -m0755 -p > //usr/lib/xen/bin > ../../../tools/cross-install -d -m0755 -p //usr/share/man/man1 > ../../../tools/cross-install -d -m0755 -p //usr/share/man/man8 > ../../../tools/cross-install -m0755 -p //usr/bin > install: too few arguments > Try `install --help' for more information. > make: *** [install] Error 1 Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
* [IA64] compilation fix caused by 17880:d3a87899985dKeir Fraser2008-06-201-7/+17
| | | | Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
* [IA64] Add a dump command to xenitpAlex Williamson2008-02-251-0/+31
| | | | Signed-off-by: Tristan Gingold <tgingold@free.fr>
* [IA64] Make xenitp more '|tee' friendlyAlex Williamson2008-02-131-0/+1
| | | | | | Flush stdout after printing the prompt. Signed-off-by: Tristan Gingold <tgingold@free.fr>
* Define CFLAGS and LDFLAGS for libxenctrl.Keir Fraser2008-01-271-3/+3
| | | | Signed-off-by: Bastian Blank <waldi@debian.org>
* Add PRIVATE_BINDIR. Use it.Keir Fraser2008-01-261-2/+2
| | | | Signed-off-by: Bastian Blank <waldi@debian.org>
* Add MANDIR, MAN1DIR and MAN8DIR. Use it.Keir Fraser2008-01-221-6/+4
| | | | Signed-off-by: Bastian Blank <waldi@debian.org>
* Add BINDIR. Use it.Keir Fraser2008-01-221-2/+2
| | | | Signed-off-by: Bastian Blank <waldi@debian.org>
* Apply PREFIX directly to LIBDIR.Keir Fraser2008-01-221-3/+2
| | | | Signed-off-by: Bastian Blank <waldi@debian.org>
* [IA64] Fix XenITP buildAlex Williamson2007-12-132-1/+1
| | | | Signed-off-by: Alex Williamson <alex.williamson@hp.com>
* [IA64] xenitp: typo fixAlex Williamson2007-10-301-1/+1
| | | | Signed-off-by: Tristan Gingold <tgingold@free.fr>
* [IA64] New features for xenitpAlex Williamson2007-10-211-87/+133
| | | | | | | | | | | Add auto-repeat feature (Just press enter to re-execute the last go/sstep/cb/disass command). Do not flush stdout in the signal handler. Single step over a breakpoint. Can quit with domain paused (quit paused) 'disp db' now displays watchpoint. Signed-off-by: Tristan Gingold <tgingold@free.fr>
* ia64: Fix help message of xenitp.Keir Fraser2007-10-101-1/+1
| | | | Signed-off-by: Akio Takebe <takebe_akio@jp.fujitsu.com>
* [IA64] xenitp improvementsAlex Williamson2007-10-021-93/+125
| | | | | | | | | | | Remove all the casts by using char * instead of unsigned char *. The go command now accept a number. The disass command can now accept a range. Number of TRs is not hard-coded. '-' (minus) can now be used in expressions. '$iip' and '$b0' can be used in expressions. Signed-off-by: Tristan Gingold <tgingold@free.fr>
* [IA64] xenitp: Display symbolic names of cr.Alex Williamson2007-10-021-0/+36
| | | | Signed-off-by: Tristan Gingold <tgingold@free.fr>
* [IA64] xenitp: correctly size vcpu_ctx arrayAlex Williamson2007-10-011-1/+1
| | | | | | vcpu_ctx[] array size should be MAX_VIRT_CPUS instead of 1. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
* [IA64] clean up xenitp build warningsAlex Williamson2007-09-261-20/+20
| | | | Signed-off-by: Alex Williamson <alex.williamson@hp.com>
* [IA64] Add xenitp toolAlex Williamson2007-09-2419-0/+19430
| | | | | | Xenitp is a low-level debugger for ia64 Signed-off-by: Tristan Gingold <tgingold@free.fr>
* Put gdb version in a variable in gdbbuild script.kfraser@localhost.localdomain2007-01-221-8/+10
| | | | Signed-off-by: Xin Li <xin.b.li@intel.com>
* [SOLARIS] Don't build ptrace code on Solaris.kfraser@localhost.localdomain2006-10-171-2/+0
| | | | Signed-off-by: Keir Fraser <keir@xensource.com>
* Remove dead pdb code from tools directory.kfraser@localhost.localdomain2006-09-2936-6315/+0
| | | | Signed-off-by: Keir Fraser <keir@xensource.com>
* [GDBSERVER] make gdbbuild exit on errorkaf24@firebug.cl.cam.ac.uk2006-09-131-0/+2
| | | | | | | | | | | If an error occurs, such as wget failing because the site (or proxy) is unavailable, the script will now terminate, rather than trying to soldier on without any real hope of success. mkbuildtree should really get the same treatment, but that can wait for another day. Signed-Off-By: Simon Horman <horms@verge.net.au>