From patchwork Fri Jan 24 21:22:21 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Burgess X-Patchwork-Id: 37536 Received: (qmail 107813 invoked by alias); 24 Jan 2020 21:22:35 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 107751 invoked by uid 89); 24 Jan 2020 21:22:35 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-21.6 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.1 spammy=viewing X-HELO: mail-wr1-f54.google.com Received: from mail-wr1-f54.google.com (HELO mail-wr1-f54.google.com) (209.85.221.54) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 24 Jan 2020 21:22:32 +0000 Received: by mail-wr1-f54.google.com with SMTP id q6so3685651wro.9 for ; Fri, 24 Jan 2020 13:22:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :in-reply-to:references; bh=yfRYOl1jRvgLm0edSlBgBG/SOSM28C/oTFQUad28B9g=; b=LiB1t4VkJ0Ajb15wyAVulUSihxI0OODjejRnQrW+bWKPgJUzSk9mz4w/9hE2rqlN03 LZBBUmmueh6gbcFJyZObF88OcHxmoqM8uI/TUa4I2N8q81sPfHzRunyFmXPnWNQRhwb6 85XBvc4Tad9kANv+xQKTngqTRbg4C9iwh3hSSc1Is2OWxkpLxe64uK/BUfjFNxW4eJV1 aV7O72QSgUedGj3NkeGzJlMDA8n64jWZaw6PVr82cZ8Zj4ANrfQCWDRoxLeoaqRidZTr G/aLV1jkAlaPhzQjE/sgmYU5ZuTCQkaePUOeInwHg8qrCupLZth1tMbBORYQXac39SjO ZrnA== Return-Path: Received: from localhost (host86-191-239-73.range86-191.btcentralplus.com. [86.191.239.73]) by smtp.gmail.com with ESMTPSA id x18sm9272102wrr.75.2020.01.24.13.22.29 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 24 Jan 2020 13:22:29 -0800 (PST) From: Andrew Burgess To: gdb-patches@sourceware.org Cc: Shahab.Vahedi@synopsys.com, palves@redhat.com, tom@tromey.com, Andrew Burgess Subject: [PATCH 2/2] gdb/tui: Disassembler scrolling of very small programs Date: Fri, 24 Jan 2020 21:22:21 +0000 Message-Id: <3073ce5588097f61f2391bdfad015a31665ce50d.1579900754.git.andrew.burgess@embecosm.com> In-Reply-To: References: In-Reply-To: References: X-IsSubscribed: yes In TUI mode, if the disassembly output for the program is less than one screen long, then currently if the user scrolls down until on the last assembly instruction is displayed and then tries to scroll up using Page-Up, the display doesn't update - they are stuck viewing the last line. If the user tries to scroll up using the Up-Arrow, then the display scrolls normally. What is happening is on the Page-Up we ask GDB to scroll backward the same number of lines as the height of the TUI ASM window. The back scanner, which looks for a good place to start disassembling, fails to find a starting address which will provide the requested number of new lines before we get back to the original starting address (which is not surprising, our whole program contains less than a screen height of instructions), as a result the back scanner gives up and returns the original starting address. When we scroll with Up-Arrow we only ask the back scanner to find 1 new instruction, which it manages to do, so this scroll works. The solution here is, when we fail to find enough instructions, to return the lowest address we did manage to find. This will ensure we jump to the lowest possible address in the disassembly output. gdb/ChangeLog: PR tui/9765 * tui/tui-disasm.c (tui_find_disassembly_address): If we don't have enough lines to fill the screen, still return the lowest address we found. gdb/testsuite/ChangeLog: PR tui/9765 * gdb.tui/tui-layout-asm-short-prog.S: New file. * gdb.tui/tui-layout-asm-short-prog.exp: New file. Change-Id: I6a6a7972c68a0559e9717fd8d82870b669a40af3 --- gdb/ChangeLog | 7 +++ gdb/testsuite/ChangeLog | 6 +++ gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.S | 22 ++++++++++ .../gdb.tui/tui-layout-asm-short-prog.exp | 51 ++++++++++++++++++++++ gdb/tui/tui-disasm.c | 2 +- 5 files changed, 87 insertions(+), 1 deletion(-) create mode 100644 gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.S create mode 100644 gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.exp diff --git a/gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.S b/gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.S new file mode 100644 index 00000000000..7705ef139aa --- /dev/null +++ b/gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.S @@ -0,0 +1,22 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2020 Free Software Foundation, Inc. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + + .global _start +_start: + .rept 5 + nop + .endr diff --git a/gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.exp b/gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.exp new file mode 100644 index 00000000000..d0b871ff762 --- /dev/null +++ b/gdb/testsuite/gdb.tui/tui-layout-asm-short-prog.exp @@ -0,0 +1,51 @@ +# Copyright 2020 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# Ensure that 'layout asm' can scroll away from the last line of a +# very short program using a page up sized scroll. + +load_lib "tuiterm.exp" + +standard_testfile tui-layout-asm-short-prog.S + +if {[build_executable "failed to prepare" ${testfile} ${srcfile} \ + {debug additional_flags=-nostdlib \ + additional_flags=-nostartfiles}] == -1} { + return -1 +} + +Term::clean_restart 24 80 $testfile +if {![Term::prepare_for_tui]} { + unsupported "TUI not supported" +} + +# This puts us into TUI mode, and should display the ASM window. +Term::command "layout asm" +Term::check_box_contents "check asm box contents" 0 0 80 15 "<_start>" + +# Record the first line of output, we'll need this later. +set first_line [Term::get_line 1] + +# Scroll forward a large amount, this should take us to the last +# instruction in the program. +Term::command "+ 13" +Term::check_box_contents "check asm box contents again" 0 0 80 15 \ + "^ *$hex\[^\n\]+\n +\n" + +# Now scroll backward again, we should return to the start of the +# program. +Term::command "- 13" +gdb_assert {[string eq "$first_line" [Term::get_line 1]]} \ + "check first line is back" diff --git a/gdb/tui/tui-disasm.c b/gdb/tui/tui-disasm.c index 726b7c27d60..547d2c9e6d6 100644 --- a/gdb/tui/tui-disasm.c +++ b/gdb/tui/tui-disasm.c @@ -268,7 +268,7 @@ tui_find_disassembly_address (struct gdbarch *gdbarch, CORE_ADDR pc, int from) if (asm_lines.size () < max_lines) { if (!possible_new_low.has_value ()) - return pc; + return new_low; /* Take the best possible match we have. */ new_low = *possible_new_low;