From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50701) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQJ3s-0002Sx-TE for qemu-devel@nongnu.org; Fri, 14 Aug 2015 13:46:37 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZQJ3r-0003gz-VN for qemu-devel@nongnu.org; Fri, 14 Aug 2015 13:46:36 -0400 Received: from mnementh.archaic.org.uk ([2001:8b0:1d0::1]:34955) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQJ3r-0003gY-OJ for qemu-devel@nongnu.org; Fri, 14 Aug 2015 13:46:35 -0400 From: Peter Maydell Date: Fri, 14 Aug 2015 18:46:28 +0100 Message-Id: <1439574392-4403-1-git-send-email-peter.maydell@linaro.org> Subject: [Qemu-devel] [PATCH v2 0/4] scripts/qemu-gdb: Split into modules List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Jan Kiszka , Markus Armbruster , "Dr. David Alan Gilbert" , Stefan Hajnoczi , patches@linaro.org This patch series splits scripts/qemu-gdb into separate files for each command, to make it easier to add new commands in future without them all going into a single huge file. Patches 1 and 2 do that split, and have been on the list before (and reviewed by Stefan). Patch 3 makes the script tell gdb not to stop on SIGUSR1, since that's our SIGIPI and happens all the time. Patch 4 adds a brief comment explaining how to source the script. NB that I haven't attempted to make the script work with automatic-sourcing based on the executable being debugged. Somebody who uses the script that way can do that :-) I've dropped the script which added support for setting breakpoints on QEMU tracepoints for the moment (pending investigating static probe point support which might render it unnecessary). Mostly I'd like to get these into master so that future debug commands can go into the right place -- IIRC David Gilbert had a patchset which added another command. thanks -- PMM Peter Maydell (4): scripts/qemu-gdb: Split MtreeCommand into its own module scripts/qemu-gdb: Split CoroutineCommand into its own file scripts/qemu-gdb: Silently pass through SIGUSR1 scripts/qemu-gdb: Add brief comment describing usage scripts/qemu-gdb.py | 146 ++++--------------------------------------- scripts/qemugdb/__init__.py | 28 +++++++++ scripts/qemugdb/coroutine.py | 91 +++++++++++++++++++++++++++ scripts/qemugdb/mtree.py | 82 ++++++++++++++++++++++++ 4 files changed, 212 insertions(+), 135 deletions(-) create mode 100644 scripts/qemugdb/__init__.py create mode 100644 scripts/qemugdb/coroutine.py create mode 100644 scripts/qemugdb/mtree.py -- 1.9.1